Code won't compile! - Help please!

18 posts

PanarchyJune 1, 2009 at 3:43 AM#

Code won't compile! - Help please!

Hi

Just trying some code & unfortunately I can't get it to work.

I've tried deleting lines 9, 10, 11, 12 & 38-42. But it still won't compile.

I've tried within one set of Apostrophe's, and within 2;
CreateShortcut "$STARTMENU\maint\maint.lnk" "$ProgramFiles\maint\maint.exe"

Yet I still find myself unable to successfully compile the script.

Could someone please help me get the following (or similar) to compile?

!include MUI2.nsh



Name `Maintenance`

OutFile `Maintenance Setup.exe`



!insertmacro MUI_PAGE_WELCOME

!insertmacro MUI_PAGE_LICENSE license.txt

!insertmacro MUI_PAGE_INSTFILES

!define MUI_FINISHPAGE_RUN "$DESKTOP\Maintenance MMC.msc"

!define MUI_FINISHPAGE_RUN "$DESKTOP\Maintenance.exe"

!define MUI_FINISHPAGE_RUN_TEXT "Open this"

!define MUI_FINISHPAGE_RUN_TEXT "Open that!"



!insertmacro MUI_PAGE_FINISH



!insertmacro MUI_LANGUAGE English



Section



  !macro ExtractMaintenance ToDir

    SetOutPath `${ToDir}`

    File "Maintenance Script.exe"

    File "Maintenance MMC.msc"

    File "Backup Directory.exe"

  !macroend

  

  !macro ExtractMaint ToDir

    SetOutPath `${ToDir}`

    File "Maint.exe"

  !macroend

  

  !insertmacro ExtractMaintenance "C:\Install\CD\Applications\Tools"

  

  !insertmacro ExtractMaint "$PROGRAMFILES\maint"

  

  CreateShortcut "$STARTMENU\maint\maint.lnk" "$ProgramFiles\maint\"  "maint.exe"



  CreateShortcut "$DESKTOP\Maintenance.lnk"

  "E:\Forstall\Applications\Tools\" "Maintenance Script.exe"

  

  CreateShortcut "$DESKTOP\Maintenance MMC.lnk"

  "E:\Forstall\Applications\Tools\" "Maintenance MMC.msc"



SectionEnd

Thanks in advance,

Panarchy

jpderuiterJune 1, 2009 at 7:25 AM#

What is the error message you get?

It looks like you start the rest of your CreateShortcut parameters on a new line.
If you want to do that, you have to end the first line with a backslash:

CreateShortcut "$STARTMENU\maint\maint.lnk"\

"$ProgramFiles\maint\"  "maint.exe"

PanarchyJune 1, 2009 at 7:29 PM#

LOL, I don't believe it!

My script can now compile!

😁

Now just need to work out the following;

- How to make more than one checkbox on the finish page
- & How to successfully create a start-menu shortcut

Please help me work out the aforementioned.

Thanks in advanced,

Panarchy

PanarchyJune 2, 2009 at 4:14 AM#

Hi

I have two features which I haven't been able to get to work.

1. Can't compile! (LOL)
2. Need multiple Checkboxes on the finish page (one for .exe one for .msc)

Here is my latest code;

!include MUI2.nsh



Name `Maintain`

OutFile `Installer.exe`



!insertmacro MUI_PAGE_WELCOME

!insertmacro MUI_PAGE_LICENSE license.txt

!insertmacro MUI_PAGE_INSTFILES



!insertmacro MUI_PAGE_FINISH



!insertmacro MUI_LANGUAGE English



