unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#17330: thanks for the comments
  2014-04-23 20:54       ` Philip Hodges
@ 2014-05-03 22:43 Philip Hodges
  2014-05-04 13:24 ` bug#17330: files.el cd-absolute overcome false negative from file-executable-p Philip Hodges
  1 sibling, 1 reply; 35+ messages in thread
From: Philip Hodges @ 2014-05-03 22:43 UTC (permalink / raw)
  To: 17330

thanks for all the discussion contributions.

I'm really happy about having my own personal same day lisp fix working for me to stop cd-absolute believing the false negative from the check_executable function. My best hope for the remaining subprocess cwd cases is to look at the samba configuration and see if I can get it improved, and maybe add a user to the cygwin passwd file.

Anything involving a rebuild and new release will take months before it is available for me to use where I need it.

For completeness, and in the spirit of DRY, whatever uses start-process and call-process *could* be refactored to share the same code to offer the same override as file-executable-p in the same situation. And while we are about it, can we please not ask if a file is executable when we really just want to know if it is a searchable directory suitable for cd.

I don't see how we can ever completely rule out false negatives, unless we are prepared to change check_executable to actually try to use the directory (cd to it, open it, list its entries, return t, whatever). Even the euidaccess man page warns against using it: "Generally, it is safer just to attempt the desired operation and handle any permission error that occurs". But I do accept that there may well be no consensus to follow through with more reliable or less gullible code. If it is just a few legacy platforms that lack euidaccess and fall back to checking the wrong uid with access, then never mind. If we can prove that the native and cygwin builds behave differently, then the offending library function can probably be fixed long before emacs can work around it. Thank you for pushing that suggestion.

My samba share is also afflicted with false negative writeable checks, as described here two years ago:
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=10257

I connect to the samba filesystem directly in Windows as a user without local administrator rights. I'll let you know if mounting it from cygwin with special acl options confers more appropriate access permissions or even works at all.




^ permalink raw reply	[flat|nested] 35+ messages in thread

end of thread, other threads:[~2021-10-23  5:09 UTC | newest]

Thread overview: 35+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E6923B15-B9A8-4B3C-ADDF-713810613ADD@bluewin.ch>
2014-05-07 18:15 ` bug#17330: files.el cd-absolute overcome false negative from file-executable-p Eli Zaretskii
2014-05-08  5:55   ` Philip Hodges
2014-05-08  7:09     ` Glenn Morris
2014-05-08 16:18     ` Eli Zaretskii
2014-05-11 10:46       ` Philip Hodges
2014-05-11 17:00         ` Eli Zaretskii
2014-05-11 18:26           ` Philip Hodges
2014-05-11 18:38             ` Glenn Morris
2014-05-11 21:59               ` Philip Hodges
2014-05-12  7:10                 ` Glenn Morris
2014-05-12  7:26                   ` Philip Hodges
2021-10-23  5:09               ` Stefan Kangas
2014-05-11 18:43             ` Eli Zaretskii
2014-05-03 22:43 bug#17330: thanks for the comments Philip Hodges
2014-05-04 13:24 ` bug#17330: files.el cd-absolute overcome false negative from file-executable-p Philip Hodges
2014-05-04 16:24   ` Eli Zaretskii
2014-05-05 22:43     ` Philip Hodges
2014-04-23 20:54       ` Philip Hodges
2014-05-03  0:24         ` Glenn Morris
2014-05-03  9:17           ` Philip Hodges
2014-05-03  9:35             ` Achim Gratz
2014-05-03 13:26               ` Eli Zaretskii
2014-05-03 13:58                 ` Achim Gratz
2014-05-03 16:37                   ` Eli Zaretskii
2014-05-03 13:40             ` Eli Zaretskii
2014-05-04  4:16             ` Glenn Morris
2014-05-04 13:01               ` Stefan Monnier
2014-05-04 16:18                 ` Eli Zaretskii
2014-05-04 18:07                 ` Glenn Morris
2014-05-04 22:44                   ` Stefan Monnier
2014-05-09  6:54                     ` Glenn Morris
2014-05-06  4:15         ` Glenn Morris
2014-05-06  7:17       ` Eli Zaretskii
2014-05-06 12:46       ` Stefan Monnier
2014-05-06 16:56         ` Glenn Morris
2014-05-11 13:55           ` Philip Hodges

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).