From: Yao Qi <qiyaoltc@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: simon.marchi@ericsson.com, gdb-patches@sourceware.org,
palves@redhat.com
Subject: Re: [PATCH 0/5] Remove a few hurdles of compiling with clang
Date: Wed, 14 Jun 2017 09:07:00 -0000 [thread overview]
Message-ID: <86bmprvsy6.fsf@gmail.com> (raw)
In-Reply-To: <83d1a751us.fsf@gnu.org> (Eli Zaretskii's message of "Tue, 13 Jun 2017 18:44:27 +0300")
Eli Zaretskii <eliz@gnu.org> writes:
> Specifically, this text sounds too vague to me to be considered
> "policy". Too much is left to judgment calls. But if everyone else
> is happy, I won't insist on prolonging this discussion.
>
It is vague because the tasks this "policy" about are not clear. It is
unlike other polices like "-Werror policy" or "GDB's #include file
policy", their tasks are quite clear, so it is possible to give clear
actionable rules. However, it is still unknown that what are we going
to do to make GDB built with compilers other than GCC. It was just
raised by Simon's patches several days ago. We need to write this vague
policy down, and don't discourage people building GDB with different
compilers.
Patches are still reviewed, and some of them may be controversial. As
we have some concrete bad/good patches, we can have some specific rules
in this policy.
>> I also looked for the place to add this policy. Looks the most relevant
>> page is https://sourceware.org/gdb/wiki/Internals%20Compiler-Warnings
>
> I would suggest to consider a new page, or maybe make this part of
> coding standards.
OK, let me create a new page, "Build GDB with different compilers" in
chapter "Miscellaneous guidelines" in
https://sourceware.org/gdb/wiki/Internals It has few to do with coding
standards. People should still write the code in the same way,
specified in our coding standards, although GDB can be built by
different compilers.
--
Yao (齐尧)
next prev parent reply other threads:[~2017-06-14 9:07 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-10 19:58 Simon Marchi
2017-06-10 19:58 ` [PATCH 3/5] gdb: Add -Wno-mismatched-tags Simon Marchi
2017-06-10 19:58 ` [PATCH 1/5] gdb: Pass -x c++ to the compiler Simon Marchi
2017-06-10 19:58 ` [PATCH 2/5] gdb: Use -Werror when checking for (un)supported warning flags Simon Marchi
2017-06-10 19:58 ` [PATCH 4/5] linux-low: Remove usage of "register" keyword Simon Marchi
2017-06-10 19:58 ` [PATCH 5/5] Add ATTRIBUTE_PRINTF to trace_start_error Simon Marchi
2017-06-14 19:49 ` Sergio Durigan Junior
2017-06-11 2:36 ` [PATCH 0/5] Remove a few hurdles of compiling with clang Eli Zaretskii
2017-06-12 7:56 ` Yao Qi
2017-06-12 14:36 ` Eli Zaretskii
2017-06-12 15:54 ` Simon Marchi
2017-06-12 16:23 ` Andrew Pinski
2017-06-12 16:35 ` Pedro Alves
2017-06-12 16:37 ` Andrew Pinski
2017-06-12 16:45 ` Pedro Alves
2017-06-12 16:55 ` Pedro Alves
2017-06-12 16:44 ` Simon Marchi
2017-06-12 16:55 ` Andrew Pinski
2017-06-12 17:00 ` Simon Marchi
2017-06-12 16:44 ` Eli Zaretskii
2017-06-13 9:14 ` Yao Qi
2017-06-13 10:23 ` Simon Marchi
2017-06-13 11:06 ` Pedro Alves
2017-06-13 11:08 ` Simon Marchi
2017-06-13 14:38 ` Eli Zaretskii
2017-06-13 17:07 ` Simon Marchi
2017-06-13 19:23 ` Eli Zaretskii
2017-06-13 20:17 ` Simon Marchi
2017-06-14 2:29 ` Eli Zaretskii
2017-06-14 10:45 ` Pedro Alves
2017-06-16 16:12 ` John Baldwin
2017-06-13 15:22 ` Yao Qi
2017-06-13 15:44 ` Eli Zaretskii
2017-06-14 9:07 ` Yao Qi [this message]
2017-06-19 8:07 ` Yao Qi
2017-06-13 10:44 ` Pedro Alves
2017-06-13 15:09 ` Joel Brobecker
2017-06-17 21:23 ` Simon Marchi
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=86bmprvsy6.fsf@gmail.com \
--to=qiyaoltc@gmail.com \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=palves@redhat.com \
--cc=simon.marchi@ericsson.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