All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] zlib-ng: new package
@ 2017-11-03  6:34 Stefan Fröberg
  2017-11-04 21:23 ` Marcus Folkesson
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Fröberg @ 2017-11-03  6:34 UTC (permalink / raw)
  To: buildroot

zlib-ng, SIMD optimized zlib

Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
---
 package/Config.in          |  1 +
 package/zlib-ng/Config.in  |  8 ++++++++
 package/zlib-ng/zlib-ng.mk | 15 +++++++++++++++
 3 files changed, 24 insertions(+)
 create mode 100644 package/zlib-ng/Config.in
 create mode 100644 package/zlib-ng/zlib-ng.mk

diff --git a/package/Config.in b/package/Config.in
index dbfb288..3e2b034 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1003,6 +1003,7 @@ menu "Compression and decompression"
 	source "package/snappy/Config.in"
 	source "package/szip/Config.in"
 	source "package/zlib/Config.in"
+	source "package/zlib-ng/Config.in"
 endmenu
 
 menu "Crypto"
diff --git a/package/zlib-ng/Config.in b/package/zlib-ng/Config.in
new file mode 100644
index 0000000..06de97c
--- /dev/null
+++ b/package/zlib-ng/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_ZLIB_NG
+	bool "zlib-ng"
+	help
+	  zlib replacement with optimizations for
+	  "next generation" systems.
+
+	  https://github.com/Dead2/zlib-ng
+
diff --git a/package/zlib-ng/zlib-ng.mk b/package/zlib-ng/zlib-ng.mk
new file mode 100644
index 0000000..eaa93cd
--- /dev/null
+++ b/package/zlib-ng/zlib-ng.mk
@@ -0,0 +1,15 @@
+################################################################################
+#
+# zlib-ng
+#
+################################################################################
+
+ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng.git
+ZLIB_NG_SOURCE = develop.zip
+ZLIB_NG_SUBDIR = zlib-ng-develop
+ZLIB_NG_EXTRACT_CMDS = unzip $(DL_DIR)/$(ZLIB_NG_SOURCE) -d $(ZLIB_NG_DIR)
+ZLIB_NG_LICENSE = Zlib
+ZLIB_NG_LICENSE_FILES = README.md
+ZLIB_NG_INSTALL_STAGING = YES
+
+$(eval $(cmake-package))
-- 
2.7.3

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

* [Buildroot] [PATCH 1/1] zlib-ng: new package
  2017-11-03  6:34 [Buildroot] [PATCH 1/1] zlib-ng: new package Stefan Fröberg
@ 2017-11-04 21:23 ` Marcus Folkesson
  2017-11-05  0:00   ` Stefan Fröberg
  2017-11-05  9:20   ` Arnout Vandecappelle
  0 siblings, 2 replies; 4+ messages in thread
From: Marcus Folkesson @ 2017-11-04 21:23 UTC (permalink / raw)
  To: buildroot

Hi Stefan,

On Fri, Nov 03, 2017 at 06:34:11AM +0000, Stefan Fr?berg wrote:
> zlib-ng, SIMD optimized zlib
> 
> Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
> ---
>  package/Config.in          |  1 +
>  package/zlib-ng/Config.in  |  8 ++++++++
>  package/zlib-ng/zlib-ng.mk | 15 +++++++++++++++
>  3 files changed, 24 insertions(+)
>  create mode 100644 package/zlib-ng/Config.in
>  create mode 100644 package/zlib-ng/zlib-ng.mk
> 
> diff --git a/package/Config.in b/package/Config.in
> index dbfb288..3e2b034 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1003,6 +1003,7 @@ menu "Compression and decompression"
>  	source "package/snappy/Config.in"
>  	source "package/szip/Config.in"
>  	source "package/zlib/Config.in"
> +	source "package/zlib-ng/Config.in"
>  endmenu
>  
>  menu "Crypto"
> diff --git a/package/zlib-ng/Config.in b/package/zlib-ng/Config.in
> new file mode 100644
> index 0000000..06de97c
> --- /dev/null
> +++ b/package/zlib-ng/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_ZLIB_NG
> +	bool "zlib-ng"
> +	help
> +	  zlib replacement with optimizations for
> +	  "next generation" systems.
> +
> +	  https://github.com/Dead2/zlib-ng
> +
> diff --git a/package/zlib-ng/zlib-ng.mk b/package/zlib-ng/zlib-ng.mk
> new file mode 100644
> index 0000000..eaa93cd
> --- /dev/null
> +++ b/package/zlib-ng/zlib-ng.mk
> @@ -0,0 +1,15 @@
> +################################################################################
> +#
> +# zlib-ng
> +#
> +################################################################################
> +
> +ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng.git
> +ZLIB_NG_SOURCE = develop.zip

