From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Fri, 28 Jan 2022 11:36:19 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nDOc7-00FsCU-89 for lore@lore.pengutronix.de; Fri, 28 Jan 2022 11:36:19 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1nDOc5-000413-Tu; Fri, 28 Jan 2022 11:36:17 +0100 Received: from ptx.hi.pengutronix.de ([2001:67c:670:100:1d::c0]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nDObz-00040W-VQ; Fri, 28 Jan 2022 11:36:11 +0100 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.92) (envelope-from ) id 1nDObz-0006KU-Kw; Fri, 28 Jan 2022 11:36:11 +0100 Date: Fri, 28 Jan 2022 11:36:11 +0100 From: Michael Olbrich To: Christian Melki Message-ID: <20220128103611.GX11273@pengutronix.de> Mail-Followup-To: Christian Melki , ptxdist@pengutronix.de References: <20220127143532.2212796-1-christian.melki@t2data.com> <20220127143532.2212796-10-christian.melki@t2data.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 11:31:02 up 48 days, 19:16, 80 users, load average: 0.18, 0.12, 0.12 User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [ptxdist] [WIP: PATCH] pps-tools: Version bump. 1.0.2 -> 1.0.3 X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false Hi On Fri, Jan 28, 2022 at 10:39:25AM +0100, Christian Melki wrote: > The question revolves around the authors use of the SYSROOT variable. > Ptxdist declares it and this is taken up as a --sysroot variable to the > toolchain, offsetting all headers etc to the ptxdist sysroot. > Which means standard headers etc are lost. > > So the question is really. Is SYSROOT a reserved variable? > I can't really make that out. Or is the authors assumption that SYSROOT > means what it means wrong? I don't think SYSROOT has a defined meaning. Some packages use it to look for stuff relative to this directory. But even that is inconsistent. And in PTXdist the whole thing is more complex: There is the sysroot in the toolchain. The toolchain headers and libraries are found there. And there is the sysroot in the BSP. So if an upstream Makefile uses SYSROOT in some way, then we just need to deal with it. Michael > On 1/28/22 10:26 AM, Michael Olbrich wrote: > > > > What's WIP here. > > > > On Thu, Jan 27, 2022 at 03:35:29PM +0100, Christian Melki wrote: > > > Update pps-tools. Very minor bugfix release. > > > Remove old patch as that one got fixed upstream. > > > Add a patch to remove inclusion of SYSROOT. > > > > > > Signed-off-by: Christian Melki > > > --- > > > ...le-fix-installation-to-empty-DESTDIR.patch | 40 ------------------- > > > patches/pps-tools-1.0.2/series | 4 -- > > > .../0001-Makefile-fix-remove-SYSROOT.patch | 14 +++++++ > > > patches/pps-tools-1.0.3/series | 1 + > > > rules/pps-tools.make | 4 +- > > > 5 files changed, 17 insertions(+), 46 deletions(-) > > > delete mode 100644 patches/pps-tools-1.0.2/0001-Makefile-fix-installation-to-empty-DESTDIR.patch > > > delete mode 100644 patches/pps-tools-1.0.2/series > > > create mode 100644 patches/pps-tools-1.0.3/0001-Makefile-fix-remove-SYSROOT.patch > > > create mode 100644 patches/pps-tools-1.0.3/series > > > > > > diff --git a/patches/pps-tools-1.0.2/0001-Makefile-fix-installation-to-empty-DESTDIR.patch b/patches/pps-tools-1.0.2/0001-Makefile-fix-installation-to-empty-DESTDIR.patch > > > deleted file mode 100644 > > > index 97498dc22..000000000 > > > --- a/patches/pps-tools-1.0.2/0001-Makefile-fix-installation-to-empty-DESTDIR.patch > > > +++ /dev/null > > > @@ -1,40 +0,0 @@ > > > -From: Thomas Petazzoni > > > -Date: Mon, 3 Dec 2018 13:55:02 +0100 > > > -Subject: [PATCH] Makefile: fix installation to empty DESTDIR > > > - > > > -When DESTDIR is empty, or at least does not contain usr/bin or > > > -usr/include, the installation fails, because install does not create > > > -those intermediate directories: > > > - > > > -$ make DESTDIR=/tmp/koin install > > > -install -m 755 -t /tmp/koin/usr/bin ppsfind ppstest ppsctl ppswatch ppsldisc > > > -install: failed to access '/tmp/koin/usr/bin': No such file or directory > > > - > > > -Using the -D option of install fixes this: > > > - > > > -$ make DESTDIR=/tmp/koin install > > > -install -D -m 755 -t /tmp/koin/usr/bin ppsfind ppstest ppsctl ppswatch ppsldisc > > > -install -D -m 644 -t /tmp/koin/usr/include/sys timepps.h > > > - > > > -Signed-off-by: Thomas Petazzoni > > > -Origin: upstream; https://github.com/redlab-i/pps-tools/commit/b3eae485a8c759d1ce1727076b2c287deb5f24e1 > > > -Signed-off-by: Roland Hieber > > > ---- > > > - Makefile | 4 ++-- > > > - 1 file changed, 2 insertions(+), 2 deletions(-) > > > - > > > -diff --git a/Makefile b/Makefile > > > -index 939466885e9f..30672f79585c 100644 > > > ---- a/Makefile > > > -+++ b/Makefile > > > -@@ -19,8 +19,8 @@ include .depend > > > - endif > > > - > > > - install : all > > > -- install -m 755 -t $(DESTDIR)/usr/bin ppsfind $(TARGETS) > > > -- install -m 644 -t $(DESTDIR)/usr/include/sys timepps.h > > > -+ install -D -m 755 -t $(DESTDIR)/usr/bin ppsfind $(TARGETS) > > > -+ install -D -m 644 -t $(DESTDIR)/usr/include/sys timepps.h > > > - > > > - uninstall : > > > - for f in $(TARGETS); do rm $(DESTDIR)/usr/bin/$$f; done > > > diff --git a/patches/pps-tools-1.0.2/series b/patches/pps-tools-1.0.2/series > > > deleted file mode 100644 > > > index 7b3c77623..000000000 > > > --- a/patches/pps-tools-1.0.2/series > > > +++ /dev/null > > > @@ -1,4 +0,0 @@ > > > -# generated by git-ptx-patches > > > -#tag:base --start-number 1 > > > -0001-Makefile-fix-installation-to-empty-DESTDIR.patch > > > -# eb24c32ce10c0f011f2d37488c8de58c - git-ptx-patches magic > > > diff --git a/patches/pps-tools-1.0.3/0001-Makefile-fix-remove-SYSROOT.patch b/patches/pps-tools-1.0.3/0001-Makefile-fix-remove-SYSROOT.patch > > > > Path header if you keep the patch. > > > > > new file mode 100644 > > > index 000000000..7b160ef9a > > > --- /dev/null > > > +++ b/patches/pps-tools-1.0.3/0001-Makefile-fix-remove-SYSROOT.patch > > > @@ -0,0 +1,14 @@ > > > +diff -urN pps-tools-1.0.3.orig/Makefile pps-tools-1.0.3/Makefile > > > +--- pps-tools-1.0.3.orig/Makefile 2022-01-27 11:20:18.590907158 +0100 > > > ++++ pps-tools-1.0.3/Makefile 2022-01-27 11:20:35.771295930 +0100 > > > +@@ -3,10 +3,6 @@ > > > + CFLAGS += -Wall -O2 -D_GNU_SOURCE > > > + CFLAGS += -ggdb > > > + CFLAGS += -fPIC > > > +-ifdef SYSROOT > > > +-CFLAGS += --sysroot $(SYSROOT) > > > +-endif > > > +- > > > > I think adding 'SYSROOT=' to PPS_TOOLS_MAKE_ENV should work instead of > > this. > > > > Michael > > > > > + LDLIBS += -lm > > > + > > > + # -- Actions section -- > > > diff --git a/patches/pps-tools-1.0.3/series b/patches/pps-tools-1.0.3/series > > > new file mode 100644 > > > index 000000000..23495d2ef > > > --- /dev/null > > > +++ b/patches/pps-tools-1.0.3/series > > > @@ -0,0 +1 @@ > > > +0001-Makefile-fix-remove-SYSROOT.patch > > > diff --git a/rules/pps-tools.make b/rules/pps-tools.make > > > index 65a06c7d1..789659154 100644 > > > --- a/rules/pps-tools.make > > > +++ b/rules/pps-tools.make > > > @@ -15,8 +15,8 @@ PACKAGES-$(PTXCONF_PPS_TOOLS) += pps-tools > > > # > > > # Paths and names > > > # > > > -PPS_TOOLS_VERSION := 1.0.2 > > > -PPS_TOOLS_MD5 := 6eddfeb151cffe8751f4766e428b8eb7 > > > +PPS_TOOLS_VERSION := 1.0.3 > > > +PPS_TOOLS_MD5 := 9b18c55efe020d02c26cd8c759ac258d > > > PPS_TOOLS := pps-tools-$(PPS_TOOLS_VERSION) > > > PPS_TOOLS_SUFFIX := tar.gz > > > PPS_TOOLS_URL := https://github.com/redlab-i/pps-tools/archive/v$(PPS_TOOLS_VERSION).$(PPS_TOOLS_SUFFIX) > > > -- > > > 2.30.2 > > > > > > > > > _______________________________________________ > > > ptxdist mailing list > > > ptxdist@pengutronix.de > > > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de > > > > > > > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de