[sword-devel] BibleCS Installer

DM Smith dmsmith555 at yahoo.com
Thu Feb 9 11:31:24 MST 2006


L.Allan-pbio wrote:
> Some comments:
> * Very good progress.
>
> * Nice implementation of the sword url link with WriteIniStr used to 
> create the link ....
>
> * HKLM SOFTWARE\Classes\sword ???
>   Seems to be set by running sword.exe .... whole series of registry 
> entries. Perhaps these should be removed also?
These are being created by the OS. Not by the installer or the executable.

Try installing and running 1.5.6 and then search for registry entries. 
There are a bunch that are managed under some notion of shared library. 
In looking at where these extra entries show up, I also see a bunch for 
stuff that I have uninstalled from other applications.

I think that the installer and uninstaller should be responsible only 
for the entries that are directly managed by by the installer and the 
application. As far as I know, the application doesn't do anything with 
writing registry keys.
>
> * The InstallManger.exe and related files are deleted during 
> UNSEC0000, so aren't available for -un.post UNSEC0001
>   I would suggest not removing the InstallManager .... handle it as a 
> different/separate uninstall .... it should be left available for 
> other "family apps"
Good catch
>   (I would suggest renaming SEC000# and UNSEC000# to more meaningful 
> names)
This is on the "cleanup" the code set of final tasks. Same with breaking 
out the defines at the top.
>
> * Still using SWORD_HOME rather than SWORD_PATH?
Still haven't gotten to look at this.
>
> * I have some concerns about over-writing an existing SWORD_PATH 
> environment variable setting. That seems appropriate for a 
> "Components" page decision (this installer doesn't use any 
> "Components" page)
I didn't think about overwriting. I'll see about adding a check to see 
if it is currently defined and pointing to an existing directory. If it 
is not then I think it should be set.

