Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Hamish Moffatt <hamish@cloud.net.au>
To: buildroot@busybox.net
Subject: [Buildroot] Makefile.autotools.in stamp files breaks multiple projects
Date: Thu, 4 Sep 2008 12:50:35 +1000	[thread overview]
Message-ID: <20080904025034.GA18148@cloud.net.au> (raw)

As previously discussed, Makefile.autotools.in creates stamp files in
$(BUILD_DIR)/$(PKG) such as .stamp_installed. This will prevent the
package being properly installed into multiple projects built out of the
same source tree. (The first project will build ok but subsequent
projects won't install the files.)

I see a few solutions;

1. Each autotools-using package could (optionally?) specify the name of
a test target file to use, instead of using an explicit stamp file.
Hans-Christian sent a patch for this already.

Obseleting (some or all of) the stamp files wouldn't be bad anyway. Many
of them could be replaced with actual targets, eg Makefile depends on
running configure (rather than using .stamp_configured).

2. The .stamp_install files could be put somewhere under $(TARGET_DIR)
instead of $(BUILD_DIR); they'd need to be stripped out by the various
generators in target/.. though.

3. Sharing $(BUILD_DIR) between multiple projects should be scrapped. In
practice, redefining BUILD_DIR = $(PROJECT_BUILD_DIR) would do it.


I think sharing $(BUILD_DIR) between multiple projects is quite broken.
The only benefit is that the packages only need to be compiled once. As
a negative though, we've got
- packages not being installed properly due to stamp files (the problem
  above).
- different projects might be using different compilers or different
  compiler flags, and the right ones won't be used for each project.
- same for different kernel headers potentially.
- We've already got exceptions for packages which can be customised,
  like busybox and the kernel.


Hamish
-- 
Hamish Moffatt VK3SB <hamish@debian.org> <hamish@cloud.net.au>

             reply	other threads:[~2008-09-04  2:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-04  2:50 Hamish Moffatt [this message]
2008-09-04  4:02 ` [Buildroot] Makefile.autotools.in stamp files breaks multipleprojects Ulf Samuelsson
2008-09-04 14:35   ` Hamish Moffatt
2008-09-05  4:30 ` [Buildroot] Makefile.autotools.in stamp files breaks multiple projects Hamish Moffatt
2008-09-08 13:39   ` Hans-Christian Egtvedt
2008-09-08 13:43     ` Hamish Moffatt

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=20080904025034.GA18148@cloud.net.au \
    --to=hamish@cloud.net.au \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox