Date: Mon, 25 Jan 2016 13:44:40 +0100
From: Natanael Copa <ncopa@alpinelinux.org>
To: Valery Kartel <valery.kartel@gmail.com>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [alpine-aports] [PATCH] main/curl: strip to subpackages and
 cleanups
Message-ID: <20160125134440.213d7b0c@ncopa-desktop.alpinelinux.org>
In-Reply-To: <1453723230-31958-1-git-send-email-valery.kartel@gmail.com>
References: <1453723230-31958-1-git-send-email-valery.kartel@gmail.com>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Mon, 25 Jan 2016 14:00:30 +0200
Valery Kartel <valery.kartel@gmail.com> wrote:

> - remove unused patch
> - move libcurl and zsh completion to subpackages
> ---
>  ...-generate-code-for-disable-manual-as-well.patch | 42 ----------------=
------
>  main/curl/APKBUILD                                 | 40 +++++++++++-----=
-----
>  2 files changed, 22 insertions(+), 60 deletions(-)
>  delete mode 100644 main/curl/0001-mkhelp-generate-code-for-disable-manua=
l-as-well.patch
>=20
> diff --git a/main/curl/0001-mkhelp-generate-code-for-disable-manual-as-we=
ll.patch b/main/curl/0001-mkhelp-generate-code-for-disable-manual-as-well.p=
atch
> deleted file mode 100644
> index d23b484..0000000
> --- a/main/curl/0001-mkhelp-generate-code-for-disable-manual-as-well.patch
> +++ /dev/null
> @@ -1,42 +0,0 @@
> -From 38d582ff541353d738858299d4a2b78bafac03ed Mon Sep 17 00:00:00 2001
> -From: Daniel Stenberg <daniel@haxx.se>
> -Date: Wed, 26 Mar 2014 13:22:10 +0100
> -Subject: [PATCH] mkhelp: generate code for --disable-manual as well
> -
> -This allows configure --disable-manual to run and build without having
> -to regenerate the src/tool_hugehelp.c file which otherwise is necessary
> -since we ship tarballs with that file present.
> -
> -Reported-by: Remi Gacogne
> -Bug: http://curl.haxx.se/bug/view.cgi?id=3D1350
> ----
> - src/mkhelp.pl | 6 +++++-
> - 1 file changed, 5 insertions(+), 1 deletion(-)
> -
> -diff --git a/src/mkhelp.pl b/src/mkhelp.pl
> -index 444b669..7ed86f7 100644
> ---- a/src/mkhelp.pl
> -+++ b/src/mkhelp.pl
> -@@ -6,7 +6,7 @@
> - #                            | (__| |_| |  _ <| |___
> - #                             \___|\___/|_| \_\_____|
> - #
> --# Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
> -+# Copyright (C) 1998 - 2014, Daniel Stenberg, <daniel@haxx.se>, et al.
> - #
> - # This software is licensed as described in the file COPYING, which
> - # you should have received as part of this distribution. The terms
> -@@ -256,6 +256,10 @@ foot();
> -=20
> - sub foot {
> -   print <<FOOT
> -+#else /* !USE_MANUAL */
> -+/* built-in manual is disabled, blank function */
> -+#include "tool_hugehelp.h"
> -+void hugehelp(void) {}
> - #endif /* USE_MANUAL */
> - FOOT
> -   ;
> ---=20
> -1.9.1
> -
> diff --git a/main/curl/APKBUILD b/main/curl/APKBUILD
> index 4962d9e..87f476d 100644
> --- a/main/curl/APKBUILD
> +++ b/main/curl/APKBUILD
> @@ -1,40 +1,29 @@
> +# Contributor: Valery Kartel <valery.kartel@gmail.com>
>  # Contributor: =C5=81ukasz Jendrysik <scadu@yandex.com>
>  # Maintainer: Natanael Copa <ncopa@alpinelinux.org>
>  pkgname=3Dcurl
>  pkgver=3D7.46.0
> -pkgrel=3D1
> +pkgrel=3D2
>  pkgdesc=3D"An URL retrival utility and library"
>  url=3D"http://curl.haxx.se"
>  arch=3D"all"
>  license=3D"MIT"
>  depends=3D"ca-certificates"
>  depends_dev=3D"zlib-dev openssl-dev libssh2-dev"
> -makedepends=3D"groff $depends_dev perl"
> -source=3D"http://curl.haxx.se/download/curl-$pkgver.tar.bz2"
> -subpackages=3D"$pkgname-doc $pkgname-dev"
> +makedepends=3D"$depends_dev groff perl"
> +source=3D"http://curl.haxx.se/download/$pkgname-$pkgver.tar.bz2"
> +subpackages=3D"$pkgname-doc $pkgname-dev libcurl $pkgname-zsh-completion=
:zshcomp"
> =20
> -_builddir=3D"$srcdir/$pkgname-$pkgver"
> -
> -prepare() {
> -	local i
> -	cd "$_builddir"
> -	for i in $source; do
> -		case $i in
> -		*.patch) msg $i; patch -p1 -i "$srcdir"/$i || return 1;;
> -		esac
> -	done
> -}

