unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Ken Brown <kbrown@cornell.edu>
To: Eli Zaretskii <eliz@gnu.org>, Katsumi Yamaoka <yamaoka@jpl.org>
Cc: 70415@debbugs.gnu.org
Subject: bug#70415: 30.0.50; [Cygwin] `lock-file' creates a symlink with funny name
Date: Mon, 22 Apr 2024 12:15:42 -0400	[thread overview]
Message-ID: <e8b9edec-2c0b-4174-8c09-f61558fe26ad@cornell.edu> (raw)
In-Reply-To: <86zftsgoxt.fsf@gnu.org>

On 4/17/2024 8:21 AM, Eli Zaretskii wrote:
>> Date: Wed, 17 Apr 2024 08:58:42 +0900
>> From: Katsumi Yamaoka <yamaoka@jpl.org>
>> Cc: 70415@debbugs.gnu.org
>>
>> On Tue, 16 Apr 2024 16:32:17 +0300, Eli Zaretskii wrote:
>>>> .#FILE -> username@hostname.1234:-11644473599
>>>> For that case `(unlock-file "FILE")' issues this warning:
>>>> Warning (unlock-file): Unlocking file: Invalid argument, FILE, ignored
>>
>>> The part after the colon is the system boot time, and is used on all
>>> Posix hosts.
>>
>> Oh, I had never seen it until recently.  Thanks.
> 
> The question is: why it started appearing only recently?  The code in
> filelock.c hasn't changed since last December, if not earlier, so if
> this started happening recently, it could only be because Gnulib's
> get_boot_time or its Cygwin implementation changed, such that it was
> previously returning zero whereas now it returns non-zero (which
> triggers the addition of the ":BOOT_TIME" part to the file name).
> 
>>>> | commit 393f58c85aeb78f814866ccaad9ae7efd3fa6766
>>>> | Author: Adam Porter <adam@alphapapa.net>
>>>> | Date:   Fri Mar 8 23:43:14 2024 -0600
>>
>>> This is extremely unlikely to have caused the problem.  It is more
>>> likely that some change in Cygwin and/or in the Gnulib's boot-time
>>> module causes this now.
>>
>> Indeed.  But the fact is that Emacs built with that revno adds
>> a colon and a negative number to a link name and the one built
>> with just before it does not.  No problem on 29.3.50 and earlier,
>> either (i.e., not add a colon and numbers).
> 
> Then please step with a debugger through the relevant code in
> filelock.c, before and after that change, and see why they are
> different.  I cannot explain that, even after doing "git log -L" on
> all of filelock.c: the most recent change of the related code was in
> Aug 2023, AFAICT, which is a very far cry from Mar 2024.
> 
> And even if we understand why you now get the ":BOOT_TIME" part,
> there's still the question why it cannot be unlinked, if Cygwin
> supports file names with a colon.  Maybe it's because Emacs creates a
> symlink, and symlinks with such names are not supported by Cygwin?

I'm still traveling for another week and won't have time to try to 
reproduce and debug this until I return.  But I did do one quick test in 
a Cygwin shell.  I created a symlink

   .#FILE -> username@hostname.1234:-11644473599

and deleted it with rm.  So symlinks like this are indeed supported by 
Cygwin.

To be continued...

Ken





  reply	other threads:[~2024-04-22 16:15 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-16 12:33 bug#70415: 30.0.50; [Cygwin] `lock-file' creates a symlink with funny name Katsumi Yamaoka
2024-04-16 13:32 ` Eli Zaretskii
2024-04-16 18:01   ` Pierre Téchoueyres
2024-04-16 22:57   ` Ken Brown
2024-04-17 12:08     ` Eli Zaretskii
2024-04-16 23:58   ` Katsumi Yamaoka
2024-04-17 12:21     ` Eli Zaretskii
2024-04-22 16:15       ` Ken Brown [this message]
2024-04-29 23:05         ` Ken Brown
2024-04-30  2:34           ` Eli Zaretskii
2024-04-30  6:45             ` Katsumi Yamaoka
2024-04-30 15:23             ` Ken Brown
2024-04-30 16:20               ` Eli Zaretskii
2024-05-01 21:20                 ` Ken Brown
2024-05-02  6:21                   ` Eli Zaretskii
2024-05-02 14:26                     ` Ken Brown
2024-05-04 17:15                       ` Ken Brown
2024-05-04 17:25                         ` Eli Zaretskii
2024-05-04 17:46                           ` Ken Brown
2024-05-04 22:01                             ` Ken Brown
2024-05-05  5:40                               ` Eli Zaretskii
2024-05-05 16:21                                 ` Ken Brown
2024-05-05 16:32                                   ` Eli Zaretskii
2024-05-05 17:02                                     ` Ken Brown
2024-05-05 18:00                                       ` Michael Albinus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-05-07  0:07                                       ` Katsumi Yamaoka
2024-05-04 17:30                         ` Ken Brown
2024-05-04 17:37                           ` Eli Zaretskii
2024-05-04 18:46                             ` Ken Brown

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=e8b9edec-2c0b-4174-8c09-f61558fe26ad@cornell.edu \
    --to=kbrown@cornell.edu \
    --cc=70415@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=yamaoka@jpl.org \
    /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 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).