diff mbox

luaposix: Use github call

Message ID 1413131033-26185-1-git-send-email-maxime.hadjinlian@gmail.com
State Accepted
Commit 71787e9d3538efcb561f780529fe395f0ca747dd
Headers show

Commit Message

Maxime Hadjinlian Oct. 12, 2014, 4:23 p.m. UTC
Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
---
 package/luaposix/luaposix.mk | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Thomas Petazzoni Oct. 12, 2014, 4:29 p.m. UTC | #1
Dear Maxime Hadjinlian,

On Sun, 12 Oct 2014 18:23:53 +0200, Maxime Hadjinlian wrote:
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> ---
>  package/luaposix/luaposix.mk | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/package/luaposix/luaposix.mk b/package/luaposix/luaposix.mk
> index 46723a0..13fc266 100644
> --- a/package/luaposix/luaposix.mk
> +++ b/package/luaposix/luaposix.mk
> @@ -5,8 +5,7 @@
>  ################################################################################
>  
>  LUAPOSIX_VERSION = 32
> -LUAPOSIX_SITE = https://github.com/luaposix/luaposix/archive
> -LUAPOSIX_SOURCE = release-v$(LUAPOSIX_VERSION).tar.gz
> +LUAPOSIX_SITE = $(call github,luaposix,luaposix,release-v$(LUAPOSIX_VERSION))
>  LUAPOSIX_LICENSE = MIT
>  LUAPOSIX_LICENSE_FILES = COPYING
>  LUAPOSIX_DEPENDENCIES = luainterpreter host-lua

Why would we want to use git instead of a simple tarball?

Thomas
Maxime Hadjinlian Oct. 12, 2014, 4:32 p.m. UTC | #2
On Sun, Oct 12, 2014 at 6:29 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Maxime Hadjinlian,
>
> On Sun, 12 Oct 2014 18:23:53 +0200, Maxime Hadjinlian wrote:
>> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
>> ---
>>  package/luaposix/luaposix.mk | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/package/luaposix/luaposix.mk b/package/luaposix/luaposix.mk
>> index 46723a0..13fc266 100644
>> --- a/package/luaposix/luaposix.mk
>> +++ b/package/luaposix/luaposix.mk
>> @@ -5,8 +5,7 @@
>>  ################################################################################
>>
>>  LUAPOSIX_VERSION = 32
>> -LUAPOSIX_SITE = https://github.com/luaposix/luaposix/archive
>> -LUAPOSIX_SOURCE = release-v$(LUAPOSIX_VERSION).tar.gz
>> +LUAPOSIX_SITE = $(call github,luaposix,luaposix,release-v$(LUAPOSIX_VERSION))
>>  LUAPOSIX_LICENSE = MIT
>>  LUAPOSIX_LICENSE_FILES = COPYING
>>  LUAPOSIX_DEPENDENCIES = luainterpreter host-lua
>
> Why would we want to use git instead of a simple tarball?
We are using the tarball, they both came from the space place.
Only we are using the github helper call, which shield us from any
change on github part.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
Yann E. MORIN Oct. 25, 2014, 12:11 a.m. UTC | #3
Maxime, All,

On 2014-10-12 18:23 +0200, Maxime Hadjinlian spake thusly:
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/luaposix/luaposix.mk | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/package/luaposix/luaposix.mk b/package/luaposix/luaposix.mk
> index 46723a0..13fc266 100644
> --- a/package/luaposix/luaposix.mk
> +++ b/package/luaposix/luaposix.mk
> @@ -5,8 +5,7 @@
>  ################################################################################
>  
>  LUAPOSIX_VERSION = 32
> -LUAPOSIX_SITE = https://github.com/luaposix/luaposix/archive
> -LUAPOSIX_SOURCE = release-v$(LUAPOSIX_VERSION).tar.gz
> +LUAPOSIX_SITE = $(call github,luaposix,luaposix,release-v$(LUAPOSIX_VERSION))
>  LUAPOSIX_LICENSE = MIT
>  LUAPOSIX_LICENSE_FILES = COPYING
>  LUAPOSIX_DEPENDENCIES = luainterpreter host-lua
> -- 
> 2.1.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Oct. 25, 2014, 12:35 a.m. UTC | #4
>>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian@gmail.com> writes:

 > Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>

Committed, thanks.
diff mbox

Patch

diff --git a/package/luaposix/luaposix.mk b/package/luaposix/luaposix.mk
index 46723a0..13fc266 100644
--- a/package/luaposix/luaposix.mk
+++ b/package/luaposix/luaposix.mk
@@ -5,8 +5,7 @@ 
 ################################################################################
 
 LUAPOSIX_VERSION = 32
-LUAPOSIX_SITE = https://github.com/luaposix/luaposix/archive
-LUAPOSIX_SOURCE = release-v$(LUAPOSIX_VERSION).tar.gz
+LUAPOSIX_SITE = $(call github,luaposix,luaposix,release-v$(LUAPOSIX_VERSION))
 LUAPOSIX_LICENSE = MIT
 LUAPOSIX_LICENSE_FILES = COPYING
 LUAPOSIX_DEPENDENCIES = luainterpreter host-lua