mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR}
@ 2014-01-24 17:59 jon
  2014-01-24 18:09 ` Jon Ringle
  0 siblings, 1 reply; 4+ messages in thread
From: jon @ 2014-01-24 17:59 UTC (permalink / raw)
  To: ptxdist; +Cc: Jon Ringle

From: Jon Ringle <jringle@gridpoint.com>

This patch allows you to setup a dev package mirror to hold *-dev.tar.gz
packages.

Upload your *-dev.tar.gz packages to your mirror, then you can avoid
rebuilding all your packages by doing:

ptxdist getdev

Signed-off-by: Jon Ringle <jringle@gridpoint.com>
---
 bin/ptxdist                         |    5 +++++
 rules/post/ptxd_make_world_get.make |    9 +++++++++
 rules/project-name.in               |    9 +++++++++
 scripts/lib/ptxd_lib_dgen.awk       |    7 +++++++
 scripts/lib/ptxd_make_get.sh        |   15 ++++++++-------
 5 files changed, 38 insertions(+), 7 deletions(-)

diff --git a/bin/ptxdist b/bin/ptxdist
index f9cbdec..663c41e 100755
--- a/bin/ptxdist
+++ b/bin/ptxdist
@@ -1789,6 +1789,11 @@ EOF
 			fi
 			exit
 			;;
+		getdev)
+			check_premake &&
+			ptxd_make_log getdev
+			exit
+			;;
 		go|images)
 			do_${cmd}
 			exit
diff --git a/rules/post/ptxd_make_world_get.make b/rules/post/ptxd_make_world_get.make
index 152674c..b99102f 100644
--- a/rules/post/ptxd_make_world_get.make
+++ b/rules/post/ptxd_make_world_get.make
@@ -26,6 +26,11 @@ $(SRCDIR)/%:
 	@$(call targetinfo)
 	@$(call get, $($@))
 
+ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),)
+$(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz:
+	@$(call targetinfo)
+	@$(call getdev, $@)
+endif
 
 $(STATEDIR)/%.get:
 	@$(call targetinfo)
@@ -59,4 +64,8 @@ get = \
 check_src = \
 	ptxd_make_check_src "$($(strip $(1))_SOURCE)" "$($(strip $(1))_MD5)"
 
+getdev = \
+	ptxd_make_get_nofail=y \
+	ptxd_make_get "$(strip $(1))" "$(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR))/$(notdir $(1))"
+
 # vim: syntax=make
diff --git a/rules/project-name.in b/rules/project-name.in
index f546458..d1cd09a 100644
--- a/rules/project-name.in
+++ b/rules/project-name.in
@@ -180,4 +180,13 @@ config PROJECT_DEVPKGDIR
 	depends on PROJECT_USE_DEVPKGS
 	default "/opt/OSELAS.BSP/${PTXCONF_PROJECT_VENDOR}/OSELAS.BSP-${PTXCONF_PROJECT_VENDOR}-${PTXCONF_PROJECT}${PTXCONF_PROJECT_VERSION}"
 
+config PROJECT_DEVMIRROR
+	string
+	prompt "Mirror for pre-build archives"
+	depends on PROJECT_USE_DEVPKGS
+	help
+	  Enter a URL where *-dev.tar.gz packages can be downloaded
+	  Use: ptxdist getdev
+
+
 endmenu
diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
index 7f790c9..6cd60e2 100644
--- a/scripts/lib/ptxd_lib_dgen.awk
+++ b/scripts/lib/ptxd_lib_dgen.awk
@@ -241,6 +241,13 @@ function write_deps_pkg_all(this_PKG, this_pkg) {
 	# .get rule
 	#
 	print "$(STATEDIR)/" this_pkg ".get: $(" this_PKG "_SOURCES)"	> DGEN_DEPS_POST;
+	print "ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR)),)" > DGEN_DEPS_POST;
+	print "ifneq ($($(" this_PKG "_DEVPKG)),NO)" > DGEN_DEPS_POST;
+	print "ifneq ($(" this_PKG "_CFGHASH),)" > DGEN_DEPS_POST;
+	print "getdev: $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/$(" this_PKG "_DEVPKG)" > DGEN_DEPS_POST;
+	print "endif" > DGEN_DEPS_POST;
+	print "endif" > DGEN_DEPS_POST;
+	print "endif" > DGEN_DEPS_POST;
 }
 
 function write_deps_pkg_active(this_PKG, this_pkg, prefix) {
diff --git a/scripts/lib/ptxd_make_get.sh b/scripts/lib/ptxd_make_get.sh
index f4d7922..1ce913f 100644
--- a/scripts/lib/ptxd_make_get.sh
+++ b/scripts/lib/ptxd_make_get.sh
@@ -285,7 +285,7 @@ ptxd_make_get() {
 			argv[${#argv[@]}]="${url}"
 			mrd=true
 			;;
-		${PTXCONF_SETUP_PTXMIRROR}/*)
+		${PTXCONF_SETUP_PTXMIRROR}/*|${PTXCONF_PROJECT_DEVMIRROR}/*)
 			# if mirror is given us to download, add it, but only once
 			if ! ${mrd}; then
 				argv[${#argv[@]}]="${url}"
@@ -397,11 +397,12 @@ ptxd_make_get() {
 		esac
 	done
 
-	echo
-	echo "Could not download package"
-	echo "URL: ${orig_argv[@]}"
-	echo
-	exit 1
+	if [ "${ptxd_make_get_nofail}" != "y" ]; then
+		echo
+		echo "Could not download package"
+		echo "URL: ${orig_argv[@]}"
+		echo
+		exit 1
+	fi
 }
-
 export -f ptxd_make_get
-- 
1.7.10.4


-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR}
  2014-01-24 17:59 [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR} jon
@ 2014-01-24 18:09 ` Jon Ringle
  2014-01-29 14:27   ` Michael Olbrich
  0 siblings, 1 reply; 4+ messages in thread
From: Jon Ringle @ 2014-01-24 18:09 UTC (permalink / raw)
  To: ptxdist

On Fri, Jan 24, 2014 at 12:59 PM,  <jon@ringle.org> wrote:
> From: Jon Ringle <jringle@gridpoint.com>
>
> This patch allows you to setup a dev package mirror to hold *-dev.tar.gz
> packages.
>
> Upload your *-dev.tar.gz packages to your mirror, then you can avoid
> rebuilding all your packages by doing:
>
> ptxdist getdev
>
> Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> ---
>  bin/ptxdist                         |    5 +++++
>  rules/post/ptxd_make_world_get.make |    9 +++++++++
>  rules/project-name.in               |    9 +++++++++
>  scripts/lib/ptxd_lib_dgen.awk       |    7 +++++++
>  scripts/lib/ptxd_make_get.sh        |   15 ++++++++-------
>  5 files changed, 38 insertions(+), 7 deletions(-)
>
> diff --git a/bin/ptxdist b/bin/ptxdist
> index f9cbdec..663c41e 100755
> --- a/bin/ptxdist
> +++ b/bin/ptxdist
> @@ -1789,6 +1789,11 @@ EOF
>                         fi
>                         exit
>                         ;;
> +               getdev)
> +                       check_premake &&
> +                       ptxd_make_log getdev
> +                       exit
> +                       ;;
>                 go|images)
>                         do_${cmd}
>                         exit
> diff --git a/rules/post/ptxd_make_world_get.make b/rules/post/ptxd_make_world_get.make
> index 152674c..b99102f 100644
> --- a/rules/post/ptxd_make_world_get.make
> +++ b/rules/post/ptxd_make_world_get.make
> @@ -26,6 +26,11 @@ $(SRCDIR)/%:
>         @$(call targetinfo)
>         @$(call get, $($@))
>
> +ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),)
> +$(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz:
> +       @$(call targetinfo)
> +       @$(call getdev, $@)
> +endif
>
>  $(STATEDIR)/%.get:
>         @$(call targetinfo)
> @@ -59,4 +64,8 @@ get = \
>  check_src = \
>         ptxd_make_check_src "$($(strip $(1))_SOURCE)" "$($(strip $(1))_MD5)"
>
> +getdev = \
> +       ptxd_make_get_nofail=y \
> +       ptxd_make_get "$(strip $(1))" "$(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR))/$(notdir $(1))"
> +
>  # vim: syntax=make
> diff --git a/rules/project-name.in b/rules/project-name.in
> index f546458..d1cd09a 100644
> --- a/rules/project-name.in
> +++ b/rules/project-name.in
> @@ -180,4 +180,13 @@ config PROJECT_DEVPKGDIR
>         depends on PROJECT_USE_DEVPKGS
>         default "/opt/OSELAS.BSP/${PTXCONF_PROJECT_VENDOR}/OSELAS.BSP-${PTXCONF_PROJECT_VENDOR}-${PTXCONF_PROJECT}${PTXCONF_PROJECT_VERSION}"
>
> +config PROJECT_DEVMIRROR
> +       string
> +       prompt "Mirror for pre-build archives"
> +       depends on PROJECT_USE_DEVPKGS
> +       help
> +         Enter a URL where *-dev.tar.gz packages can be downloaded
> +         Use: ptxdist getdev
> +
> +
>  endmenu
> diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
> index 7f790c9..6cd60e2 100644
> --- a/scripts/lib/ptxd_lib_dgen.awk
> +++ b/scripts/lib/ptxd_lib_dgen.awk
> @@ -241,6 +241,13 @@ function write_deps_pkg_all(this_PKG, this_pkg) {
>         # .get rule
>         #
>         print "$(STATEDIR)/" this_pkg ".get: $(" this_PKG "_SOURCES)"   > DGEN_DEPS_POST;
> +       print "ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR)),)" > DGEN_DEPS_POST;
> +       print "ifneq ($($(" this_PKG "_DEVPKG)),NO)" > DGEN_DEPS_POST;
> +       print "ifneq ($(" this_PKG "_CFGHASH),)" > DGEN_DEPS_POST;
> +       print "getdev: $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/$(" this_PKG "_DEVPKG)" > DGEN_DEPS_POST;
> +       print "endif" > DGEN_DEPS_POST;
> +       print "endif" > DGEN_DEPS_POST;
> +       print "endif" > DGEN_DEPS_POST;
>  }
>
>  function write_deps_pkg_active(this_PKG, this_pkg, prefix) {
> diff --git a/scripts/lib/ptxd_make_get.sh b/scripts/lib/ptxd_make_get.sh
> index f4d7922..1ce913f 100644
> --- a/scripts/lib/ptxd_make_get.sh
> +++ b/scripts/lib/ptxd_make_get.sh
> @@ -285,7 +285,7 @@ ptxd_make_get() {
>                         argv[${#argv[@]}]="${url}"
>                         mrd=true
>                         ;;
> -               ${PTXCONF_SETUP_PTXMIRROR}/*)
> +               ${PTXCONF_SETUP_PTXMIRROR}/*|${PTXCONF_PROJECT_DEVMIRROR}/*)

I moved this to where you asked me to, but now since mrd is
conditionally set, it is always replacing PTXCONF_SETUP_PTXMIRROR for
the DEVMIRROR. When I had it on the previous case in the switch, it
would not do that. This brings me back to the other issue I'm having
http://article.gmane.org/gmane.comp.embedded.ptxdist.devel/10980

>                         # if mirror is given us to download, add it, but only once
>                         if ! ${mrd}; then
>                                 argv[${#argv[@]}]="${url}"
> @@ -397,11 +397,12 @@ ptxd_make_get() {
>                 esac
>         done
>
> -       echo
> -       echo "Could not download package"
> -       echo "URL: ${orig_argv[@]}"
> -       echo
> -       exit 1
> +       if [ "${ptxd_make_get_nofail}" != "y" ]; then
> +               echo
> +               echo "Could not download package"
> +               echo "URL: ${orig_argv[@]}"
> +               echo
> +               exit 1
> +       fi
>  }
> -
>  export -f ptxd_make_get
> --
> 1.7.10.4
>

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR}
  2014-01-24 18:09 ` Jon Ringle
