This thread contains a patchset. You're looking at the original emails,
but you may wish to use the patch review UI.
Review patch
4
2
[alpine-devel] [PATCH] main/pcre: enabled unicode properties (UCP)
---
main/pcre/APKBUILD | 3 ++ -
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/main/pcre/APKBUILD b/main/pcre/APKBUILD
index ff18dbf..e4f0430 100644
--- a/main/pcre/APKBUILD
+++ b/main/pcre/APKBUILD
@@ -1,7 +1,7 @@
# Maintainer: Natanael Copa <ncopa@alpinelinux.org>
pkgname=pcre
pkgver=8.11
- pkgrel=0
+ pkgrel=1
pkgdesc="Perl-compatible regular expression library"
url="http://pcre.sourceforge.net"
arch="x86 x86_64"
@@ -27,6 +27,7 @@ build() {
cd "$_builddir"
./configure --prefix=/usr \
--enable-utf8 \
+ --enable-unicode-properties \
--with-match-limit-recursion=8192 \
--htmldir=/usr/share/doc/$pkgname-$pkgver/html \
--docdir=/usr/share/doc/$pkgname-$pkgver
--
1.7.3.3
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
On Fri, 31 Dec 2010 20:51:18 -0600
Matt Smith <mcs@darkregion.net > wrote:
> ---
> main/pcre/APKBUILD | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/main/pcre/APKBUILD b/main/pcre/APKBUILD
> index ff18dbf..e4f0430 100644
> --- a/main/pcre/APKBUILD
...
> ./configure --prefix=/usr \
> --enable-utf8 \
> + --enable-unicode-properties \
> --with-match-limit-recursion=8192 \
...
Will this change break ABI compatibility?
-nc
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
On Mon, 3 Jan 2011 14:49:47 +0100
Natanael Copa <ncopa@alpinelinux.org > wrote:
> On Fri, 31 Dec 2010 20:51:18 -0600
> Matt Smith <mcs@darkregion.net > wrote:
>
> > ---
> > main/pcre/APKBUILD | 3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/main/pcre/APKBUILD b/main/pcre/APKBUILD
> > index ff18dbf..e4f0430 100644
> > --- a/main/pcre/APKBUILD
> ...
> > ./configure --prefix=/usr \
> > --enable-utf8 \
> > + --enable-unicode-properties \
> > --with-match-limit-recursion=8192 \
> ...
>
> Will this change break ABI compatibility?
apparently no. Applied. Thanks!
-nc
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
On Tuesday, January 4, 2011 10:08am, "Natanael Copa" <ncopa@alpinelinux.org > said:
> On Mon, 3 Jan 2011 14:49:47 +0100
> Natanael Copa <ncopa@alpinelinux.org > wrote:
>
>> On Fri, 31 Dec 2010 20:51:18 -0600
>> Matt Smith <mcs@darkregion.net > wrote:
>>
>> > ---
>> > main/pcre/APKBUILD | 3 ++-
>> > 1 files changed, 2 insertions(+), 1 deletions(-)
>> >
>> > diff --git a/main/pcre/APKBUILD b/main/pcre/APKBUILD
>> > index ff18dbf..e4f0430 100644
>> > --- a/main/pcre/APKBUILD
>> ...
>> > ./configure --prefix=/usr \
>> > --enable-utf8 \
>> > + --enable-unicode-properties \
>> > --with-match-limit-recursion=8192 \
>> ...
>>
>> Will this change break ABI compatibility?
>
> apparently no. Applied. Thanks!
I'm sorry I haven't been active recently, but I'm assuming it wasn't a problem?
Thanks,
Matt
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
On Fri, 7 Jan 2011 20:51:15 -0600 (CST)
"Matt Smith" <mcs@darkregion.net > wrote:
> >> > diff --git a/main/pcre/APKBUILD b/main/pcre/APKBUILD
> >> > index ff18dbf..e4f0430 100644
> >> > --- a/main/pcre/APKBUILD
> >> ...
> >> > ./configure --prefix=/usr \
> >> > --enable-utf8 \
> >> > + --enable-unicode-properties \
> >> > --with-match-limit-recursion=8192 \
> >> ...
> >>
> >> Will this change break ABI compatibility?
> >
> > apparently no. Applied. Thanks!
>
> I'm sorry I haven't been active recently, but I'm assuming it wasn't
> a problem?
so far no problems.
There have been similar changes, i think the --enable-utf8 or similar
that broke every package linked to pcre. We just need to be careful
sometimes.
Thanks!
-nc
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---