This should be in a separate nsh file so all SWORD family applications 
can use it.
>
>
> * Uninstaller may not want to remove SWORD_PATH environment variable. 
> Leave it around for other "sword-api family apps" to make use of.
>
> * May not want to get rid of mods.d and leave it for other "family 
> apps" that are making use of SWORD_PATH. I think the 
> InstallManager.exe is the appropriate way to get rid of mods.d files 
> (to keep them sync'ed ok with contents of subdirectory modules?)
I think that there should be a dialog asking the user whether the sword 
modules should be removed, if this is the last application in the SWORD 
family to be removed. If the answer is yes then the modules and env var 
are removed. Otherwise they are left intact.

This will be part of the final solution. OK?

This should be in a separate nsh file so all SWORD family applications 
can use it.
>
> * Baffled by what is happening in the SwordUtil::AllDone function to 
> read Field 4 and Field 5 of ioSpecial.ini. I only see 3 fields in 
> ioSpecial.ini
Its baffling because it relies upon the actual execution of what should 
be "black box" code.

I studied the code in NSIS/contrib/Modern UI/system.nsh
It uses the ioSpecial.ini (which has only the 3 fields as you noted) as 
a starting point.
This file is copied to $NSISPLUGINDIR (generally in temp) and then 
manipulated by the various MUI macros.
The check boxes on the Finish page can either be reboot or it can have 
one, the other or both of "run" and "readme".
Based upon this Field 4 is for reboot, and if reboot is not specified, 
then run and if run is not specified then readme.
If run and readme are specified then these become fields 4 and 5 
respectively.

Now if the MUI changes, then all bets are off. These could be fields 6 
and 10. Or have meaningful names.
I should probably put a check in against MUI_VERSION and abort the 
compile if it is otherwise.

What the AllDone done is called after the user (un)checks the RUN and 
README options and then clicks on "Finish" (I think that's the button label)

So at this time, the MUI code is checking for Field 4 and 5. It then 
sequentially runs the RUN action followed by the README action.
The run action is executed using Exec, which is non-blocking or Call, in 
the case of a function.
The readme action is executed using ExecShell on "open", which is 
non-blocking or Call, in the case of the function.

So I created a function AllDone that is called by both the RUN and 
README callbacks. This function is a run once.

Since ExecShell is non-blocking, I changed the execution of the readme 
to explicitly use notepad and to use ExecWait. The significant 
difference here is that if the user has set emacs, vim or some other 
text editor as their default *.txt open action, it will not be used. And 
it now waits until the user is done.

If the user chose to run the application, I bring up the installmgr and 
then the application. (using the code you supplied)


>
>   I think a simpler solution ... would let the readme.txt and the 
> InstallManager show up at the same time ...
>     It might be useful to be able to see the readme.txt instructions 
> for using InstallManager at the same time that InstallManager appears.
>  Would use ExecWait for InstallManager, then Exec for sword.exe
>  (in other words, standard handling of readme.txt, and special 
> function for InstallManager and then sword.exe)
Troy has requested that the readme precede the installer as it is in the 
current 1.5.6 installer. (See the original request at the bottom of this 
e-mail)

Let me say, I don't like my solution, but I don't know another way to 
satisfy Troy's request. I don't have a problem with Troy's request, I 
just don't like using internals of a "black box" to create a solution. 
The other solution would be to create a custom page that would execute 
the AllDone function once and use variable under that page's control. My 
take is that I would want to use the internationalized strings that the 
MUI provides and that would require digging into MUI's internals and 
also duplicate a lot of code that is already provide by MUI. It's a 
matter of trade offs. I can add more comments to the function if that 
would help.

The primary purpose of the readme is to let users in countries hostile 
to believers know that the installer is going to connect in an insecure 
manner to the internet.

The problem that I have with running both at the same time is that 
sometimes the readme is on top and sometimes it is on the bottom. The 
user may not see the readme until after they use the InstallMgr.

As an aside, the way we handle this in JSword is that the application 
looks to see if there are any modules installed and if not then prompts 
the user to install some. At this time we don't warn the user, because 
JSword was delivered initially via WebStart.
>
>
> * Uninstaller leaves registry entry App Paths\Sword.exe (maybe ok?)
It wasn't supposed to.

But if we leave the modules, we should leave some key regarding them 
behind. See below for a suggestion that the key SOFTWARE/CrossWire be used.

>
>
> * Overall, the installer is using three registry "subhives" (and 
> sword.exe uses another one). Perhaps it could settle on one, and have 
> all the applicable information there? I/we/you could check with the 
> nsis forum to see what is "best practice".
>
> HKLM SOFTWARE\CrossWire\The SWORD Project

This is part of at least two installer generators. NSIS Eclipse plugin 
and one other, I don't remember.
This is also in many of the examples as a place to keep configuration 
information.
This is fairly standard. Lots of other apps under SOFTWARE.

The idea here is that if all the other apps like LCDBible put stuff 
under SOFTWARE/CrossWire, then we can discover whether the application 
is the last to be uninstalled. If it is not the last then the 
uninstaller shouldn't delete mods.d and modules or even offer the user a 
choice.

I didn't know that the Sword executables are statically linked except 
for the icudtXX.dll, which only contains data. This means that sharing 
is limited to mods.d, modules and possibly icudtXX.dll.

So I think that this as a mechanism should work for the LCD bible.

> HKLM SOFTWARE\Microsoft\Windows\CurrentVersion\App Paths\Sword.exe
This is standard and follows what was done previously.
The NSIS command InstallDirRegKey goes after this key as a prior 
installation and uses that as the new default INSTDIR.
Lots of other stuff under Apps Paths to model after.
> HKLM SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\The SWORD 
> Project

This is standard and follows what was done previously.
Also this is where Add/Remove Programs looks for uninstallers.

> HKLM SOFTWARE\Classes\sword (used by sword.exe for icon and perhaps 
> file association?)
See above.
>
> WriteRegDWord HKLM "${PRODUCT_REGKEY}\Components" Main 1
> WriteRegStr   HKLM "${PRODUCT_INST_KEY}"   Path            $INSTDIR
> WriteRegStr   HKLM "${PRODUCT_UNINST_KEY}" DisplayName "${PRODUCT_TITLE}"
>
> * I'd like to do some more testing of Win2000 and WinXp in "limited 
> user" privilege mode, or did we decide not to be concerned about that? 
> I suspect that use of HKLM by the installer (and sword.exe itself) may 
> make it difficult or impossible for a "limited user" to be able to 
> install and run.
I've been looking for an example of a page that presents radio buttons 
for two options:
O Install for everyone that uses this machine (requires administrator 
priveledges)
O Install only for me.
The result of this page would be to call:
SetShellVarContext all
or
SetShellVarContext current

Then everywhere HKLM is used, it would be replaced with SHCTX.

The only trick to doing this is the detection of a prior release.
One would need to check both HKLM and HKCU.
Based upon the results of this check the default for the page would be set.

If it did exist under HKLM and not HKCU and the user chose HKCU, then we 
would need to delete the HKLM registry entries. (and visa versa)

If you can get this working fairly quickly, I'd be glad to use it, but I 
am running out of time and there is a sense of urgency to getting 
something working.

According to Troy there are about 20 requests for CDs each month and it 
takes time to master another batch. Since we are close to a release, it 
would make sense to wait on new CD requests. But not too long.

And for this CD, I would like to put JSword/BibleDesktop on it. So I 
have some work to do in that arena.

Also, the KJV that's distributed as part of the starter pack for 1.5.6 
has known problems. While the current KJV module also has problems it is 
much better. With 4000+ downloads of the starter pack each month, it 
would be good to get the installer in place very soon.

>
> ----- Original Message ----- From: "DM Smith" <dmsmith555 at yahoo.com>
> To: "SWORD Developers' Collaboration Forum" <sword-devel at crosswire.org>
> Sent: Thursday, February 09, 2006 5:56 AM
> Subject: Re: [sword-devel] BibleCS Installer
>
>
>> I have updated the installer as before.
>> This now launches readme (if requested) and then the InstallMgr 
>> followed by Sword.exe (if program run requested)
>> I think that all the registry keys are correct.
>> I think that all the links are correct.
>> I fixed a problem where the InstallMgr --uninstall was called even if 
>> the user chose to cancel the uninstall.
>> I added the splash screen to the uninstall.
>>
>> So, while it does it differently internally, we now have the same 
>> functionality of the simple installer. StarterPack is next.
>>
>> There may be a little more that needs to be done along with code 
>> commenting and simple refactoring, Then I think this is sufficient to 
>> get a first release out.
>>
>> Monday, should be a reasonable milestone.
>>
>> Lynn, I think there are enough hooks in the registry for the 
>> discovery of Sword to use its files in another project. I am still 
>> trying to figure out SWORD_PATH and SWORD_HOME.
>> Troy, please take a critical look at it and see what else must be 
>> done before a release.
>>
>> DM Smith wrote:
>>> I have uploaded a current copy of the installer script that I am 
>>> working on to:
>>> http://www.crosswire.org/bibledesktop/biblecs/biblecs/installer/SwordSetup.nsi 
>>>
>>>
>>> I have also uploaded the installer:
>>> http://www.crosswire.org/bibledesktop/biblecs/biblecs/installer/SwordSetup.exe 
>>>
>>>
>>> Give it a try and let me know what what you think.
>>>
>>> I will be adding the ability to chose whether to create a Start Menu 
>>> Group (may already be present), a desktop shortcut and a Quick 
>>> Launch shortcut. And the launching of the executable and the readme 
>>> at the end don't work right.
>>>
>>> DM Smith wrote:
>>>> Troy,
>>>>    Lynn and I have been working on this. Our approach has been a 
>>>> little different. My goal is to get a basic installer for just 
>>>> BibleCS and his has been to create a family of installers for all 
>>>> derivative Sword API Windows applications. These are not at odds 
>>>> with each other and I think we are nearly there with at least a 
>>>> first pass.
>>>>
>>>> To finish the first pass, we need a couple of things:
>>>> 1) All the compile artifacts of the build (e.g. dlls, exes, ...) as 
>>>> I don't have a C++ compile environment. Preferably, these should be 
>>>> what will be used by the installer. Also, if you could give me the 
>>>> paths for these using the checkout scheme below that would be 
>>>> great. Currently, I am stubbing this with 1.5.6.
>>>
>>> I am now stubbing with the latest betas in the alpha download area. 
>>> These require icudt28L.dll and not icudt34dll. The latter is what is 
>>> in source code control.
>>>
>>>> 2) What are the other files that must be included? And for dlls, 
>>>> what versions?
>>>> (For 1 and 2, if you could create an directory layout of what the 
>>>> final install should be like that might be easier. Essentially, 
>>>> this would be a zip of what the installation directory would 
>>>> contain right after installation when no modules are installed.)
>>>
>>> Rooted at http://www.crosswire.org/bibledesktop/biblecs is the 
>>> "image" of what the installer uses.
>>> This is a subset of the checkout Troy mentioned below. However, I 
>>> used a tag for checking out Sword, not trunk.
>>>
>>> I am not sure of the placement of sword.exe or InstallManager.exe.
>>> I have placed sword.exe in biblecs/stage and InstallManager.exe in 
>>> apps/InstallMgr
>>>
>>>> 3) What is the minimum set of files to provide the Sword API for 
>>>> other applications? (I want to create a SwordAPI installer that 
>>>> will install the absolute minimum that a derivative application 
>>>> must have present. This will then be called by those applications 
>>>> installers, if the SwordAPI has not been installed.) Does it need 
>>>> icudt34.dll? Does it need the install manager? How about all the 
>>>> subdirectories in biblecs/.../stage/?
>>>> 4) In an earlier note, Lynn gave a path to a patch to allow for the 
>>>> modules to be located elsewhere. Is this something that can be 
>>>> included in 1.5.8 or should it wait for a later release.
>>>> 5) NSIS needs (un)installer icons that work. I am terrible at 
>>>> artwork, so I am not the one to do it. NSIS requires that the size 
>>>> of the installer and uninstaller icons be the same.
>>>
>>> I found that the installer icons were taken from NSIS. I am using 
>>> these directly now and have fixed the mismatch problem. I also have 
>>> fully branded the installer with the "orange" theme.
>>>
>>>>
>>>> Can you give me check in privs for the installer directory? Then I 
>>>> can work in there. I think that the following addition to the svn 
>>>> authorization file, would work:
>>>> [biblecs:/trunk/installer]
>>>> dmsmith=rw
>>>>
>>>> Right now this is where the installer is at:
>>>> 1) Offers the user a choice as to where to install the program and 
>>>> what Start Menu Group to create.
>>>>    It defaults to the same default locations that 1.5.6 uses.
>>>> 2) The installation
>>>>    a) Installs the files
>>>>    b) creates the registry entries
>>>>       i) The uninstaller is listed in Add/Remove Programs
>>>>       ii) Keys and values are created so that other programs can 
>>>> discover where Sword is installed.
>>>
>>> It still needs to create a couple of keys.
>>>
>>>>    c) creates a desktop shortcut
>>>>        Does not offer the user a choice about this one.
>>>>        Should also probably install to the Quick Start task bar. 
>>>> And it should be a choice, too.
>>>
>>> I have not added the choice for these.
>>>
>>>>    d) creates shortcuts in the Start Menu for
>>>>        i) the Sword program
>>>>        ii) the Sword Install Manager
>>>>        iii) the website
>>>>        iv) the uninstaller
>>>>       It probably should also create a shortcut to help.
>>>
>>> Shortcut for help now present.
>>>
>>>>    e) It adds/changes SWORD_HOME environment variable to point to 
>>>> the installation directory.
>>>>    The installer assumes that prior versions have been deleted. 
>>>> This may be bad.
>>>>    Are there any files that *must* be removed for the install to 
>>>> the same directory to be successful?
>>>>    For example, is it bad to have icudtXX.dll and icudtYY.dll in 
>>>> the same directory?
>>> It appears that the executables require a particular version, so 
>>> this won't be a problem.
>>> However, it may still be best to do a "clean" upgrade.
>>>> 3) It creates an uninstaller that
>>>>    a) runs InstallManager -uninstall
>>>>       currently, this is broken
>>>
>>> This now works, but both the readme and the installer come up at the 
>>> same time rather than serially.
>>> I think I know a way to fix this.
>>>
>>>>    b) Removes all files that the installer installed. Leaving all 
>>>> files that the program or user created.
>>>>    c) Removes all shortcuts (that have not been moved)
>>>>    c) Removes empty folders
>>>>    d) Removes registry entries.
>>>>    The uninstaller does not determine whether there are any 
>>>> dependent, derivative applications and act appropriately.
>>>>    These will have a registry key located under 
>>>> HKLM\software\CrossWire.
>>>>    It should be the responsibility of the last removed derivative 
>>>> application to remove the SwordAPI.
>>>>
>>>> Is there anything missing from the above? Must haves? Should haves? 
>>>> Nice to have?
>>>>
>>>> The last thing needed before the installer(s) is(are) released is 
>>>> robust testing. I know there are somethings that don't work yet and 
>>>> some of the paths are wrong. And there may be some things, I won't 
>>>> find because I always test the same small set of paths! So after I 
>>>> get the bugs out that I know, testing help would be appreciated.
>>>>
>>>> In His Service,
>>>>    DM
>>>>
>>>> Troy A. Griffitts wrote:
>>>>> Hey guys,
>>>>>    I spend today trying to build a new installer for the upcoming 
>>>>> release of BibleCS.  I'm abandoning the commercial InstallShield 
>>>>> installer we currently use and am trying to make everything work 
>>>>> using the free NullSoft installer.  I gave up after trying to 
>>>>> learn the scripting language and downloaded Eclipse and the 
>>>>> NullSoft Installer plugin for Eclipse.  I used a wizard and got it 
>>>>> pretty far.  Had to hand edit the generated script to get it to 
>>>>> compile.
>>>>>
>>>>> There are a few things missing:
>>>>>    Launch Readme.txt and wait for close BEFORE launching 
>>>>> InstallManager.exe
>>>>>    Registry keys correct so Windows Module self-installers 
>>>>> (website, module download, windows) know where to unzip themselves.
>>>>>    Uninstaller successfully launching "$INSTDIR/InstallManager.exe 
>>>>> -uninstall" to uninstall all modules
>>>>>
>>>>>    I've checked everything in: 
>>>>> https://crosswire.org/svn/biblecs/trunk/installer   so if you'd 
>>>>> like to give it a shot, please help yourself!  It expects the 
>>>>> normal: sword, icu-sword, biblecs directories all checked out at 
>>>>> the same level.
>>>>>
>>>>> svn co https://crosswire.org/svn/sword/trunk sword
>>>>> svn co https://crosswire.org/svn/icu-sword/trunk icu-sword
>>>>> svn co https://crosswire.org/svn/biblecs/trunk biblecs
>>>>>
>>>>>    I'd really welcome the help with this.
>>>>>
>>>>>       -Troy.
>>>
>>> _______________________________________________
>>> sword-devel mailing list: sword-devel at crosswire.org
>>> http://www.crosswire.org/mailman/listinfo/sword-devel
>>> Instructions to unsubscribe/change your settings at above page
>>>
>>>
>>
>> _______________________________________________
>> sword-devel mailing list: sword-devel at crosswire.org
>> http://www.crosswire.org/mailman/listinfo/sword-devel
>> Instructions to unsubscribe/change your settings at above page
>>
>
> _______________________________________________
> sword-devel mailing list: sword-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/sword-devel
> Instructions to unsubscribe/change your settings at above page
>


More information about the sword-devel mailing list