all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Ken Brown <kbrown@cornell.edu>
Cc: Keith David Bershatsky <esq@lawlist.com>,
	Andreas Schwab <schwab@suse.de>, Brady Trainor <brady@bradyt.com>,
	Michael Albinus <michael.albinus@gmx.de>,
	24441@debbugs.gnu.org
Subject: bug#24441: 24.5; rename directory in dired to change case
Date: Mon, 14 Nov 2016 12:33:27 -0800	[thread overview]
Message-ID: <afaab017-9f11-7e62-bb53-89f910706303@cs.ucla.edu> (raw)
In-Reply-To: <m2poo5u9c9.fsf@bradyt.com>

[-- Attachment #1: Type: text/plain, Size: 1351 bytes --]

Three comments about that fix.

1. Emacs seems to conflate whether a file system is case-sensitive, and 
whether it is case-preserving. Darwin pathconf distinguishes between 
_PC_CASE_SENSITIVE and _PC_CASE_PRESERVING. For example, perhaps the 
following code in dired-do-create-files:

                (if (and (file-name-case-insensitive-p (car fn-list))
                     (eq op-symbol 'move)
                     dired-one-file

should also check whether (car fn-list) is on a case-preserving file 
system (if not, there's no point going ahead, as the user can't change 
the case anyway). I suppose this would require adding a Lisp predicate 
for _PC_CASE_PRESERVING. (Possibly the MS-Windows code already deals 
with this, and Cygwin and macOS are currently broken.)

2. From my reading of the Apple documentation, the code taken from that 
circa-2007 website should have problems on some file systems. It does 
not match what Wine does in a similar situation.

3. Nowadays, I hope pathconf works on Apple; that's simpler. If not, I 
think the Wine approach should be better. So I installed the attached 
patch to try to do that. If you have access to a case-insensitive file 
system on macOS, please try it out as-is, or compiled with 
-DDARWIN_OS_CASE_SENSITIVE_FIXME=1 or 
-DDARWIN_OS_CASE_SENSITIVE_FIXME=2, and let me know how it works.


[-- Attachment #2: 0001-Improve-case-insensitive-checks-Bug-24441.patch --]
[-- Type: application/x-patch, Size: 6857 bytes --]

  parent reply	other threads:[~2016-11-14 20:33 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-15  3:32 bug#24441: 24.5; rename directory in dired to change case Brady Trainor
2016-09-15  9:05 ` Tino Calancha
2016-09-15  9:28 ` Andreas Schwab
2016-09-15 13:43   ` Drew Adams
2016-09-15 14:49     ` Eli Zaretskii
2016-09-15 14:31   ` Eli Zaretskii
2016-11-14 20:33 ` Paul Eggert [this message]
2016-11-15 19:59   ` Ken Brown
2016-11-15 20:30     ` Eli Zaretskii
2016-11-30 22:22       ` Ken Brown
2017-05-21  6:17 ` Paul Eggert
     [not found] <<m2poo5u9c9.fsf@bradyt.com>
     [not found] ` <<mvmeg4la4w6.fsf@hawking.suse.de>
     [not found]   ` <<3c49fbe5-9ae0-4ae2-8fa0-3c44fa85c981@default>
     [not found]     ` <<83d1k56wwt.fsf@gnu.org>
2016-09-15 14:57       ` Drew Adams
2016-09-15 15:20         ` Eli Zaretskii
2016-09-15 16:03           ` Ken Brown
2016-09-15 16:36             ` Eli Zaretskii
2016-11-10 22:25               ` Ken Brown
2016-11-11  1:42                 ` Ken Brown
2016-11-11  8:27                   ` Eli Zaretskii
2016-11-11 16:23                     ` Michael Albinus
2016-11-11 21:42                     ` Ken Brown
2016-11-12  7:29                       ` Eli Zaretskii
2016-11-13 13:14                         ` Ken Brown
2016-11-15 19:58                           ` Michael Albinus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=afaab017-9f11-7e62-bb53-89f910706303@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=24441@debbugs.gnu.org \
    --cc=brady@bradyt.com \
    --cc=esq@lawlist.com \
    --cc=kbrown@cornell.edu \
    --cc=michael.albinus@gmx.de \
    --cc=schwab@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.