Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Kai Tietz <ktietz70@googlemail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: asmwarrior@gmail.com, tromey@redhat.com, yao@codesourcery.com,
		gdb-patches@sourceware.org
Subject: Re: [MinGW-w64]Build gdb/ctf.c failed
Date: Mon, 25 Mar 2013 15:18:00 -0000	[thread overview]
Message-ID: <CAEwic4Z0kRjmUuEh3y5h6uMCCSyzxwScGwzspD2jwxP1xYx3rA@mail.gmail.com> (raw)
In-Reply-To: <8338vj3i1w.fsf@gnu.org>

2013/3/25 Eli Zaretskii <eliz@gnu.org>:
>> Date: Mon, 25 Mar 2013 10:15:14 +0100
>> From: Kai Tietz <ktietz70@googlemail.com>
>> Cc: asmwarrior@gmail.com, tromey@redhat.com, yao@codesourcery.com,
>>       gdb-patches@sourceware.org
>>
>> >> You are driveling ...
>> >
>> > Very mature.  Thanks a lot.
>>
>> Be welcome.
>
> I'm not, actually.
>
>> Sorry, but I want to insist on such apparent things.  I am wondering
>> about your arguments, due you should know better.
>
> I have no idea what I "should know better".  If you wonder about my
> arguments, you can always ask for clarifications.  We are both
> writing in a language that isn't our first one, so misunderstanding is
> much more probable than anything else.

Well, some of this might be caused by foreign tongue.  What I meant by
... well let me say confused instead of "drivel" ... its English
meaning is more harsh as I wanted to say:

First cite of your reply within that thread I want to reference is:

"Is mkdir declared in MinGW64 unistd.h?  If so, can you please show its
declaration from unistd.h?

Also, what other headers are included in MinGW64 unistd.h?

I dislike using Windows-specific headers in general source files.
MinGW unistd.h (which is absent from MS SDK's) was created for that
very purpose.  If MinGW64 is breaking that, IMO it's a MinGW64 bug
that needs to be reported to them, while we look for the cleanest way
of fixing this annoyance."

So, you claimed MinGW-w64 did something wrong ... well, if we wouldn't
declare mkdir here, indeed it would be worth a bug-report ...

And then you ware asking the following question to Asmwarrior:
"Isn't _mkdir also declared in io.h?"

Well, so I assume that you are meaning _mkdir in first offensive cite,
as this function was actual used by Yao Qi's patch.

After you got the reply by Yao Qi you wrote:
"Too bad.  Gratuitous differences between the different MinGW variants
are likely to become maintenance headaches in the long run.  Like in
this case."

So what differences you were talking about?  The function mkdir is
declared for MinGW.org, and for MinGW-w64.  It is the POSIX compliant
API name and both ventures are declaring it in an POSIX-helper header.
 So you mean yet _mkdir?

The few mingw posix extensions are there in order to make porting a
bit easier.  if you want to use a posix function, include the
corresponding posix header; if you want to a MS API, use the header MS
defines the function/interface lives in.  it wouldn't really help
portability (in either direction) to support including a posix header,
and getting a MS API function, so mingw doesn't lay its headers that
way.  The header io.h isn't a POSIX one, and therefore you should just
expect what actual is documented by vendor (in msdn) for it and not
what one implementation mightz does.

Kai


  reply	other threads:[~2013-03-25 12:43 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-15  2:02 New ARI warning Fri Mar 15 02:02:12 UTC 2013 in -D 2013-03-15-gmt GDB Administrator
2013-03-15  9:05 ` Yao Qi
2013-03-15  9:48   ` Compilation failure for mingw64 target (was New ARI warning Fri Mar 15 02:02:12 UTC 2013 in -D 2013-03-15-gmt) Pierre Muller
2013-03-15 11:06     ` Yao Qi
2013-03-15 11:24       ` Pierre Muller
2013-03-15 14:05       ` Yao Qi
2013-03-15 14:43       ` Yao Qi
2013-03-15 16:52         ` Tom Tromey
2013-03-15 16:55           ` Pierre Muller
     [not found]           ` <20313.4872871034$1363366373@news.gmane.org>
2013-03-15 17:01             ` Tom Tromey
2013-03-15 18:22               ` Eli Zaretskii
2013-03-15 18:24                 ` Tom Tromey
2013-03-15 18:33                   ` Eli Zaretskii
2013-03-15 18:53                     ` Mike Frysinger
2013-03-15 18:58                       ` Eli Zaretskii
2013-03-15 19:45                         ` Mike Frysinger
2013-03-16  4:23                     ` [PATCH 0/3] Fix build failure caused by ctf.c Yao Qi
2013-03-16  4:23                       ` [PATCH 3/3] Don't use unportable macros Yao Qi
2013-03-19 17:12                         ` Doug Evans
2013-03-20  2:48                           ` Yao Qi
2013-03-20 15:17                             ` Doug Evans
2013-03-21  1:58                               ` Yao Qi
2013-03-20 17:36                             ` Eli Zaretskii
2013-03-20 17:40                               ` Tom Tromey
2013-03-25  4:08                                 ` [MinGW-w64]Build gdb/ctf.c failed asmwarrior
2013-03-25  6:41                                   ` asmwarrior
2013-03-25  8:28                                   ` Yao Qi
2013-03-25  8:39                                     ` Eli Zaretskii
2013-03-25  8:40                                   ` Eli Zaretskii
2013-03-25  9:15                                     ` Kai Tietz
2013-03-25  9:42                                       ` Yao Qi
2013-03-25 10:12                                         ` Kai Tietz
2013-03-25 12:43                                         ` Eli Zaretskii
2013-03-25 14:40                                           ` Yao Qi
2013-03-25  9:54                                       ` Eli Zaretskii
2013-03-25 10:50                                         ` Kai Tietz
2013-03-25 13:12                                           ` Eli Zaretskii
2013-03-25 13:21                                             ` Kai Tietz
2013-03-25 14:25                                               ` Eli Zaretskii
2013-03-25 15:18                                                 ` Kai Tietz [this message]
2013-03-25 15:44                                                   ` Eli Zaretskii
2013-03-25 16:14                                                     ` Kai Tietz
2013-03-25 16:25                                                       ` Eli Zaretskii
2013-03-16  4:23                       ` [PATCH 2/3] Write CTF in host byte order Yao Qi
2013-03-19 21:26                         ` Joel Brobecker
2013-03-20  3:48                           ` Yao Qi
2013-03-16  7:35                       ` [PATCH 1/3] Import mkdir module Yao Qi
2013-03-19 16:32                         ` Doug Evans
2013-03-19 17:20                           ` Eli Zaretskii
2013-03-19 21:23                           ` Joel Brobecker
2013-03-15 16:22   ` New ARI warning Fri Mar 15 02:02:12 UTC 2013 in -D 2013-03-15-gmt Pedro Alves

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=CAEwic4Z0kRjmUuEh3y5h6uMCCSyzxwScGwzspD2jwxP1xYx3rA@mail.gmail.com \
    --to=ktietz70@googlemail.com \
    --cc=asmwarrior@gmail.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@redhat.com \
    --cc=yao@codesourcery.com \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox