This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: is cruel

[redirected to cygwin-apps]

Can we keep the 1.7 stuff out of the cygwin list for now, please?

Either use cygwin-apps for the packaging discussions or cygwin-developers
for the debugging and bug fixing discussions, or cygwin-patches for...
erm...  patches.


On Apr 28 15:01, Eric Blake wrote:
> In testing cygwin 1.7.0, I noticed that /etc/profile.d/ blindly 
> slams /bin at the front of PATH.  This is cruel to users, since it breaks the 
> 1.5.x behavior of defaulting to /usr/local/bin at the front of PATH.  It also 
> pollutes the user's environment with SYSCONFDIR, FSTABDIR, and FSTAB.  Since 
> this file is sourced by /etc/profile, it seems like the fastest fix would be 
> making the entire body of the file behave as a subshell, so as not to pollute 
> the parent shell.
> --- /etc/profile.d/	2008-04-28 08:59:55.695117400 -0600
> +++ /etc/profile.d/	2008-04-28 09:00:03.117039900 -0600
> @@ -7,6 +7,7 @@
>  #
>  # The registry mount points are stored in a file /etc/fstab.d/<username>
>  #
> +(
>  export PATH="/bin:$PATH"
> @@ -95,3 +96,4 @@
>      echo " 0 0"
>    fi >> ${FSTAB}
>  fi
> +)

Ouch, you're right.  I actually forgot that this setting isn't just
local to the script.

Thanks for the hint,

Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]