public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Zack Weinberg <zackw@panix.com>
To: Joseph Myers <joseph@codesourcery.com>
Cc: GNU C Library <libc-alpha@sourceware.org>
Subject: Re: Add script to build many glibc configurations
Date: Fri, 18 Nov 2016 16:28:00 -0000	[thread overview]
Message-ID: <CAKCAbMi=PpLoahBsx_twVtKPHBpUUZ4c9VH7Tivd2NkRouHMRQ@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.20.1611171747300.21214@digraph.polyomino.org.uk>

On Thu, Nov 17, 2016 at 12:51 PM, Joseph Myers <joseph@codesourcery.com> wrote:
> On Thu, 17 Nov 2016, Zack Weinberg wrote:
>> On 11/09/2016 11:27 AM, Joseph Myers wrote:
>> > This patch adds a Python (3.5 or later) script to build many different
>> > configurations of glibc ...
>>
>> Since this does its own glibc checkout, it's not clear to me how one
>> should use it to test a patch(set)...
>
> You patch the script's glibc checkout (or switch it to a different branch,
> or whatever) at some point after the checkout step and before running the
> "glibcs" build...

Thanks, I have it working now.  I'd like to mention some more places
where the script could be more ergonomic for patch testing:

* Please don't delete the build tree unless configure, build, and
check all succeed, so that one can dig into a failure beyond what's
visible in the logs.  Add an option to preserve the build tree even if
all three succeeded, to facilitate additional testing (for instance,
an "installed stripped libraries are unchanged" test - if that fails
one will want to go look at the object files).
* Please find some way of not needing to make a copy of the glibc
source tree for each build configuration, so that one can make edits
to .../src/glibc and immediately do incremental rebuilds in all
failing configurations.
* Please make sure that command lines written to logfiles are properly
quoted, so that they can be copied and pasted directly into a shell.
(The problem case I know about is CC="x-y-z-gcc -mthis -mthat" on a
configure command line, which currently gets written to the logfile
without any quotes.)  (Python stdlib has a function `shlex.quote` that
will help.)

I'm kibitzing a lot, so let me say that this script is really, really
useful and I can see lots of cases where it will make complex changes
easier to validate.  Thanks for writing it.

zw

  reply	other threads:[~2016-11-18 16:28 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-09 16:27 Joseph Myers
2016-11-10 14:27 ` Joseph Myers
2016-11-10 16:44   ` Joseph Myers
2016-11-23 17:36   ` Chris Metcalf
2016-11-10 17:09 ` Steve Ellcey
2016-11-10 17:22   ` Joseph Myers
2016-11-10 17:39     ` Steve Ellcey
2016-11-11 15:20 ` Joseph Myers
2016-11-11 19:37   ` Carlos O'Donell
2016-11-14 15:06   ` Mike Frysinger
2016-11-14 15:23     ` Joseph Myers
2016-11-14 19:27       ` Mike Frysinger
2016-11-14 23:28         ` Joseph Myers
2016-11-14 23:57     ` Joseph Myers
2016-11-17 16:52 ` Zack Weinberg
2016-11-17 17:26   ` Zack Weinberg
2016-11-17 17:47     ` Joseph Myers
2016-11-17 17:54     ` Andreas Schwab
2016-11-17 17:51   ` Joseph Myers
2016-11-18 16:28     ` Zack Weinberg [this message]
2016-11-18 18:27       ` Joseph Myers

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='CAKCAbMi=PpLoahBsx_twVtKPHBpUUZ4c9VH7Tivd2NkRouHMRQ@mail.gmail.com' \
    --to=zackw@panix.com \
    --cc=joseph@codesourcery.com \
    --cc=libc-alpha@sourceware.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.
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).