Opened 16 years ago

Closed 16 years ago

#855 closed defect (fixed)

Move serverconfig.ini configuration out of wix include file

Reported by: jng Owned by:
Priority: low Milestone: 2.1
Component: General Version:
Severity: trivial Keywords: installer wix
Cc: External ID:

Description

The current wix installer uses the default serverconfig.ini

However the installer does not change the paths in that file during installation to match the paths defined in the installer.

Attached is a patch that will write the correct paths to serverconfig.ini during installation.

This requires that the default serverconfig.ini is replaced with one that has all the paths removed (also attached)

Attachments (2)

mgserver.patch (14.7 KB ) - added by jng 16 years ago.
diff for FDO and MapGuide server wix files
serverconfig.ini (27.4 KB ) - added by jng 16 years ago.
serverconfig.ini used for wix

Download all attachments as: .zip

Change History (5)

by jng, 16 years ago

Attachment: mgserver.patch added

diff for FDO and MapGuide server wix files

by jng, 16 years ago

Attachment: serverconfig.ini added

serverconfig.ini used for wix

comment:1 by jbirch, 16 years ago

Jackie,

The current installer include files for the bin directory included a section that configured the serverconfig.ini file. I'm guessing that when you regenerated the file include this got blown away?

mapguide/browser/trunk/Installer/Libraries/MapGuide%20Server/FileIncludes/incBinFiles.wxs

Could you please make sure that your patch duplicates this functionality? It would likely to be best to pull this out into a separate area so that this doesn't get walked on every time the include files are regenned.

comment:2 by jng, 16 years ago

Summary: Wix Installer does not configure serverconfig.iniMove serverconfig.ini configuration out of wix include file

Hmmm, I saw the paraffin batch files and assumed these files were disposable (since the batch files will overwrite them), so I never looked at the file that was gonna get trashed :-)

Yes this patch is essentially a relocation of that functionality into a non-volatile area. Having seen the original file now, I'll put up another patch. One that doesn't scatter these IniFile elements all over the place.

Summary changed to reflect the true issue.

comment:3 by jng, 16 years ago

Resolution: fixed
Status: newclosed

Fixed with submission r3644

Note: See TracTickets for help on using tickets.