i like to keep the prepare function in case we need to add a patch
later.

For example see commit:
http://git.alpinelinux.org/cgit/aports/commit/?id=3D81070afb94cd7454dd87cad=
fb0d0faed7e556de8

I spent almost a day to debug the issue that commit is supposed to
solve. However, the patch was actually not applied:

http://git.alpinelinux.org/cgit/aports/commit/?id=3Dd836f6e8fb1d5165bf70839=
a7c953a56568848d7

So, I prefer to keep it unless there is a good reason to remove it.


> +_builddir=3D"$srcdir"/$pkgname-$pkgver
>  build() {
>  	cd "$_builddir"
>  	./configure \
>  		--build=3D$CBUILD \
>  		--host=3D$CHOST \
>  		--prefix=3D/usr \
> -		--mandir=3D/usr/share/man \
> -		--without-libidn \
>  		--enable-ipv6 \
>  		--enable-unix-sockets \
> -		--disable-ldap \

I don't like removing the --disable-* things. It will prevent to
unintentionally link against ldap and libidn in case it happens to be
installed at developer box.

otherwise i think its ok.


> +		--disable-static \
>  		|| return 1
>  	make || return 1
>  }
> @@ -44,6 +33,21 @@ package() {
>  	make DESTDIR=3D"$pkgdir" install || return 1
>  }
> =20
> +libcurl() {
> +	pkgdesc=3D"The multiprotocol file transfer library"
> +	mkdir -p "$subpkgdir"/usr
> +	mv "$pkgdir"/usr/lib "$subpkgdir"/usr
> +}
> +
> +zshcomp() {
> +	pkgdesc=3D"Zsh completion for $pkgname"
> +	arch=3D"noarch"
> +	depends=3D""
> +	install_if=3D"$pkgname=3D$pkgver-r$pkgrel zsh"
> +	mkdir -p "$subpkgdir"/usr/
> +	mv "$pkgdir"/usr/share "$subpkgdir"/usr
> +}
> +
>  md5sums=3D"9979f989a2a9930d10f1b3deeabc2148  curl-7.46.0.tar.bz2"
>  sha256sums=3D"b7d726cdd8ed4b6db0fa1b474a3c59ebbbe4dcd4c61ac5e7ade0e0270d=
3195ad  curl-7.46.0.tar.bz2"
>  sha512sums=3D"ff47216a2b4a2d14a2e98e0e87867897a9bf059c58d2d73aa47a69ccd9=
4977367e2de15bea641abfb00e236a8b016ee869675c2cc702f7919da5fe612bd8b5e1  cur=
l-7.46.0.tar.bz2"



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---