* [Buildroot] [PATCH] pkg-download: remove explicit PKG_VERSION from github helper
@ 2017-08-28 9:30 Marcin Nowakowski
2017-08-28 20:49 ` Arnout Vandecappelle
0 siblings, 1 reply; 3+ messages in thread
From: Marcin Nowakowski @ 2017-08-28 9:30 UTC (permalink / raw)
To: buildroot
Currently it is always required to add package version as an argument to
the github helper. Since the version is always defined as PKG_VERSION,
drop this argument and generate it atuomatically inside the helper
routine.
To make the transition easier the github helper function is extended to
support both 2 and 3 argument variants (ie. either use the provided
package version argument or automatically substitute with PKG_VERSION if
not available).
Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>
---
docs/manual/adding-packages-tips.txt | 2 +-
package/pkg-download.mk | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/manual/adding-packages-tips.txt b/docs/manual/adding-packages-tips.txt
index 19577fa..30ec768 100644
--- a/docs/manual/adding-packages-tips.txt
+++ b/docs/manual/adding-packages-tips.txt
@@ -138,7 +138,7 @@ past, the 'github' helper function should be used as shown below.
------------------------
# Use a tag or a full commit ID
FOO_VERSION = v1.0
-FOO_SITE = $(call github,<user>,<package>,$(FOO_VERSION))
+FOO_SITE = $(call github,<user>,<package>)
------------------------
.Notes
diff --git a/package/pkg-download.mk b/package/pkg-download.mk
index fbbc2d7..38c8f35 100644
--- a/package/pkg-download.mk
+++ b/package/pkg-download.mk
@@ -52,8 +52,8 @@ notdomain = $(patsubst $(call domain,$(1),$(2))$(call domainseparator,$(2))%,%,$
# default domainseparator is /, specify alternative value as first argument
domainseparator = $(if $(1),$(1),/)
-# github(user,package,version): returns site of GitHub repository
-github = https://github.com/$(1)/$(2)/archive/$(3)
+# github(user,package[,version]): returns site of GitHub repository
+github = https://github.com/$(1)/$(2)/archive/$(if $(3),$(3),$($(call UPPERCASE,$(pkgname))_VERSION))
# Expressly do not check hashes for those files
# Exported variables default to immediately expanded in some versions of
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] pkg-download: remove explicit PKG_VERSION from github helper
2017-08-28 9:30 [Buildroot] [PATCH] pkg-download: remove explicit PKG_VERSION from github helper Marcin Nowakowski
@ 2017-08-28 20:49 ` Arnout Vandecappelle
2017-08-29 7:18 ` Marcin Nowakowski
0 siblings, 1 reply; 3+ messages in thread
From: Arnout Vandecappelle @ 2017-08-28 20:49 UTC (permalink / raw)
To: buildroot
Hi Marcin,
On 28-08-17 11:30, Marcin Nowakowski wrote:
> Currently it is always required to add package version as an argument to
> the github helper. Since the version is always defined as PKG_VERSION,
> drop this argument and generate it atuomatically inside the helper
automatically
> routine.
Good idea!
>
> To make the transition easier the github helper function is extended to
> support both 2 and 3 argument variants (ie. either use the provided
> package version argument or automatically substitute with PKG_VERSION if
> not available).
It's not just to make the transition easier, it is also needed because you can
use the github helper outside the _SITE variable as well. See for example
ci40_defconfig.
>
> Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>
> ---
> docs/manual/adding-packages-tips.txt | 2 +-
> package/pkg-download.mk | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/docs/manual/adding-packages-tips.txt b/docs/manual/adding-packages-tips.txt
> index 19577fa..30ec768 100644
> --- a/docs/manual/adding-packages-tips.txt
> +++ b/docs/manual/adding-packages-tips.txt
> @@ -138,7 +138,7 @@ past, the 'github' helper function should be used as shown below.
> ------------------------
> # Use a tag or a full commit ID
> FOO_VERSION = v1.0
> -FOO_SITE = $(call github,<user>,<package>,$(FOO_VERSION))
> +FOO_SITE = $(call github,<user>,<package>)
> ------------------------
>
> .Notes
> diff --git a/package/pkg-download.mk b/package/pkg-download.mk
> index fbbc2d7..38c8f35 100644
> --- a/package/pkg-download.mk
> +++ b/package/pkg-download.mk
> @@ -52,8 +52,8 @@ notdomain = $(patsubst $(call domain,$(1),$(2))$(call domainseparator,$(2))%,%,$
> # default domainseparator is /, specify alternative value as first argument
> domainseparator = $(if $(1),$(1),/)
>
> -# github(user,package,version): returns site of GitHub repository
> -github = https://github.com/$(1)/$(2)/archive/$(3)
> +# github(user,package[,version]): returns site of GitHub repository
> +github = https://github.com/$(1)/$(2)/archive/$(if $(3),$(3),$($(call UPPERCASE,$(pkgname))_VERSION))
I think the condition can be simplified to $(or $(3),$($(PKG)_VERSION))
We're trying to use $(PKG) as much as possible. It should expand to the
"current" package in any context where the github helper could be used.
Regards,
Arnout
>
> # Expressly do not check hashes for those files
> # Exported variables default to immediately expanded in some versions of
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] pkg-download: remove explicit PKG_VERSION from github helper
2017-08-28 20:49 ` Arnout Vandecappelle
@ 2017-08-29 7:18 ` Marcin Nowakowski
0 siblings, 0 replies; 3+ messages in thread
From: Marcin Nowakowski @ 2017-08-29 7:18 UTC (permalink / raw)
To: buildroot
Hi Arnout,
On 28.08.2017 22:49, Arnout Vandecappelle wrote:
> Hi Marcin,
>
> On 28-08-17 11:30, Marcin Nowakowski wrote:
>> Currently it is always required to add package version as an argument to
>> the github helper. Since the version is always defined as PKG_VERSION,
>> drop this argument and generate it atuomatically inside the helper
> automatically
>> routine.
>
> Good idea!
>
>>
>> To make the transition easier the github helper function is extended to
>> support both 2 and 3 argument variants (ie. either use the provided
>> package version argument or automatically substitute with PKG_VERSION if
>> not available).
>
> It's not just to make the transition easier, it is also needed because you can
> use the github helper outside the _SITE variable as well. See for example
> ci40_defconfig.
I didn't realise it was used there that way. In that case it's even more
useful to do it that way than I initially thought ;)
>>
>> Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>
>> ---
>> docs/manual/adding-packages-tips.txt | 2 +-
>> package/pkg-download.mk | 4 ++--
>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/docs/manual/adding-packages-tips.txt b/docs/manual/adding-packages-tips.txt
>> index 19577fa..30ec768 100644
>> --- a/docs/manual/adding-packages-tips.txt
>> +++ b/docs/manual/adding-packages-tips.txt
>> @@ -138,7 +138,7 @@ past, the 'github' helper function should be used as shown below.
>> ------------------------
>> # Use a tag or a full commit ID
>> FOO_VERSION = v1.0
>> -FOO_SITE = $(call github,<user>,<package>,$(FOO_VERSION))
>> +FOO_SITE = $(call github,<user>,<package>)
>> ------------------------
>>
>> .Notes
>> diff --git a/package/pkg-download.mk b/package/pkg-download.mk
>> index fbbc2d7..38c8f35 100644
>> --- a/package/pkg-download.mk
>> +++ b/package/pkg-download.mk
>> @@ -52,8 +52,8 @@ notdomain = $(patsubst $(call domain,$(1),$(2))$(call domainseparator,$(2))%,%,$
>> # default domainseparator is /, specify alternative value as first argument
>> domainseparator = $(if $(1),$(1),/)
>>
>> -# github(user,package,version): returns site of GitHub repository
>> -github = https://github.com/$(1)/$(2)/archive/$(3)
>> +# github(user,package[,version]): returns site of GitHub repository
>> +github = https://github.com/$(1)/$(2)/archive/$(if $(3),$(3),$($(call UPPERCASE,$(pkgname))_VERSION))
>
> I think the condition can be simplified to $(or $(3),$($(PKG)_VERSION))
> We're trying to use $(PKG) as much as possible. It should expand to the
> "current" package in any context where the github helper could be used.
Unfortunately PKG is not defined in this context:
in pkg-generic:
ifeq ($$(patsubst %/,ERROR,$$($(2)_SITE)),ERROR)
$$(error $(2)_SITE ($$($(2)_SITE)) cannot have a trailing slash)
endif
so I think I'll need to stick to my original solution (but thanks for
the suggestion to use $(or ...) - it does simplify it a little bit.
Marcin
> Regards,
> Arnout
>
>
>>
>> # Expressly do not check hashes for those files
>> # Exported variables default to immediately expanded in some versions of
>>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-29 7:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-28 9:30 [Buildroot] [PATCH] pkg-download: remove explicit PKG_VERSION from github helper Marcin Nowakowski
2017-08-28 20:49 ` Arnout Vandecappelle
2017-08-29 7:18 ` Marcin Nowakowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.