~alpine/aports

2 2

[alpine-aports] [PATCH] testing/lttng-tools: upgrade to 2.10.6

Michael Jeanson <mjeanson@efficios.com>
Details
Message ID
<20190320171453.13904-1-mjeanson@efficios.com>
Sender timestamp
1553102093
DKIM signature
missing
Download raw message
Patch: +43 -9
---
 ...db67743d08f418235fee43747bb9d42d62ce.patch | 29 +++++++++++++++++++
 testing/lttng-tools/APKBUILD                  | 23 +++++++++------
 2 files changed, 43 insertions(+), 9 deletions(-)
 create mode 100644 testing/lttng-tools/9193db67743d08f418235fee43747bb9d42d62ce.patch

diff --git a/testing/lttng-tools/9193db67743d08f418235fee43747bb9d42d62ce.patch b/testing/lttng-tools/9193db67743d08f418235fee43747bb9d42d62ce.patch
new file mode 100644
index 0000000000..bc3c5382e3
--- /dev/null
+++ b/testing/lttng-tools/9193db67743d08f418235fee43747bb9d42d62ce.patch
@@ -0,0 +1,29 @@
From 9193db67743d08f418235fee43747bb9d42d62ce Mon Sep 17 00:00:00 2001
From: Michael Jeanson <mjeanson@efficios.com>
Date: Fri, 15 Mar 2019 11:19:27 -0400
Subject: [PATCH] fix: tests: link libpause_consumer on liblttng-ctl

This preload test library uses symbols from liblttng-ctl which are
resolved when preloaded by GLIBC but not by MUSL.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
---
 tests/regression/tools/notification/Makefile.am | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tests/regression/tools/notification/Makefile.am b/tests/regression/tools/notification/Makefile.am
index 33e95886d..b869bdb1d 100644
--- a/tests/regression/tools/notification/Makefile.am
+++ b/tests/regression/tools/notification/Makefile.am
@@ -20,7 +20,10 @@ FORCE_SHARED_LIB_OPTIONS = -module -shared -avoid-version \
 			   -rpath $(abs_builddir)
 
 libpause_consumer_la_SOURCES = consumer_testpoints.c
-libpause_consumer_la_LIBADD = $(top_builddir)/src/common/libcommon.la $(DL_LIBS)
+libpause_consumer_la_LIBADD = \
+	$(top_builddir)/src/common/libcommon.la \
+	$(top_builddir)/src/lib/lttng-ctl/liblttng-ctl.la \
+	$(DL_LIBS)
 libpause_consumer_la_LDFLAGS = $(FORCE_SHARED_LIB_OPTIONS)
 noinst_LTLIBRARIES = libpause_consumer.la
 
diff --git a/testing/lttng-tools/APKBUILD b/testing/lttng-tools/APKBUILD
index 08e344eae6..9d4c8c1c8c 100644
--- a/testing/lttng-tools/APKBUILD
+++ b/testing/lttng-tools/APKBUILD
@@ -1,7 +1,7 @@
# Contributor: Michael Jeanson <mjeanson@efficios.com>
# Maintainer: Michael Jeanson <mjeanson@efficios.com>
pkgname=lttng-tools
pkgver=2.9.6
pkgver=2.10.6
pkgrel=0
pkgdesc="LTTng 2.0 control and utility programs"
url="https://lttng.org"
@@ -14,7 +14,7 @@ makedepends="$depends_dev bash grep coreutils automake autoconf"
install=""
subpackages="$pkgname-dev $pkgname-doc"
source="https://lttng.org/files/$pkgname/$pkgname-$pkgver.tar.bz2
	musl-has-no-dlmopen.patch"
	9193db67743d08f418235fee43747bb9d42d62ce.patch"

builddir="$srcdir/$pkgname-$pkgver"

@@ -22,25 +22,30 @@ build() {
	cd "$builddir"

	unset LDFLAGS
	autoreconf -i # Because we patch a Makefile.am
	./configure \
		--build=$CBUILD \
		--host=$CHOST \
		--prefix=/usr \
		--sysconfdir=/etc \
		--localstatedir=/var \
		|| return 1
	make V=1 || return 1
	make -j1 check || return 1
		--localstatedir=/var
	make V=1
}

check() {
	cd "$builddir"

	make -j1 check
}

