Menu

#411 browsed network root directory isn't accepted

2.0 Series
closed-fixed
Scripting (211)
5
2005-11-26
2005-10-19
Anonymous
No

On windows, creating a shortcut installer for an access
mdb, e.g., to \\mycomp\\databaseInstDir

I set AllowRootDirInstall true in the script, but
coming back from the browse which is filtered using
.onVerifyInstDir the Install button is grayed out.

Placing StrCpy $INSTDIR "$INSTDIR\" in .onVerifyInstDir
allows installation.

The bug is either that it will only install (or reveal
the functionality to install) to a network path ending
with a backslash, or that the return value from the
browse function is missing the backslash.

Aidan
amorgan \ vfemail \ net

SCRIPT excerpt:

...

AllowRootDirInstall true
InstallDir ""
DirText "Setup must be provided with the network
location of the database " \
"Network Location of Database" \
"" \
"Select the network location of the Database"

...

//from windows help documentation
Function .onVerifyInstDir
IfFileExists $INSTDIR\database.mdb PathGood
Abort
PathGood:
;next line is my own
StrCpy $INSTDIR "$INSTDIR\"
FunctionEnd

Page directory
Page instfiles

.....

Discussion

  • Amir Szekely

    Amir Szekely - 2005-10-20

    Logged In: YES
    user_id=584402

    Bug report #1331310 says this happens with 2.10.

     
  • Amir Szekely

    Amir Szekely - 2005-10-22

    Logged In: YES
    user_id=584402

    I can reproduce the problem. I hope I'll get to it soon,
    before 2.11.

     
  • Amir Szekely

    Amir Szekely - 2005-11-26

    Logged In: YES
    user_id=584402

    Thanks, fixed.

     
  • Amir Szekely

    Amir Szekely - 2005-11-26
    • assigned_to: nobody --> kichik
    • summary: StrCpy $INSTDIR "$INSTDIR\" to allow root install - hack? --> browsed network root directory isn't accepted
    • status: open --> closed-fixed
     
  • Amir Szekely

    Amir Szekely - 2005-11-26

    Logged In: YES
    user_id=584402

    btw, there's no need for .onVerifyInstDir to reproduce this.
    Simply browsing for a network share root is enough.

     

Log in to post a comment.