Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
@ 2020-09-29  0:13 Logarithmus
  2020-09-29  0:25 ` [PR REVIEW] " ericonr
                   ` (15 more replies)
  0 siblings, 16 replies; 17+ messages in thread
From: Logarithmus @ 2020-09-29  0:13 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

There is a new pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 660 bytes --]

From 6953175106838a0f0c95ee9291b059181d8972a0 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even
 on musl targets

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..e086020a468 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys
+export PCRE2_SYS_STATIC=0

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

* Re: [PR REVIEW] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
@ 2020-09-29  0:25 ` ericonr
  2020-09-29  0:29 ` [PR PATCH] [Updated] " Logarithmus
                   ` (14 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: ericonr @ 2020-09-29  0:25 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#discussion_r496306404

Comment:
Make a note that this is only necessary for musl

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
  2020-09-29  0:25 ` [PR REVIEW] " ericonr
@ 2020-09-29  0:29 ` Logarithmus
  2020-09-29  0:32 ` Logarithmus
                   ` (13 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-09-29  0:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 550 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 680 bytes --]

From a7dcddd07d15c84b4068603c333fd6dda649aeea Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 on
 musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
  2020-09-29  0:25 ` [PR REVIEW] " ericonr
  2020-09-29  0:29 ` [PR PATCH] [Updated] " Logarithmus
@ 2020-09-29  0:32 ` Logarithmus
  2020-09-29  2:34 ` ericonr
                   ` (12 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-09-29  0:32 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 550 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 1374 bytes --]

From a7dcddd07d15c84b4068603c333fd6dda649aeea Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH 1/2] common/build-helper/rust.sh: dynamic linking to libpcre2
 on musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

From d986821ad42d997008186ee97d4b15ce06dbeeed Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 03:32:25 +0300
Subject: [PATCH 2/2] ripgrep: revbump to use system-provided libpcre2 on musl

---
 srcpkgs/ripgrep/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/ripgrep/template b/srcpkgs/ripgrep/template
index 339d2de3a39..b2ec6ad3d34 100644
--- a/srcpkgs/ripgrep/template
+++ b/srcpkgs/ripgrep/template
@@ -1,7 +1,7 @@
 # Template file for 'ripgrep'
 pkgname=ripgrep
 version=12.1.1