I guess the intention is to point to
ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng/archive

Otherwise it will try to download
http://github/Dead2/zlib-ng.git/develop.zip
which is incorrect.

However, this will result in a file ./dl/develop.zip, which says nothing
about package or version.

I think it is better to use git:
ZLIB_NG_SITE_METHOD = git

and specify version
ZLIB_NG_VERSION = xxxxx


> +ZLIB_NG_SUBDIR = zlib-ng-develop
> +ZLIB_NG_EXTRACT_CMDS = unzip $(DL_DIR)/$(ZLIB_NG_SOURCE) -d $(ZLIB_NG_DIR)

This is affected by the comment above

> +ZLIB_NG_LICENSE = Zlib
> +ZLIB_NG_LICENSE_FILES = README.md

The license file is LICENSE.md, not README.md.


Best regards
Marcus Folkesson

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

* [Buildroot] [PATCH 1/1] zlib-ng: new package
  2017-11-04 21:23 ` Marcus Folkesson
@ 2017-11-05  0:00   ` Stefan Fröberg
  2017-11-05  9:20   ` Arnout Vandecappelle
  1 sibling, 0 replies; 4+ messages in thread
From: Stefan Fröberg @ 2017-11-05  0:00 UTC (permalink / raw)
  To: buildroot

Hi Marcus

Okay, will do another patch soon.

-S-

4.11.2017, 23:23, Marcus Folkesson kirjoitti:
> Hi Stefan,
>
> On Fri, Nov 03, 2017 at 06:34:11AM +0000, Stefan Fr?berg wrote:
>> zlib-ng, SIMD optimized zlib
>>
>> Signed-off-by: Stefan Fr?berg <stefan.froberg@petroprogram.com>
>> ---
>>   package/Config.in          |  1 +
>>   package/zlib-ng/Config.in  |  8 ++++++++
>>   package/zlib-ng/zlib-ng.mk | 15 +++++++++++++++
>>   3 files changed, 24 insertions(+)
>>   create mode 100644 package/zlib-ng/Config.in
>>   create mode 100644 package/zlib-ng/zlib-ng.mk
>>
>> diff --git a/package/Config.in b/package/Config.in
>> index dbfb288..3e2b034 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -1003,6 +1003,7 @@ menu "Compression and decompression"
>>   	source "package/snappy/Config.in"
>>   	source "package/szip/Config.in"
>>   	source "package/zlib/Config.in"
>> +	source "package/zlib-ng/Config.in"
>>   endmenu
>>   
>>   menu "Crypto"
>> diff --git a/package/zlib-ng/Config.in b/package/zlib-ng/Config.in
>> new file mode 100644
>> index 0000000..06de97c
>> --- /dev/null
>> +++ b/package/zlib-ng/Config.in
>> @@ -0,0 +1,8 @@
>> +config BR2_PACKAGE_ZLIB_NG
>> +	bool "zlib-ng"
>> +	help
>> +	  zlib replacement with optimizations for
>> +	  "next generation" systems.
>> +
>> +	  https://github.com/Dead2/zlib-ng
>> +
>> diff --git a/package/zlib-ng/zlib-ng.mk b/package/zlib-ng/zlib-ng.mk
>> new file mode 100644
>> index 0000000..eaa93cd
>> --- /dev/null
>> +++ b/package/zlib-ng/zlib-ng.mk
>> @@ -0,0 +1,15 @@
>> +################################################################################
>> +#
>> +# zlib-ng
>> +#
>> +################################################################################
>> +
>> +ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng.git
>> +ZLIB_NG_SOURCE = develop.zip
> I guess the intention is to point to
> ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng/archive
>
> Otherwise it will try to download
> http://github/Dead2/zlib-ng.git/develop.zip
> which is incorrect.
>
> However, this will result in a file ./dl/develop.zip, which says nothing
> about package or version.
>
> I think it is better to use git:
> ZLIB_NG_SITE_METHOD = git
>
> and specify version
> ZLIB_NG_VERSION = xxxxx
>
>
>> +ZLIB_NG_SUBDIR = zlib-ng-develop
>> +ZLIB_NG_EXTRACT_CMDS = unzip $(DL_DIR)/$(ZLIB_NG_SOURCE) -d $(ZLIB_NG_DIR)
> This is affected by the comment above
>
>> +ZLIB_NG_LICENSE = Zlib
>> +ZLIB_NG_LICENSE_FILES = README.md
> The license file is LICENSE.md, not README.md.
>
>
> Best regards
> Marcus Folkesson

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

* [Buildroot] [PATCH 1/1] zlib-ng: new package
  2017-11-04 21:23 ` Marcus Folkesson
  2017-11-05  0:00   ` Stefan Fröberg
@ 2017-11-05  9:20   ` Arnout Vandecappelle
  1 sibling, 0 replies; 4+ messages in thread
