Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0
@ 2021-04-08  1:46 sgn
  2021-04-08  1:50 ` [PR REVIEW] " sgn
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: sgn @ 2021-04-08  1:46 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sgn/void-packages libxslt-fix-config-recoll-bump-1.30.0
https://github.com/void-linux/void-packages/pull/30086

libxslt: fix cross config; recoll update to 1.30.0
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-libxslt-fix-config-recoll-bump-1.30.0-30086.patch --]
[-- Type: text/x-diff, Size: 4428 bytes --]

From a339b4e22f7e8242b16a24ea36443dce10d8b147 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
 <congdanhqx@gmail.com>
Date: Thu, 8 Apr 2021 08:12:32 +0700
Subject: [PATCH 1/2] libxslt: fix xslt-config

---
 srcpkgs/libxslt/template | 27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)

diff --git a/srcpkgs/libxslt/template b/srcpkgs/libxslt/template
index e38b28fff11b..42d80f08ac98 100644
--- a/srcpkgs/libxslt/template
+++ b/srcpkgs/libxslt/template
@@ -1,10 +1,10 @@
 # Template file for 'libxslt'
 pkgname=libxslt
 version=1.1.34
-revision=3
+revision=4
 build_style=gnu-configure
 configure_args="--disable-static --disable-dependency-tracking"
-hostmakedepends="libtool"
+hostmakedepends="libtool python-devel libxml2-python pkg-config"
 makedepends="python-devel libxml2-devel libxml2-python libgcrypt-devel"
 short_desc="XSLT parser library from the GNOME project"
 maintainer="Orphaned <orphan@voidlinux.org>"
@@ -15,33 +15,26 @@ distfiles="http://xmlsoft.org/sources/libxslt-${version}.tar.gz
 checksum="98b1bd46d6792925ad2dfe9a87452ea2adebf69dcb9919ffd55bf926a7f93f7f
  98b1bd46d6792925ad2dfe9a87452ea2adebf69dcb9919ffd55bf926a7f93f7f"
 
-if [ "$CROSS_BUILD" ]; then
-	hostmakedepends="libtool automake gettext-devel ${makedepends}"
-	pre_build() {
-		sed -e "s|/usr/\(include/python2.7\)|$XBPS_CROSS_BASE/\1|g" \
-			-e "s|/usr/\(lib/python2.7/site-packages\)|$XBPS_CROSS_BASE/\1|g" \
-			-i python/Makefile
-	}
-fi
-
 post_configure() {
 	# Remove missing seperators and errors
 	# on don't know how to make target, needed
 	# by Usage:
 	# Makefile is created after configure so fix here
 	find ${wrksrc} -type f -name Makefile | xargs sed -i '/Usage/,/--version/d'
+	if [ "$CROSS_BUILD" ]; then
+		sed -e "s|/usr/include/python2.7|$XBPS_CROSS_BASE/&|g" \
+			-e "s|/usr/lib/python2.7/site-packages|$XBPS_CROSS_BASE/&|g" \
+			-i python/Makefile
+	fi
 }
 
 post_install() {
-	# Remove references to the install(1) wrapper.
-	vsed -e "s,${XBPS_WRAPPERDIR},/usr/bin,g" -i ${DESTDIR}/usr/bin/xslt-config
 	if [ "$CROSS_BUILD" ]; then
 		# Remove $XBPS_CROSS_BASE in pkg-config
+		vsed -i -e "s,-I${XBPS_CROSS_BASE}/usr/include,-I\${includedir}," \
+			$DESTDIR/usr/bin/xslt-config
 		vsed -i -e "s,$XBPS_CROSS_BASE,,g" \
-			$DESTDIR/usr/bin/xslt-config \
-			$DESTDIR/usr/lib/pkgconfig/libxslt.pc \
-			$DESTDIR/usr/lib/pkgconfig/libexslt.pc \
-			$DESTDIR/usr/lib/xsltConf.sh
+			$DESTDIR/usr/lib/pkgconfig/libexslt.pc
 	fi
 	vlicense COPYING
 }

From 5475a32a1beb0aea52d6f2dbd709b39c03332040 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
 <congdanhqx@gmail.com>
Date: Tue, 6 Apr 2021 22:46:16 +0700
Subject: [PATCH 2/2] recoll: update to 1.30.0.

---
 srcpkgs/recoll/template | 21 +++++----------------
 1 file changed, 5 insertions(+), 16 deletions(-)

diff --git a/srcpkgs/recoll/template b/srcpkgs/recoll/template
index 98fda41bbcad..8c5de6966a46 100644
--- a/srcpkgs/recoll/template
+++ b/srcpkgs/recoll/template
@@ -1,12 +1,13 @@
 # Template file for 'recoll'
 pkgname=recoll
-version=1.26.1
-revision=2
+version=1.30.0
+revision=1
 build_style=gnu-configure
 build_helper=qmake
 configure_args="--disable-python-module --disable-python-chm
  --enable-recollq --disable-x11mon"
-hostmakedepends="automake libtool gettext-devel qt5-qmake"
+hostmakedepends="automake libtool gettext-devel qt5-qmake pkg-config
+ qt5-host-tools"
 makedepends="xapian-core-devel zlib-devel libuuid-devel qt5-webkit-devel qt5-devel
  aspell-devel libxslt-devel libxml2-devel"
 short_desc="Full text search tool based on Xapian backend"
@@ -14,16 +15,4 @@ maintainer="Rui Abreu Ferreira <raf-ep@gmx.com>"
 license="GPL-2.0-or-later"
 homepage="https://www.lesbonscomptes.com/recoll/"
 distfiles="https://www.lesbonscomptes.com/${pkgname}/${pkgname}-${version}.tar.gz"
-checksum=438f251c24baf954020cf3298872c74982f4c4abffd10197d7700db83072e732
-
-if [ "$CROSS_BUILD" ]; then
-	hostmakedepends+=" qt5-host-tools qt5-devel qt5-webkit-devel"
-
-	pre_configure() {
-		# xslt-config has the libxml2 hardcoded to /usr/include/libxml2
-		# use cross path instead.
-		vsed -e 's;-I/usr/include/libxml2;-I${includedir}/libxml2;' \
-			-e 's;libs="-lxslt -L/usr/lib -lxml2";libs="-lxslt -lxml2";g' \
-			-i ${XBPS_WRAPPERDIR}/xslt-config
-	}
-fi
+checksum=17a8e684a8d5560dc7b44cd4b2962cb46476a75bdc107f622051235076d11885

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

* Re: [PR REVIEW] libxslt: fix cross config; recoll update to 1.30.0
  2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
@ 2021-04-08  1:50 ` sgn
  2021-04-08  9:53 ` Veganizer
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: sgn @ 2021-04-08  1:50 UTC (permalink / raw)
  To: ml

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

New review comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/30086#discussion_r609200814

Comment:
Those `vsed` was obsoleted by adding `pkg-config`

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

* Re: libxslt: fix cross config; recoll update to 1.30.0
  2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
  2021-04-08  1:50 ` [PR REVIEW] " sgn
@ 2021-04-08  9:53 ` Veganizer
  2021-04-08 10:00 ` Veganizer
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Veganizer @ 2021-04-08  9:53 UTC (permalink / raw)
  To: ml

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

New comment by Veganizer on void-packages repository

https://github.com/void-linux/void-packages/pull/30086#issuecomment-815624809

Comment:
I think we need a musl patch with adding following code in fstreewalk.h here:

```cpp
#ifndef	__dev_t_defined
#include <sys/types.h>
#endif /* __dev_t_defined */
```

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

* Re: libxslt: fix cross config; recoll update to 1.30.0
  2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
  2021-04-08  1:50 ` [PR REVIEW] " sgn
  2021-04-08  9:53 ` Veganizer
@ 2021-04-08 10:00 ` Veganizer
  2021-04-08 12:39 ` [PR PATCH] [Updated] " sgn
  2021-04-14 23:41 ` [PR PATCH] [Merged]: " sgn
  4 siblings, 0 replies; 6+ messages in thread
From: Veganizer @ 2021-04-08 10:00 UTC (permalink / raw)
  To: ml

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

New comment by Veganizer on void-packages repository

https://github.com/void-linux/void-packages/pull/30086#issuecomment-815624809

Comment:
I think we need a musl patch with adding following code in fstreewalk.h here:

```cpp
#include <sys/types.h>
```

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

* Re: [PR PATCH] [Updated] libxslt: fix cross config; recoll update to 1.30.0
  2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
                   ` (2 preceding siblings ...)
  2021-04-08 10:00 ` Veganizer
@ 2021-04-08 12:39 ` sgn
  2021-04-14 23:41 ` [PR PATCH] [Merged]: " sgn
  4 siblings, 0 replies; 6+ messages in thread
From: sgn @ 2021-04-08 12:39 UTC (permalink / raw)
  To: ml

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

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

https://github.com/sgn/void-packages libxslt-fix-config-recoll-bump-1.30.0
https://github.com/void-linux/void-packages/pull/30086

libxslt: fix cross config; recoll update to 1.30.0
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-libxslt-fix-config-recoll-bump-1.30.0-30086.patch --]
[-- Type: text/x-diff, Size: 5444 bytes --]

From 89c42f51d3103948a5f4fd05dcf10175553542cb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
 <congdanhqx@gmail.com>
Date: Thu, 8 Apr 2021 08:12:32 +0700
Subject: [PATCH 1/2] libxslt: fix xslt-config

---
 srcpkgs/libxslt/template | 27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)

diff --git a/srcpkgs/libxslt/template b/srcpkgs/libxslt/template
index e38b28fff11b..42d80f08ac98 100644
--- a/srcpkgs/libxslt/template
+++ b/srcpkgs/libxslt/template
@@ -1,10 +1,10 @@
 # Template file for 'libxslt'
 pkgname=libxslt
 version=1.1.34
-revision=3
+revision=4
 build_style=gnu-configure
 configure_args="--disable-static --disable-dependency-tracking"
-hostmakedepends="libtool"
+hostmakedepends="libtool python-devel libxml2-python pkg-config"
 makedepends="python-devel libxml2-devel libxml2-python libgcrypt-devel"
 short_desc="XSLT parser library from the GNOME project"
 maintainer="Orphaned <orphan@voidlinux.org>"
@@ -15,33 +15,26 @@ distfiles="http://xmlsoft.org/sources/libxslt-${version}.tar.gz
 checksum="98b1bd46d6792925ad2dfe9a87452ea2adebf69dcb9919ffd55bf926a7f93f7f
  98b1bd46d6792925ad2dfe9a87452ea2adebf69dcb9919ffd55bf926a7f93f7f"
 
-if [ "$CROSS_BUILD" ]; then
-	hostmakedepends="libtool automake gettext-devel ${makedepends}"
-	pre_build() {
-		sed -e "s|/usr/\(include/python2.7\)|$XBPS_CROSS_BASE/\1|g" \
-			-e "s|/usr/\(lib/python2.7/site-packages\)|$XBPS_CROSS_BASE/\1|g" \
-			-i python/Makefile
-	}
-fi
-
 post_configure() {
 	# Remove missing seperators and errors
 	# on don't know how to make target, needed
 	# by Usage:
 	# Makefile is created after configure so fix here
 	find ${wrksrc} -type f -name Makefile | xargs sed -i '/Usage/,/--version/d'
+	if [ "$CROSS_BUILD" ]; then
+		sed -e "s|/usr/include/python2.7|$XBPS_CROSS_BASE/&|g" \
+			-e "s|/usr/lib/python2.7/site-packages|$XBPS_CROSS_BASE/&|g" \
+			-i python/Makefile
+	fi
 }
 
 post_install() {
-	# Remove references to the install(1) wrapper.
-	vsed -e "s,${XBPS_WRAPPERDIR},/usr/bin,g" -i ${DESTDIR}/usr/bin/xslt-config
 	if [ "$CROSS_BUILD" ]; then
 		# Remove $XBPS_CROSS_BASE in pkg-config
+		vsed -i -e "s,-I${XBPS_CROSS_BASE}/usr/include,-I\${includedir}," \
+			$DESTDIR/usr/bin/xslt-config
 		vsed -i -e "s,$XBPS_CROSS_BASE,,g" \
-			$DESTDIR/usr/bin/xslt-config \
-			$DESTDIR/usr/lib/pkgconfig/libxslt.pc \
-			$DESTDIR/usr/lib/pkgconfig/libexslt.pc \
-			$DESTDIR/usr/lib/xsltConf.sh
+			$DESTDIR/usr/lib/pkgconfig/libexslt.pc
 	fi
 	vlicense COPYING
 }

From 74ebda92b8bf5495b91fa79b7ee6dbb96a927264 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
 <congdanhqx@gmail.com>
Date: Tue, 6 Apr 2021 22:46:16 +0700
Subject: [PATCH 2/2] recoll: update to 1.30.0.

---
 .../patches/{musl-pxattr.patch => musl.patch} | 14 +++++++++++++
 srcpkgs/recoll/template                       | 21 +++++--------------
 2 files changed, 19 insertions(+), 16 deletions(-)
 rename srcpkgs/recoll/patches/{musl-pxattr.patch => musl.patch} (52%)

diff --git a/srcpkgs/recoll/patches/musl-pxattr.patch b/srcpkgs/recoll/patches/musl.patch
similarity index 52%
rename from srcpkgs/recoll/patches/musl-pxattr.patch
rename to srcpkgs/recoll/patches/musl.patch
index 22cee3331302..c57aca4a565d 100644
--- a/srcpkgs/recoll/patches/musl-pxattr.patch
+++ b/srcpkgs/recoll/patches/musl.patch
@@ -1,3 +1,17 @@
+Index: utils/fstreewalk.cpp
+===================================================================
+--- utils/fstreewalk.cpp.orig
++++ utils/fstreewalk.cpp
+@@ -17,6 +17,7 @@
+ 
+ #include "autoconfig.h"
+ 
++#include <sys/types.h>
+ #include <stdio.h>
+ 
+ #include <errno.h>
+Index: utils/pxattr.cpp
+===================================================================
 --- utils/pxattr.cpp	2017-07-03 14:14:46.000000000 +0200
 +++ utils/pxattr.cpp	2017-12-17 14:34:29.955674837 +0100
 @@ -44,7 +44,7 @@
diff --git a/srcpkgs/recoll/template b/srcpkgs/recoll/template
index 98fda41bbcad..8c5de6966a46 100644
--- a/srcpkgs/recoll/template
+++ b/srcpkgs/recoll/template
@@ -1,12 +1,13 @@
 # Template file for 'recoll'
 pkgname=recoll
-version=1.26.1
-revision=2
+version=1.30.0
+revision=1
 build_style=gnu-configure
 build_helper=qmake
 configure_args="--disable-python-module --disable-python-chm
  --enable-recollq --disable-x11mon"
-hostmakedepends="automake libtool gettext-devel qt5-qmake"
+hostmakedepends="automake libtool gettext-devel qt5-qmake pkg-config
+ qt5-host-tools"
 makedepends="xapian-core-devel zlib-devel libuuid-devel qt5-webkit-devel qt5-devel
  aspell-devel libxslt-devel libxml2-devel"
 short_desc="Full text search tool based on Xapian backend"
@@ -14,16 +15,4 @@ maintainer="Rui Abreu Ferreira <raf-ep@gmx.com>"
 license="GPL-2.0-or-later"
 homepage="https://www.lesbonscomptes.com/recoll/"
 distfiles="https://www.lesbonscomptes.com/${pkgname}/${pkgname}-${version}.tar.gz"
-checksum=438f251c24baf954020cf3298872c74982f4c4abffd10197d7700db83072e732
-
-if [ "$CROSS_BUILD" ]; then
-	hostmakedepends+=" qt5-host-tools qt5-devel qt5-webkit-devel"
-
-	pre_configure() {
-		# xslt-config has the libxml2 hardcoded to /usr/include/libxml2
-		# use cross path instead.
-		vsed -e 's;-I/usr/include/libxml2;-I${includedir}/libxml2;' \
-			-e 's;libs="-lxslt -L/usr/lib -lxml2";libs="-lxslt -lxml2";g' \
-			-i ${XBPS_WRAPPERDIR}/xslt-config
-	}
-fi
+checksum=17a8e684a8d5560dc7b44cd4b2962cb46476a75bdc107f622051235076d11885

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

* Re: [PR PATCH] [Merged]: libxslt: fix cross config; recoll update to 1.30.0
  2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
                   ` (3 preceding siblings ...)
  2021-04-08 12:39 ` [PR PATCH] [Updated] " sgn
@ 2021-04-14 23:41 ` sgn
  4 siblings, 0 replies; 6+ messages in thread
From: sgn @ 2021-04-14 23:41 UTC (permalink / raw)
  To: ml

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

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

libxslt: fix cross config; recoll update to 1.30.0
https://github.com/void-linux/void-packages/pull/30086

Description:
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

end of thread, other threads:[~2021-04-14 23:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08  1:46 [PR PATCH] libxslt: fix cross config; recoll update to 1.30.0 sgn
2021-04-08  1:50 ` [PR REVIEW] " sgn
2021-04-08  9:53 ` Veganizer
2021-04-08 10:00 ` Veganizer
2021-04-08 12:39 ` [PR PATCH] [Updated] " sgn
2021-04-14 23:41 ` [PR PATCH] [Merged]: " sgn

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).