Forgot a REVISION bump there.

On 12/28/22 11:55, Volker Schlecht wrote:
Attached is a proposal to build benchmarks/tsung with erlang25.

I pulled in an upstream patch to fix an already failing testcase that was already reported by gnezdo@:
https://github.com/processone/tsung/issues/385

The only remaining failing testcase has apparently been failing already in previous versions.

benchmarks/tsung is the only remaining non-broken port that depends on erlang21, so if we could get tsung updated, we could possibly drop erlang/21 from ports ... it won't build databases/riak anyway.
Index: Makefile
===================================================================
RCS file: /cvs/ports/benchmarks/tsung/Makefile,v
retrieving revision 1.18
diff -u -p -r1.18 Makefile
--- Makefile	13 Nov 2022 15:27:39 -0000	1.18
+++ Makefile	28 Dec 2022 22:28:17 -0000
@@ -2,7 +2,7 @@ COMMENT=	multi-protocol distributed load
 
 V=		1.7.0
 DISTNAME=	tsung-$V
-REVISION=	3
+REVISION=	4
 
 CATEGORIES=	benchmarks net
 
@@ -15,7 +15,7 @@ MASTER_SITES=		http://tsung.erlang-proje
 
 MODULES=		lang/erlang \
 			lang/python
-MODERL_VERSION=		21
+MODERL_VERSION=		25
 MODPY_BUILDDEP= 	No
 
 RUN_DEPENDS=		graphics/py-matplotlib${MODPY_FLAVOR} \
Index: patches/patch-src_test_ts_test_utils_erl
===================================================================
RCS file: patches/patch-src_test_ts_test_utils_erl
diff -N patches/patch-src_test_ts_test_utils_erl
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ patches/patch-src_test_ts_test_utils_erl	28 Dec 2022 22:28:17 -0000
@@ -0,0 +1,14 @@
+Pull in an upstream fix for a failing testcase with erlang >= 21
+https://github.com/processone/tsung/commit/e53a59d16df5a2d3781ef9d4f0171ad0ca5bc931
+Index: src/test/ts_test_utils.erl
+--- src/test/ts_test_utils.erl.orig
++++ src/test/ts_test_utils.erl
+@@ -48,7 +48,7 @@ mkey1search_string_test()->
+     ?assertEqual(["bar","caps"],ts_utils:mkey1search(Data,"foo")).
+ 
+ datestr_test()->
+-    ?assertEqual(["2013","10","17",45,"19","41"],ts_utils:datestr({{2013,10,17},{19,41,29}})).
++    ?assertEqual("20131017-1941",lists:flatten(ts_utils:datestr({{2013,10,17},{19,41,29}}))).
+ 
+ export_text_test()->
+     ?assertEqual("foo",ts_utils:export_text("foo")).

Reply via email to