[gentoo-commits] repo/gentoo:master commit in: net-im/bitlbee/files/

[gentoo-commits] repo/gentoo:master commit in: net-im/bitlbee/files/

[Sam James]

commit:     9234f9e1d6a7a7c9b4bd9f4fd11c33b1bd33bee1
Author:     Petr Vaněk <arkamar <AT> atlas <DOT> cz>
AuthorDate: Thu Aug 18 09:25:51 2022 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Tue Aug 23 04:50:22 2022 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9234f9e1

net-im/bitlbee: fix tests in 3.6-r3 and live

It was necessary to adapt system-json-parser patches more with
LDFLAGS_CHECK variable which is used when ./tests/check binary is
linked.

Fixes: 90a3d5fff1c0 ("net-im/bitlbee: Use system json-parser")
Fixes: 2f4aeedeec9c ("net-im/bitlbee: sync live")
Closes: https://bugs.gentoo.org/865667
Signed-off-by: Petr Vaněk <arkamar <AT> atlas.cz>
Signed-off-by: Sam James <sam <AT> gentoo.org>

 .../files/bitlbee-3.6-system-json-parser.patch     | 23 ++++++++++++++++---
 .../files/bitlbee-9999-system-json-parser.patch    | 26 +++++++++++++++++-----
 2 files changed, 41 insertions(+), 8 deletions(-)

diff --git a/net-im/bitlbee/files/bitlbee-3.6-system-json-parser.patch b/net-im/bitlbee/files/bitlbee-3.6-system-json-parser.patch
index 02d57e02d6d2..0a9c197d763d 100644
--- a/net-im/bitlbee/files/bitlbee-3.6-system-json-parser.patch
+++ b/net-im/bitlbee/files/bitlbee-3.6-system-json-parser.patch
@@ -1,13 +1,16 @@
 This is adapted patch from upstream PR 151. Original PR defines
 --external-json-parser which breaks configuration, see [1]. It was changed to
 --externaljsonparser and $external_json_parser to $externaljsonparser.
+Additionally, LDFLAGS_CHECK variable was introduced in order to make
+tests working, see [2] and #865667.
 
 [1] https://github.com/bitlbee/bitlbee/pull/151#issuecomment-1201181905
+[2] https://github.com/bitlbee/bitlbee/pull/151#issuecomment-1219254239
 
 Upstream-PR: https://github.com/bitlbee/bitlbee/pull/151
 
 diff --git a/configure b/configure
-index 4640c806..77b1213b 100755
+index fd8d363a..39495515 100755
 --- a/configure
 +++ b/configure
 @@ -47,6 +47,7 @@ plugins=1
@@ -26,7 +29,7 @@ index 4640c806..77b1213b 100755
  
  
  --target=...	Cross compilation target 		same as host
-@@ -398,6 +400,20 @@ else
+@@ -398,6 +400,21 @@ else
  fi
  echo 'EVENT_HANDLER=events_'$events'.o' >> Makefile.settings
  
@@ -41,13 +44,14 @@ index 4640c806..77b1213b 100755
 +if [ "$externaljsonparser" = "1" ]; then
 +    echo "CFLAGS+=$(pkg-config --cflags json-parser)" >> Makefile.settings
 +    echo "LDFLAGS_BITLBEE+=$(pkg-config --libs json-parser)" >> Makefile.settings
