Created attachment 218315 [details] Patch for irssi (WIP) Update irssi to snapshot 1.3.20200911 Includes various bug fixes including ones related to Glib and SSL/TLS Compile and run-tested FreeBSD 13.0-CURRENT #0 r364979 (AMD64) Poudriere testport OK 12.1-RELEASE (amd64)
I forgot to mention, this also switches to Meson Known issues: Capsicum fail during configure stage: Compiler stderr: /usr/ports/irc/irssi/work/irssi-561619f/_build/meson-private/tmp4ziugp_n/testfile.c:17:18: error: "__builtin_nvlist_create_found not found" #error "__builtin_nvlist_create_found not found" ^ 1 error generated. Portlint complains about TRUECOLOR option and I can't figure out why WARN: Makefile: TRUECOLOR is listed in OPTIONS_DEFINE, but no PORT_OPTIONS:MTRUECOLOR appears. Help appreciated Additional notes, I know that not using releases aren't ideal but upstream at least for now has no intention releasing a new version.
Created attachment 218324 [details] Patch for irssi v2 (WIP) Use naming schema in used in example 5.14 (Porters Handbook), suggested by jrm@ Use YES/NO ports framework logic (5.13.3.5.3 in Ports handbook), suggested by jrm@ (I missed that paragraph when I looked at it) portlint till complains about TRUECOLOR (seems to be a false positive) Capsicum still doesn't build
Created attachment 218328 [details] Patch for irssi v3 Fixes compilation with capsicum
^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field. Thanks!
Hi, There isn't one except for the commit log which is too long (GitHub cuts it at 250 commits) https://github.com/irssi/irssi/compare/1.2.2...master Best regards, Daniel
Regarding users: irssi-dcc_send_limiter - Build, Run dep (testport OK) irssi-fish - Build, Run dep (testport OK) irssi-icb - Build, Run dep (testport OK) irssi-xmpp - Doesn't compile with new Glib and seems to be broken in general looking at GitHub irssi-scripts - Just copies files (scripts) p5-POE-Session-Irssi - Run dep twirssi - Run dep irssi-themes - Run dep
Reference to glib fix: https://github.com/irssi/irssi/issues/1180
Friendly ping
Hi, Apologies for the silence here, things have been busy recently. I should have some time to look at this properly later in the week. Thanks, -David
Sounds great, thanks!
Any progress on this? Best regards, Daniel
maintainer timeout, 24d
Sorry about this, I forgot again with a ton of stuff happening recently. I'm not sure how comfortable I am to be chasing arbitrary upstream commits. It would be great if they could just tag a new release :( I'll try to give this patch a test later, definitely this time. -David
Upstream doesn't seem to have any intentions of tagging a new version anytime soon. https://github.com/irssi/irssi/issues/1222 If you want to cherry-pick commits instead that's fine with me although I would expect that it would quickly become quite messy.
Unfortunately, I cannot build this patch in a FreeBSD 12.2 `poudriere testport` at the moment. The build fails while testing to see if Perl is available: Checking if "working Perl support" compiles: NO meson.build:377:6: ERROR: Problem encountered: error linking with perl libraries Everything looks correct, and the problem seems to be that -Werror=unused-command-line-argument is being introduced to the cc command line somewhere. This is causing the following error during the test for Perl: cc: error: -Wl,-R/usr/local/lib/perl5/5.32/mach/CORE: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -Wl,-E: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -Wl,--start-group: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -lperl: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -lpthread: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -lm: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -lcrypt: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -lutil: 'linker' input unused [-Werror,-Wunused-command-line-argument] cc: error: -Wl,--end-group: 'linker' input unused [-Werror,-Wunused-command-line-argument] Manipulating the command line in an interactive poudriere session to add -Wno-unused-command-line-argument to the end of the build command allows this particular test to pass. Sadly, adding that same argument to CPPFLAGS does not work to avoid this issue as the arguments are order dependent, and -Wno-unused-command-line-argument appears before -Werror=unused-command-line-argument in the command. The full failing command is: cc -I/usr/local/include -I/usr/local/lib/perl5/5.32/mach/CORE -L/usr/local/lib -L/usr/local/lib/perl5/5.32/mach/CORE -I/usr/include /wrkdirs/usr/ports/irc/irssi/work/irssi-17c6c22/_build/meson-private/tmpskfkg4ld/testfile.c -o /wrkdirs/usr/ports/irc/irssi/work/irssi-17c6c22/_build/meson-private/tmpskfkg4ld/output.obj -pipe -c -O2 -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing -isystem /usr/local/include -D_FILE_OFFSET_BITS=64 -O0 -Werror=unknown-warning-option -Werror=unused-command-line-argument -Werror=ignored-optimization-argument -DHAS_FPSETMASK -DHAS_FLOATINGPOINT_H -DUSE_THREAD_SAFE_LOCALE -fno-strict-aliasing -fstack-protector-strong -fPIC -Wl,-R/usr/local/lib/perl5/5.32/mach/CORE -pthread -Wl,-E -fstack-protector-strong -Wl,--start-group -lperl -lpthread -lm -lcrypt -lutil -Wl,--end-group I am unsure where the various -W flags are being introduced. grepping both the irssi source and /usr/ports/Mk doesn't turn anything up, nor does manually executing some of the commands that Meson seems to be running to work out the Perl LDFLAGS and CFLAGS. I believe it's being introduced as an entry in perl_rpath_flags, but as I'm unfamiliar with Meson, I'm having real issues debugging it. -David
David, I think the -Werror... comes from meson: $ pkg info -lq meson | xargs grep -n unused-command Binary file /usr/local/lib/python3.7/site-packages/mesonbuild/compilers/mixins/__pycache__/clang.cpython-37.opt-1.pyc matches Binary file /usr/local/lib/python3.7/site-packages/mesonbuild/compilers/mixins/__pycache__/clang.cpython-37.pyc matches /usr/local/lib/python3.7/site-packages/mesonbuild/compilers/mixins/clang.py:83: myargs.extend(['-Werror=unknown-warning-option', '-Werror=unused-command-line-argument']) I know little about meson and cannot recommend anything on it. It appears to be misguided though because this way meson commits to chasing each and every compiler warning behaviour change.
The quick and dirty workaround for this meson idiosyncrasy is USE_GCC=yes.
Breakage is caused by Meson 0.56.0 , it works fine with previous version 0.55.3 I'll have a look during the weekend.
Meson 0.56.0 build failure is being tracked here: https://github.com/irssi/irssi/issues/1235
Cool, I've subscribed to the Github issue so I can be alerted when upstream decide fixes this issue. I may also take a look at patching the meson.build or porting specific patches to 1.2.x, but I think at this point I'd rather take the 1.3-dev route as these Meson changes are going to be needed in the future anyway, and you've already taken the time to do the work :) -David
Created attachment 219689 [details] Patch for irssi v4 Update to 1.3 snapshot 20201114 Add fix for meson 0.56.0 an higher Reference: https://github.com/irssi/irssi/issues/1235 Compile and run-tested FreeBSD 13.0-CURRENT #0 r364979 (AMD64) Poudriere testport OK 12.2-RELEASE (amd64)
There's something odd going on with FreeBSD 11.4, both capsicum and openssl fails to link properly. I would appreciate if someone could have a look at it.
Created attachment 219719 [details] Patch for irssi v5 Fixes compilation on FreeBSD 11 (11.4 tested) Force lld linker as bfd breaks compilation Compile and run-tested FreeBSD 13.0-CURRENT #0 r364979 (amd64) Run-time tested on 11.4 by CrtxReavr (irc) (amd64) Poudriere testport OK 12.2-RELEASE (amd64) Poudriere testport OK 11.4-RELEASE (amd64)
Friendly ping :-)
Committed, Thanks!
A commit references this bug: Author: fernape Date: Sun Nov 22 18:33:13 UTC 2020 New revision: 556069 URL: https://svnweb.freebsd.org/changeset/ports/556069 Log: irc/irssi: Update to snapshot 1.3 (20201114) Includes various bug fixes including ones related to Glib and SSL/TLS PR: 249910 Submitted by: daniel.engberg.lists@pyret.net Approved by: dor.bsd@xm0.uk (maintainer) Changes: head/irc/irssi/Makefile head/irc/irssi/distinfo head/irc/irssi/files/patch-Makefile.in head/irc/irssi/files/patch-configure.ac head/irc/irssi/files/patch-meson.build head/irc/irssi/files/patch-perl-Makefile head/irc/irssi/pkg-plist
It would be nice to mention (pkg-message?) that you now have to explicitly load the perl module using ~/.irssi/startup according to https://github.com/irssi/irssi/issues/1203#issuecomment-651986498 as it somewhat breaks the POLA.
A commit references this bug: Author: fernape Date: Tue Nov 24 12:33:00 UTC 2020 New revision: 556156 URL: https://svnweb.freebsd.org/changeset/ports/556156 Log: irc/irssi: Add pkg-message Building with meson and ninja requires to load perl in the configuration file. PR: 249910 Reported by: yuripv@yuripv.dev Changes: head/irc/irssi/pkg-message
(In reply to Yuri Pankov from comment #27) Done. Thanks!
This is badly phrased as Perl support is optional.
(In reply to daniel.engberg.lists from comment #30) Well, it seems since the change to use meson, perl is not optional anymore. See bug #251371
Just a heads up, looks like development has come to a halt in main repo and (main active?) releases are being done in https://github.com/ailin-nemui/irssi I have a patch if someone wants to give it a go