unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Richard Copley <rcopley@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Angelo Graziosi <angelo.g0@libero.it>,
	Emacs Development <emacs-devel@gnu.org>
Subject: Re: Suspicious warning in W64 build
Date: Sat, 9 Sep 2017 12:17:14 +0100	[thread overview]
Message-ID: <CAPM58oisvOXcYH7sUTqpAxCwcM-j+qSHQVt=JgFLRUcpUm-xLg@mail.gmail.com> (raw)
In-Reply-To: <83h8wcw3td.fsf@gnu.org>

On 9 September 2017 at 11:42, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: Richard Copley <rcopley@gmail.com>
>> Date: Sat, 9 Sep 2017 10:36:28 +0100
>> Cc: Angelo Graziosi <angelo.g0@libero.it>, Emacs Development <emacs-devel@gnu.org>
>>
>> >> That conversion happens often, e.g., in normalize_filename.
>> >
>> > What conversion is that?
>>
>> The down-casing of the drive letter in a native or MSYS file name.
>
> But find-file-noselect insists on comparing file names as simple
> case-sensitive strings.  Which will fail in a few more interesting
> cases.
>
>> > For starters, can you or Angelo tell where did that upper-case "C:/"
>> > originate from?
>>
>> You can see the warnings, for example, by invoking configure by
>> absolute file name with an upper-case drive letter before running make.
>>
>> Usually one uses a relative path and the problem can't arise.
>>
>> I used to see this when I used a complicated Perl script to build
>> Emacs. The upper-case drive letter was just an artifact. It was
>> derived from the current directory of the calling command prompt.
>
> So you are saying that upper-case C:/ comes from the user?  And if the
> user configures Emacs with a lower-case c:/, the problem will never
> happen?  If so, would binding find-file-suppress-same-file-warnings
> non-nil in autoload-find-generated-file solve the problem?  (This is
> less drastic than invoking find-file-noselect with a non-nil 2nd
> argument.)
>
>> > Also, are there other places in the build process
>> > that display file names with the upper-case drive letter?
>>
>> Yes. Transcript attached.
>
> I see a lot of warnings in that log.  It's a pity no one reports them,
> let alone works on fixing them.  (I don't see any of them on my
> systems.)

I assumed nobody was interested, since there were so many. My
mistake, sorry.

Reporting the warnings by email and answering questions about what
happens when one makes various changes isn't the most convenient
edit-compile-test cycle I've seen. You have your reasons for using an
old version of the compiler. Can you install MSYS2 and MinGW-W64
somewhere just for build testing? It would be less frustrating.

At least some of the "-Wformat=" warnings are misleading.
Emacs has to use MSVC's rules for format strings, but GCC
warns based on the C standard rules implemented in GCC.

Thanks.



  parent reply	other threads:[~2017-09-09 11:17 UTC|newest]

