[Buildroot] svn commit: trunk/buildroot/target/linux

[Jean-Christian de Rivaz <jc@eclis.ch>]

ulf at uclibc.org a ?crit :
> Author: ulf
> Date: 2008-04-15 10:10:24 -0700 (Tue, 15 Apr 2008)
> New Revision: 21739
> 
> Log:
> Reversing bad tftpboot patch resulting in two identical targets
> 
> Modified:
>    trunk/buildroot/target/linux/Makefile.in.advanced

Ouch! Can you please explain why did you re-introduce this bug ?
The BR2_LINUX_COPYTO variable have absolutely no way to work with a hard 
coded "/tftpboot/". Just try to copy to /var/lib/tftpboot for example.

Please expose a test case so this "resulting in two identical targets" 
issue can be resolved the right way. Because reversing this patch is 
just plain wrong.

> Changeset:
> Modified: trunk/buildroot/target/linux/Makefile.in.advanced
> ===================================================================
> --- trunk/buildroot/target/linux/Makefile.in.advanced	2008-04-15 08:27:24 UTC (rev 21738)
> +++ trunk/buildroot/target/linux/Makefile.in.advanced	2008-04-15 17:10:24 UTC (rev 21739)
> @@ -230,7 +230,7 @@
>  endif
>  
>  ifeq ($(BR2_LINUX_COPYTO_TFTPBOOT),y)
> -LINUX26_TARGETS+=$(BR2_LINUX_COPYTO)/$(LINUX26_KERNEL_NAME)
> +LINUX26_TARGETS+=/tftpboot/$(LINUX26_KERNEL_NAME)
>  endif
>  
>  ifneq ($(strip $(subst ",,$(BR2_LINUX_COPYTO))),)
> @@ -437,9 +437,9 @@
>  	mkdir -p $(TARGET_DIR)/boot
>  	cp -dpf .config $(TARGET_DIR)/boot/busybox.config
>  
> -$(BR2_LINUX_COPYTO)/$(LINUX26_KERNEL_NAME): $(LINUX26_KERNEL)
> -	mkdir -p $(BR2_LINUX_COPYTO)
> -	cp -dpf $(LINUX26_KERNEL) $(BR2_LINUX_COPYTO)/$(LINUX26_KERNEL_NAME)
> +/tftpboot/$(LINUX26_KERNEL_NAME): $(LINUX26_KERNEL)
> +	mkdir -p /tftpboot
> +	cp -dpf $(LINUX26_KERNEL) /tftpboot/$(LINUX26_KERNEL_NAME)
>  
>  $(LINUX_COPYTO)/$(LINUX26_KERNEL_NAME): $(LINUX26_KERNEL)
>  	mkdir -p $(LINUX_COPYTO)
> 
> _______________________________________________
> buildroot mailing list
> buildroot at uclibc.org
> http://busybox.net/mailman/listinfo/buildroot
> 

-- 
Jean-Christian de Rivaz