Bug 4265 - Windows installer no longer populates the registry
Summary: Windows installer no longer populates the registry
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Client platforms (show other bugs)
Version: 3.2.0
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.0.0
Assignee: Peter Åstrand
URL:
Keywords: ossman_tester
Depends on:
Blocks:
 
Reported: 2012-04-18 13:05 CEST by Peter Åstrand
Modified: 2012-11-28 12:26 CET (History)
0 users

See Also:
Acceptance Criteria:


Attachments

Description Peter Åstrand cendio 2012-04-18 13:05:29 CEST
Apparently, the Windows installer no longer imports settings.reg to the registry. Only customized installers does. I believe older clients did and I think the installer should. See also Issue 13225.
Comment 1 Peter Åstrand cendio 2012-08-07 13:42:59 CEST
nsis does not have direct support for importing to the registry from a .REG file. In general, those folks seems not to like those solutions. Instead, there's a hack that converts from .REG to nsis format, see:

http://nsis.sourceforge.net/Reg2Nsis_-_convert_registry_info_into_NSIS_commands

I like .REG files, though. The difference between program logic and a data format might be subtle, but is real. Since customers needs to edit the registry data in order to create customized clients, it's much easier for them to work with the .REG than with NSIS program code. Besides, it's good if we can change the installer without re-train customers. Additionally, one way of working is to create settings.reg by exporting from the registry. 

The earlier installer called "regedit /s <file>". I think we should move to reg.exe instead, since it's a real console/command line tool. Included with XP and later.
Comment 2 Peter Åstrand cendio 2012-08-08 09:50:02 CEST
Fixed in 25571.
Comment 3 Pierre Ossman cendio 2012-10-04 14:40:11 CEST
Did not work when I tried with build 3671.
Comment 4 Peter Åstrand cendio 2012-10-04 16:09:42 CEST
(In reply to comment #3)
> Did not work when I tried with build 3671.

Works, but since we only have a 32 bit client and the Win 7 machine in the lab is 64 bit, the settings (correctly) ends up below Wow6432Node.
Comment 5 Pierre Ossman cendio 2012-10-26 13:37:48 CEST
Verified on Vista.
Comment 6 Peter Åstrand cendio 2012-10-29 16:03:41 CET
From bug 2344, does not work on XP Embedded, since it lacks the reg.exe command.
Comment 7 Peter Åstrand cendio 2012-10-29 16:09:43 CET
(In reply to comment #6)
> From bug 2344, does not work on XP Embedded, since it lacks the reg.exe
> command.

Switched to regedit.exe in 26096. Need to re-test on all Windows platforms.
Comment 8 Pierre Ossman cendio 2012-11-05 15:01:12 CET
Works fine now. Tested on XP, XPe, Vista, 7.
Comment 9 Pierre Ossman cendio 2012-11-08 13:12:26 CET
settings.reg doesn't seem to get uninstalled.
Comment 10 Peter Åstrand cendio 2012-11-08 14:04:18 CET
(In reply to comment #9)
> settings.reg doesn't seem to get uninstalled.

r26149.
Comment 11 Pierre Ossman cendio 2012-11-12 14:36:36 CET
Retested on XPe, XP and 7.

Note You need to log in before you can comment on or make changes to this bug.