Skip to content

[Xamarin.Android.Build.Tasks] Improve the logic of GetAdditionalResourcesFromAssemblies' file validation - #292

Merged
dellis1972 merged 1 commit into
dotnet:masterfrom
zappolowski:SpeedUpValidationOfDownloadedFiles
Nov 14, 2016
Merged

dellis1972 merged 1 commit into
dotnet:masterfrom
zappolowski:SpeedUpValidationOfDownloadedFiles

Conversation

@zappolowski

Copy link
Copy Markdown
Contributor

Validation of downloaded files in the
GetAdditionalResourcesFromAssemblies build task is currently rehashing
the same file several times during a build leading to increased build
times.

By storing the computed hash and reusing it in subsequent invocations
validation can be speed up drastically:

$ tail -n1 build_{orig,patched}.log
==> build_orig.log <==
Time Elapsed 00:07:50.9875230

==> build_patched.log <==
Time Elapsed 00:02:41.3661180

Validation of downloaded files in the
GetAdditionalResourcesFromAssemblies build task is currently rehashing
the same file several times during a build leading to increased build
times.

By storing the computed hash and reusing it in subsequent invocations
validation can be speed up drastically:

$ tail -n1 build_{orig,patched}.log
==> build_orig.log <==
Time Elapsed 00:07:50.9875230

==> build_patched.log <==
Time Elapsed 00:02:41.3661180
@monojenkins

Copy link
Copy Markdown

Hello! I'm the build bot for the Mono project.

I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.

Contributors can ignore this message.

@dnfclas

dnfclas commented Nov 13, 2016

Copy link
Copy Markdown

Hi @zappolowski, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@zappolowski zappolowski changed the title Improve the logic of GetAdditionalResourcesFromAssemblies' file validation [Xamarin.Android.Build.Tasks] Improve the logic of GetAdditionalResourcesFromAssemblies' file validation Nov 13, 2016
@dellis1972

Copy link
Copy Markdown
Contributor

Looks good merging

@dellis1972
dellis1972 merged commit 2f690ca into dotnet:master Nov 14, 2016
@zappolowski
zappolowski deleted the SpeedUpValidationOfDownloadedFiles branch November 15, 2016 19:40
jonpryor pushed a commit that referenced this pull request Nov 18, 2016
Validation of downloaded files in the
GetAdditionalResourcesFromAssemblies build task is currently rehashing
the same file several times during a build leading to increased build
times.

By storing the computed hash and reusing it in subsequent invocations
validation can be speed up drastically:

$ tail -n1 build_{orig,patched}.log
==> build_orig.log <==
Time Elapsed 00:07:50.9875230

==> build_patched.log <==
Time Elapsed 00:02:41.3661180
radical pushed a commit that referenced this pull request May 8, 2018
This options tells the tool to generate the marshal methods only for
the given type. It comes handy when woking with big assemblies like
`Mono.Android.dll` and one is interested only in results for the
given type.
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants