Old 13th February 2006, 21:05   #1
sissy
Guest
 
Posts: n/a
My Thoughts

Hi Guys,

Ive reading the forums here and looking at a few scripts and have decided to take the plunge.

I dont do many installers and when i do i have always used InnoSetup for it and whilst its an awesome product i think NSIS may be even better.

As with the grip of some people here the syntax is horrible and the docs arent the best, however the forum seems to make up for that with its skilled members.

So im going to stick with NSIS and try to learn it and come back with a more detailed report sometime soon.

Something i noticed on a few scripts out there in the uninstall section was -

RMDir /r "$INSTDIR\*.*"
RMDir "$INSTDIR"

Isnt that absolute madness? because as soon as someone installs into say x:\Program Files\ with no APP FOLDER name, on uninstall its going to be HORRIBLE

Theres probably a few more traps for the unexperienced too.

Thanks for your time.

Sissy
  Reply With Quote
Old 13th February 2006, 22:22   #2
dienjd
Senior Member
 
Join Date: Oct 2005
Posts: 189
RMDir "$INSTDIR" will only remove the $INSTDIR directory if it's empty. However, RMDir /r "$INSTDIR\*.*" would be very nasty if $INSTDIR got set to program files or windows.
dienjd is offline   Reply With Quote
Old 15th February 2006, 14:40   #3
Programmer
Junior Member
 
Join Date: Feb 2006
Posts: 4
Try these examples. I've still got a bunch to go, but I hope these might be a decent start. Let me know if they're useful.
Attached Files
File Type: zip tutorials.zip (18.9 KB, 254 views)
Programmer is offline   Reply With Quote
Old 16th February 2006, 15:31   #4
iceman_k
NSIS Dev
 
iceman_k's Avatar
 
Join Date: Feb 2003
Location: Boston, MA, U.S.A.
Posts: 455
Perhaps NSIS should include a check that dangerous commands like RmDir do not operate on Special folders like $PROGRAMFILES, $DESKTOP, $STARTMENU, $WINDIR, etc.
Or if it does, automatically popup a warning to the user and give them a chance to cancel the setup or uninstaller.

Cheers,
Iceman_K

EclipseNSIS - An NSIS IDE for the Eclipse Platform | My contributions to the wiki
iceman_k is offline   Reply With Quote
Reply
Go Back   Winamp & Shoutcast Forums > Developer Center > NSIS Discussion

Thread Tools Search this Thread
Search this Thread:

Advanced Search
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump