From: Joseph Myers <joseph@codesourcery.com>
To: Simon Marchi <simon.marchi@polymtl.ca>
Cc: <gdb-patches@sourceware.org>, <binutils@sourceware.org>,
Simon Marchi <simon.marchi@ericsson.com>
Subject: Re: [PATCH 1/2] Bump to autoconf 2.69 and automake 1.15.1
Date: Fri, 15 Jun 2018 17:29:00 -0000 [thread overview]
Message-ID: <alpine.DEB.2.20.1806151717390.20961@digraph.polyomino.org.uk> (raw)
In-Reply-To: <20180615004309.29913-1-simon.marchi@polymtl.ca>
Questions:
* Are all the shared files and directories fully in sync between the GCC
and binutils-gdb trees before this patch? Shared directories include
config, intl, parts of include, libdecnumber, libiberty, zlib, for
example.
* Where you are changing shared files and directories, do any changes to
*non-generated* files other than config/override.m4 depend on the autoconf
/ automake updates, or would such changes work equally well in the GCC
tree even in the absence of a version update there?
If proposing to change only one tree at a time I think it's important to
take care to minimise the extra costs introduced for people synchronizing
changes between the two trees while the versions are out of sync. To me,
that indicates that the shared files and directories should be fully in
sync before any changes making them deliberately out of sync are applied,
and that changes to non-generated files other than config/override.m4
should go in both places if they work in both places, so that the
differences immediately after the change is applied are only the required
ones (i.e. config/override.m4 and the generated files), so that anyone
then merging a subsequent change in future knows they expect to get back
to exactly that set of differences and no more.
(Shared files in the newlib-cygwin tree have been out of sync for a lot
longer. So, while I think that tree also ought to have shared files in
sync, with changes being applied to all three trees (I don't know if
newlib-cygwin has any changes not present in the other trees), I don't
think it's immediately relevant to changes in the binutils-gdb tree right
now.)
--
Joseph S. Myers
joseph@codesourcery.com
next prev parent reply other threads:[~2018-06-15 17:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-15 0:43 Simon Marchi
2018-06-15 0:46 ` [PATCH 2/2] Generated files Simon Marchi
2018-06-15 3:00 ` Simon Marchi
2018-06-15 14:22 ` [PATCH 1/2] Bump to autoconf 2.69 and automake 1.15.1 Nick Clifton
2018-06-15 14:38 ` Simon Marchi
2018-06-15 14:58 ` Nick Clifton
2018-06-15 15:14 ` Simon Marchi
2018-06-15 15:30 ` Nick Clifton
2018-06-15 15:38 ` Simon Marchi
2018-06-15 15:54 ` Nick Clifton
2018-06-16 1:39 ` Alan Modra
2018-06-16 3:56 ` Simon Marchi
2018-06-16 7:17 ` Andreas Schwab
2018-06-18 2:46 ` Alan Modra
2018-06-18 3:05 ` Simon Marchi
2018-06-18 9:42 ` Nick Clifton
2018-06-15 17:29 ` Joseph Myers [this message]
2018-06-16 3:05 ` Simon Marchi
2018-06-18 15:12 ` Joseph Myers
2018-06-18 15:32 ` Simon Marchi
2018-06-16 22:41 ` Andreas K. Huettel
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=alpine.DEB.2.20.1806151717390.20961@digraph.polyomino.org.uk \
--to=joseph@codesourcery.com \
--cc=binutils@sourceware.org \
--cc=gdb-patches@sourceware.org \
--cc=simon.marchi@ericsson.com \
--cc=simon.marchi@polymtl.ca \
/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