From: Arnout Vandecappelle @ 2017-11-05  9:20 UTC (permalink / raw)
  To: buildroot



On 04-11-17 22:23, Marcus Folkesson wrote:
> Hi Stefan,
> 
> On Fri, Nov 03, 2017 at 06:34:11AM +0000, Stefan Fr?berg wrote:
[snip]
>> +ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng.git
>> +ZLIB_NG_SOURCE = develop.zip
> 
> I guess the intention is to point to
> ZLIB_NG_SITE = https://github.com/Dead2/zlib-ng/archive
> 
> Otherwise it will try to download
> http://github/Dead2/zlib-ng.git/develop.zip
> which is incorrect.
> 
> However, this will result in a file ./dl/develop.zip, which says nothing
> about package or version.
> 
> I think it is better to use git:
> ZLIB_NG_SITE_METHOD = git

 No, use the github helper [1]. And don't download the zip file, download the
tarball. And add a hash file. Make sure the hash file contains a hash for the
license file as well.


 Regards,
 Arnout

[1] https://buildroot.org/downloads/manual/manual.html#github-download-url

> 
> and specify version
> ZLIB_NG_VERSION = xxxxx
> 
> 
>> +ZLIB_NG_SUBDIR = zlib-ng-develop
>> +ZLIB_NG_EXTRACT_CMDS = unzip $(DL_DIR)/$(ZLIB_NG_SOURCE) -d $(ZLIB_NG_DIR)
> 
> This is affected by the comment above
> 
>> +ZLIB_NG_LICENSE = Zlib
>> +ZLIB_NG_LICENSE_FILES = README.md
> 
> The license file is LICENSE.md, not README.md.
> 
> 
> Best regards
> Marcus Folkesson
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
> 

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

end of thread, other threads:[~2017-11-05  9:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-03  6:34 [Buildroot] [PATCH 1/1] zlib-ng: new package Stefan Fröberg
2017-11-04 21:23 ` Marcus Folkesson
2017-11-05  0:00   ` Stefan Fröberg
2017-11-05  9:20   ` Arnout Vandecappelle

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.