GNU bug report logs - #5771
23.1; compile command with cd to new dir

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 5771 in the body.
You can then email your comments to 5771 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Kevin Ryde <user42 <at> zip.com.au>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.1; compile command with cd to new dir
Date: Fri, 26 Mar 2010 09:27:07 +1100

Evaluating

    (compile "mkdir /tmp/newdir;\n cd /tmp/newdir;\n echo hi")

gets an emacs error

    => /tmp/newdir/: no such directory

and doesn't run the command.  I hoped that no matter what the command
string contains that emacs would start it.

I suppose `compilation-start' attempts to set default-directory to the
cd it sees, but doesn't notice the command itself will create it.


A similar thing happens if you've got a hairy shell expression in the
cd,

    (compile "cd ${FOO:-${BAR}};\n echo hi")

    => emacs error

I suppose in this case the expression is more than substitute-env-vars
understands.

I think if the apparent directory in a command doesn't exist then
compilation-start shouldn't try to use it.



In GNU Emacs 23.1.1 (i486-pc-linux-gnu, GTK+
Version 2.16.5)
 of 2009-09-14 on raven, modified by Debian
configured using `configure  '--build=i486-linux-gnu' '--host=i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.1/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.1/leim' '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g -O2' 'LDFLAGS=-g' 'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_AU
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default-enable-multibyte-characters: t

Message #8 received at 5771 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> stupidchicken.com>
To: Kevin Ryde <user42 <at> zip.com.au>
Cc: 5771 <at> debbugs.gnu.org
Subject: Re: 23.1; compile command with cd to new dir
Date: Sun, 28 Mar 2010 15:53:58 -0400

>     (compile "mkdir /tmp/newdir;\n cd /tmp/newdir;\n echo hi")
>
> gets an emacs error
>
>     => /tmp/newdir/: no such directory
>
> and doesn't run the command.  I hoped that no matter what the command
> string contains that emacs would start it.

The problem was that compile had this faulty regexp test:

(string-match "^\\s *cd\\(?:\\s +\\(\\S +?\\)\\)?\\s *[;&\n]" command)

The `^' matches characters after newlines, which is not what's intended;
we want to match only the beginning of the string, with `\''.  I've
checked in a fix; thanks for the bug report.

Message #13 received at 5771 <at> debbugs.gnu.org (full text, mbox):

From: Kevin Ryde <user42 <at> zip.com.au>
To: 5771 <at> debbugs.gnu.org
Cc: Chong Yidong <cyd <at> stupidchicken.com>
Subject: Re: 23.1; compile command with cd to new dir
Date: Wed, 31 Mar 2010 11:14:06 +1100

Chong Yidong <cyd <at> stupidchicken.com> writes:
>
> The `^' matches characters after newlines, which is not what's intended;

Really?  I suppose the changelog entry reads like that (ChangeLog.11
2004-09-12 Daniel Pfeiffer).  You'd be very tempted to allow a "cd"
after simple stuff like an echo though.

> I've checked in a fix; thanks for the bug report.

The second case too, with a hairy shell expression in the cd?  Just
enough that the command is started of course, no need to worry if it
does anything good or bad.


(Apologies for not trying it myself, the bloat of bizaare makes the head
inaccessible to the peasants :-(

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.