* [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils
@ 2018-10-10 8:05 Alexander Dahl
2018-10-10 11:04 ` Roland Hieber
0 siblings, 1 reply; 7+ messages in thread
From: Alexander Dahl @ 2018-10-10 8:05 UTC (permalink / raw)
To: distrokit; +Cc: ptxdist
[-- Attachment #1.1: Type: text/plain, Size: 1251 bytes --]
Hei hei,
while I could build current ptxdist-2018.10.0 based DistroKit on one
host, I have problems building the very same BSP (DistroKit @
DistroKit-2018.03.0-30-g78f0dbf) on another, quite similar host
(Debian GNU/Linux 9 (stretch), amd64). There seems to be a problem
with the host-coreutils package (selected by systemd). Note: the coreutils-package itself is
not selected. See:
-----------------------------
target: coreutils-8.29.tar.xz
-----------------------------
error: empty parameter to 'ptxd_make_get'
make: *** [/mnt/trash/src/coreutils-8.29.tar.xz] Error 1
/usr/local/lib/ptxdist-2018.10.0/rules/post/ptxd_make_world_get.make:17: recipe for target '/mnt/trash/src/coreutils-8.29.tar.xz' failed
Calling ptxdist with -v gives no more output. How can I debug this?
The host, where I could build DistroKit has coreutils-8.29.tar.xz
already downloaded, the host where it fails has not.
Greets
Alex
--
/"\ ASCII RIBBON | »With the first link, the chain is forged. The first
\ / CAMPAIGN | speech censured, the first thought forbidden, the
X AGAINST | first freedom denied, chains us all irrevocably.«
/ \ HTML MAIL | (Jean-Luc Picard, quoting Judge Aaron Satie)
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 91 bytes --]
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-10 8:05 [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils Alexander Dahl @ 2018-10-10 11:04 ` Roland Hieber 2018-10-10 12:21 ` Roland Hieber 0 siblings, 1 reply; 7+ messages in thread From: Roland Hieber @ 2018-10-10 11:04 UTC (permalink / raw) To: distrokit, ptxdist Ohai, I set my source download folder to ./src/ and I can reproduce this with various other packages as well (kernel, coreutils, systemd, networkmanager), and even on PTXdist 2018.01.0…?! I added some debug echos to ptxdist/rules/post/ptxd_make_world_get.make: ------------------------------------------------ 8<--------------------- --- a/rules/post/ptxd_make_world_get.make +++ b/rules/post/ptxd_make_world_get.make @@ -15,7 +15,8 @@ # $(SRCDIR)/%: →@$(call targetinfo) - →@$(call get, $($@)) + →echo target '$$(SRCDIR)/%'; echo params: $@; echo $$($$@) = $($@); set -x; \ + →$(call get, $($@)) ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),) $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz: ------------------------------------------------ >8--------------------- With this, I get the following output: $ ptxdist get networkmanager ----------------------------------- target: NetworkManager-1.8.4.tar.xz ----------------------------------- echo target '$(SRCDIR)/%'; echo params: src/NetworkManager-1.8.4.tar.xz; echo $($@) = ; set -x; \ ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "" "" target $(SRCDIR)/% params: src/NetworkManager-1.8.4.tar.xz = + ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src + ptxd_make_get '' '' + local -a argv + local ptxmirror_url + local path= + shift + local -a orig_argv + orig_argv=("${@}") + '[' -z '' ']' + echo + echo 'error: empty parameter to '\''ptxd_make_get'\''' + echo + exit 1 error: empty parameter to 'ptxd_make_get' /ptx/work/dude/WORK_B/rhi/ptxdist/rules/post/ptxd_make_world_get.make:17: recipe for target 'src/NetworkManager-1.8.4.tar.xz' failed make: *** [src/NetworkManager-1.8.4.tar.xz] Error 1 I think that the $($@) expression should result in "NETWORKMANAGER", so ptxd_make_get knows about what package to download. As far as I understand, this should be a generated rule in DistroKit/platform-$name/state/ptxd_dgen_deps.post, but I cannot find it there... I'll try a bisect on ptxdist to find out where it first went wrong. - Roland On Wed, Oct 10, 2018 at 10:05:20AM +0200, Alexander Dahl wrote: > Hei hei, > > while I could build current ptxdist-2018.10.0 based DistroKit on one > host, I have problems building the very same BSP (DistroKit @ > DistroKit-2018.03.0-30-g78f0dbf) on another, quite similar host > (Debian GNU/Linux 9 (stretch), amd64). There seems to be a problem > with the host-coreutils package (selected by systemd). Note: the coreutils-package itself is > not selected. See: > > ----------------------------- > target: coreutils-8.29.tar.xz > ----------------------------- > > > error: empty parameter to 'ptxd_make_get' > > make: *** [/mnt/trash/src/coreutils-8.29.tar.xz] Error 1 > /usr/local/lib/ptxdist-2018.10.0/rules/post/ptxd_make_world_get.make:17: recipe for target '/mnt/trash/src/coreutils-8.29.tar.xz' failed > > Calling ptxdist with -v gives no more output. How can I debug this? > > The host, where I could build DistroKit has coreutils-8.29.tar.xz > already downloaded, the host where it fails has not. > > Greets > Alex > > -- > /"\ ASCII RIBBON | »With the first link, the chain is forged. The first > \ / CAMPAIGN | speech censured, the first thought forbidden, the > X AGAINST | first freedom denied, chains us all irrevocably.« > / \ HTML MAIL | (Jean-Luc Picard, quoting Judge Aaron Satie) > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-10 11:04 ` Roland Hieber @ 2018-10-10 12:21 ` Roland Hieber 2018-10-10 13:22 ` Alexander Dahl 0 siblings, 1 reply; 7+ messages in thread From: Roland Hieber @ 2018-10-10 12:21 UTC (permalink / raw) To: distrokit, ptxdist On Wed, Oct 10, 2018 at 01:04:01PM +0200, Roland Hieber wrote: > Ohai, > > I set my source download folder to ./src/ and I can reproduce this with > various other packages as well (kernel, coreutils, systemd, > networkmanager), and even on PTXdist 2018.01.0…?! > > I added some debug echos to ptxdist/rules/post/ptxd_make_world_get.make: > > ------------------------------------------------ 8<--------------------- > --- a/rules/post/ptxd_make_world_get.make > +++ b/rules/post/ptxd_make_world_get.make > @@ -15,7 +15,8 @@ > # > $(SRCDIR)/%: > →@$(call targetinfo) > - →@$(call get, $($@)) > + →echo target '$$(SRCDIR)/%'; echo params: $@; echo $$($$@) = $($@); set -x; \ > + →$(call get, $($@)) > > ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),) > $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz: > ------------------------------------------------ >8--------------------- > > With this, I get the following output: > > $ ptxdist get networkmanager > > ----------------------------------- > target: NetworkManager-1.8.4.tar.xz > ----------------------------------- > > echo target '$(SRCDIR)/%'; echo params: src/NetworkManager-1.8.4.tar.xz; echo $($@) = ; set -x; \ > ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "" "" > target $(SRCDIR)/% > params: src/NetworkManager-1.8.4.tar.xz Ah, there is the crux in the matter: I set PTXCONF_SETUP_SRCDIR=./src, but the ./ is missing here, because make internally transforms the name. Everything works fine if I set the source directory to /tmp/src. - Roland > = > + ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src > + ptxd_make_get '' '' > + local -a argv > + local ptxmirror_url > + local path= > + shift > + local -a orig_argv > + orig_argv=("${@}") > + '[' -z '' ']' > + echo > + echo 'error: empty parameter to '\''ptxd_make_get'\''' > + echo > + exit 1 > > error: empty parameter to 'ptxd_make_get' > > /ptx/work/dude/WORK_B/rhi/ptxdist/rules/post/ptxd_make_world_get.make:17: recipe for target 'src/NetworkManager-1.8.4.tar.xz' failed > make: *** [src/NetworkManager-1.8.4.tar.xz] Error 1 > > > I think that the $($@) expression should result in "NETWORKMANAGER", so > ptxd_make_get knows about what package to download. As far as I > understand, this should be a generated rule in > DistroKit/platform-$name/state/ptxd_dgen_deps.post, but I cannot find it > there... I'll try a bisect on ptxdist to find out where it first went > wrong. > > - Roland > > > On Wed, Oct 10, 2018 at 10:05:20AM +0200, Alexander Dahl wrote: > > Hei hei, > > > > while I could build current ptxdist-2018.10.0 based DistroKit on one > > host, I have problems building the very same BSP (DistroKit @ > > DistroKit-2018.03.0-30-g78f0dbf) on another, quite similar host > > (Debian GNU/Linux 9 (stretch), amd64). There seems to be a problem > > with the host-coreutils package (selected by systemd). Note: the coreutils-package itself is > > not selected. See: > > > > ----------------------------- > > target: coreutils-8.29.tar.xz > > ----------------------------- > > > > > > error: empty parameter to 'ptxd_make_get' > > > > make: *** [/mnt/trash/src/coreutils-8.29.tar.xz] Error 1 > > /usr/local/lib/ptxdist-2018.10.0/rules/post/ptxd_make_world_get.make:17: recipe for target '/mnt/trash/src/coreutils-8.29.tar.xz' failed > > > > Calling ptxdist with -v gives no more output. How can I debug this? > > > > The host, where I could build DistroKit has coreutils-8.29.tar.xz > > already downloaded, the host where it fails has not. > > > > Greets > > Alex > > > > -- > > /"\ ASCII RIBBON | »With the first link, the chain is forged. The first > > \ / CAMPAIGN | speech censured, the first thought forbidden, the > > X AGAINST | first freedom denied, chains us all irrevocably.« > > / \ HTML MAIL | (Jean-Luc Picard, quoting Judge Aaron Satie) > > > > > _______________________________________________ > > ptxdist mailing list > > ptxdist@pengutronix.de > > > -- > Roland Hieber | r.hieber@pengutronix.de | > Pengutronix e.K. | https://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > _______________________________________________ > ptxdist mailing list > ptxdist@pengutronix.de -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-10 12:21 ` Roland Hieber @ 2018-10-10 13:22 ` Alexander Dahl 2018-10-11 14:28 ` [ptxdist] [DistroKit] " Roland Hieber 0 siblings, 1 reply; 7+ messages in thread From: Alexander Dahl @ 2018-10-10 13:22 UTC (permalink / raw) To: Roland Hieber; +Cc: distrokit, ptxdist [-- Attachment #1.1: Type: text/plain, Size: 3505 bytes --] Hei hei, On Wed, Oct 10, 2018 at 02:21:48PM +0200, Roland Hieber wrote: > On Wed, Oct 10, 2018 at 01:04:01PM +0200, Roland Hieber wrote: > > Ohai, > > > > I set my source download folder to ./src/ and I can reproduce this with > > various other packages as well (kernel, coreutils, systemd, > > networkmanager), and even on PTXdist 2018.01.0…?! I don't think that is the same problem. I set PTXCONF_SETUP_SRCDIR to /mnt/trash/src first, and to the default ${PTXDIST_WORKSPACE}/src then. The problem is reproducible for packages where you build the host, but not the target package, e.g. FOO=n and HOST_FOO=y. For DistroKit master that is the case for acl, coreutils and maybe more. > > I added some debug echos to ptxdist/rules/post/ptxd_make_world_get.make: > > > > ------------------------------------------------ 8<--------------------- > > --- a/rules/post/ptxd_make_world_get.make > > +++ b/rules/post/ptxd_make_world_get.make > > @@ -15,7 +15,8 @@ > > # > > $(SRCDIR)/%: > > →@$(call targetinfo) > > - →@$(call get, $($@)) > > + →echo target '$$(SRCDIR)/%'; echo params: $@; echo $$($$@) = $($@); set -x; \ > > + →$(call get, $($@)) > > > > ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),) > > $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz: > > ------------------------------------------------ >8--------------------- Nice. Let me show you two example outputs from my side, first is from a usual target only package, second triggers the problem: -------------------------- target: glib-2.56.1.tar.xz -------------------------- echo target '$(SRCDIR)/%'; echo params: /home/alex/src/DistroKit/src/glib-2.56.1.tar.xz; echo $($@) = GLIB; set -x; \ ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "/home/alex/src/DistroKit/src/glib-2.56.1.tar.xz" "http://ftp.gnome.o rg/pub/GNOME/sources/glib/2.56/glib-2.56.1.tar.xz" target $(SRCDIR)/% params: /home/alex/src/DistroKit/src/glib-2.56.1.tar.xz = GLIB ----------------------------- target: acl-2.2.52.src.tar.gz ----------------------------- echo target '$(SRCDIR)/%'; echo params: /home/alex/src/DistroKit/src/acl-2.2.52.src.tar.gz; echo $($@) = ; set -x; \ ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "" "" target $(SRCDIR)/% params: /home/alex/src/DistroKit/src/acl-2.2.52.src.tar.gz = > > I think that the $($@) expression should result in "NETWORKMANAGER", so > > ptxd_make_get knows about what package to download. As far as I > > understand, this should be a generated rule in > > DistroKit/platform-$name/state/ptxd_dgen_deps.post, but I cannot find it > > there... I'll try a bisect on ptxdist to find out where it first went > > wrong. If I compare the above output, the same happens here. In the second case ACL is missing while in the first case GLIB is there. I could reproduce this on another build host. So to say it clear again: download of a tarball fails, if the host package is selected, but the normal target package is not. If the tarball is already there, build is successful. Greets Alex -- /"\ ASCII RIBBON | »With the first link, the chain is forged. The first \ / CAMPAIGN | speech censured, the first thought forbidden, the X AGAINST | first freedom denied, chains us all irrevocably.« / \ HTML MAIL | (Jean-Luc Picard, quoting Judge Aaron Satie) [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 91 bytes --] _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [DistroKit] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-10 13:22 ` Alexander Dahl @ 2018-10-11 14:28 ` Roland Hieber 2018-10-11 20:05 ` Roland Hieber 0 siblings, 1 reply; 7+ messages in thread From: Roland Hieber @ 2018-10-11 14:28 UTC (permalink / raw) To: distrokit, ptxdist On Wed, Oct 10, 2018 at 03:22:13PM +0200, Alexander Dahl wrote: > Hei hei, > > On Wed, Oct 10, 2018 at 02:21:48PM +0200, Roland Hieber wrote: > > On Wed, Oct 10, 2018 at 01:04:01PM +0200, Roland Hieber wrote: > > > Ohai, > > > > > > I set my source download folder to ./src/ and I can reproduce this with > > > various other packages as well (kernel, coreutils, systemd, > > > networkmanager), and even on PTXdist 2018.01.0…?! > > I don't think that is the same problem. I set PTXCONF_SETUP_SRCDIR to > /mnt/trash/src first, and to the default ${PTXDIST_WORKSPACE}/src > then. The problem is reproducible for packages where you build the > host, but not the target package, e.g. FOO=n and HOST_FOO=y. For > DistroKit master that is the case for acl, coreutils and maybe more. Indeed, I can also reproduce this with ${PTXDIST_WORKSPACE}/src. I guess HOST_* packages are missing the mapping from the HOST_*_SOURCE to the *_SOURCE variables, but I cannot see right now where and when this broke. I'll try a bisect later today. - Roland > > > > I added some debug echos to ptxdist/rules/post/ptxd_make_world_get.make: > > > > > > ------------------------------------------------ 8<--------------------- > > > --- a/rules/post/ptxd_make_world_get.make > > > +++ b/rules/post/ptxd_make_world_get.make > > > @@ -15,7 +15,8 @@ > > > # > > > $(SRCDIR)/%: > > > →@$(call targetinfo) > > > - →@$(call get, $($@)) > > > + →echo target '$$(SRCDIR)/%'; echo params: $@; echo $$($$@) = $($@); set -x; \ > > > + →$(call get, $($@)) > > > > > > ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),) > > > $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz: > > > ------------------------------------------------ >8--------------------- > > Nice. Let me show you two example outputs from my side, first is from > a usual target only package, second triggers the problem: > > -------------------------- > target: glib-2.56.1.tar.xz > -------------------------- > > echo target '$(SRCDIR)/%'; echo params: /home/alex/src/DistroKit/src/glib-2.56.1.tar.xz; echo $($@) = GLIB; set -x; \ > ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "/home/alex/src/DistroKit/src/glib-2.56.1.tar.xz" "http://ftp.gnome.o > rg/pub/GNOME/sources/glib/2.56/glib-2.56.1.tar.xz" > target $(SRCDIR)/% > params: /home/alex/src/DistroKit/src/glib-2.56.1.tar.xz > = GLIB > > > ----------------------------- > target: acl-2.2.52.src.tar.gz > ----------------------------- > > echo target '$(SRCDIR)/%'; echo params: /home/alex/src/DistroKit/src/acl-2.2.52.src.tar.gz; echo $($@) = ; set -x; \ > ptxd_make_get_mirror=http://www.pengutronix.de/software/ptxdist/temporary-src ptxd_make_get "" "" > target $(SRCDIR)/% > params: /home/alex/src/DistroKit/src/acl-2.2.52.src.tar.gz > = > > > > I think that the $($@) expression should result in "NETWORKMANAGER", so > > > ptxd_make_get knows about what package to download. As far as I > > > understand, this should be a generated rule in > > > DistroKit/platform-$name/state/ptxd_dgen_deps.post, but I cannot find it > > > there... I'll try a bisect on ptxdist to find out where it first went > > > wrong. > > If I compare the above output, the same happens here. In the second > case ACL is missing while in the first case GLIB is there. > > I could reproduce this on another build host. > > So to say it clear again: download of a tarball fails, if the host > package is selected, but the normal target package is not. If the > tarball is already there, build is successful. > > Greets > Alex > > -- > /"\ ASCII RIBBON | »With the first link, the chain is forged. The first > \ / CAMPAIGN | speech censured, the first thought forbidden, the > X AGAINST | first freedom denied, chains us all irrevocably.« > / \ HTML MAIL | (Jean-Luc Picard, quoting Judge Aaron Satie) > _______________________________________________ > DistroKit mailing list > DistroKit@pengutronix.de -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [DistroKit] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-11 14:28 ` [ptxdist] [DistroKit] " Roland Hieber @ 2018-10-11 20:05 ` Roland Hieber 2018-10-12 11:59 ` Michael Olbrich 0 siblings, 1 reply; 7+ messages in thread From: Roland Hieber @ 2018-10-11 20:05 UTC (permalink / raw) To: distrokit, ptxdist, Michael Olbrich On Thu, Oct 11, 2018 at 04:28:43PM +0200, Roland Hieber wrote: > On Wed, Oct 10, 2018 at 03:22:13PM +0200, Alexander Dahl wrote: > > On Wed, Oct 10, 2018 at 02:21:48PM +0200, Roland Hieber wrote: > > > On Wed, Oct 10, 2018 at 01:04:01PM +0200, Roland Hieber wrote: > > [...] I set PTXCONF_SETUP_SRCDIR to > > /mnt/trash/src first, and to the default ${PTXDIST_WORKSPACE}/src > > then. The problem is reproducible for packages where you build the > > host, but not the target package, e.g. FOO=n and HOST_FOO=y. For > > DistroKit master that is the case for acl, coreutils and maybe more. > > Indeed, I can also reproduce this with ${PTXDIST_WORKSPACE}/src. I guess > HOST_* packages are missing the mapping from the HOST_*_SOURCE to the > *_SOURCE variables, but I cannot see right now where and when this > broke. I'll try a bisect later today. I did a git bisect run with a script that deletes $SRCDIR/coreutils* followed by ptxdist get host-coreutils. It results in commit 400bee10 [a] as the first commit which no longer downloads coreutils*tar.gz and only prints "error: empty parameter to 'ptxd_make_get'". That commit unfortunately landed in ptxdist-2018.10.0. I'm not so versed in the inner workings of ptxd_lib_dgen.awk - mol, do you have an idea what needs to be done here? It looks to me that if we don't want to reintroduce the $(<PKG>_SOURCE) := <PKG> variables, we need to teach the '$(SRCDIR)/%' target [b] how to infer the HOST_<PKG> prefix from the source archive name, even if <pkg> is not selected. But I have no idea how to do that. - Roland [b]: see https://git.pengutronix.de/cgit/ptxdist/tree/rules/post/ptxd_make_world_get.make#n16 [a]: this patch: commit 400bee10bdb3f6ab29304365a19ce05929f09436 Author: Michael Olbrich <m.olbrich@pengutronix.de> Date: 2018-09-28 19:02 ptxd_lib_dgen: make sure $(<PKG>_SOURCE) refers to the primary package To get the package for a source archive we define for each package: $(<PKG>_SOURCE) := <PKG> With host and cross packages that share the source with the target package, the same variable may be defined multiple times. Which one 'wins' is undefined. Avoid this by only defining the variable for the target package. Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de> diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk index 888d989ee..5437d497c 100644 --- a/scripts/lib/ptxd_lib_dgen.awk +++ b/scripts/lib/ptxd_lib_dgen.awk @@ -338,7 +338,13 @@ function write_deps_pkg_active(this_PKG, this_pkg, prefix) { # # default deps # + if (prefix != "") { + target_PKG = gensub(/^HOST_|^CROSS_/, "", 1, this_PKG); + print "ifneq ($(" this_PKG "_SOURCE),$(" target_PKG "_SOURCE))" > DGEN_DEPS_POST; + } print "$(if $(" this_PKG "_SOURCE),$(eval $(" this_PKG "_SOURCE) := " this_PKG "))" > DGEN_DEPS_POST; + if (prefix != "") + print "endif" > DGEN_DEPS_POST; print "$(foreach src,$(" this_PKG "_SOURCES)," \ "$(eval $(STATEDIR)/" this_pkg ".get:" "$(STATEDIR)/" this_pkg ".$(notdir $(src)).stamp))" > DGEN_DEPS_POST; print "$(STATEDIR)/" this_pkg ".extract: " "$(STATEDIR)/" this_pkg ".get" > DGEN_DEPS_POST; -- Roland Hieber | r.hieber@pengutronix.de | Pengutronix e.K. | https://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim | Phone: +49-5121-206917-5086 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [ptxdist] [DistroKit] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils 2018-10-11 20:05 ` Roland Hieber @ 2018-10-12 11:59 ` Michael Olbrich 0 siblings, 0 replies; 7+ messages in thread From: Michael Olbrich @ 2018-10-12 11:59 UTC (permalink / raw) To: ptxdist, distrokit On Thu, Oct 11, 2018 at 10:05:56PM +0200, Roland Hieber wrote: > On Thu, Oct 11, 2018 at 04:28:43PM +0200, Roland Hieber wrote: > > On Wed, Oct 10, 2018 at 03:22:13PM +0200, Alexander Dahl wrote: > > > On Wed, Oct 10, 2018 at 02:21:48PM +0200, Roland Hieber wrote: > > > > On Wed, Oct 10, 2018 at 01:04:01PM +0200, Roland Hieber wrote: > > > [...] I set PTXCONF_SETUP_SRCDIR to > > > /mnt/trash/src first, and to the default ${PTXDIST_WORKSPACE}/src > > > then. The problem is reproducible for packages where you build the > > > host, but not the target package, e.g. FOO=n and HOST_FOO=y. For > > > DistroKit master that is the case for acl, coreutils and maybe more. > > > > Indeed, I can also reproduce this with ${PTXDIST_WORKSPACE}/src. I guess > > HOST_* packages are missing the mapping from the HOST_*_SOURCE to the > > *_SOURCE variables, but I cannot see right now where and when this > > broke. I'll try a bisect later today. > > I did a git bisect run with a script that deletes $SRCDIR/coreutils* > followed by ptxdist get host-coreutils. It results in commit 400bee10 [a] > as the first commit which no longer downloads coreutils*tar.gz and only > prints "error: empty parameter to 'ptxd_make_get'". That commit > unfortunately landed in ptxdist-2018.10.0. > > I'm not so versed in the inner workings of ptxd_lib_dgen.awk - mol, do > you have an idea what needs to be done here? It looks to me that if we > don't want to reintroduce the $(<PKG>_SOURCE) := <PKG> variables, we need to > teach the '$(SRCDIR)/%' target [b] how to infer the HOST_<PKG> prefix > from the source archive name, even if <pkg> is not selected. But I have > no idea how to do that. The problem is, that '$(<PKG>_SOURCE) := <PKG>' is not set for the target package because it's disabled. And it's not set for the host package because they share the source (that condition was introduced in the commit you found). The solution is to define '$(<PKG>_SOURCE) := <PKG>' for the host package if the target package is disabled. This should be fixed in master. Michael -- Pengutronix 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-10-12 11:59 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-10-10 8:05 [ptxdist] Recent DistroKit with ptxdist 2018.10.0 fails to get host-coreutils Alexander Dahl 2018-10-10 11:04 ` Roland Hieber 2018-10-10 12:21 ` Roland Hieber 2018-10-10 13:22 ` Alexander Dahl 2018-10-11 14:28 ` [ptxdist] [DistroKit] " Roland Hieber 2018-10-11 20:05 ` Roland Hieber 2018-10-12 11:59 ` Michael Olbrich
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox