public inbox for crossgcc@sourceware.org
 help / color / mirror / Atom feed
From: "Jasmin J." <jasmin@anw.at>
To: crossgcc@sourceware.org
Subject: Re: [RFC] Refactor autoconf options and build scripts
Date: Sun, 13 Sep 2015 22:59:00 -0000	[thread overview]
Message-ID: <55F5FFE2.5010505@anw.at> (raw)
In-Reply-To: <20150912200305.69b80a14@free-electrons.com>

Thomas,

> support only a smaller subset of the gcc/binutils/C library versions,
> remove seldom used configuration options, etc.
My main goal is use CT-NG to build an arm-none-eabi compiler. This is currently
not full supported by CT-NG, at least with all the options the original build
script is using. But I am very happy to have all the current options and
selections which makes my life easier.

A solution might be to enable some of the options only in experimental mode or
to introduce a new mode, which allows only a few version combinations. If the
new "Expert Mode" is selected, then you will get all the options to change the
versions.

> For example, Crosstool-NG
> does not ensure that the proper cloog/mpc/gmp version is selected for a
> given version of gcc
At least for ClooG/isl I just implemented this. But you are right, who has the
time to check all the possible combinations.

> I think you might be confusing complexity with length. The current build 
> scripts may be long, but if it's just a list of:
> 
> if [ "${CT_CC_CXA_ATEXIT}" = "y" ]; then 
> extra_config+=("--enable-__cxa_atexit") else 
> extra_config+=("--disable-__cxa_atexit") fi
> 
> Then it is trivial to understand. If you replace that by something
> "generic" that is shorter but in fact trickier to understand, there is no
> real benefit.
100-gcc.sh uses "do_gcc_core_backend" for both the core and the final gcc,
when building for bare-metal. I don't know why, but the do_gcc_core_backend
function does not set the options for libssp and libquadmath and the handling
of "CT_CC_GCC_USE_LTO" is different, too.
So a generic "base class" for the gcc build might help to reduce such
inconsistencies, because I guess this difference for both compilers is not
intended.

I guess it depends how Bryan wants to implement the generic script and how it
can be tailored in the specific script. For example, I want to build newlib
in two variants. This requires an additional build step "newlib-nano". However
I will implement this, it is a complete copy of the newlib step with only 2..3
options changed/added. Having here a generic build script would reduce the work
to implement this.

> Now that you've moved to github for the code, maybe
> https://travis-ci.org might help (at least for the non-randconfig cases)?
Although this is intended for program regression tests, it can be used for
CT-NG with this options (I guess):

before_install:
	- sudo apt-get update -qq
	- sudo apt-get install -qq [packages list]
 -> used to install the packages required for CT-NG

install: ./tests/config_cg_ng.sh
 -> will do bootstrap, configure --enable-local and make

script: ./tests/run-tests.sh
 -> will execute CT-NG with the predefined configurations

Note:
 a) ./tests is a new subdirectory for the travis-ci scripts.
 b) Someone needs to define predefined configs for the different variants to
    test and enter them into run-tests.sh

Got this info about the configuration from here:
  http://docs.travis-ci.com/user/customizing-the-build/

BR
   Jasmin

--
For unsubscribe information see http://sourceware.org/lists.html#faq

      parent reply	other threads:[~2015-09-13 22:59 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-08 17:32 Bryan Hundven
2015-09-08 23:42 ` Jasmin J.
2015-09-12 18:03 ` Thomas Petazzoni
2015-09-12 21:23   ` Bryan Hundven
2015-09-13 14:13     ` Trevor Woerner
2015-09-13 23:19     ` Jasmin J.
2015-09-13 23:35       ` Jasmin J.
2015-09-14  0:31         ` Jasmin J.
2015-09-15 14:21           ` Jean-Marie Lemetayer
2015-09-15 14:39             ` Trevor Woerner
2015-09-15 15:22               ` Jean-Marie Lemetayer
2015-09-15 18:30                 ` Jasmin J.
2015-09-15 20:38                   ` Jasmin J.
2015-09-15 20:49                   ` Bryan Hundven
2015-09-15 22:46                     ` Jasmin J.
2015-09-15 23:21                       ` Bryan Hundven
2015-09-16 11:17                         ` Jasmin J.
2015-09-16 11:45                         ` Jean-Marie Lemetayer
2015-09-16 11:47                           ` Jean-Marie Lemetayer
2015-09-16 13:20                             ` Jasmin J.
2015-09-16 13:47                               ` Jean-Marie Lemetayer
2015-09-16 14:50                                 ` Jasmin J.
2015-09-21 22:55                                   ` Jasmin J.
2015-09-22  8:17                                     ` Jean-Marie Lemetayer
2015-09-22 21:18                                       ` Bryan Hundven
2015-09-13 22:59   ` Jasmin J. [this message]

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=55F5FFE2.5010505@anw.at \
    --to=jasmin@anw.at \
    --cc=crossgcc@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).