-revision=1
+revision=2
 build_style=cargo
 configure_args="--features=pcre2"
 hostmakedepends="ruby-asciidoctor pkg-config"

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (2 preceding siblings ...)
  2020-09-29  0:32 ` Logarithmus
@ 2020-09-29  2:34 ` ericonr
  2020-10-05 14:33 ` [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 " Logarithmus
                   ` (11 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: ericonr @ 2020-09-29  2:34 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 189 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-700389726

Comment:
@q66 do you know of any issues this could cause?

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (3 preceding siblings ...)
  2020-09-29  2:34 ` ericonr
@ 2020-10-05 14:33 ` Logarithmus
  2020-10-05 15:25 ` Logarithmus
                   ` (10 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-05 14:33 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 4414 bytes --]

From 6a9ca72eb0f1c1a903e5ad5da7f31c4cf099b796 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Mon, 28 Sep 2020 08:51:45 +0300
Subject: [PATCH 1/3] New package: cross-avr-gdb

---
 srcpkgs/cross-avr-gdb/template | 80 ++++++++++++++++++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 srcpkgs/cross-avr-gdb/template

diff --git a/srcpkgs/cross-avr-gdb/template b/srcpkgs/cross-avr-gdb/template
new file mode 100644
index 00000000000..e01a4bf6bfa
--- /dev/null
+++ b/srcpkgs/cross-avr-gdb/template
@@ -0,0 +1,80 @@
+# Template file for 'cross-${_triplet}-${_pkgname}'
+_triplet=avr
+_pkgname=gdb
+pkgname=cross-${_triplet}-${_pkgname}
+version=9.2
+revision=1
+wrksrc=${_pkgname}-${version}
+build_style=gnu-configure
+pycompile_dirs="/usr/share/gdb/python"
+configure_args="\
+	--target=${_triplet} \
+	--program-prefix=${_triplet}- \
+	--disable-werror \
+	--disable-nls \
+	--with-system-readline \
+	--with-system-gdbinit=/etc/gdb/gdbinit \
+	--with-system-zlib $(vopt_enable gdbserver) \
+	$(vopt_if static 'CFLAGS=-static CXXFLAGS=-static LDFLAGS=-static') \
+	$(vopt_if python --with-python=/usr/bin/python3)"
+hostmakedepends="texinfo perl $(vopt_if python python3-devel)"
+makedepends="ncurses-devel zlib-devel readline-devel expat-devel
+	$(vopt_if python 'python3-devel gettext-libs')"
+depends=gdb
+short_desc="GNU Debugger for AVR"
+maintainer="Artur Sinila <opensource@logarithmus.dev>"
+license="GPL-3.0-or-later"
+homepage="https://www.gnu.org/software/gdb/"
+distfiles="${GNU_SITE}/${_pkgname}/${_pkgname}-${version}.tar.xz"
+checksum=360cd7ae79b776988e89d8f9a01c985d0b1fa21c767a4295e5f88cb49175c555
+python_version=3
+
+if [ "${CROSS_BUILD}" ]; then
+	# Make python3.x detection work in cross builds
+	CFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+	CXXFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+fi
+CFLAGS+=" -fcommon"
+CXXFLAGS+=" -fcommon"
+# Package build options
+build_options="gdbserver static python"
+desc_option_gdbserver="Enable support for building GDB server"
+# By default, don't enable any of build options
+#build_options_default="gdbserver"
+# Both options cannot be enabled at the same time
+vopt_conflict gdbserver static
+
+post_extract() {
+	vsed -i 's,sgidefs.h,asm/sgidefs.h,' gdb/mips-linux-nat.c
+	mkdir -p build
+}
+
+do_configure() {
+	cd build
+	../configure ${configure_args/with-sysroot/with-build-sysroot} \
+		CFLAGS_FOR_BUILD="${CFLAGS_FOR_BUILD} -fcommon"
+	export gl_cv_func_gettimeofday_clobber=no
+	export gl_cv_func_working_strerror=yes
+	export gl_cv_func_strerror_0_works=yes
+}
+
+do_build() {
+	cd build
+	make ${makejobs} all
+}
+
+do_install() {
+	cd build
+	make DESTDIR=${DESTDIR} ${makejobs} install
+	# resolve conflicts with binutils
+	rm -rf ${DESTDIR}/usr/include ${DESTDIR}/usr/lib
+	rm -rf ${DESTDIR}/usr/lib64
+	for f in bfd configure standards; do
+		rm -f ${DESTDIR}/usr/share/info/${f}.info*
+	done
+}
+
+post_install() {
+	# resolve conflicts with binutils and native gdb
+	rm -fr usr/{share/{locale,gdb,info},include}
+}

From 9eabf45824138638c9eb57ff1fe35dbaab520608 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH 2/3] common/build-helper/rust.sh: dynamic linking to libpcre2
 on musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

From b2a73cc81bb205560259c4b931b5a057859c8516 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 03:32:25 +0300
Subject: [PATCH 3/3] ripgrep: revbump to use system-provided libpcre2 on musl

---
 srcpkgs/ripgrep/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/ripgrep/template b/srcpkgs/ripgrep/template
index 339d2de3a39..b2ec6ad3d34 100644
--- a/srcpkgs/ripgrep/template
+++ b/srcpkgs/ripgrep/template
@@ -1,7 +1,7 @@
 # Template file for 'ripgrep'
 pkgname=ripgrep
 version=12.1.1
-revision=1
+revision=2
 build_style=cargo
 configure_args="--features=pcre2"
 hostmakedepends="ruby-asciidoctor pkg-config"

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (4 preceding siblings ...)
  2020-10-05 14:33 ` [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 " Logarithmus
@ 2020-10-05 15:25 ` Logarithmus
  2020-10-05 15:27 ` Logarithmus
                   ` (9 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-05 15:25 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 4414 bytes --]

From 65154c329e70f05e6b559871d91d9f8e8c74f2aa Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Mon, 28 Sep 2020 08:51:45 +0300
Subject: [PATCH 1/3] New package: cross-avr-gdb

---
 srcpkgs/cross-avr-gdb/template | 80 ++++++++++++++++++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 srcpkgs/cross-avr-gdb/template