package() {
	cd "$builddir"

	make DESTDIR="$pkgdir" install || return 1
	make DESTDIR="$pkgdir" install
	mkdir -p "$pkgdir"/usr/share/licenses/$pkgname
	mv "$pkgdir"/usr/share/doc/$pkgname/LICENSE \
		"$pkgdir"/usr/share/licenses/$pkgname/
}

sha512sums="5fc9da4280587c4dc3c695f713e3ae51ad46a9bd0c82d4241a95f406e2644ef296d46c507171b7adc549a933f3abc64c993106287a84a0b8c6901b3b122a0646  lttng-tools-2.9.6.tar.bz2
cb5136007b0d8ea0a84557ab025e709dc4c5d818272f057a9f6c78ed7cf5104b9d904a1b8c5e8674e3455fe85dc28bb890c190cc04f0c1d9c29d87b40dadb88b  musl-has-no-dlmopen.patch"
sha512sums="d33379687755b1c97036d69abcf778278adc099e8c44f94b1f1c610011bf8f0da66a2743f39cb9798c72a2e6218bb31c4d392ff4269dbf0d6a694294458f231d  lttng-tools-2.10.6.tar.bz2
63fa29f2f6ee7065cc8c44663428ecf2b8c1d4557214e6aa04a863168f73bbe532f2ee7d55fc23a30fda1ab03c4434b857f3778a7807eba573e01959b2288ac9  9193db67743d08f418235fee43747bb9d42d62ce.patch"
-- 
2.21.0



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---
Michael Jeanson <mjeanson@efficios.com>
Details
Message ID
<2b09aa15-64d8-e3ae-a746-c0931a9ec886@efficios.com>
In-Reply-To
<20190321110229.78647928@ncopa-desktop.copa.dup.pw> (view parent)
Sender timestamp
1553179996
DKIM signature
missing
Download raw message
On 2019-03-21 6:02 a.m., Natanael Copa wrote:
> On Wed, 20 Mar 2019 12:14:53 -0500
> Michael Jeanson <mjeanson@efficios.com> wrote:
> 
> Hi!
>   
>> @@ -22,25 +22,30 @@ build() {
>>  	cd "$builddir"
>>  
>>  	unset LDFLAGS
>> +	autoreconf -i # Because we patch a Makefile.am
> 
> We also need to add libtool to makedepends for this.

I'll fix this.

> 
>>  	./configure \
>>  		--build=$CBUILD \
>>  		--host=$CHOST \
>>  		--prefix=/usr \
>>  		--sysconfdir=/etc \
>> -		--localstatedir=/var \
>> -		|| return 1
>> -	make V=1 || return 1
>> -	make -j1 check || return 1
>> +		--localstatedir=/var
>> +	make V=1
>> +}
>> +
>> +check() {
>> +	cd "$builddir"
>> +
>> +	make -j1 check
>>  }
> 
> The test suite hangs for me here.

You also need the updated ust version I posted earlier with the
_SC_NPROCESSORS_CONF patch. If it still fails can you tell me which test
hangs or send me the logs.

I run the full test suite here in a VM with the virt kernel without issues.

> 
> -nc
> 

Regards,

Michael


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---
Natanael Copa <ncopa@alpinelinux.org>
Details
Message ID
<20190321110229.78647928@ncopa-desktop.copa.dup.pw>
In-Reply-To
<20190320171453.13904-1-mjeanson@efficios.com> (view parent)
Sender timestamp
1553162549
DKIM signature
missing
Download raw message
On Wed, 20 Mar 2019 12:14:53 -0500
Michael Jeanson <mjeanson@efficios.com> wrote:

Hi!
  
> @@ -22,25 +22,30 @@ build() {
>  	cd "$builddir"
>  
>  	unset LDFLAGS
> +	autoreconf -i # Because we patch a Makefile.am

We also need to add libtool to makedepends for this.

>  	./configure \
>  		--build=$CBUILD \
>  		--host=$CHOST \
>  		--prefix=/usr \
>  		--sysconfdir=/etc \
> -		--localstatedir=/var \
> -		|| return 1
> -	make V=1 || return 1
> -	make -j1 check || return 1
> +		--localstatedir=/var
> +	make V=1
> +}
> +
> +check() {
> +	cd "$builddir"
> +
> +	make -j1 check
>  }

The test suite hangs for me here.

-nc


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