@ 2014-01-29 14:27   ` Michael Olbrich
  2014-02-01 20:37     ` Jon Ringle
  0 siblings, 1 reply; 4+ messages in thread
From: Michael Olbrich @ 2014-01-29 14:27 UTC (permalink / raw)
  To: ptxdist

On Fri, Jan 24, 2014 at 01:09:13PM -0500, Jon Ringle wrote:
> On Fri, Jan 24, 2014 at 12:59 PM,  <jon@ringle.org> wrote:
> > From: Jon Ringle <jringle@gridpoint.com>
> >
> > This patch allows you to setup a dev package mirror to hold *-dev.tar.gz
> > packages.
> >
> > Upload your *-dev.tar.gz packages to your mirror, then you can avoid
> > rebuilding all your packages by doing:
> >
> > ptxdist getdev
> >
> > Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> > ---
> >  bin/ptxdist                         |    5 +++++
> >  rules/post/ptxd_make_world_get.make |    9 +++++++++
> >  rules/project-name.in               |    9 +++++++++
> >  scripts/lib/ptxd_lib_dgen.awk       |    7 +++++++
> >  scripts/lib/ptxd_make_get.sh        |   15 ++++++++-------
> >  5 files changed, 38 insertions(+), 7 deletions(-)
> >
> > diff --git a/bin/ptxdist b/bin/ptxdist
> > index f9cbdec..663c41e 100755
> > --- a/bin/ptxdist
> > +++ b/bin/ptxdist
> > @@ -1789,6 +1789,11 @@ EOF
> >                         fi
> >                         exit
> >                         ;;
> > +               getdev)
> > +                       check_premake &&
> > +                       ptxd_make_log getdev
> > +                       exit
> > +                       ;;
> >                 go|images)
> >                         do_${cmd}
> >                         exit
> > diff --git a/rules/post/ptxd_make_world_get.make b/rules/post/ptxd_make_world_get.make
> > index 152674c..b99102f 100644
> > --- a/rules/post/ptxd_make_world_get.make
> > +++ b/rules/post/ptxd_make_world_get.make
> > @@ -26,6 +26,11 @@ $(SRCDIR)/%:
> >         @$(call targetinfo)
> >         @$(call get, $($@))
> >
> > +ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),)
> > +$(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz:
> > +       @$(call targetinfo)
> > +       @$(call getdev, $@)
> > +endif
> >
> >  $(STATEDIR)/%.get:
> >         @$(call targetinfo)
> > @@ -59,4 +64,8 @@ get = \
> >  check_src = \
> >         ptxd_make_check_src "$($(strip $(1))_SOURCE)" "$($(strip $(1))_MD5)"
> >
> > +getdev = \
> > +       ptxd_make_get_nofail=y \
> > +       ptxd_make_get "$(strip $(1))" "$(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR))/$(notdir $(1))"
> > +
> >  # vim: syntax=make
> > diff --git a/rules/project-name.in b/rules/project-name.in
> > index f546458..d1cd09a 100644
> > --- a/rules/project-name.in
> > +++ b/rules/project-name.in
> > @@ -180,4 +180,13 @@ config PROJECT_DEVPKGDIR
> >         depends on PROJECT_USE_DEVPKGS
> >         default "/opt/OSELAS.BSP/${PTXCONF_PROJECT_VENDOR}/OSELAS.BSP-${PTXCONF_PROJECT_VENDOR}-${PTXCONF_PROJECT}${PTXCONF_PROJECT_VERSION}"
> >
> > +config PROJECT_DEVMIRROR
> > +       string
> > +       prompt "Mirror for pre-build archives"
> > +       depends on PROJECT_USE_DEVPKGS
> > +       help
> > +         Enter a URL where *-dev.tar.gz packages can be downloaded
> > +         Use: ptxdist getdev
> > +
> > +
> >  endmenu
> > diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
> > index 7f790c9..6cd60e2 100644
> > --- a/scripts/lib/ptxd_lib_dgen.awk
> > +++ b/scripts/lib/ptxd_lib_dgen.awk
> > @@ -241,6 +241,13 @@ function write_deps_pkg_all(this_PKG, this_pkg) {
> >         # .get rule
> >         #
> >         print "$(STATEDIR)/" this_pkg ".get: $(" this_PKG "_SOURCES)"   > DGEN_DEPS_POST;
> > +       print "ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR)),)" > DGEN_DEPS_POST;
> > +       print "ifneq ($($(" this_PKG "_DEVPKG)),NO)" > DGEN_DEPS_POST;
> > +       print "ifneq ($(" this_PKG "_CFGHASH),)" > DGEN_DEPS_POST;
> > +       print "getdev: $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/$(" this_PKG "_DEVPKG)" > DGEN_DEPS_POST;
> > +       print "endif" > DGEN_DEPS_POST;
> > +       print "endif" > DGEN_DEPS_POST;
> > +       print "endif" > DGEN_DEPS_POST;
> >  }
> >
> >  function write_deps_pkg_active(this_PKG, this_pkg, prefix) {
> > diff --git a/scripts/lib/ptxd_make_get.sh b/scripts/lib/ptxd_make_get.sh
> > index f4d7922..1ce913f 100644
> > --- a/scripts/lib/ptxd_make_get.sh
> > +++ b/scripts/lib/ptxd_make_get.sh
> > @@ -285,7 +285,7 @@ ptxd_make_get() {
> >                         argv[${#argv[@]}]="${url}"
> >                         mrd=true
> >                         ;;
> > -               ${PTXCONF_SETUP_PTXMIRROR}/*)
> > +               ${PTXCONF_SETUP_PTXMIRROR}/*|${PTXCONF_PROJECT_DEVMIRROR}/*)
> 
> I moved this to where you asked me to, but now since mrd is
> conditionally set, it is always replacing PTXCONF_SETUP_PTXMIRROR for
> the DEVMIRROR. When I had it on the previous case in the switch, it
> would not do that. This brings me back to the other issue I'm having
> http://article.gmane.org/gmane.comp.embedded.ptxdist.devel/10980

I've reworked the URL handling and applied your patch with a small change
in ptxd_make_world_get.make. And I've implemented the whitelist.
Can you please test if it works as expexted now?

Michael

> >                         # if mirror is given us to download, add it, but only once
> >                         if ! ${mrd}; then
> >                                 argv[${#argv[@]}]="${url}"
> > @@ -397,11 +397,12 @@ ptxd_make_get() {
> >                 esac
> >         done
> >
> > -       echo
> > -       echo "Could not download package"
> > -       echo "URL: ${orig_argv[@]}"
> > -       echo
> > -       exit 1
> > +       if [ "${ptxd_make_get_nofail}" != "y" ]; then
> > +               echo
> > +               echo "Could not download package"
> > +               echo "URL: ${orig_argv[@]}"
> > +               echo
> > +               exit 1
> > +       fi
> >  }
> > -
> >  export -f ptxd_make_get
> > --
> > 1.7.10.4
> >
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
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] 4+ messages in thread

* Re: [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR}
  2014-01-29 14:27   ` Michael Olbrich
@ 2014-02-01 20:37     ` Jon Ringle
  0 siblings, 0 replies; 4+ messages in thread
From: Jon Ringle @ 2014-02-01 20:37 UTC (permalink / raw)
  To: ptxdist

On Wed, Jan 29, 2014 at 9:27 AM, Michael Olbrich
<m.olbrich@pengutronix.de> wrote:
> On Fri, Jan 24, 2014 at 01:09:13PM -0500, Jon Ringle wrote:
>> On Fri, Jan 24, 2014 at 12:59 PM,  <jon@ringle.org> wrote:
>> > From: Jon Ringle <jringle@gridpoint.com>
>> >
>> > This patch allows you to setup a dev package mirror to hold *-dev.tar.gz
>> > packages.
>> >
>> > Upload your *-dev.tar.gz packages to your mirror, then you can avoid
>> > rebuilding all your packages by doing:
>> >
>> > ptxdist getdev
>> >
>> > Signed-off-by: Jon Ringle <jringle@gridpoint.com>
>> > ---
>> >  bin/ptxdist                         |    5 +++++
>> >  rules/post/ptxd_make_world_get.make |    9 +++++++++
>> >  rules/project-name.in               |    9 +++++++++
>> >  scripts/lib/ptxd_lib_dgen.awk       |    7 +++++++
>> >  scripts/lib/ptxd_make_get.sh        |   15 ++++++++-------
>> >  5 files changed, 38 insertions(+), 7 deletions(-)
>> >
>> > diff --git a/bin/ptxdist b/bin/ptxdist
>> > index f9cbdec..663c41e 100755
>> > --- a/bin/ptxdist
>> > +++ b/bin/ptxdist
>> > @@ -1789,6 +1789,11 @@ EOF
>> >                         fi
>> >                         exit
>> >                         ;;
>> > +               getdev)
>> > +                       check_premake &&
>> > +                       ptxd_make_log getdev
>> > +                       exit
>> > +                       ;;
>> >                 go|images)
>> >                         do_${cmd}
>> >                         exit
>> > diff --git a/rules/post/ptxd_make_world_get.make b/rules/post/ptxd_make_world_get.make
>> > index 152674c..b99102f 100644
>> > --- a/rules/post/ptxd_make_world_get.make
>> > +++ b/rules/post/ptxd_make_world_get.make
>> > @@ -26,6 +26,11 @@ $(SRCDIR)/%:
>> >         @$(call targetinfo)
>> >         @$(call get, $($@))
>> >
>> > +ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR)),)
>> > +$(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/%-dev.tar.gz:
>> > +       @$(call targetinfo)
>> > +       @$(call getdev, $@)
>> > +endif
>> >
>> >  $(STATEDIR)/%.get:
>> >         @$(call targetinfo)
>> > @@ -59,4 +64,8 @@ get = \
>> >  check_src = \
>> >         ptxd_make_check_src "$($(strip $(1))_SOURCE)" "$($(strip $(1))_MD5)"
>> >
>> > +getdev = \
>> > +       ptxd_make_get_nofail=y \
>> > +       ptxd_make_get "$(strip $(1))" "$(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR))/$(notdir $(1))"
>> > +
>> >  # vim: syntax=make
>> > diff --git a/rules/project-name.in b/rules/project-name.in
>> > index f546458..d1cd09a 100644
>> > --- a/rules/project-name.in
>> > +++ b/rules/project-name.in
>> > @@ -180,4 +180,13 @@ config PROJECT_DEVPKGDIR
>> >         depends on PROJECT_USE_DEVPKGS
>> >         default "/opt/OSELAS.BSP/${PTXCONF_PROJECT_VENDOR}/OSELAS.BSP-${PTXCONF_PROJECT_VENDOR}-${PTXCONF_PROJECT}${PTXCONF_PROJECT_VERSION}"
>> >
>> > +config PROJECT_DEVMIRROR
>> > +       string
>> > +       prompt "Mirror for pre-build archives"
>> > +       depends on PROJECT_USE_DEVPKGS
>> > +       help
>> > +         Enter a URL where *-dev.tar.gz packages can be downloaded
>> > +         Use: ptxdist getdev
>> > +
>> > +
>> >  endmenu
>> > diff --git a/scripts/lib/ptxd_lib_dgen.awk b/scripts/lib/ptxd_lib_dgen.awk
>> > index 7f790c9..6cd60e2 100644
>> > --- a/scripts/lib/ptxd_lib_dgen.awk
>> > +++ b/scripts/lib/ptxd_lib_dgen.awk
>> > @@ -241,6 +241,13 @@ function write_deps_pkg_all(this_PKG, this_pkg) {
>> >         # .get rule
>> >         #
>> >         print "$(STATEDIR)/" this_pkg ".get: $(" this_PKG "_SOURCES)"   > DGEN_DEPS_POST;
>> > +       print "ifneq ($(call remove_quotes, $(PTXCONF_PROJECT_DEVMIRROR)),)" > DGEN_DEPS_POST;
>> > +       print "ifneq ($($(" this_PKG "_DEVPKG)),NO)" > DGEN_DEPS_POST;
>> > +       print "ifneq ($(" this_PKG "_CFGHASH),)" > DGEN_DEPS_POST;
>> > +       print "getdev: $(call remove_quotes, $(PTXCONF_PROJECT_DEVPKGDIR))/$(" this_PKG "_DEVPKG)" > DGEN_DEPS_POST;
>> > +       print "endif" > DGEN_DEPS_POST;
>> > +       print "endif" > DGEN_DEPS_POST;
>> > +       print "endif" > DGEN_DEPS_POST;
>> >  }
>> >
>> >  function write_deps_pkg_active(this_PKG, this_pkg, prefix) {
>> > diff --git a/scripts/lib/ptxd_make_get.sh b/scripts/lib/ptxd_make_get.sh
>> > index f4d7922..1ce913f 100644
>> > --- a/scripts/lib/ptxd_make_get.sh
>> > +++ b/scripts/lib/ptxd_make_get.sh
>> > @@ -285,7 +285,7 @@ ptxd_make_get() {
>> >                         argv[${#argv[@]}]="${url}"
>> >                         mrd=true
>> >                         ;;
>> > -               ${PTXCONF_SETUP_PTXMIRROR}/*)
>> > +               ${PTXCONF_SETUP_PTXMIRROR}/*|${PTXCONF_PROJECT_DEVMIRROR}/*)
>>
>> I moved this to where you asked me to, but now since mrd is
>> conditionally set, it is always replacing PTXCONF_SETUP_PTXMIRROR for
>> the DEVMIRROR. When I had it on the previous case in the switch, it
>> would not do that. This brings me back to the other issue I'm having
>> http://article.gmane.org/gmane.comp.embedded.ptxdist.devel/10980
>
> I've reworked the URL handling and applied your patch with a small change
> in ptxd_make_world_get.make. And I've implemented the whitelist.
> Can you please test if it works as expexted now?

Yes, the whitelist is working as expected. Thanks!

-- 
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-02-01 20:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-24 17:59 [ptxdist] [PATCHv4] ptxdist getdev: Download dev packages from ${PTXCONF_PROJECT_DEVMIRROR} jon
2014-01-24 18:09 ` Jon Ringle
2014-01-29 14:27   ` Michael Olbrich
2014-02-01 20:37     ` Jon Ringle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox