All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@sunsite.dk>
To: buildroot@busybox.net
Subject: [Buildroot] [git commit master 1/1] Use sys-root rather than sysroot for the sysroot/staging_dir
Date: Tue, 25 Jan 2011 22:51:22 +0100	[thread overview]
Message-ID: <87wrlszm79.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <201101252239.36666.yann.morin.1998@anciens.enib.fr> (Yann E. MORIN's message of "Tue, 25 Jan 2011 22:39:36 +0100")

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@anciens.enib.fr> writes:

 Yann> Peter, All,
 Yann> On Tuesday 25 January 2011 16:08:50 Peter Korsgaard wrote:
 >> The name of the sysroot directory is arbitrary, but as ct-ng uses sys-root,
 >> let's use that as well for consistency.

 Yann> As Thomas said on IRC, I would suggest to revert this. Is there
 Yann> anything in BR that depends on the sysroot being named 'sysroot'
 Yann> or 'sys-root' at all, or was that just an eye-candy change?
 Yann> (genuine question)

No, as mentioned - The name is arbitrary, but as the ct-ng version we're
currently using hardcodes it as sys-root, then we either need to use
sys-root or add symlinks if we want ct-ng to install directly into
HOST_DIR/usr to get BR's and ct-ng's idea of $tuple/sys-root to match.

 Yann> I will push the change in crosstool-NG that makes this
 Yann> configurable, and the default is now 'sysroot', instead of the
 Yann> old hard-coded 'sys-root'.

Ok, good.

 Yann> Then, it will be a matter of taste what to set it to, with BR forcing it
 Yann> to whatever suits (I believe just 'sysroot').

Yeah. Until very recently noone really cared as STAGING_DIR used to be
directly in output/.

 Yann> Also, note that not all toolchains set this to any of 'sysroot'
 Yann> or 'sys-root'.  For example, the CodeSourcery toolchains set this
 Yann> to 'libc' for the main sysroot, and to libc/${variant} for each
 Yann> supported variant's sysroot.

Yes, but that's OK. This change was just to get the ct-ng toolchain to
behave more similar to the internal one.

-- 
Bye, Peter Korsgaard

      parent reply	other threads:[~2011-01-25 21:51 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-25 15:08 [Buildroot] [git commit master 1/1] Use sys-root rather than sysroot for the sysroot/staging_dir Peter Korsgaard
2011-01-25 21:39 ` Yann E. MORIN
2011-01-25 21:50   ` Thomas Petazzoni
2011-01-25 23:25     ` Yann E. MORIN
2011-01-25 21:51   ` Peter Korsgaard [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=87wrlszm79.fsf@macbook.be.48ers.dk \
    --to=jacmet@sunsite.dk \
    --cc=buildroot@busybox.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.