[NSIS-Bugs] [ nsis-Patches-1120399 ] nsis 204 install.sh installation under Linux

[SourceForge.net <no...@so...>]

Patches item #1120399, was opened at 2005-02-10 23:58
Message generated for change (Comment added) made by kichik
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373087&aid=1120399&group_id=22049

Category: None
Group: Stable
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Jean-Paul Saman (jpsaman)
Assigned to: Amir Szekely (kichik)
Summary: nsis 204 install.sh installation under Linux

Initial Comment:
The stable NSIS version 204 fails to properly install
under Linux. This patch solves the installation issues
and adds NSIS library path the the  dynamic loader
cache configuration. For instance when installing in
"/usr/local" the installscript produces the file
/etc/ld.so.conf.d/nsis.conf . The contents of that
script is: "/usr/local/lib/NSIS" and the program
/sbin/ldconfig should be run afterwards.

----------------------------------------------------------------------

>Comment By: Amir Szekely (kichik)
Date: 2005-02-25 15:26

Message:
Logged In: YES 
user_id=584402

OK, changes made in latest CVS version.

But I'd still like to know, out of curiosity, why
$(tempfile) doesn't work. Can you try it again, only with
parenthesis instead of curly brackets? The script itself
uses parenthesis.

----------------------------------------------------------------------

Comment By: Jean-Paul Saman (jpsaman)
Date: 2005-02-23 10:39

Message:
Logged In: YES 
user_id=518287

/bin/sh is linked to bash on Fedora Core 3

[jpsaman@somepc ~]$ ls -l /bin/sh
lrwxrwxrwx  1 root root 4 Feb 15 12:19 /bin/sh -> bash

${tempfile} is by default empty. 
[jpsaman@somepc ~]$ echo ${tempfile}

It might be less error prone if you used the name of the
final destination script before installing it on the right
place.



----------------------------------------------------------------------

Comment By: Amir Szekely (kichik)
Date: 2005-02-17 21:24

Message:
Logged In: YES 
user_id=584402

lib is indeed a weird place to put NSIS. I'll change it.

It seems $(tempfile) is a feature of bash. What is /bin/sh
linked to on your FC3?

----------------------------------------------------------------------

Comment By: Jean-Paul Saman (jpsaman)
Date: 2005-02-17 10:53

Message:
Logged In: YES 
user_id=518287

You are right LD does not need to know the path to NSIS. I 
assumed it would because it installed in a library directory. 
Maybe a /usr/share/NSIS should be better a better choice.

I changed $(tempfile) because it was undefined on my FC3 
Linux system. The installation exited at that empty variable.

----------------------------------------------------------------------

Comment By: Amir Szekely (kichik)
Date: 2005-02-15 20:36

Message:
Logged In: YES 
user_id=584402

The DIRS problems was already fixed in the latest CVS version.

Why did you change $(tempfile) to "tmp_nsis.sh"?

Why does ld need to know about the location of NSIS?
makensis doesn't use any loadable objects from that directory.

----------------------------------------------------------------------

Comment By: Jean-Paul Saman (jpsaman)
Date: 2005-02-14 11:40

Message:
Logged In: YES 
user_id=518287

Oops. Patch attached

----------------------------------------------------------------------

Comment By: Amir Szekely (kichik)
Date: 2005-02-11 00:06

Message:
Logged In: YES 
user_id=584402

No patch is attached, please try again. Don't forget to
check the checkbox.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373087&aid=1120399&group_id=22049