* bug#27167: eshell/sudo doesn't handle absolute command names correctly
@ 2017-05-31 16:35 Yegor Timoshenko
2017-06-07 12:53 ` npostavs
0 siblings, 1 reply; 4+ messages in thread
From: Yegor Timoshenko @ 2017-05-31 16:35 UTC (permalink / raw)
To: 27167
Example (in M-x eshell):
$ touch example
$ /bin/cat example
$ sudo /bin/cat example
/sudo:root@localhost:/Users/yegortimoshenko/bin/cat: command not found
E.g. eshell/sudo implies that command name is relative to `pwd`, which should not be the case.
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#27167: eshell/sudo doesn't handle absolute command names correctly
2017-05-31 16:35 bug#27167: eshell/sudo doesn't handle absolute command names correctly Yegor Timoshenko
@ 2017-06-07 12:53 ` npostavs
2017-06-07 18:42 ` Yegor Timoshenko
0 siblings, 1 reply; 4+ messages in thread
From: npostavs @ 2017-06-07 12:53 UTC (permalink / raw)
To: Yegor Timoshenko; +Cc: 27167
tags 27167 unreproducible
quit
Yegor Timoshenko <yegortimoshenko@gmail.com> writes:
> Example (in M-x eshell):
>
> $ touch example
> $ /bin/cat example
> $ sudo /bin/cat example
> /sudo:root@localhost:/Users/yegortimoshenko/bin/cat: command not found
Works for me, can you reproduce this from 'emacs -Q'? What Emacs
version are you using?
Welcome to the Emacs shell
~/src $ touch example
~/src $ sudo /bin/cat example
[sudo] password for npostavs:
~/src $
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#27167: eshell/sudo doesn't handle absolute command names correctly
2017-06-07 12:53 ` npostavs
@ 2017-06-07 18:42 ` Yegor Timoshenko
2017-06-11 12:48 ` npostavs
0 siblings, 1 reply; 4+ messages in thread
From: Yegor Timoshenko @ 2017-06-07 18:42 UTC (permalink / raw)
To: npostavs; +Cc: 27167
Still reproducible, even with -Q flag. Affected versions:
GNU Emacs 25.2.1 (x86_64-apple-darwin13.4.0, NS appkit-1265.21 Version 10.9.5 (Build 13F1911)) of 2017-04-21
GNU Emacs 25.2.1 (x86_64-apple-darwin16, NS appkit-1504.83 Version 10.12.5 (Build 16F73)) of 2017-06-07
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#27167: eshell/sudo doesn't handle absolute command names correctly
2017-06-07 18:42 ` Yegor Timoshenko
@ 2017-06-11 12:48 ` npostavs
0 siblings, 0 replies; 4+ messages in thread
From: npostavs @ 2017-06-11 12:48 UTC (permalink / raw)
To: Yegor Timoshenko; +Cc: 27167
tags 27167 = fixed
close 27167 26.1
# I got 27167 and 27169 mixed up before
tags 27169 = unreproducible
quit
I pushed the fix to master [1: cf557fa9c5]. I had posted it to the
wrong bug [2]. This bug I could reproduce (with the caveat that 'eshell/sudo'
needs to be used instead of 'sudo', OP explained they made this alias in [3]).
[1: cf557fa9c5]: 2017-06-11 07:39:40 -0400
Let eshell/sudo handle absolute command names (Bug#27167)
http://git.savannah.gnu.org/cgit/emacs.git/commit/?id=cf557fa9c53bb8795ddc744319e067515a9dba67
[2]: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=27169#18
[3]: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=27169#26
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-06-11 12:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-31 16:35 bug#27167: eshell/sudo doesn't handle absolute command names correctly Yegor Timoshenko
2017-06-07 12:53 ` npostavs
2017-06-07 18:42 ` Yegor Timoshenko
2017-06-11 12:48 ` npostavs
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).