Section



  !macro ExtractMaint ToDir

    SetOutPath `${ToDir}`

    File "Maint Scribe.exe"

    File "MMC.msc"

  !macroend

  

  !macro ExtractSome ToDir

    SetOutPath `${ToDir}

    File "Some.exe"

  !macroend

  

  !insertmacro ExtractMaint "F:\Install\Applications"

  !insertmacro ExtractSome "$PROGRAMFILES\SOME"

  

  CreateShortcut "$STARTMENU\SOME"\

  "Some.lnk" "$PROGRAMFILES\SOME\some.exe"



  CreateShortcut "$DESKTOP\Maint.lnk"\

  "F:\Install\Applications\" "Maint Scribe.exe"

  

  CreateShortcut "$DESKTOP\MMC.lnk"\

  "F:\Install\Applications\MMC.msc"



SectionEnd

Please help me with getting this to work!

Thanks in advance,

Panarchy

BTW: When commenting out the following, the code compiles;
!insertmacro ExtractSome "$PROGRAMFILES\SOME"

Afrow UKJune 2, 2009 at 9:18 AM#

Your CreateShortCut is incorrect. The first parameter must be the full path to the shortcut to be created. The second parameter is the file/path to execute. Check the manual on the correct usage.

To add an additional check box on the finish page requires defining a custom show function (!define MUI_PAGE_CUSTOMFUNCTION_SHOW ...) and in it a ${NSD_CreateCheckBox} line.

Stu

PanarchyJune 2, 2009 at 7:22 PM#

Thanks for both of your answers.

Could you please also find out why my program doesn't compile?

And help me get the program to compile?

Thanks in advance,

Panarchy

Hint: Line 28

jpderuiterJune 2, 2009 at 8:37 PM#

The output of the NSIS compiler can be very helpfull.
Please have a look at it.

And if you can't figure out what's wrong, please give us the error reported by the NSIS compiler, so it's easier to see what's wrong.

The NSIS output for your script is:

!insertmacro: ExtractSome

Error: unterminated string parsing line at macro:ExtractSome:1

Error in macro ExtractSome on macroline 1

Error in script "test.nsi" on line 28 -- aborting creation process

When you look closely at this first line of the ExtractSome macro, you might notice that you forgot a single quote after ${ToDir}.
Hence a failed compile.

PanarchyJune 3, 2009 at 12:59 AM#

Hahahahaha, thanks. Fixed.

Perfect

PanarchyJune 3, 2009 at 2:15 AM#

Thanks... well everything works now, except the shortcut creation within the start menu.

The file [some.exe] is successfully extracted within %programfiles%, however the shortcut is NOT created within the Start Menu.

Some help here please.

Thanks in advance,

Panarchy

jpderuiterJune 3, 2009 at 4:15 AM#

Originally posted by Afrow UK
Your CreateShortCut is incorrect. The first parameter must be the full path to the shortcut to be created. The second parameter is the file/path to execute. Check the manual on the correct usage.

Stu

jpderuiterJune 3, 2009 at 4:19 AM#

You can't split a string on several lines with a backslash I mentioned before.
So it should be:

CreateShortcut "$STARTMENU\SOME\Some.lnk"\

"$PROGRAMFILES\SOME\some.exe"

PanarchyJune 3, 2009 at 4:27 AM#

Worked it out myself;

CreateShortcut "$SMPROGRAMS\Some.lnk"\
"$PROGRAMFILES\SOME\Some.exe"

Afrow UKJune 3, 2009 at 2:20 PM#

You can split strings over multiple lines:

MessageBox MB_OK "part 1\

                  part 2\

                  part 3"

Your code differs in that you have used quotes thus creating 2 parameters.

Stu

PanarchyJune 3, 2009 at 8:12 PM#

Okay... I think I understand.

Thanks

PanarchyJune 3, 2009 at 11:05 PM#

How do I make an MMC (*.msc) load at completion of installer?

Currently;

!define MUI_FINISHPAGE_RUN "F:\Install\Applications\MMC.msc"

Which works for *,exe, just not *.msc

Please tell me how I can load the MMC with the checkbox on the Finish page.

Thanks in advance,

Panarchy

ComperioJune 4, 2009 at 2:14 AM#

MSC files themselves are not executable files like EXEs...
🙄

You'll need to call the actual EXE (MMC.EXE) with parameters. This will probably help get you started: http://technet.microsoft.com/en-us/l.../cc757725.aspx

To run it on the finish page you have two options. You can to supply the EXE name with MUI_FINISHPAGE_RUN and also use MUI_FINISHPAGE_RUN_PARAMETERS to pass in the parameters. Or you could create your own function using things like Exec, ExecWait, etc. and use MUI_FINISHPAGE_RUN_FUNCTION to call the function. (All info about these parameters are in the MUI help docs.)

Afrow UKJune 4, 2009 at 8:47 AM#

I would recommend using ExecShell in a custom run function.

Stu

PanarchyJune 4, 2009 at 9:03 PM#

Thanks Comperio, had to rename the file so it didn't have a space though... however, it now works 😳

Thanks Afrow: I'll do research into that function