all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
@ 2022-07-22 21:10 Ken Brown
  2022-07-23  1:10 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Ken Brown @ 2022-07-22 21:10 UTC (permalink / raw)
  To: 56712; +Cc: Po Lu

The test in the subject fails as follows on Cygwin:

Test x-dnd-tests-do-direct-save backtrace:
   ert--should-signal-hook(ert-test-failed (((should (equal (x-dnd-test
   signal(ert-test-failed (((should (equal (x-dnd-tests-call-xds-conver
   ert-fail(((should (equal (x-dnd-tests-call-xds-converter) '(STRING .
   (if (unwind-protect (setq value-19 (apply fn-17 args-18)) (setq form
   (let (form-description-21) (if (unwind-protect (setq value-19 (apply
   (let ((value-19 'ert-form-evaluation-aborted-20)) (let (form-descrip
   (let* ((fn-17 #'equal) (args-18 (condition-case err (let ((signal-ho
   (prog1 'XdndActionDirectSave (let* ((fn-10 #'eq) (args-11 (condition
   x-begin-drag(("XdndDirectSave0" "text/uri-list") XdndActionDirectSav
   x-dnd-do-direct-save("/tmp/x-dnd-test7T4xoz" "x-dnd-test-targetDdUAT
   (list (x-dnd-do-direct-save original-file x-dnd-tests-xds-name nil n
   (let ((signal-hook-function #'ert--should-signal-hook)) (list (x-dnd
   (condition-case err (let ((signal-hook-function #'ert--should-signal
   (let* ((fn-47 #'eq) (args-48 (condition-case err (let ((signal-hook-
   (progn (if (file-exists-p original-file) nil (write-region "" 0 orig
   (unwind-protect (progn (if (file-exists-p original-file) nil (write-
   (progn (unwind-protect (progn (if (file-exists-p original-file) nil
   (let* ((--cl-gui-set-selection-- #'ignore)) (progn (unwind-protect (
   (let ((x-dnd-tests-xds-include-hostname include-hostname) (x-dnd-tes
   x-dnd-tests-do-direct-save-internal(nil)
   (closure (t) nil (x-dnd-tests-do-direct-save-internal nil) (x-dnd-te
   ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
   ert-run-test(#s(ert-test :name x-dnd-tests-do-direct-save :documenta
   ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
   ert-run-tests((not (or (tag :unstable) (tag :nativecomp))) #f(compil
   ert-run-tests-batch((not (or (tag :unstable) (tag :nativecomp))))
   ert-run-tests-batch-and-exit((not (or (tag :unstable) (tag :nativeco
   eval((ert-run-tests-batch-and-exit '(not (or (tag :unstable) (tag :n
   command-line-1(("-L" ":../../master/test" "-l" "ert" "-l" "lisp/x-dn
   command-line()
   normal-top-level()
Test x-dnd-tests-do-direct-save condition:
     (ert-test-failed
      ((should
        (equal
         (x-dnd-tests-call-xds-converter)
         '(STRING . "S")))
       :form
       (equal
        (STRING . "E")
        (STRING . "S"))
       :value nil :explanation
       (cdr
        (array-elt 0
                   (different-atoms ... ...)))))

The failure first occurred in the commit that introduced the test.  Let me know 
what I can do to help debug this.

Ken


In GNU Emacs 29.0.50 (build 42, x86_64-pc-cygwin, GTK+ Version 3.22.28, cairo 
version 1.17.4)
  of 2022-07-22 built on moufang2
Repository revision: 3534a8e15c971d4dc901e029f8c0f6ba68fcecd5
Repository branch: master
Windowing system distributor 'The Cygwin/X Project', version 11.0.12101003
Configured using:
  'configure 'CFLAGS=-g3 -O0''

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GSETTINGS HARFBUZZ JPEG JSON
LCMS2 LIBOTF LIBXML2 M17N_FLT MODULES NOTIFY GFILENOTIFY PDUMPER PNG
RSVG SOUND THREADS TIFF TOOLKIT_SCROLL_BARS WEBP X11 XDBE XIM XINPUT2
XPM GTK3 ZLIB

Important settings:
   value of $LANG: en_US.UTF-8
   locale-coding-system: utf-8-unix





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-22 21:10 bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin Ken Brown
@ 2022-07-23  1:10 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-23 17:30   ` Ken Brown
  0 siblings, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-23  1:10 UTC (permalink / raw)
  To: Ken Brown; +Cc: 56712

Ken Brown <kbrown@cornell.edu> writes:

> The failure first occurred in the commit that introduced the test.
> Let me know what I can do to help debug this.

Thanks.  That means this code in x-dnd.el has failed:

          (progn
            (copy-file x-dnd-xds-current-file
                       local-name t)
            (when (equal x-dnd-xds-current-file
                         dnd-last-dragged-remote-file)
              (dnd-remove-last-dragged-remote-file)))

So it would be good to get the detailed error message from that progn.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-23  1:10 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-23 17:30   ` Ken Brown
  2022-07-23 17:41     ` Ken Brown
  2022-07-24  2:26     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 2 replies; 20+ messages in thread
From: Ken Brown @ 2022-07-23 17:30 UTC (permalink / raw)
  To: Po Lu; +Cc: 56712

On 7/22/2022 9:10 PM, Po Lu wrote:
> Ken Brown <kbrown@cornell.edu> writes:
> 
>> The failure first occurred in the commit that introduced the test.
>> Let me know what I can do to help debug this.
> 
> Thanks.  That means this code in x-dnd.el has failed:
> 
>            (progn
>              (copy-file x-dnd-xds-current-file
>                         local-name t)
>              (when (equal x-dnd-xds-current-file
>                           dnd-last-dragged-remote-file)
>                (dnd-remove-last-dragged-remote-file)))
> 
> So it would be good to get the detailed error message from that progn.

Sorry, I'm not immediately seeing how to do this in the context of the test. 
Could you be more explicit?

In the meantime, I tried

    make -C test x-dnd-tests TEST_INTERACTIVE=yes

and the *ert* buffer reported

F x-dnd-tests-do-direct-save
     (error "Invalid selection type: XdndSelection")

Does that help?

By the way, I should have said in my original report that I do an out-of-tree 
build.  I'll try an in-tree build after sending this message to see if I still 
get the test failure.

Ken





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-23 17:30   ` Ken Brown
@ 2022-07-23 17:41     ` Ken Brown
  2022-07-24  2:26     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  1 sibling, 0 replies; 20+ messages in thread
From: Ken Brown @ 2022-07-23 17:41 UTC (permalink / raw)
  To: Po Lu; +Cc: 56712

On 7/23/2022 1:30 PM, Ken Brown wrote:
> On 7/22/2022 9:10 PM, Po Lu wrote:
>> Ken Brown <kbrown@cornell.edu> writes:
>>
>>> The failure first occurred in the commit that introduced the test.
>>> Let me know what I can do to help debug this.
>>
>> Thanks.  That means this code in x-dnd.el has failed:
>>
>>            (progn
>>              (copy-file x-dnd-xds-current-file
>>                         local-name t)
>>              (when (equal x-dnd-xds-current-file
>>                           dnd-last-dragged-remote-file)
>>                (dnd-remove-last-dragged-remote-file)))
>>
>> So it would be good to get the detailed error message from that progn.
> 
> Sorry, I'm not immediately seeing how to do this in the context of the test. 
> Could you be more explicit?
> 
> In the meantime, I tried
> 
>     make -C test x-dnd-tests TEST_INTERACTIVE=yes
> 
> and the *ert* buffer reported
> 
> F x-dnd-tests-do-direct-save
>      (error "Invalid selection type: XdndSelection")
> 
> Does that help?
> 
> By the way, I should have said in my original report that I do an out-of-tree 
> build.  I'll try an in-tree build after sending this message to see if I still 
> get the test failure.

That wasn't the problem.  I get the same failure with an in-tree build.

Ken





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-23 17:30   ` Ken Brown
  2022-07-23 17:41     ` Ken Brown
@ 2022-07-24  2:26     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24  5:50       ` Eli Zaretskii
  1 sibling, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-24  2:26 UTC (permalink / raw)
  To: Ken Brown; +Cc: 56712

Ken Brown <kbrown@cornell.edu> writes:

> Sorry, I'm not immediately seeing how to do this in the context of the
> test. Could you be more explicit?

I suggest adding some code here, in x-dnd.el:

      (condition-case nil
          (progn
            (copy-file x-dnd-xds-current-file
                       local-name t)
            (when (equal x-dnd-xds-current-file
                         dnd-last-dragged-remote-file)
              (dnd-remove-last-dragged-remote-file)))
        (:success '(STRING . "S"))
        (error '(STRING . "E")) <--- add code here

to print out the error that is causing it to reply with "E".

> Does that help?

No, unfortunately not.  The test is not useful interactively.

> By the way, I should have said in my original report that I do an
> out-of-tree build.  I'll try an in-tree build after sending this
> message to see if I still get the test failure.

Thanks.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  2:26     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-24  5:50       ` Eli Zaretskii
  2022-07-24  7:23         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Eli Zaretskii @ 2022-07-24  5:50 UTC (permalink / raw)
  To: Po Lu; +Cc: 56712, kbrown

> Cc: 56712@debbugs.gnu.org
> Date: Sun, 24 Jul 2022 10:26:30 +0800
> From:  Po Lu via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
> 
> Ken Brown <kbrown@cornell.edu> writes:
> 
> > Sorry, I'm not immediately seeing how to do this in the context of the
> > test. Could you be more explicit?
> 
> I suggest adding some code here, in x-dnd.el:
> 
>       (condition-case nil
>           (progn
>             (copy-file x-dnd-xds-current-file
>                        local-name t)
>             (when (equal x-dnd-xds-current-file
>                          dnd-last-dragged-remote-file)
>               (dnd-remove-last-dragged-remote-file)))
>         (:success '(STRING . "S"))
>         (error '(STRING . "E")) <--- add code here
> 
> to print out the error that is causing it to reply with "E".

Maybe it is worth our while to convert this to a debugging code which
would collect the information given some variable setting?  Similar to
what other packages do, like Tramp?





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  5:50       ` Eli Zaretskii
@ 2022-07-24  7:23         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24  7:42           ` Michael Albinus
  2022-07-24  8:55           ` Eli Zaretskii
  0 siblings, 2 replies; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-24  7:23 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 56712, kbrown

Eli Zaretskii <eliz@gnu.org> writes:

>>       (condition-case nil
>>           (progn
>>             (copy-file x-dnd-xds-current-file
>>                        local-name t)
>>             (when (equal x-dnd-xds-current-file
>>                          dnd-last-dragged-remote-file)
>>               (dnd-remove-last-dragged-remote-file)))
>>         (:success '(STRING . "S"))
>>         (error '(STRING . "E")) <--- add code here
>> 
>> to print out the error that is causing it to reply with "E".
>
> Maybe it is worth our while to convert this to a debugging code which
> would collect the information given some variable setting?  Similar to
> what other packages do, like Tramp?

I'm not sure what you mean by that.  Could you explain which part of the
Tramp code has such debug functionality?

Thanks.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  7:23         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-24  7:42           ` Michael Albinus
  2022-07-24  8:55           ` Eli Zaretskii
  1 sibling, 0 replies; 20+ messages in thread
From: Michael Albinus @ 2022-07-24  7:42 UTC (permalink / raw)
  To: 56712; +Cc: luangruo, eliz, kbrown

Po Lu via "Bug reports for GNU Emacs, the Swiss army knife of text
editors" <bug-gnu-emacs@gnu.org> writes:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>>>       (condition-case nil
>>>           (progn
>>>             (copy-file x-dnd-xds-current-file
>>>                        local-name t)
>>>             (when (equal x-dnd-xds-current-file
>>>                          dnd-last-dragged-remote-file)
>>>               (dnd-remove-last-dragged-remote-file)))
>>>         (:success '(STRING . "S"))
>>>         (error '(STRING . "E")) <--- add code here
>>>
>>> to print out the error that is causing it to reply with "E".
>>
>> Maybe it is worth our while to convert this to a debugging code which
>> would collect the information given some variable setting?  Similar to
>> what other packages do, like Tramp?
>
> I'm not sure what you mean by that.  Could you explain which part of the
> Tramp code has such debug functionality?

All the calls of `tramp-message' and `tramp-error', controlled by the
debug level in `tramp-verbose'.

> Thanks.

Best regars, Michael.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  7:23         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24  7:42           ` Michael Albinus
@ 2022-07-24  8:55           ` Eli Zaretskii
  2022-07-24 10:55             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24 13:33             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  1 sibling, 2 replies; 20+ messages in thread
From: Eli Zaretskii @ 2022-07-24  8:55 UTC (permalink / raw)
  To: Po Lu; +Cc: 56712, kbrown

> From: Po Lu <luangruo@yahoo.com>
> Cc: kbrown@cornell.edu,  56712@debbugs.gnu.org
> Date: Sun, 24 Jul 2022 15:23:06 +0800
> 
> > Maybe it is worth our while to convert this to a debugging code which
> > would collect the information given some variable setting?  Similar to
> > what other packages do, like Tramp?
> 
> I'm not sure what you mean by that.  Could you explain which part of the
> Tramp code has such debug functionality?

I meant the concept: the user sets some variable, and that causes
debug information to be collected in some buffer.  The variable could
be a boolean, or it could allow specifying different levels of debug
info to collect.  See tramp-verbose for an example.  (I'm sure Michael
can describe these Tramp facilities in more detail if needed.)

Thanks.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  8:55           ` Eli Zaretskii
@ 2022-07-24 10:55             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24 13:33             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  1 sibling, 0 replies; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-24 10:55 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 56712, kbrown

Eli Zaretskii <eliz@gnu.org> writes:

> I meant the concept: the user sets some variable, and that causes
> debug information to be collected in some buffer.  The variable could
> be a boolean, or it could allow specifying different levels of debug
> info to collect.  See tramp-verbose for an example.  (I'm sure Michael
> can describe these Tramp facilities in more detail if needed.)
>
> Thanks.

Sure, I will arrange that.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24  8:55           ` Eli Zaretskii
  2022-07-24 10:55             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-24 13:33             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-24 15:40               ` Ken Brown
  1 sibling, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-24 13:33 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 56712, kbrown

Eli Zaretskii <eliz@gnu.org> writes:

> I meant the concept: the user sets some variable, and that causes
> debug information to be collected in some buffer.  The variable could
> be a boolean, or it could allow specifying different levels of debug
> info to collect.  See tramp-verbose for an example.  (I'm sure Michael
> can describe these Tramp facilities in more detail if needed.)
>
> Thanks.

Now done.  Ken, could you please re-run the test and show the detailed
error that is signalled?

Thanks in advance.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24 13:33             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-24 15:40               ` Ken Brown
  2022-07-25  1:02                 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Ken Brown @ 2022-07-24 15:40 UTC (permalink / raw)
  To: Po Lu, Eli Zaretskii; +Cc: 56712

On 7/24/2022 9:33 AM, Po Lu wrote:
> Now done.  Ken, could you please re-run the test and show the detailed
> error that is signalled?

Thanks.  The error is now clear:

Test x-dnd-tests-do-direct-save condition:
     (file-missing "Opening output file" "No such file or directory" 
"//tmp/x-dnd-testvZQUCZ/x-dnd-test-targetKiqqPT")

According to POSIX, the meaning of exactly 2 slashes at the beginning of a path 
name is system dependent.  On Cygwin it's interpreted as a UNC path.  That 
explains the error, but I don't immediately see where the extra slash comes 
from.  (There are plenty of references to UNC paths and to "file://" and 
"file:///" in dnd.el, so maybe there's some code in there that incorrectly 
assumes that MS-Windows is the only platform on which exactly 2 initial slashes 
has a special meaning.)

Ken





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-24 15:40               ` Ken Brown
@ 2022-07-25  1:02                 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-25  1:42                   ` Ken Brown
  0 siblings, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-25  1:02 UTC (permalink / raw)
  To: Ken Brown; +Cc: Eli Zaretskii, 56712

Ken Brown <kbrown@cornell.edu> writes:

> On 7/24/2022 9:33 AM, Po Lu wrote:
>> Now done.  Ken, could you please re-run the test and show the detailed
>> error that is signalled?
>
> Thanks.  The error is now clear:
>
> Test x-dnd-tests-do-direct-save condition:
>     (file-missing "Opening output file" "No such file or directory"
>     "//tmp/x-dnd-testvZQUCZ/x-dnd-test-targetKiqqPT")
>
> According to POSIX, the meaning of exactly 2 slashes at the beginning
> of a path name is system dependent.  On Cygwin it's interpreted as a
> UNC path.  That explains the error, but I don't immediately see where
> the extra slash comes from.  (There are plenty of references to UNC
> paths and to "file://" and "file:///" in dnd.el, so maybe there's some
> code in there that incorrectly assumes that MS-Windows is the only
> platform on which exactly 2 initial slashes has a special meaning.)
>
> Ken

Unless system-type can be windows-nt on Cygwin, I don't think that can
happen.  I will look into this, thanks.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-25  1:02                 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-25  1:42                   ` Ken Brown
  2022-07-25  2:10                     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Ken Brown @ 2022-07-25  1:42 UTC (permalink / raw)
  To: Po Lu; +Cc: Eli Zaretskii, 56712

On 7/24/2022 9:02 PM, Po Lu wrote:
> Ken Brown <kbrown@cornell.edu> writes:
> 
>> On 7/24/2022 9:33 AM, Po Lu wrote:
>>> Now done.  Ken, could you please re-run the test and show the detailed
>>> error that is signalled?
>>
>> Thanks.  The error is now clear:
>>
>> Test x-dnd-tests-do-direct-save condition:
>>      (file-missing "Opening output file" "No such file or directory"
>>      "//tmp/x-dnd-testvZQUCZ/x-dnd-test-targetKiqqPT")
>>
>> According to POSIX, the meaning of exactly 2 slashes at the beginning
>> of a path name is system dependent.  On Cygwin it's interpreted as a
>> UNC path.  That explains the error, but I don't immediately see where
>> the extra slash comes from.  (There are plenty of references to UNC
>> paths and to "file://" and "file:///" in dnd.el, so maybe there's some
>> code in there that incorrectly assumes that MS-Windows is the only
>> platform on which exactly 2 initial slashes has a special meaning.)
>>
>> Ken
> 
> Unless system-type can be windows-nt on Cygwin, I don't think that can
> happen.

No, system-type is cygwin on Cygwin.

>  I will look into this, thanks.

Thanks.

Ken





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-25  1:42                   ` Ken Brown
@ 2022-07-25  2:10                     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-25 16:24                       ` Ken Brown
  0 siblings, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-25  2:10 UTC (permalink / raw)
  To: Ken Brown; +Cc: Eli Zaretskii, 56712

Ken Brown <kbrown@cornell.edu> writes:

> Thanks.

It's a bug in the test, which I (hopefully) fixed.  Please give it
another try.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-25  2:10                     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-25 16:24                       ` Ken Brown
  2022-07-25 16:53                         ` Eli Zaretskii
  0 siblings, 1 reply; 20+ messages in thread
From: Ken Brown @ 2022-07-25 16:24 UTC (permalink / raw)
  To: Po Lu; +Cc: Eli Zaretskii, 56712-done

On 7/24/2022 10:10 PM, Po Lu wrote:
> It's a bug in the test, which I (hopefully) fixed.  Please give it
> another try.

It's fixed, thanks.  So I guessed right about there being a "file:///" that 
should have been "file://", but I didn't see it staring me in the face in the 
test file!

Closing.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-25 16:24                       ` Ken Brown
@ 2022-07-25 16:53                         ` Eli Zaretskii
  2022-07-26  2:10                           ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Eli Zaretskii @ 2022-07-25 16:53 UTC (permalink / raw)
  To: Ken Brown; +Cc: luangruo, 56712

> Date: Mon, 25 Jul 2022 12:24:21 -0400
> Cc: Eli Zaretskii <eliz@gnu.org>, 56712-done@debbugs.gnu.org
> From: Ken Brown <kbrown@cornell.edu>
> 
> On 7/24/2022 10:10 PM, Po Lu wrote:
> > It's a bug in the test, which I (hopefully) fixed.  Please give it
> > another try.
> 
> It's fixed, thanks.  So I guessed right about there being a "file:///" that 
> should have been "file://", but I didn't see it staring me in the face in the 
> test file!

FTR: if that test should ever run on native MS-Windows, we'd need that
third slash, because file://c:/foo/bar is not valid on Windows, you
must say file:///c:/foo/bar.

So from my POV, it is best to make that code system-dependent, with
windows-nt using 3 slashes, not 2.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-25 16:53                         ` Eli Zaretskii
@ 2022-07-26  2:10                           ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2022-07-26  2:38                             ` Eli Zaretskii
  0 siblings, 1 reply; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-26  2:10 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 56712, Ken Brown

Eli Zaretskii <eliz@gnu.org> writes:

> FTR: if that test should ever run on native MS-Windows, we'd need that
> third slash, because file://c:/foo/bar is not valid on Windows, you
> must say file:///c:/foo/bar.
>
> So from my POV, it is best to make that code system-dependent, with
> windows-nt using 3 slashes, not 2.

The code is specific to X -- as evidenced by the name.  Doesn't Cygwin
(the only way I know of to get an X server on MS Windows) use Unix-style
file names everywhere?

Thanks.





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-26  2:10                           ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2022-07-26  2:38                             ` Eli Zaretskii
  2022-07-26  2:52                               ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 20+ messages in thread
From: Eli Zaretskii @ 2022-07-26  2:38 UTC (permalink / raw)
  To: Po Lu; +Cc: 56712, kbrown

> From: Po Lu <luangruo@yahoo.com>
> Cc: Ken Brown <kbrown@cornell.edu>,  56712@debbugs.gnu.org
> Date: Tue, 26 Jul 2022 10:10:49 +0800
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > FTR: if that test should ever run on native MS-Windows, we'd need that
> > third slash, because file://c:/foo/bar is not valid on Windows, you
> > must say file:///c:/foo/bar.
> >
> > So from my POV, it is best to make that code system-dependent, with
> > windows-nt using 3 slashes, not 2.
> 
> The code is specific to X -- as evidenced by the name.  Doesn't Cygwin
> (the only way I know of to get an X server on MS Windows) use Unix-style
> file names everywhere?

Yes, thus my "if this test should ever run" condition.

IOW, the code produces 3 slashes by concatenating 2 strings, but the
results are only correct on Posix filesystems.  So at least a comment
is in order there, because how long would it take, you think, for
someone to copy/paste that code into an entirely different context?





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

* bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin
  2022-07-26  2:38                             ` Eli Zaretskii
@ 2022-07-26  2:52                               ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 0 replies; 20+ messages in thread
From: Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2022-07-26  2:52 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 56712, kbrown

Eli Zaretskii <eliz@gnu.org> writes:

> Yes, thus my "if this test should ever run" condition.
>
> IOW, the code produces 3 slashes by concatenating 2 strings, but the
> results are only correct on Posix filesystems.  So at least a comment
> is in order there, because how long would it take, you think, for
> someone to copy/paste that code into an entirely different context?

I see, I'll add a comment there.






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

end of thread, other threads:[~2022-07-26  2:52 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-22 21:10 bug#56712: 29.0.50; x-dnd-tests-do-direct-save fails on Cygwin Ken Brown
2022-07-23  1:10 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-23 17:30   ` Ken Brown
2022-07-23 17:41     ` Ken Brown
2022-07-24  2:26     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-24  5:50       ` Eli Zaretskii
2022-07-24  7:23         ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-24  7:42           ` Michael Albinus
2022-07-24  8:55           ` Eli Zaretskii
2022-07-24 10:55             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-24 13:33             ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-24 15:40               ` Ken Brown
2022-07-25  1:02                 ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-25  1:42                   ` Ken Brown
2022-07-25  2:10                     ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-25 16:24                       ` Ken Brown
2022-07-25 16:53                         ` Eli Zaretskii
2022-07-26  2:10                           ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-07-26  2:38                             ` Eli Zaretskii
2022-07-26  2:52                               ` Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors

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.