[Buildroot] svn commit: trunk/buildroot

Bernhard Fischer rep.dot.nop at gmail.com
Mon Jul 7 15:25:46 UTC 2008

On Mon, Jul 07, 2008 at 04:44:25PM +0200, Peter Korsgaard wrote:
>>>>>> "Bernhard" == Bernhard Fischer <rep.dot.nop at gmail.com> writes:
> Bernhard> On Mon, Jul 07, 2008 at 02:13:44PM +0200, Ulf Samuelsson wrote:
> >> Hash: SHA1
> >> 
> >> Peter Korsgaard skrev:
> >>> That breaks the source-check/external-deps check in
> >>> Makefile.autotools.in (r22607).
> >>> 
> >> 
> >> Is not something like the following to be preferred?
> Bernhard> What were you trying to fix initially?
>Me or Ulf? For me, see r22607. Makefile.autotools.in is kinda broken as
>the download target is executed even if the download exists (E.G. it

Ok that's broken.

	$(WGET) -P $(DL_DIR) $($(PKG)_SITE)/$($(PKG)_SOURCE)
	# or $(if $(BR2_PRIMARY_SITE),$(BR2_PRIMARY_SITE),$($(PKG)_SITE))/$($(PKG)_SOURCE) if you want to support that primary site stuff
	touch $@

	$(foreach p,$($(PKG)_PATCH),$(WGET) -P $(DL_DIR) $($(PKG)_SITE)/$p)

$(BUILD_DIR)/%/.stamp_downloaded: |\
	$(BUILD_DIR)/%/.stamp_download_tarball \

>uses a phony target instead of the exact file name), so source-check /

The phony has to go, of course. There is absolutely no sane reason for it.

>external-deps doesn't work like it does for the other packages.

My lack of interrest to support external stuff is deep and profound.

More information about the buildroot mailing list