Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Stefan Fröberg" <stefan.froberg@petroprogram.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] zlib-ng: new package
Date: Sun, 05 Nov 2017 02:00:57 +0200	[thread overview]
Message-ID: <59FE54B9.8080400@petroprogram.com> (raw)
In-Reply-To: <20171104212316.GA4491@gmail.com>

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

  reply	other threads:[~2017-11-05  0:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2017-11-05  9:20   ` Arnout Vandecappelle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=59FE54B9.8080400@petroprogram.com \
    --to=stefan.froberg@petroprogram.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox