Bug 4435 - Inno installer should not ask about overwriting Mercurial.ini
Summary: Inno installer should not ask about overwriting Mercurial.ini
Status: RESOLVED FIXED
Alias: None
Product: Mercurial
Classification: Unclassified
Component: Mercurial (show other bugs)
Version: 3.2
Hardware: PC Windows
: normal bug
Assignee: Bugzilla
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-04 03:35 UTC by techtonik
Modified: 2015-01-24 08:58 UTC (History)
3 users (show)

See Also:
Python Version: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description techtonik 2014-11-04 03:35 UTC
I am always confused when .exe installer asks me if I want to overwrite Mercurial.ini  I pretty sure I never edit any file in "Program Files", because there is my personal setting in "Users" and even if I need to make some configuration global the proper way for this is to place file in "All users" and look it up from there.
Comment 1 Pascal Quantin 2014-11-04 04:25 UTC
I tend to disagree with you: if someone is crazy enough to edit the file in the installation folder (was quite easy to do so in Windows XP for example), he certainly does not want to have his profile erased on the next upgrade. Better safe than sorry.
Note that I fully agree with you and any modification should be done in the profile folder.
Comment 2 techtonik 2014-11-04 05:25 UTC
Why not to ship Mercurial.ini.in and let people rename it if they want their custom config?

The dialog is unhelpful, because it doesn't show the diff, so you can't tell if you need to overwrite it for new version or not anyway.
Comment 3 kiilerix 2014-11-04 05:30 UTC
Mercuria.ini is documented as something the user can customize. http://selenic.com/hg/rev/c4ce077588d0 introduce a new way for packagers to ship config that users never should modify and silently could/should be overwritten.

Proper handling of that in the .exe installers requires some windows users to step up and maintain it.
Comment 4 techtonik 2014-11-04 05:43 UTC
Deprecating and removing Mercurial.ini seems like a way to go then.
Comment 5 HG Bot 2014-11-04 17:46 UTC
Fixed by http://selenic.com/repo/hg/rev/5ec7b2cb577b
Pascal Quantin <pascal.quantin@gmail.com>
win32: remove Mercurial.ini file from Inno Setup installer (issue4435)

(please test the fix)
Comment 6 techtonik 2014-11-06 03:15 UTC
And how can I test this? https://bitbucket.org/tortoisehg/thg-winbuild/downloads/ linked from http://mercurial.selenic.com/wiki/Download#Windows seems defunkt.

I also found some references to Mercurial.ini that are likely to be revisited.
https://stackoverflow.com/questions/2658419/mercurial-win-instal-no-ini-files
http://mercurial.selenic.com/wiki/WindowsInstall

Mercurial.ini seems like a kludge. `hgrc` and `.hgrc` seem perfectly fine. Those people who can't open non-ini file from Windows Explorer use TortoiseHg anyway. http://www.selenic.com/mercurial/hgrc.5.html
Comment 8 Matt Mackall 2015-01-22 15:04 UTC
Bulk testing -> fixed
Comment 9 techtonik 2015-01-24 08:58 UTC
Sorry, wasn't able to test this - not on Windows ATM. But it is nice
to see this fixed.