Re: X Strike Force X.Org X11 SVN commit: r889 - in trunk/debian: . local
On Wed, Nov 23, 2005 at 02:22:08PM -0500, X Strike Force SVN Repository Admin wrote:
> Modified: trunk/debian/changelog
> ===================================================================
> --- trunk/debian/changelog 2005-11-22 03:22:39 UTC (rev 888)
> +++ trunk/debian/changelog 2005-11-23 19:22:02 UTC (rev 889)
> @@ -5,6 +5,9 @@
> xserver-xorg, not xserver-xorg-dbg. Thanks Steve Langasek and Ryan Murray.
> * Remove obsolete "| xlibs (> 4.1.0)" from various shlibs packaging files at
> the request of the release team
> + * Change ls to /bin/ls in Xsession to prevent login problems if ls is
> + locally aliased to ls --color=auto. Thanks January Weiner, David
> + Vernazobres. (closes: #340443, #337650)
>
> Modified: trunk/debian/local/Xsession
> ===================================================================
> --- trunk/debian/local/Xsession 2005-11-22 03:22:39 UTC (rev 888)
> +++ trunk/debian/local/Xsession 2005-11-23 19:22:02 UTC (rev 889)
> @@ -56,7 +56,7 @@
> internal_errormsg "run_parts() called, but \"$1\" does not exist or is" \
> "not a directory."
> fi
> - for F in $(ls $1); do
> + for F in $(/bin/ls $1); do
> if expr "$F" : '[[:alnum:]_-]\+$' > /dev/null 2>&1; then
> if [ -f "$1/$F" ]; then
> echo "$1/$F"
Hi David,
Is there any reason why you prefer this solution over 'run-parts --list',
which had been mentioned in #337650?
Denis
Reply to: