~alpine/aports

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
1

[alpine-aports] [PATCH] main/aspell: /usr/lib/aspell symlink fix, some build fixes, some staff moved to subpackages

Details
Message ID
<1447070424-1851-1-git-send-email-valery.kartel@gmail.com>
Sender timestamp
1447070424
DKIM signature
missing
Download raw message
Patch: +39 -20
---
 main/aspell/APKBUILD      | 47 +++++++++++++++++++++++++++++++++--------------
 main/aspell/libmath.patch | 12 ++++++------
 2 files changed, 39 insertions(+), 20 deletions(-)

diff --git a/main/aspell/APKBUILD b/main/aspell/APKBUILD
index 436a3a1..c4534ef 100644
--- a/main/aspell/APKBUILD
+++ b/main/aspell/APKBUILD
@@ -1,13 +1,14 @@
# Maintainer: Natanael Copa <ncopa@alpinelinux.org>
# Contributor: Valery Kartel <valery.kartel@gmail.com>
pkgname=aspell
pkgver=0.60.6.1
_pkgmajorver=${pkgver%.*}
pkgrel=3
_pkgmajorver=0.60
pkgrel=4
pkgdesc="A spell checker designed to eventually replace Ispell"
url="http://aspell.net/"
arch="all"
license="LGPL2+"
subpackages="$pkgname-dev $pkgname-doc"
subpackages="$pkgname-compat $pkgname-utils $pkgname-dev $pkgname-doc"
depends=
makedepends="ncurses-dev perl"
install=
@@ -18,20 +19,16 @@ _builddir="$srcdir/$pkgname-$pkgver"

prepare() {
	cd "$_builddir"
	update_config_sub || return 1
	for i in $source; do
		case $i in
		*.patch) msg $i; patch -p1 -i "$srcdir"/$i || return 1;;
		esac
	done
}

build() { 
	cd "$_builddir"
	./configure \
		--build=$CBUILD \
		--host=$CHOST \
		--prefix=/usr \
		|| return 1

	# we want add -lm to linker flag for libaspell so wil build that
	# separately. this is just an ugly workaround
	make libaspell.la LDFLAGS="$LDFLAGS -lm" || return 1
	./configure --prefix=/usr || return 1
	make || return 1
}

@@ -40,5 +37,27 @@ package() {
	make DESTDIR="$pkgdir" install || return 1
	ln -s $pkgname-${_pkgmajorver} "$pkgdir"/usr/lib/$pkgname || return 1
}

compat() {
	pkgdesc="aspell compatibility scripts for ispell and unix spell"
	arch="noarch"
	install -d "$subpkgdir"/usr/bin
	cd "$_builddir"/scripts
	cp spell ispell "$subpkgdir"/usr/bin/
}

utils() {
	pkgdesc="aspell misc utils"
	depends="perl"
	arch="all"
	install -d "$subpkgdir"/usr/bin
	cd "$pkgdir"/usr/bin
	mv pre* aspell-import run-with-aspell word-list-compress "$subpkgdir"/usr/bin/
}

md5sums="e66a9c9af6a60dc46134fdacf6ce97d7  aspell-0.60.6.1.tar.gz
715f4e138ab33b27201d90cbc98b4fb3  libmath.patch"
2b866e5365aca89510c8c84359007ba5  libmath.patch"
sha256sums="f52583a83a63633701c5f71db3dc40aab87b7f76b29723aeb27941eff42df6e1  aspell-0.60.6.1.tar.gz
a20ea65e3a88123db9e9cd89af71962472a548113c79860d28e74494d65cdad7  libmath.patch"
sha512sums="f310c7590be98406589b5c26ca36a2ecfe4733f0b40fd6c176b96b7955ef2b5cd0ec9a3d770cf132146ae7a896042b4b698945112995ee1ae66adcfa5542247f  aspell-0.60.6.1.tar.gz
e390fc1004d59d895ae3cde198265fa769efa1ec729a00d2d8ceca63a4a10d93857a7fef9a1bb7c9434502630e3ae85f2bc2c207c7dffab78469eb344e2275ea  libmath.patch"
diff --git a/main/aspell/libmath.patch b/main/aspell/libmath.patch
index 3fad33a..71f1b26 100644
--- a/main/aspell/libmath.patch
+++ b/main/aspell/libmath.patch
@@ -1,11 +1,11 @@
--- a/Makefile.orig	2009-05-21 08:22:51.000000000 +0000
+++ b/Makefile	2009-05-21 08:23:06.000000000 +0000
@@ -568,7 +568,7 @@
--- old/Makefile.in	2011-07-04 11:58:49.000000000 +0300
+++ new/Makefile.in	2015-11-09 13:20:24.823333353 +0200
@@ -569,7 +569,7 @@
 	lib/string_list-c.cpp lib/find_speller.cpp lib/speller-c.cpp \
 	lib/string_pair_enumeration-c.cpp lib/new_checker.cpp \
 	modules/filter/url.cpp $(am__append_3)
-libaspell_la_LIBADD = $(LTLIBINTL) $(PTHREAD_LIB)
+libaspell_la_LIBADD = $(LTLIBINTL) $(PTHREAD_LIB) -lm
 libaspell_la_LDFLAGS = -version-info 16:4:1 -no-undefined
 #libaspell_la_LDFLAGS = -version-info 16:4:0 -no-undefined
 libpspell_la_SOURCES = lib/dummy.cpp
 @INCREMENTED_SONAME_FALSE@libaspell_la_LDFLAGS = -version-info 16:5:1 -no-undefined
 @INCREMENTED_SONAME_TRUE@libaspell_la_LDFLAGS = -version-info 16:5:0 -no-undefined
 @PSPELL_COMPATIBILITY_TRUE@libpspell_la_SOURCES = lib/dummy.cpp
-- 
2.6.1



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---
Natanael Copa <ncopa@alpinelinux.org>
Details
Message ID
<20151110092136.46361d67@ncopa-desktop.alpinelinux.org>
In-Reply-To
<1447070424-1851-1-git-send-email-valery.kartel@gmail.com> (view parent)
Sender timestamp
1447143696
DKIM signature
missing
Download raw message
On Mon,  9 Nov 2015 14:00:24 +0200
Valery Kartel <valery.kartel@gmail.com> wrote:

> ---
>  main/aspell/APKBUILD      | 47 +++++++++++++++++++++++++++++++++--------------
>  main/aspell/libmath.patch | 12 ++++++------
>  2 files changed, 39 insertions(+), 20 deletions(-)

Thanks. I applied and cleaned it up a bit. see comments below.


> 
> diff --git a/main/aspell/APKBUILD b/main/aspell/APKBUILD
> index 436a3a1..c4534ef 100644
> --- a/main/aspell/APKBUILD
> +++ b/main/aspell/APKBUILD
> @@ -1,13 +1,14 @@
>  # Maintainer: Natanael Copa <ncopa@alpinelinux.org>
> +# Contributor: Valery Kartel <valery.kartel@gmail.com>
>  pkgname=aspell
>  pkgver=0.60.6.1
> -_pkgmajorver=${pkgver%.*}
> -pkgrel=3
> +_pkgmajorver=0.60
> +pkgrel=4

setting _pkgmajorver hard will make things break if we do 'abump
aspell-0.61.0' in future. I prefer to have things like this extracted
from pkgver.

>  pkgdesc="A spell checker designed to eventually replace Ispell"
>  url="http://aspell.net/"
>  arch="all"
>  license="LGPL2+"
> -subpackages="$pkgname-dev $pkgname-doc"
> +subpackages="$pkgname-compat $pkgname-utils $pkgname-dev $pkgname-doc"
>  depends=
>  makedepends="ncurses-dev perl"
>  install=
> @@ -18,20 +19,16 @@ _builddir="$srcdir/$pkgname-$pkgver"
>  
>  prepare() {
>  	cd "$_builddir"
> -	update_config_sub || return 1

we need the update_config_sub or build breaks.

> +	for i in $source; do
> +		case $i in
> +		*.patch) msg $i; patch -p1 -i "$srcdir"/$i || return 1;;
> +		esac
> +	done
>  }
>  
>  build() { 
>  	cd "$_builddir"
> -	./configure \
> -		--build=$CBUILD \
> -		--host=$CHOST \
> -		--prefix=/usr \
> -		|| return 1
> -

we normally want to keep the --build=$CBUILD --host=$CHOST as those are
needed in case we want support crosscompile in future.


> -	# we want add -lm to linker flag for libaspell so wil build that
> -	# separately. this is just an ugly workaround
> -	make libaspell.la LDFLAGS="$LDFLAGS -lm" || return 1
> +	./configure --prefix=/usr || return 1
>  	make || return 1
>  }
>  
> @@ -40,5 +37,27 @@ package() {
>  	make DESTDIR="$pkgdir" install || return 1
>  	ln -s $pkgname-${_pkgmajorver} "$pkgdir"/usr/lib/$pkgname || return 1
>  }
> +
> +compat() {
> +	pkgdesc="aspell compatibility scripts for ispell and unix spell"
> +	arch="noarch"
> +	install -d "$subpkgdir"/usr/bin
> +	cd "$_builddir"/scripts
> +	cp spell ispell "$subpkgdir"/usr/bin/
> +}
> +
> +utils() {
> +	pkgdesc="aspell misc utils"
> +	depends="perl"
> +	arch="all"
> +	install -d "$subpkgdir"/usr/bin
> +	cd "$pkgdir"/usr/bin
> +	mv pre* aspell-import run-with-aspell word-list-compress "$subpkgdir"/usr/bin/
> +}
> +

good!

Thanks!

>  md5sums="e66a9c9af6a60dc46134fdacf6ce97d7  aspell-0.60.6.1.tar.gz
> -715f4e138ab33b27201d90cbc98b4fb3  libmath.patch"
> +2b866e5365aca89510c8c84359007ba5  libmath.patch"
> +sha256sums="f52583a83a63633701c5f71db3dc40aab87b7f76b29723aeb27941eff42df6e1  aspell-0.60.6.1.tar.gz
> +a20ea65e3a88123db9e9cd89af71962472a548113c79860d28e74494d65cdad7  libmath.patch"
> +sha512sums="f310c7590be98406589b5c26ca36a2ecfe4733f0b40fd6c176b96b7955ef2b5cd0ec9a3d770cf132146ae7a896042b4b698945112995ee1ae66adcfa5542247f  aspell-0.60.6.1.tar.gz
> +e390fc1004d59d895ae3cde198265fa769efa1ec729a00d2d8ceca63a4a10d93857a7fef9a1bb7c9434502630e3ae85f2bc2c207c7dffab78469eb344e2275ea  libmath.patch"
> diff --git a/main/aspell/libmath.patch b/main/aspell/libmath.patch
> index 3fad33a..71f1b26 100644
> --- a/main/aspell/libmath.patch
> +++ b/main/aspell/libmath.patch
> @@ -1,11 +1,11 @@
> ---- a/Makefile.orig	2009-05-21 08:22:51.000000000 +0000
> -+++ b/Makefile	2009-05-21 08:23:06.000000000 +0000
> -@@ -568,7 +568,7 @@
> +--- old/Makefile.in	2011-07-04 11:58:49.000000000 +0300
> ++++ new/Makefile.in	2015-11-09 13:20:24.823333353 +0200
> +@@ -569,7 +569,7 @@
>   	lib/string_list-c.cpp lib/find_speller.cpp lib/speller-c.cpp \
>   	lib/string_pair_enumeration-c.cpp lib/new_checker.cpp \
>   	modules/filter/url.cpp $(am__append_3)
>  -libaspell_la_LIBADD = $(LTLIBINTL) $(PTHREAD_LIB)
>  +libaspell_la_LIBADD = $(LTLIBINTL) $(PTHREAD_LIB) -lm
> - libaspell_la_LDFLAGS = -version-info 16:4:1 -no-undefined
> - #libaspell_la_LDFLAGS = -version-info 16:4:0 -no-undefined
> - libpspell_la_SOURCES = lib/dummy.cpp
> + @INCREMENTED_SONAME_FALSE@libaspell_la_LDFLAGS = -version-info 16:5:1 -no-undefined
> + @INCREMENTED_SONAME_TRUE@libaspell_la_LDFLAGS = -version-info 16:5:0 -no-undefined
> + @PSPELL_COMPATIBILITY_TRUE@libpspell_la_SOURCES = lib/dummy.cpp

-nc


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---
Reply to thread Export thread (mbox)