[Buildroot] [PATCH 3/4] setserial: convert to autotargets and fix bug 2269

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Jul 27 08:35:06 UTC 2010


On Tue, 27 Jul 2010 00:50:58 +0200
Peter Korsgaard <jacmet at uclibc.org> wrote:

> >>>>> "Thomas" == Thomas Petazzoni
> >>>>> <thomas.petazzoni at free-electrons.com> writes:
> 
>  Thomas> +Index: setserial-2.17/Makefile.in
>  Thomas> +===================================================================
>  Thomas> +--- setserial-2.17.orig/Makefile.in
>  Thomas> ++++ setserial-2.17/Makefile.in
>  Thomas> +@@ -26,9 +26,11 @@
>  Thomas> + 	nroff -man setserial.8 > setserial.cat
>  Thomas> + 
>  Thomas> + install: setserial setserial.8
>  Thomas> +-	$(INSTALL_PROGRAM) setserial $(DESTDIR)/bin
>  Thomas> ++	mkdir -p $(DESTDIR)/usr/bin
>  Thomas> ++	$(INSTALL_PROGRAM) setserial $(DESTDIR)/usr/bin
>  Thomas> + 	$(STRIP) $(DESTDIR)/bin/setserial
> 
> That strip should go.

Well, it was here in the setserial Makefile. While I agree that we
shouldn't explicitly add strip commands in Buildroot .mk files, I don't
think it's worth the effort removing those strip commands from within
the package.

> You're also not renaming setserial-tiocghayesesp-build-fix.patch, so
> we're back with 5a2844c748.

Right, sorry.

Thanks for having fixed those issues.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com



More information about the buildroot mailing list