public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Kai Tietz <ktietz70@googlemail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Kai Tietz <ktietz@redhat.com>,
	Jan Kratochvil <jan.kratochvil@redhat.com>,
	sellcey@imgtec.com, 	Joel Brobecker <brobecker@adacore.com>,
	Yao Qi <yao@codesourcery.com>,
		"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [patch] compile: rm -rf -> ftw()+rmdir()+unlink() [Re: [patch] compile: Fix MinGW build]
Date: Thu, 18 Dec 2014 20:55:00 -0000	[thread overview]
Message-ID: <CAEwic4ZoQKAcTpjmzXA2kHfRcitU4PeA_FGT14rptVresVgFhw@mail.gmail.com> (raw)
In-Reply-To: <83k31ou1mi.fsf@gnu.org>

2014-12-18 21:47 GMT+01:00 Eli Zaretskii <eliz@gnu.org>:
>> Date: Thu, 18 Dec 2014 15:14:48 -0500 (EST)
>> From: Kai Tietz <ktietz@redhat.com>
>> Cc: Jan Kratochvil <jan.kratochvil@redhat.com>, sellcey@imgtec.com,
>>         brobecker@adacore.com, yao@codesourcery.com,
>>         gdb-patches@sourceware.org
>>
>> > > >From the Fedora point of view MinGW64 32-bit mode seems to be a superset
>> > > >of
>> > > MinGW32 so why to care about MinGW32 anymore?  Or what do I miss?
>> >
>> > That _I_ use MinGW32?
>>
>> That is actually your problem, isn't it?
>
> I don't see it as a problem, necessarily.
>
>> The mingw-w64 target support ftw, so why not simply allow it for targets providing it, and other targets can be covered by gnulib?
>
> Sure, why not?  I wasn't objecting to that, I just provided
> information, since Jan seemed to think ftw is available everywhere.
>
>> What libraries "mingw-w64" breaks often?!?  Could you please go in detail?  I am curious to hear that, as all distributors I know (Fedora, Debian, OpenSuse, ArchLinux, ...) haven't reported this.  Or is that just one thing you have a "gut" feeling about?
>
> The latest that I saw is this:
>
>   http://lists.gnu.org/archive/html/bug-gnulib/2014-12/msg00186.html
>
> And I remember a few more lately.

Hmm, this isn't something we got reported at all.  As you are using
pthread-library based toolchain (this is a build-option, and not a
mandatory mingw-w64 thing at all) I don't see that this is a mingw-w64
venture issue at all.  Btw we (on mingw-w64) strongly recomment to use
winpthread instead, as it solves some quirks existing with other
pthread-implementations available for Win32 ... anyway later is
off-topic.

> But look, I don't want to argue, I specifically said that.  Jan asked
> why not forget about MinGW32, and I gave _my_ reasons.  You don't have
> to agree, and we don't have to convince each other.  My only request
> is that GDB doesn't drop MinGW32 support.

Sure, no problem about that.  Nevertheless I have a problem if you try
to tell people things not true.

>> Just one point here I got curious about. What you mean by ABI?  The ABI of mingw-targets is the same for all targets using gcc.  So what ABI-differences you are talking about?!?
>
> Exception handling across DLLs is one difference I know of.

This is again a build-option of gcc, and has nothing directly to do
with mingw-w64.  For example, you can find for 32-bit dw2 based
exception-handling toolchain provided by mingw-builds projects (and
some others providing for 32-bit dw2 too).  For 64-bit there is SEH,
which replaces dw2 completely.

>> > Even if there were no problems with MinGW64, I don't think we should
>> > stop supporting MinGW32 just like that, it is still a live project,
>> > and I, for one, is quite happy with it.  I hope GDB will not drop its
>> > support any time soon.
>>
>> No problem about this, but why blocking things not related to MinGW.org?
>
> I didn't, it's a misunderstanding.  Sorry if I caused it.

Ok, np.

Kai

  reply	other threads:[~2014-12-18 20:55 UTC|newest]

