All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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 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.