From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0] ident=Debian-exim) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1Xduid-0002hO-St for ptxdist@pengutronix.de; Tue, 14 Oct 2014 07:32:23 +0200 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.80) (envelope-from ) id 1Xduid-0005mN-Nk for ptxdist@pengutronix.de; Tue, 14 Oct 2014 07:32:23 +0200 Date: Tue, 14 Oct 2014 07:32:23 +0200 From: Michael Olbrich Message-ID: <20141014053223.GA3143@pengutronix.de> References: <1412649737-28949-1-git-send-email-jon@ringle.org> <1412649737-28949-2-git-send-email-jon@ringle.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: Subject: Re: [ptxdist] [PATCH 2/2] host-coreutils: build touch Reply-To: ptxdist@pengutronix.de List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist@pengutronix.de On Tue, Oct 14, 2014 at 01:23:36AM -0400, Jon Ringle wrote: > I've found that the below patch isn't good enough. The HOST_COREUTILS > only gets installed if systemd is selected. But I need the 'touch' > provided by HOST_COREUTILS to get installed before any targetinstall > rule gets run. I'm not sure how to make this happen though... Indeed. '--no-dereference' for touch was introduced in 2009. That's rather old. Maybe we should just check and only use it, if it's available. The original commit was made to minimize the differences between two builds of a BSP. Having a few links with changing timestaps is not too bad. But I think we should add 'touch' to coreutils checks in configure.ac. Michael > On Mon, Oct 6, 2014 at 10:42 PM, wrote: > > From: Jon Ringle > > > > As of commit ec2b023f4b5207eec76f711d8d8e5fe2854fee6c, ptxdist now depends > > upon touch having --no-dereference, but older versions of touch do not have > > this option, resulting int the following error message: > > > > touch: unrecognized option '--no-dereference' > > Try `touch --help' for more information. > > > > So, we will install one that does support --no-dereference > > > > Signed-off-by: Jon Ringle > > --- > > rules/host-coreutils.make | 6 +++--- > > rules/post/virtual.make | 4 ++++ > > 2 files changed, 7 insertions(+), 3 deletions(-) > > > > diff --git a/rules/host-coreutils.make b/rules/host-coreutils.make > > index f6fe941..2bcd8a0 100644 > > --- a/rules/host-coreutils.make > > +++ b/rules/host-coreutils.make > > @@ -25,7 +25,7 @@ HOST_COREUTILS_SKIP := \ > > nohup nproc numfmt od paste pathchk pinky pr printenv printf ptx \ > > pwd readlink realpath rm rmdir runcon seq sha1sum sha224sum \ > > sha256sum sha384sum sha512sum shred shuf sleep sort split stat \ > > - stdbuf stty sum sync tac tail tee test [ timeout touch tr true \ > > + stdbuf stty sum sync tac tail tee test [ timeout tr true \ > > truncate tsort tty uname unexpand uniq unlink uptime users vdir \ > > wc who whoami yes > > > > @@ -42,7 +42,7 @@ HOST_COREUTILS_CONF_OPT := \ > > --disable-libsmack \ > > --disable-xattr \ > > --disable-libcap \ > > - --enable-install-program=ln \ > > + --enable-install-program=ln,touch \ > > --enable-no-install-program=$(subst $(space),$(comma),$(strip $(HOST_COREUTILS_SKIP))) \ > > --disable-nls \ > > --without-openssl \ > > @@ -55,7 +55,7 @@ HOST_COREUTILS_CONF_OPT := \ > > $(STATEDIR)/host-coreutils.install: > > @$(call targetinfo) > > @$(call world/install, HOST_COREUTILS) > > - @find $(HOST_COREUTILS_PKGDIR) -type f -executable ! -name ln | xargs rm > > + @find $(HOST_COREUTILS_PKGDIR) -type f -executable ! \( -name ln -o -name touch \) | xargs rm > > @$(call touch) > > > > # vim: syntax=make > > diff --git a/rules/post/virtual.make b/rules/post/virtual.make > > index 2d5a2b5..5bc7232 100644 > > --- a/rules/post/virtual.make > > +++ b/rules/post/virtual.make > > @@ -41,6 +41,10 @@ ifdef PTXCONF_HOST_CHRPATH > > $(STATEDIR)/virtual-host-tools.install: $(STATEDIR)/host-chrpath.install.post > > endif > > > > +ifdef PTXCONF_HOST_COREUTILS > > +$(STATEDIR)/virtual-host-tools.install: $(STATEDIR)/host-coreutils.install.post > > +endif > > + > > $(STATEDIR)/virtual-host-tools.install: > > @$(call targetinfo) > > @$(call touch) > > -- > > 1.8.5.4 > > > > -- > ptxdist mailing list > ptxdist@pengutronix.de > -- engutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- ptxdist mailing list ptxdist@pengutronix.de