Thread overview: 123+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-05 13:38 Suspicious warning in W64 build Angelo Graziosi
2017-09-05 14:04 ` Richard Copley
2017-09-07 15:16 ` Eli Zaretskii
2017-09-07 15:42   ` Angelo Graziosi
2017-09-07 17:52     ` Richard Copley
2017-09-07 17:58       ` Richard Copley
2017-09-07 19:00         ` Angelo Graziosi
2017-09-07 19:21           ` Richard Copley
2017-09-09  4:58             ` Herring, Davis
2017-09-09  9:55               ` Richard Copley
2017-09-09 10:20                 ` Eli Zaretskii
2017-09-09 11:24                   ` Angelo Graziosi
2017-09-09 13:25                     ` Eli Zaretskii
2017-09-09 11:16                 ` Angelo Graziosi
2017-09-07 18:58       ` Eli Zaretskii
2017-09-07 19:26         ` Paul Eggert
2017-09-07 19:50           ` Richard Copley
2017-09-07 20:02             ` Richard Copley
2017-09-08  6:49               ` Eli Zaretskii
2017-09-08  8:02                 ` Eli Zaretskii
2017-09-08 19:31                   ` Richard Copley
2017-09-08 20:17                     ` Eli Zaretskii
2017-09-08 21:08                       ` Richard Copley
2017-09-08 21:37                         ` Richard Copley
2017-09-09  7:37                           ` Eli Zaretskii
2017-09-08 22:20                         ` Richard Copley
2017-09-09  7:33                         ` Eli Zaretskii
2017-09-09  9:36                           ` Richard Copley
2017-09-09 10:42                             ` Eli Zaretskii
2017-09-09 10:52                               ` Eli Zaretskii
2017-09-09 11:17                               ` Richard Copley [this message]
2017-09-09 16:07                                 ` Eli Zaretskii
2017-09-10  1:01                                   ` Richard Copley
2017-09-10 14:40                                     ` Eli Zaretskii
2017-09-10 19:14                                       ` Richard Copley
2017-09-10 19:38                                         ` Angelo Graziosi
2017-09-11 16:17                                           ` Eli Zaretskii
2017-09-11 22:21                                             ` Angelo Graziosi
2017-09-11 16:39                                           ` Óscar Fuentes
2017-09-11 17:20                                             ` Eli Zaretskii
2017-09-12 17:49                                   ` Eli Zaretskii
2017-09-12 18:01                                     ` Fabrice Popineau
2017-09-12 18:37                                       ` Richard Copley
2017-09-12 18:59                                         ` Eli Zaretskii
2017-09-12 19:14                                           ` Richard Copley
2017-09-12 18:38                                       ` Eli Zaretskii
2017-09-14 17:47                                         ` Eli Zaretskii
2017-09-14 19:34                                           ` Richard Copley
2017-09-15  8:54                                             ` Eli Zaretskii
2017-09-15 23:05                                               ` Richard Copley
2017-09-16  6:40                                                 ` Eli Zaretskii
2017-09-16  8:19                                                   ` Richard Copley
2017-09-16  8:34                                                     ` Richard Copley
2017-09-16  8:54                                                       ` Eli Zaretskii
2017-09-16  9:07                                                         ` Richard Copley
2017-09-16 11:54                                                           ` Fabrice Popineau
2017-09-16  8:52                                                     ` Eli Zaretskii
2017-09-15  8:59                                             ` Eli Zaretskii
2017-09-15 14:43                                               ` Eli Zaretskii
2017-09-17  6:42                                                 ` Paul Eggert
2017-09-17  7:14                                                   ` Richard Copley
2017-09-17 14:31                                                     ` Eli Zaretskii
2017-09-17  6:40                                               ` Paul Eggert
2017-09-17 14:29                                                 ` Eli Zaretskii
2017-09-17 16:39                                                   ` Fabrice Popineau
2017-09-17 16:52                                                     ` Eli Zaretskii
2017-09-18  0:26                                                   ` Paul Eggert
2017-09-18 11:47                                                     ` Fabrice Popineau
2017-09-18 14:46                                                       ` Eli Zaretskii
2017-09-18  0:01                                                 ` Richard Stallman
2017-09-14 19:36                                           ` Fabrice Popineau
2017-09-14 21:17                                           ` Andy Moreton
2017-09-15  6:55                                             ` Fabrice Popineau
2017-09-15  9:12                                               ` Eli Zaretskii
2017-09-15 15:33                                                 ` Fabrice Popineau
2017-09-15 15:45                                                   ` Eli Zaretskii
2017-09-15 18:15                                                     ` Fabrice Popineau
2017-09-15 19:00                                                       ` Eli Zaretskii
2017-09-15 21:02                                                         ` Fabrice Popineau
2017-09-16  7:45                                                           ` Eli Zaretskii
2017-09-17  7:01                                                             ` Paul Eggert
2017-09-17 14:31                                                               ` Eli Zaretskii
2017-09-17 14:52                                                                 ` Philipp Stephani
2017-09-17 22:34                                                                   ` Paul Eggert
2017-09-17 17:07                                                                 ` Paul Eggert
2017-09-17 17:14                                                                   ` Eli Zaretskii
2017-09-17 18:53                                                                     ` Paul Eggert
2017-09-17 19:30                                                                       ` Eli Zaretskii
2017-09-17 20:34                                                                         ` Paul Eggert
2017-09-18  2:30                                                                           ` Eli Zaretskii
2017-09-18  4:52                                                                             ` Paul Eggert
2017-09-18 14:41                                                                               ` Eli Zaretskii
2017-09-18 17:35                                                                                 ` Paul Eggert
2017-09-18 17:58                                                                                   ` Andy Moreton
2017-09-19  9:05                                                                                     ` Paul Eggert
2017-09-18 18:01                                                                                   ` Eli Zaretskii
2017-09-17 20:45                                                   ` Paul Eggert
2017-09-16 13:17                                                 ` Andy Moreton
2017-09-16 13:46                                                   ` Eli Zaretskii
2017-09-16 18:57                                                     ` Richard Copley
2017-09-16 19:21                                                       ` Eli Zaretskii
2017-09-15  9:03                                             ` Eli Zaretskii
2017-09-09  8:49                       ` Angelo Graziosi
2017-09-09 10:37                         ` Eli Zaretskii
2017-09-09 11:32                           ` Angelo Graziosi
2017-09-09 13:28                             ` Eli Zaretskii
2017-09-09 13:33                               ` Fabrice Popineau
2017-09-09 14:55                               ` Angelo Graziosi
2017-09-09 16:37                                 ` Eli Zaretskii
2017-09-09 18:38                                   ` Angelo Graziosi
2017-09-09 18:59                                     ` Eli Zaretskii
2017-09-09 21:29                                       ` Angelo Graziosi
2017-09-10 14:56                                         ` Eli Zaretskii
2017-09-10 15:45                                           ` Angelo Graziosi
2017-09-10 16:02                                             ` Eli Zaretskii
2017-09-10 18:45                                               ` Angelo Graziosi
2017-09-10 19:43                                                 ` Eli Zaretskii
2017-09-09 15:40                           ` Angelo Graziosi
2017-09-09 16:40                             ` Eli Zaretskii
2017-09-09 18:33                               ` Fabrice Popineau
2017-09-07 20:20           ` Eli Zaretskii
2017-09-07 21:59             ` Angelo Graziosi
2017-09-08  8:01               ` Eli Zaretskii

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='CAPM58oisvOXcYH7sUTqpAxCwcM-j+qSHQVt=JgFLRUcpUm-xLg@mail.gmail.com' \
    --to=rcopley@gmail.com \
    --cc=angelo.g0@libero.it \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.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).