This is an archive of the discontinued Mercurial Phabricator instance.

packaging: remove hg.exe.local file
ClosedPublic

Authored by indygreg on Oct 23 2019, 9:54 PM.

Details

Summary

<exe>.local files are used by Windows to set up DLL
redirection. But these files are ignored if you embed an
application manifest in your binary, which we do.

So the existence of this file serves no purpose. So we remove
it.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

indygreg created this revision.Oct 23 2019, 9:54 PM

From the referenced URL:

If the application has a manifest, then any .local files are ignored.

We seem to add a manifest now, so maybe this can go away instead?

indygreg abandoned this revision.Oct 24 2019, 11:28 PM

From the referenced URL:

If the application has a manifest, then any .local files are ignored.

We seem to add a manifest now, so maybe this can go away instead?

Nice catch!

I confirmed that we are embedding an application manifest in hg.exe. So we can delete this file. I'll update the series.

indygreg reclaimed this revision.Oct 24 2019, 11:28 PM
indygreg retitled this revision from packaging: create empty hg.exe.local file to packaging: remove hg.exe.local file.Oct 24 2019, 11:34 PM
indygreg edited the summary of this revision. (Show Details)
indygreg updated this revision to Diff 17399.
mharbison72 accepted this revision.Oct 24 2019, 11:47 PM

<exe>.manifest files are used by Windows to set up DLL

s/manifest/local/ ?

indygreg edited the summary of this revision. (Show Details)Oct 25 2019, 12:22 AM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.