++    echo "LDFLAGS_CHECK+=$(pkg-config --libs json-parser)" >> Makefile.settings
 +fi
 +
 +
  detect_gnutls()
  {
  	if $PKG_CONFIG --exists gnutls; then
-@@ -979,6 +995,12 @@ else
+@@ -979,6 +996,12 @@ else
  	echo '  systemd disabled.'
  fi
  
@@ -77,6 +81,19 @@ index fc46aedc..ad95d9cb 100644
  
  LFLAGS += -r
  
+diff --git a/tests/Makefile b/tests/Makefile
+index 76122a2f..2235bf1d 100644
+--- a/tests/Makefile
++++ b/tests/Makefile
+@@ -20,7 +20,7 @@ test_objs = check.o check_util.o check_nick.o check_md5.o check_arc.o check_irc.
+ 
+ check: $(test_objs) $(addprefix ../, $(main_objs)) ../protocols/protocols.o ../lib/lib.o
+ 	@echo '*' Linking $@
+-	$(VERBOSE) $(CC) $(CFLAGS) -o $@ $^ $(LFLAGS) $(EFLAGS)
++	$(VERBOSE) $(CC) $(CFLAGS) -o $@ $^ $(LFLAGS) $(LDFLAGS_CHECK) $(EFLAGS)
+ 
+ %.o: $(_SRCDIR_)%.c
+ 	@echo '*' Compiling $<
 -- 
 2.35.1
 

diff --git a/net-im/bitlbee/files/bitlbee-9999-system-json-parser.patch b/net-im/bitlbee/files/bitlbee-9999-system-json-parser.patch
index 07f2878525ec..d0020307cfa3 100644
--- a/net-im/bitlbee/files/bitlbee-9999-system-json-parser.patch
+++ b/net-im/bitlbee/files/bitlbee-9999-system-json-parser.patch
@@ -1,13 +1,16 @@
 This is adapted patch from upstream PR 151. Original PR defines
 --external-json-parser which breaks configuration, see [1]. It was changed to
 --externaljsonparser and $external_json_parser to $externaljsonparser.
+Additionally, LDFLAGS_CHECK variable was introduced in order to make
+tests working, see [2] and #865667.
 
 [1] https://github.com/bitlbee/bitlbee/pull/151#issuecomment-1201181905
+[2] https://github.com/bitlbee/bitlbee/pull/151#issuecomment-1219254239
 
 Upstream-PR: https://github.com/bitlbee/bitlbee/pull/151
 
 diff --git a/configure b/configure
-index 4640c806..77b1213b 100755
+index 2291e9a4..3ba40933 100755
 --- a/configure
 +++ b/configure
 @@ -47,6 +47,7 @@ plugins=1
@@ -18,7 +21,7 @@ index 4640c806..77b1213b 100755
  ssl=auto
  
  pam=0
-@@ -148,6 +149,7 @@ Option		Description				Default
+@@ -154,6 +155,7 @@ Option		Description				Default
  --events=...	Event handler (glib, libevent)		$events
  --ssl=...	SSL library to use (gnutls, nss, openssl, auto)
  							$ssl
@@ -26,7 +29,7 @@ index 4640c806..77b1213b 100755
  
  
  --target=...	Cross compilation target 		same as host
-@@ -398,6 +400,20 @@ else
+@@ -404,6 +406,21 @@ else
  fi
  echo 'EVENT_HANDLER=events_'$events'.o' >> Makefile.settings
  
@@ -41,13 +44,14 @@ index 4640c806..77b1213b 100755
 +if [ "$externaljsonparser" = "1" ]; then
 +    echo "CFLAGS+=$(pkg-config --cflags json-parser)" >> Makefile.settings
 +    echo "LDFLAGS_BITLBEE+=$(pkg-config --libs json-parser)" >> Makefile.settings
++    echo "LDFLAGS_CHECK+=$(pkg-config --libs json-parser)" >> Makefile.settings
 +fi
 +
 +
  detect_gnutls()
  {
  	if $PKG_CONFIG --exists gnutls; then
-@@ -979,6 +995,12 @@ else
+@@ -993,6 +1010,12 @@ else
  	echo '  systemd disabled.'
  fi
  
@@ -60,7 +64,6 @@ index 4640c806..77b1213b 100755
  echo '  Using python: '$PYTHON
  echo '  Using event handler: '$events
  echo '  Using SSL library: '$ssl
- #echo '  Building with these storage backends: '$STORAGES
 diff --git a/lib/Makefile b/lib/Makefile
 index fc46aedc..ad95d9cb 100644
 --- a/lib/Makefile
@@ -78,6 +81,19 @@ index fc46aedc..ad95d9cb 100644
  
  LFLAGS += -r
  
+diff --git a/tests/Makefile b/tests/Makefile
+index 76122a2f..2235bf1d 100644
+--- a/tests/Makefile
++++ b/tests/Makefile
+@@ -20,7 +20,7 @@ test_objs = check.o check_util.o check_nick.o check_md5.o check_arc.o check_irc.
+ 
+ check: $(test_objs) $(addprefix ../, $(main_objs)) ../protocols/protocols.o ../lib/lib.o
+ 	@echo '*' Linking $@
+-	$(VERBOSE) $(CC) $(CFLAGS) -o $@ $^ $(LFLAGS) $(EFLAGS)
++	$(VERBOSE) $(CC) $(CFLAGS) -o $@ $^ $(LFLAGS) $(LDFLAGS_CHECK) $(EFLAGS)
+ 
+ %.o: $(_SRCDIR_)%.c
+ 	@echo '*' Compiling $<
 -- 
 2.35.1