Opened 18 years ago

Closed 18 years ago

#7504 closed defect (fixed)

command lines arguments to "makensis" are handled differently on win32 and darwin if they have spaces

Reported by: francois@… Owned by: landonf (Landon Fuller)
Priority: Normal Milestone:
Component: ports Version: 1.2
Keywords: Cc: francois@…, markd@…
Port:

Description

The port of NSIS ("makensis") does not handle correctly command-line arguments that contain spaces in a property value. For example, if I have a script that uses a "MY_CAPTION" property, I cannot set a property with a space in the command line. All this: makensis '-DMY_CAPTION=my hello' test.nsi makensis "-DMY_CAPTION=my hello" test.nsi makensis -DMY_CAPTION="my hello" test.nsi makensis -DMY_CAPTION='my hello' test.nsi etc.. will not work. In all cases, NSIS will believe that the property finishes after "my" and will think that "hello" is the name of the script to run; it gives the following output:

Command line defined: "MY_CAPTION=my\"

Can't open script "hello"

The same thing works OK on windows.

If you want, I can work on it, I've look where it is in the code, and it's probably easy to fix. Just, before I work on it, can I know if it would be released very soon, and if there's a chance that someone would fix the other bug I entered (bug #7502), and would do a new release of nsis port ? :-)

thanks

Attachments (1)

test.nsi (89 bytes) - added by francois@… 18 years ago.
The script I used. Note that if you don't have space in MY_CAPTION, you'll still hit bug# 7502

Download all attachments as: .zip

Change History (5)

Changed 18 years ago by francois@…

Attachment: test.nsi added

The script I used. Note that if you don't have space in MY_CAPTION, you'll still hit bug# 7502

comment:1 Changed 18 years ago by francois@…

Cc: francois@… added

comment:2 Changed 18 years ago by blb@…

Owner: changed from darwinports-bugs@… to landonf@…

Assigning to maintainer.

comment:3 Changed 18 years ago by markd@…

Cc: markd@… added

The port has been updated to 2.17. Did that fix the problem?

comment:4 Changed 18 years ago by markd@…

Resolution: fixed
Status: newclosed

New port was released. Reopen if there are still problems.

Note: See TracTickets for help on using tickets.