From: Grant Erickson <gerickson@nuovations.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] Remove $(VERSION_FILE) from PHONY Target List
Date: Fri, 09 May 2008 09:00:07 -0700 [thread overview]
Message-ID: <C449C317.F16E%gerickson@nuovations.com> (raw)
In-Reply-To: <20080509082246.9F8FA247F5@gemini.denx.de>
On 5/9/08 1:22 AM, Wolfgang Denk wrote:
> In message <C42519C4.E8EE%gerickson@nuovations.com> you wrote:
>> When building against non-local, non-disk-backed file systems (e.g. NFS,
>> tmpfs), the u-boot build can iterate forever, attempting to re-generate
>> "include/autoconf.mk". This occurs because $(VERSION_FILE) (aka
>> ${ROOT}/u-boot/build/include/version_autogenerated.h) is always regarded as
>> out-of-date because it is in the .PHONY target list, even though it's a real
>> file and seems to need to be only created once and only once.
>>
>> This patch removes $(VERSION_FILE) from the .PHONY target list and has been
>> verified to work with various flavors and builds of make-3.81 against NFS,
>> ext2fs, ext3fs and tmpfs file systems.
>>
>> More detail at:
>>
>> http://sourceforge.net/mailarchive/message.php?msg_id=C4180895.E556%25gerick
>> son%40nuovations.com
>
> This may fix your immediate problem, but it is wrong. Assume you have
> built U-Boot from some specific version, then the git commit ID will
> be shown as part of the version string. Assume you now edit a file
> and run "make" again. The intention is that this new build shows the
> "-dirty" marker in the version string, plus the new build date.
>
> Your change breaks this behaviour.
>
> I just posted another patch: ``Avoid infinite loop "Generating
> include/autoconf.mk" '' which is supposed to fix the problem, without
> unwanted side effects.
>
> Please test.
Wolfgang,
This patch works effectively and successfully when tested against NFSv3,
EXT2, EXT3 and TMPFS file systems with make v3.79, v.3.80 and v3.81.
Thanks,
Grant
prev parent reply other threads:[~2008-05-09 16:00 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-11 20:36 [U-Boot-Users] [PATCH] Remove $(VERSION_FILE) from PHONY Target List Grant Erickson
2008-04-12 19:29 ` Mike Frysinger
2008-04-13 6:20 ` Grant Erickson
2008-04-13 6:43 ` Mike Frysinger
2008-05-09 8:22 ` Wolfgang Denk
2008-05-09 16:00 ` Grant Erickson [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=C449C317.F16E%gerickson@nuovations.com \
--to=gerickson@nuovations.com \
--cc=u-boot@lists.denx.de \
/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