diff --git a/srcpkgs/cross-avr-gdb/template b/srcpkgs/cross-avr-gdb/template
new file mode 100644
index 00000000000..e01a4bf6bfa
--- /dev/null
+++ b/srcpkgs/cross-avr-gdb/template
@@ -0,0 +1,80 @@
+# Template file for 'cross-${_triplet}-${_pkgname}'
+_triplet=avr
+_pkgname=gdb
+pkgname=cross-${_triplet}-${_pkgname}
+version=9.2
+revision=1
+wrksrc=${_pkgname}-${version}
+build_style=gnu-configure
+pycompile_dirs="/usr/share/gdb/python"
+configure_args="\
+	--target=${_triplet} \
+	--program-prefix=${_triplet}- \
+	--disable-werror \
+	--disable-nls \
+	--with-system-readline \
+	--with-system-gdbinit=/etc/gdb/gdbinit \
+	--with-system-zlib $(vopt_enable gdbserver) \
+	$(vopt_if static 'CFLAGS=-static CXXFLAGS=-static LDFLAGS=-static') \
+	$(vopt_if python --with-python=/usr/bin/python3)"
+hostmakedepends="texinfo perl $(vopt_if python python3-devel)"
+makedepends="ncurses-devel zlib-devel readline-devel expat-devel
+	$(vopt_if python 'python3-devel gettext-libs')"
+depends=gdb
+short_desc="GNU Debugger for AVR"
+maintainer="Artur Sinila <opensource@logarithmus.dev>"
+license="GPL-3.0-or-later"
+homepage="https://www.gnu.org/software/gdb/"
+distfiles="${GNU_SITE}/${_pkgname}/${_pkgname}-${version}.tar.xz"
+checksum=360cd7ae79b776988e89d8f9a01c985d0b1fa21c767a4295e5f88cb49175c555
+python_version=3
+
+if [ "${CROSS_BUILD}" ]; then
+	# Make python3.x detection work in cross builds
+	CFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+	CXXFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+fi
+CFLAGS+=" -fcommon"
+CXXFLAGS+=" -fcommon"
+# Package build options
+build_options="gdbserver static python"
+desc_option_gdbserver="Enable support for building GDB server"
+# By default, don't enable any of build options
+#build_options_default="gdbserver"
+# Both options cannot be enabled at the same time
+vopt_conflict gdbserver static
+
+post_extract() {
+	vsed -i 's,sgidefs.h,asm/sgidefs.h,' gdb/mips-linux-nat.c
+	mkdir -p build
+}
+
+do_configure() {
+	cd build
+	../configure ${configure_args/with-sysroot/with-build-sysroot} \
+		CFLAGS_FOR_BUILD="${CFLAGS_FOR_BUILD} -fcommon"
+	export gl_cv_func_gettimeofday_clobber=no
+	export gl_cv_func_working_strerror=yes
+	export gl_cv_func_strerror_0_works=yes
+}
+
+do_build() {
+	cd build
+	make ${makejobs} all
+}
+
+do_install() {
+	cd build
+	make DESTDIR=${DESTDIR} ${makejobs} install
+	# resolve conflicts with binutils
+	rm -rf ${DESTDIR}/usr/include ${DESTDIR}/usr/lib
+	rm -rf ${DESTDIR}/usr/lib64
+	for f in bfd configure standards; do
+		rm -f ${DESTDIR}/usr/share/info/${f}.info*
+	done
+}
+
+post_install() {
+	# resolve conflicts with binutils and native gdb
+	rm -fr usr/{share/{locale,gdb,info},include}
+}

From 84f91a015b7abe24ee55153c3c7436c65d6e26bb Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH 2/3] common/build-helper/rust.sh: dynamic linking to libpcre2
 on musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

From 48f9ee450cdae1f5486137da02c51f2b5a9e2310 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 03:32:25 +0300
Subject: [PATCH 3/3] ripgrep: revbump to use system-provided libpcre2 on musl

---
 srcpkgs/ripgrep/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/ripgrep/template b/srcpkgs/ripgrep/template
index 339d2de3a39..b2ec6ad3d34 100644
--- a/srcpkgs/ripgrep/template
+++ b/srcpkgs/ripgrep/template
@@ -1,7 +1,7 @@
 # Template file for 'ripgrep'
 pkgname=ripgrep
 version=12.1.1
