tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r929958 - /tomcat/tc5.5.x/trunk/STATUS.txt
Date Thu, 01 Apr 2010 13:21:04 GMT
On 01/04/2010 14:16, kkolinko@apache.org wrote:
> Author: kkolinko
> Date: Thu Apr  1 13:16:35 2010
> New Revision: 929958
> 
> URL: http://svn.apache.org/viewvc?rev=929958&view=rev
> Log:
> vote
> 
> Modified:
>     tomcat/tc5.5.x/trunk/STATUS.txt
> 
> Modified: tomcat/tc5.5.x/trunk/STATUS.txt
> URL: http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=929958&r1=929957&r2=929958&view=diff
> ==============================================================================
> --- tomcat/tc5.5.x/trunk/STATUS.txt (original)
> +++ tomcat/tc5.5.x/trunk/STATUS.txt Thu Apr  1 13:16:35 2010
> @@ -152,4 +152,9 @@ PATCHES PROPOSED TO BACKPORT:
>    CATALINA_HOME so any issues will be visible then.
>    Patch provided by mdietze
>    +1: markt
> +   0: kkolinko: When CDPATH is set, it creates an ambiguity in behaviour of
> +      "cd". That is why there is output there.
> +      By the way, the patch that you are proposing is probably the following
> +      (there is no link above):
> +       http://svn.apache.org/viewvc?rev=928732&view=rev

The user may actually want the behaviour triggered by setting CDPATH
hence why I opted to swallow the output rather than unset it. If the
behaviour is unexpected, the display of CATALINA_HOME will at least show
what is going on.

That said, I had a hard time coming up with a use case where this was
even an issue, let alone one where setting CDPATH was useful. If this
doesn't gather enough votes then I'd be content with a WONTFIX resolution.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message