Win32 - installer.sh - adding profile.d

Christian Stimming stimming at tuhh.de
Thu Dec 21 09:53:03 EST 2006


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andreas Köhler schrieb:
>>> Before applying this, I would like to hear Andreas' take on this: You
>>> intentionally added only the printing of the lines that should be added
>>> to install.sh, instead of directly writing into that file. Was there a
>>> specific reason for doing so? 
>>> 
>> Right now we don't modify /etc/profile.d/installer by hand at all.
>> We /JUST/ ask the user to make the changes.  I agree that this is what
>> we should continue to do, so I agree with Andreas to not modify
>> /etc/profile.d ourselves.
>>
>> I'll note that some people might not want /etc/profile.d modified
>> by the build script; they should be allowed to do so on their own.
> 
> Seconded.
> Another point is that we cannot change the running shells environment
> from a subshell, so newly opened shells would have different environment
> variables set.
> 
> I will not like the patch without very good reasons :-)

Agreed. That's what I thought as well.

In any case I cannot see why this would be a problem during install.sh
or dist.sh - those will set their appropriate path variables from
custom.sh themselves, anyway. If not, it's a bug in these scripts and
should be fixed there.

Conclusion: Thanks to hfelton for the suggestion, but we believe the
installation is better organized by not modifying /etc/profile.d from
the scripts in general.

Christian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQCVAwUBRYqfz2XAi+BfhivFAQKrHwP/aArW8lGgwIXayaxPXQERXlKfrog9VEHH
QgcEFvz8uVy13+xx8zxhOHZkfexiBbqL3E2GKU8MxJbvs1caMJwsHJ9VEpCY8H/Z
APfA8ZlX53qj+QKOhcHzhhSSmcgJl4V6APeX1nN/ps2qNqfLs/9AHgpBcSlTLtTi
zlmpKJCxKdY=
=gX+w
-----END PGP SIGNATURE-----


More information about the gnucash-devel mailing list