-revision=1
+revision=2
 build_style=cargo
 configure_args="--features=pcre2"
 hostmakedepends="ruby-asciidoctor pkg-config"

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (5 preceding siblings ...)
  2020-10-05 15:25 ` Logarithmus
@ 2020-10-05 15:27 ` Logarithmus
  2020-10-05 15:41 ` Logarithmus
                   ` (8 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-05 15:27 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 4414 bytes --]

From 58eccd26046b776338ef758e49497209b780647a Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Mon, 28 Sep 2020 08:51:45 +0300
Subject: [PATCH 1/3] New package: cross-avr-gdb

---
 srcpkgs/cross-avr-gdb/template | 80 ++++++++++++++++++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 srcpkgs/cross-avr-gdb/template

diff --git a/srcpkgs/cross-avr-gdb/template b/srcpkgs/cross-avr-gdb/template
new file mode 100644
index 00000000000..e01a4bf6bfa
--- /dev/null
+++ b/srcpkgs/cross-avr-gdb/template
@@ -0,0 +1,80 @@
+# Template file for 'cross-${_triplet}-${_pkgname}'
+_triplet=avr
+_pkgname=gdb
+pkgname=cross-${_triplet}-${_pkgname}
+version=9.2
+revision=1
+wrksrc=${_pkgname}-${version}
+build_style=gnu-configure
+pycompile_dirs="/usr/share/gdb/python"
+configure_args="\
+	--target=${_triplet} \
+	--program-prefix=${_triplet}- \
+	--disable-werror \
+	--disable-nls \
+	--with-system-readline \
+	--with-system-gdbinit=/etc/gdb/gdbinit \
+	--with-system-zlib $(vopt_enable gdbserver) \
+	$(vopt_if static 'CFLAGS=-static CXXFLAGS=-static LDFLAGS=-static') \
+	$(vopt_if python --with-python=/usr/bin/python3)"
+hostmakedepends="texinfo perl $(vopt_if python python3-devel)"
+makedepends="ncurses-devel zlib-devel readline-devel expat-devel
+	$(vopt_if python 'python3-devel gettext-libs')"
+depends=gdb
+short_desc="GNU Debugger for AVR"
+maintainer="Artur Sinila <opensource@logarithmus.dev>"
+license="GPL-3.0-or-later"
+homepage="https://www.gnu.org/software/gdb/"
+distfiles="${GNU_SITE}/${_pkgname}/${_pkgname}-${version}.tar.xz"
+checksum=360cd7ae79b776988e89d8f9a01c985d0b1fa21c767a4295e5f88cb49175c555
+python_version=3
+
+if [ "${CROSS_BUILD}" ]; then
+	# Make python3.x detection work in cross builds
+	CFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+	CXXFLAGS="-I${XBPS_CROSS_BASE}/${py3_inc}"
+fi
+CFLAGS+=" -fcommon"
+CXXFLAGS+=" -fcommon"
+# Package build options
+build_options="gdbserver static python"
+desc_option_gdbserver="Enable support for building GDB server"
+# By default, don't enable any of build options
+#build_options_default="gdbserver"
+# Both options cannot be enabled at the same time
+vopt_conflict gdbserver static
+
+post_extract() {
+	vsed -i 's,sgidefs.h,asm/sgidefs.h,' gdb/mips-linux-nat.c
+	mkdir -p build
+}
+
+do_configure() {
+	cd build
+	../configure ${configure_args/with-sysroot/with-build-sysroot} \
+		CFLAGS_FOR_BUILD="${CFLAGS_FOR_BUILD} -fcommon"
+	export gl_cv_func_gettimeofday_clobber=no
+	export gl_cv_func_working_strerror=yes
+	export gl_cv_func_strerror_0_works=yes
+}
+
+do_build() {
+	cd build
+	make ${makejobs} all
+}
+
+do_install() {
+	cd build
+	make DESTDIR=${DESTDIR} ${makejobs} install
+	# resolve conflicts with binutils
+	rm -rf ${DESTDIR}/usr/include ${DESTDIR}/usr/lib
+	rm -rf ${DESTDIR}/usr/lib64
+	for f in bfd configure standards; do
+		rm -f ${DESTDIR}/usr/share/info/${f}.info*
+	done
+}
+
+post_install() {
+	# resolve conflicts with binutils and native gdb
+	rm -fr usr/{share/{locale,gdb,info},include}
+}