Thread overview: 93+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-23 19:27 [PATCH v4 00/14] let gdb reuse gcc's C compiler Jan Kratochvil
2014-11-23 19:27 ` [PATCH v4 02/14] add gcc/gdb interface files Jan Kratochvil
2014-11-23 19:27 ` [PATCH v4 03/14] add some missing ops to DWARF assembler Jan Kratochvil
2014-11-23 19:27 ` [PATCH v4 01/14] introduce ui_file_write_for_put Jan Kratochvil
2014-11-23 19:27 ` [PATCH v4 04/14] add make_unqualified_type Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 09/14] split dwarf2_fetch_cfa_info from dwarf2_compile_expr_to_ax Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 06/14] add infcall_mmap and gcc_target_options gdbarch methods Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 13/14] add s390_gcc_target_options Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 11/14] export dwarf2_reg_to_regnum_or_error Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 12/14] add linux_infcall_mmap Jan Kratochvil
2014-12-15 12:41   ` Ulrich Weigand
2014-12-15 13:49     ` Yao Qi
2014-12-15 16:48       ` Jan Kratochvil
2014-12-16  5:27         ` Yao Qi
2014-11-23 19:28 ` [PATCH v4 05/14] add dummy frame destructor Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 08/14] introduce call_function_by_hand_dummy Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 10/14] make dwarf_expr_frame_base_1 public Jan Kratochvil
2014-11-23 19:28 ` [PATCH v4 07/14] add gnu_triplet_regexp gdbarch method Jan Kratochvil
2014-11-23 19:29 ` [PATCH v4 14/14] the "compile" command Jan Kratochvil
2014-12-14  6:05   ` Yao Qi
2014-12-14  7:53     ` Jan Kratochvil
2014-12-14  8:54       ` Yao Qi
2014-12-14 18:24         ` [mingw rfc] Add mkdtemp to gdb/gnulib/ [Re: [PATCH v4 14/14] the "compile" command] Jan Kratochvil
2014-12-15  3:16           ` Yao Qi
2014-12-15 12:44             ` Joel Brobecker
2014-12-15 13:11               ` Kai Tietz
2014-12-15 13:24               ` Yao Qi
2014-12-15 17:12               ` Jan Kratochvil
2014-12-15 17:48                 ` Eli Zaretskii
2014-12-15 18:14                 ` Joel Brobecker
2014-12-15 18:21                   ` Jan Kratochvil
2014-12-15 18:35                     ` Joel Brobecker
2014-12-15 18:40                       ` Jan Kratochvil
2014-12-15 18:57                         ` Eli Zaretskii
2014-12-15 22:28                           ` [patch] compile: Fix MinGW build [Re: [mingw rfc] Add mkdtemp to gdb/gnulib/] Jan Kratochvil
2014-12-15 22:56                             ` Jan Kratochvil
2014-12-16  3:40                               ` Eli Zaretskii
2014-12-16  9:04                                 ` Kai Tietz
2014-12-17 21:02                                   ` [patch] compile: rm -rf -> ftw()+rmdir()+unlink() [Re: [patch] compile: Fix MinGW build] Jan Kratochvil
2014-12-17 22:07                                     ` Steve Ellcey
2014-12-17 22:42                                     ` Pedro Alves
2014-12-18 18:14                                       ` Jan Kratochvil
2014-12-18 18:24                                         ` Pedro Alves
2014-12-18 18:41                                           ` Jan Kratochvil
2014-12-18 19:07                                             ` Pedro Alves
2014-12-18 21:05                                               ` Jan Kratochvil
2014-12-18 21:11                                                 ` Eli Zaretskii
2014-12-18 21:24                                                   ` Jan Kratochvil
2014-12-18 22:18                                                     ` Sergio Durigan Junior
2014-12-18 23:06                                                     ` Pedro Alves
2014-12-19  0:12                                                       ` Jan Kratochvil
2014-12-19 10:48                                                         ` Pedro Alves
2014-12-19  8:24                                                     ` Eli Zaretskii
2014-12-19 12:59                                             ` Joel Brobecker
2014-12-18 20:26                                         ` Eli Zaretskii
2014-12-18  5:38                                     ` Yao Qi
2014-12-18 17:23                                     ` Eli Zaretskii
2014-12-18 17:31                                       ` Jan Kratochvil
2014-12-18 17:40                                         ` Eli Zaretskii
2014-12-18 20:15                                           ` Kai Tietz
2014-12-18 20:47                                             ` Eli Zaretskii
2014-12-18 20:55                                               ` Kai Tietz [this message]
2014-12-18 23:47                                             ` Pedro Alves
2014-12-19  9:28                                               ` Kai Tietz
2014-12-19 19:25                                               ` Jan Kratochvil
2014-12-19 20:55                                                 ` Jan Kratochvil
2014-12-20 12:26                                                 ` Joel Brobecker
2014-12-18 20:56                                     ` Sergio Durigan Junior
2014-12-17 17:30                                 ` [patch] compile: Fix MinGW build [Re: [mingw rfc] Add mkdtemp to gdb/gnulib/] Steve Ellcey
2014-12-16  3:37                             ` Eli Zaretskii
2014-12-16  9:06                               ` Kai Tietz
2014-12-17 19:18                               ` Jan Kratochvil
2014-12-17 19:31                                 ` Eli Zaretskii
2014-12-17 19:37                                   ` Jan Kratochvil
2014-12-17 20:34                                   ` Pedro Alves
2014-12-18 17:22                                     ` Eli Zaretskii
2014-12-18 18:00                                       ` Pedro Alves
2014-12-16 12:40                             ` Pierre Muller
2014-12-17 18:21                               ` Jan Kratochvil
2014-12-17  1:22                             ` Yao Qi
2014-12-17 10:57                               ` Pedro Alves
2014-12-17 11:40                                 ` Yao Qi
2014-12-17 11:43                                   ` Pedro Alves
2014-12-17 11:56                                     ` Yao Qi
2014-12-17 19:15                                     ` Jan Kratochvil
2014-12-17 11:16                             ` Pedro Alves
2014-12-17 18:17                               ` [patchv2] " Jan Kratochvil
2014-12-17 18:33                                 ` Pedro Alves
2014-12-17 19:12                                   ` [commit] " Jan Kratochvil
2014-12-05 18:29 ` [patch 15/14] GDB/GCC compile function pointers [Re: [PATCH v4 00/14] let gdb reuse gcc's C compiler] Jan Kratochvil
2014-12-12 21:51   ` obsolete: " Jan Kratochvil
2014-12-12 14:40 ` [PATCH v4 00/14] let gdb reuse gcc's C compiler Pedro Alves
2014-12-12 21:46   ` [commit all 14 parts] " Jan Kratochvil

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=CAEwic4ZoQKAcTpjmzXA2kHfRcitU4PeA_FGT14rptVresVgFhw@mail.gmail.com \
    --to=ktietz70@googlemail.com \
    --cc=brobecker@adacore.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=ktietz@redhat.com \
    --cc=sellcey@imgtec.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).