From 9f62d9b01be1a4ea70c888cff4fa277769dc66d8 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH 2/3] common/build-helper/rust.sh: dynamic linking to libpcre2
 on musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

From 1f08ba93625602a2ba49126abd233c3cbc4ea125 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 03:32:25 +0300
Subject: [PATCH 3/3] ripgrep: revbump to use system-provided libpcre2 on musl

---
 srcpkgs/ripgrep/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/ripgrep/template b/srcpkgs/ripgrep/template
index 339d2de3a39..b2ec6ad3d34 100644
--- a/srcpkgs/ripgrep/template
+++ b/srcpkgs/ripgrep/template
@@ -1,7 +1,7 @@
 # Template file for 'ripgrep'
 pkgname=ripgrep
 version=12.1.1
-revision=1
+revision=2
 build_style=cargo
 configure_args="--features=pcre2"
 hostmakedepends="ruby-asciidoctor pkg-config"

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

* Re: [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (6 preceding siblings ...)
  2020-10-05 15:27 ` Logarithmus
@ 2020-10-05 15:41 ` Logarithmus
  2020-10-05 18:49 ` Logarithmus
                   ` (7 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-05 15:41 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

There is an updated pull request by Logarithmus against master on the void-packages repository

https://github.com/Logarithmus/void-packages rust-pcre-sys
https://github.com/void-linux/void-packages/pull/25182

common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

A patch file from https://github.com/void-linux/void-packages/pull/25182.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rust-pcre-sys-25182.patch --]
[-- Type: text/x-diff, Size: 1374 bytes --]

From 02200ef82e044a3a9c7a6d83ec6fd47178d4169e Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 02:55:05 +0300
Subject: [PATCH 1/2] common/build-helper/rust.sh: dynamic linking to libpcre2
 on musl

---
 common/build-helper/rust.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/build-helper/rust.sh b/common/build-helper/rust.sh
index a62503583f3..6fcd3bd7d9e 100644
--- a/common/build-helper/rust.sh
+++ b/common/build-helper/rust.sh
@@ -38,3 +38,6 @@ export SODIUM_INC_DIR="${XBPS_CROSS_BASE}/usr/lib"
 
 # openssl-sys
 export OPENSSL_NO_VENDOR=1
+
+# pcre2-sys, only necessary for musl targets
+export PCRE2_SYS_STATIC=0

From fc12a2645299ce754c12776963615a374c39bb94 Mon Sep 17 00:00:00 2001
From: Artur Sinila <personal@logarithmus.dev>
Date: Tue, 29 Sep 2020 03:32:25 +0300
Subject: [PATCH 2/2] ripgrep: revbump to use system-provided libpcre2 on musl

---
 srcpkgs/ripgrep/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/ripgrep/template b/srcpkgs/ripgrep/template
index 339d2de3a39..b2ec6ad3d34 100644
--- a/srcpkgs/ripgrep/template
+++ b/srcpkgs/ripgrep/template
@@ -1,7 +1,7 @@
 # Template file for 'ripgrep'
 pkgname=ripgrep
 version=12.1.1
-revision=1
+revision=2
 build_style=cargo
 configure_args="--features=pcre2"
 hostmakedepends="ruby-asciidoctor pkg-config"

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (7 preceding siblings ...)
  2020-10-05 15:41 ` Logarithmus
@ 2020-10-05 18:49 ` Logarithmus
  2020-10-05 18:57 ` ericonr
                   ` (6 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-05 18:49 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 204 bytes --]

New comment by Logarithmus on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-703820767

Comment:
> @q66 do you know of any issues this could cause?

Ping!

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (8 preceding siblings ...)
  2020-10-05 18:49 ` Logarithmus
@ 2020-10-05 18:57 ` ericonr
  2020-10-06 14:30 ` sgn
                   ` (5 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: ericonr @ 2020-10-05 18:57 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 185 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-703825244

Comment:
@sgn also asked to wait on merging this one.

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (9 preceding siblings ...)
  2020-10-05 18:57 ` ericonr
@ 2020-10-06 14:30 ` sgn
  2020-10-06 17:15 ` Logarithmus
                   ` (4 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: sgn @ 2020-10-06 14:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 276 bytes --]

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-704309320

Comment:
> @sgn also asked to wait on merging this one.

The check for `pcre2` also fail for musl, so, I don't have any objection for this change.

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (10 preceding siblings ...)
  2020-10-06 14:30 ` sgn
@ 2020-10-06 17:15 ` Logarithmus
  2020-10-06 17:15 ` Logarithmus
                   ` (3 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-06 17:15 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 339 bytes --]

New comment by Logarithmus on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-704423921

Comment:
> > @sgn also asked to wait on merging this one.
> 
> The check for `pcre2` also fail for musl, so, I don't have any objection for this change.
What "check for `pcre2`" are you talking about?

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (11 preceding siblings ...)
  2020-10-06 17:15 ` Logarithmus
@ 2020-10-06 17:15 ` Logarithmus
  2020-10-06 17:20 ` Logarithmus
                   ` (2 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-06 17:15 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 341 bytes --]

New comment by Logarithmus on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-704423921

Comment:
> > @sgn also asked to wait on merging this one.
> 
> The check for `pcre2` also fail for musl, so, I don't have any objection for this change.

What "check for `pcre2`" are you talking about?

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (12 preceding siblings ...)
  2020-10-06 17:15 ` Logarithmus
@ 2020-10-06 17:20 ` Logarithmus
  2020-10-09 20:11 ` ericonr
  2020-10-15  6:09 ` [PR PATCH] [Merged]: " ericonr
  15 siblings, 0 replies; 17+ messages in thread
From: Logarithmus @ 2020-10-06 17:20 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 487 bytes --]

New comment by Logarithmus on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-704423921

Comment:
> > @sgn also asked to wait on merging this one.
> 
> The check for `pcre2` also fail for musl, so, I don't have any objection for this change.

What "check for `pcre2`" are you talking about? On my laptop with `x86_64-musl` `ripgrep`, built with PCRE2_SYS_STATIC=0, works just fine with -P flag. All CI checks on GitHub have also passed.

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

* Re: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (13 preceding siblings ...)
  2020-10-06 17:20 ` Logarithmus
@ 2020-10-09 20:11 ` ericonr
  2020-10-15  6:09 ` [PR PATCH] [Merged]: " ericonr
  15 siblings, 0 replies; 17+ messages in thread
From: ericonr @ 2020-10-09 20:11 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 213 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/25182#issuecomment-706381603

Comment:
It's probably the checks being run with `xbps-src -Q pcre2`, at a guess.

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

* Re: [PR PATCH] [Merged]: common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
  2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
                   ` (14 preceding siblings ...)
  2020-10-09 20:11 ` ericonr
@ 2020-10-15  6:09 ` ericonr
  15 siblings, 0 replies; 17+ messages in thread
From: ericonr @ 2020-10-15  6:09 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 375 bytes --]

There's a merged pull request on the void-packages repository

common/build-helper/rust.sh: dynamic linking to libpcre2 on musl
https://github.com/void-linux/void-packages/pull/25182

Description:
By default, `pcre2-sys` crate use static linking on `musl` targets. This patch fixes such behavior and all Rust crates will automatically use system-provided `libpcre2` on musl.

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

end of thread, other threads:[~2020-10-15  6:09 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-29  0:13 [PR PATCH] common/build-helper/rust.sh: dynamic linking to libpcre2 even on musl Logarithmus
2020-09-29  0:25 ` [PR REVIEW] " ericonr
2020-09-29  0:29 ` [PR PATCH] [Updated] " Logarithmus
2020-09-29  0:32 ` Logarithmus
2020-09-29  2:34 ` ericonr
2020-10-05 14:33 ` [PR PATCH] [Updated] common/build-helper/rust.sh: dynamic linking to libpcre2 " Logarithmus
2020-10-05 15:25 ` Logarithmus
2020-10-05 15:27 ` Logarithmus
2020-10-05 15:41 ` Logarithmus
2020-10-05 18:49 ` Logarithmus
2020-10-05 18:57 ` ericonr
2020-10-06 14:30 ` sgn
2020-10-06 17:15 ` Logarithmus
2020-10-06 17:15 ` Logarithmus
2020-10-06 17:20 ` Logarithmus
2020-10-09 20:11 ` ericonr
2020-10-15  6:09 ` [PR PATCH] [Merged]: " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).