Update devel/readline to 8.0 - Bump PORTREVISION of dependent ports for shlib change Changes: https://tiswww.case.edu/php/chet/readline/CHANGES
please provide a patch that applies
In the URL line as before. https://people.FreeBSD.org/~sunpoet/patch/devel-readline.txt
this patch fails to apply
Regenerate/simplify the patch and remove all PORTREVISION bump. https://people.FreeBSD.org/~sunpoet/patch/devel-readline-v2.txt
New failure on 11.2 amd64: http://package22.nyi.freebsd.org/data/112amd64-default-PR236156-7-8/2019-03-05_21h33m35s/logs/errors/folks-0.11.4_1.log
Patch for devel/readline itself is updated. Request for another exp-run. net-im/folks builds successfully with this patch.
Exp-run looks fine
A commit references this bug: Author: sunpoet Date: Tue Apr 9 14:05:10 UTC 2019 New revision: 498476 URL: https://svnweb.freebsd.org/changeset/ports/498476 Log: Update devel/readline to 8.0 - Bump PORTREVISION of dependent ports for shlib change Changes: https://tiswww.case.edu/php/chet/readline/CHANGES PR: 236156 Exp-run by: antoine Changes: head/Mk/Uses/readline.mk head/Mk/bsd.ruby.mk head/audio/ardour5/Makefile head/audio/cutmp3/Makefile head/audio/ecasound/Makefile head/audio/fluidsynth/Makefile head/audio/guitarix-lv2/Makefile head/audio/jack/Makefile head/audio/kid3-qt5/Makefile head/audio/lash/Makefile head/audio/libsmf/Makefile head/audio/mpiosh/Makefile head/audio/ncmpcpp/Makefile head/audio/supercollider/Makefile head/audio/xmms2/Makefile head/audio/yoshimi/Makefile head/biology/mrbayes/Makefile head/cad/abc/Makefile head/cad/iverilog/Makefile head/cad/linuxcnc-devel/Makefile head/cad/ngspice_rework/Makefile head/comms/gnokii/Makefile head/comms/hamlib/Makefile head/comms/obexapp/Makefile head/comms/wsjtx/Makefile head/comms/zssh/Makefile head/databases/cego/Makefile head/databases/clickhouse/Makefile head/databases/freetds/Makefile head/databases/gdbm/Makefile head/databases/gqlplus/Makefile head/databases/grass7/Makefile head/databases/libgda5/Makefile head/databases/libodbc++/Makefile head/databases/mariadb101-server/Makefile head/databases/mariadb102-server/Makefile head/databases/mariadb103-server/Makefile head/databases/mariadb55-client/Makefile head/databases/mdbtools/Makefile head/databases/mysql55-server/Makefile head/databases/opendbx/Makefile head/databases/percona55-server/Makefile head/databases/pg_citus/Makefile head/databases/postgres-xl/Makefile head/databases/postgresql11-server/Makefile head/databases/postgresql95-plperl/Makefile head/databases/pspg/Makefile head/databases/riak/Makefile head/databases/sqlcached/Makefile head/databases/sqlite2/Makefile head/databases/sqlite3/Makefile head/databases/sqlitestudio/Makefile head/databases/sqlrelay/Makefile head/databases/sqsh/Makefile head/databases/tarantool/Makefile head/databases/virtuoso/Makefile head/databases/xrootd/Makefile head/deskutils/genius/Makefile head/deskutils/pal/Makefile head/deskutils/tasksh/Makefile head/devel/ald/Makefile head/devel/arduino-avrdude/Makefile head/devel/avrdude/Makefile head/devel/bossa/Makefile head/devel/cdecl/Makefile head/devel/cgdb/Makefile head/devel/clewn/Makefile head/devel/devtodo/Makefile head/devel/elf/Makefile head/devel/elfsh/Makefile head/devel/gdb/Makefile head/devel/gnatcoll/Makefile head/devel/grv/Makefile head/devel/hs-readline/Makefile head/devel/libepp-nicbr/Makefile head/devel/libnfc/Makefile head/devel/libreadline-java/Makefile head/devel/libvirt/Makefile head/devel/mspdebug/Makefile head/devel/omake/Makefile head/devel/opendht/Makefile head/devel/openzz/Makefile head/devel/p5-Term-ReadLine-Gnu/Makefile head/devel/pcre/Makefile head/devel/pcre2/Makefile head/devel/physfs/Makefile head/devel/pure-readline/Makefile head/devel/readline/Makefile head/devel/readline/distinfo head/devel/readline/pkg-plist head/devel/remake/Makefile head/devel/rlwrap/Makefile head/devel/rudiments/Makefile head/devel/tclreadline/Makefile head/devel/tdl/Makefile head/devel/tig/Makefile head/devel/urjtag/Makefile head/devel/yosys/Makefile head/emulators/bochs/Makefile head/emulators/hatari/Makefile head/emulators/vice/Makefile head/ftp/hsftp/Makefile head/ftp/lftp/Makefile head/ftp/quftp/Makefile head/ftp/yafc/Makefile head/games/eif/Makefile head/games/freeciv/Makefile head/games/ggz-txt-client/Makefile head/games/gnubg/Makefile head/games/gnurobots/Makefile head/games/hoichess/Makefile head/games/wesnoth/Makefile head/graphics/bugle/Makefile head/graphics/gphoto2/Makefile head/graphics/s10sh/Makefile head/lang/Gofer/Makefile head/lang/algol68g/Makefile head/lang/chaiscript/Makefile head/lang/cint/Makefile head/lang/gawk/Makefile head/lang/gjs/Makefile head/lang/guile2/Makefile head/lang/io/Makefile head/lang/kawa/Makefile head/lang/lci/Makefile head/lang/librep/Makefile head/lang/lua53/Makefile head/lang/mdk/Makefile head/lang/micropython/Makefile head/lang/mujs/Makefile head/lang/newlisp/Makefile head/lang/newlisp-devel/Makefile head/lang/nickle/Makefile head/lang/ohugs/Makefile head/lang/php71/Makefile head/lang/php72/Makefile head/lang/php73/Makefile head/lang/picoc/Makefile head/lang/python27/Makefile head/lang/python35/Makefile head/lang/python36/Makefile head/lang/python37/Makefile head/lang/rubinius/Makefile head/lang/scm/Makefile head/lang/sdcc/Makefile head/lang/spidermonkey17/Makefile head/lang/spidermonkey170/Makefile head/lang/spidermonkey185/Makefile head/lang/spidermonkey24/Makefile head/lang/spidermonkey38/Makefile head/lang/spidermonkey52/Makefile head/lang/swi-pl/Makefile head/lang/yap/Makefile head/lang/yap-devel/Makefile head/mail/abook/Makefile head/mail/mailfromd/Makefile head/mail/mailutils/Makefile head/mail/nmh/Makefile head/mail/nmzmail/Makefile head/math/R/Makefile head/math/SCIP/Makefile head/math/aamath/Makefile head/math/asymptote/Makefile head/math/bcal/Makefile head/math/calc/Makefile head/math/cvc4/Makefile head/math/eukleides/Makefile head/math/eval/Makefile head/math/fityk/Makefile head/math/giacxcas/Makefile head/math/gnuplot/Makefile head/math/gretl/Makefile head/math/gri/Makefile head/math/lambda/Makefile head/math/libqalculate/Makefile head/math/mathomatic/Makefile head/math/octave/Makefile head/math/pari/Makefile head/math/pspp/Makefile head/math/py-rpy2/Makefile head/math/sfst/Makefile head/math/spar/Makefile head/math/topaz/Makefile head/math/wcalc/Makefile head/misc/lxi-tools/Makefile head/misc/nnn/Makefile head/misc/pinfo/Makefile head/misc/tempcontrol/Makefile head/misc/ytree/Makefile head/multimedia/xine/Makefile head/net/afpfs-ng/Makefile head/net/bird/Makefile head/net/bird2/Makefile head/net/c3270/Makefile head/net/ccxstream/Makefile head/net/freeradius3/Makefile head/net/frr3/Makefile head/net/frr4/Makefile head/net/frr5/Makefile head/net/frr6/Makefile head/net/glusterfs/Makefile head/net/kamailio/Makefile head/net/msend/Makefile head/net/nfsshell/Makefile head/net/ocserv/Makefile head/net/quagga/Makefile head/net/samba46/Makefile head/net/samba47/Makefile head/net/samba48/Makefile head/net/socat/Makefile head/net/yaz/Makefile head/net/zebra/Makefile head/net-im/folks/Makefile head/net-im/freetalk/Makefile head/net-im/profanity/Makefile head/net-im/telegram/Makefile head/net-im/zephyr/Makefile head/net-mgmt/argus3-clients/Makefile head/net-mgmt/scli/Makefile head/net-mgmt/send/Makefile head/net-p2p/amule/Makefile head/net-p2p/amule-devel/Makefile head/net-p2p/microdc2/Makefile head/net-p2p/monero-cli/Makefile head/net-p2p/shx/Makefile head/palm/pilot-link/Makefile head/polish/ekg2/Makefile head/polish/kadu/Makefile head/print/fontforge/Makefile head/science/gnudatalanguage/Makefile head/science/qcl/Makefile head/science/silo/Makefile head/security/authforce/Makefile head/security/gnomint/Makefile head/security/gnupg/Makefile head/security/gnupg1/Makefile head/security/heimdal/Makefile head/security/kc24/Makefile head/security/krb5-115/Makefile head/security/krb5-116/Makefile head/security/krb5-117/Makefile head/security/krb5-devel/Makefile head/security/osiris/Makefile head/security/ossec-hids-local/Makefile head/security/softether/Makefile head/security/softether-devel/Makefile head/security/softether5/Makefile head/security/tinc-devel/Makefile head/security/wpa_supplicant/Makefile head/shells/es/Makefile head/shells/esh/Makefile head/shells/pear-PHP_Shell/Makefile head/shells/rc/Makefile head/shells/sparforte/Makefile head/sysutils/afflib/Makefile head/sysutils/aimage/Makefile head/sysutils/android-file-transfer/Makefile head/sysutils/apachetop/Makefile head/sysutils/bacula9-server/Makefile head/sysutils/bareos-server/Makefile head/sysutils/bareos16-server/Makefile head/sysutils/djmount/Makefile head/sysutils/hstr/Makefile head/sysutils/ipmitool/Makefile head/sysutils/lnav/Makefile head/sysutils/openipmi/Makefile head/sysutils/pwsafe/Makefile head/sysutils/renameutils/Makefile head/sysutils/riak-cs/Makefile head/sysutils/slurm-wlm/Makefile head/sysutils/stanchion/Makefile head/sysutils/xfsprogs/Makefile head/sysutils/xorriso/Makefile head/textproc/augeas/Makefile head/textproc/hunspell/Makefile head/textproc/refdb/Makefile head/textproc/sdcv/Makefile head/textproc/ucto/Makefile head/www/edbrowse/Makefile head/www/itop/Makefile head/www/matomo/Makefile head/www/mediawiki127/Makefile head/www/mediawiki130/Makefile head/www/mediawiki131/Makefile head/www/mediawiki132/Makefile head/www/varnish4/Makefile head/www/varnish6/Makefile head/x11-wm/fvwm2/Makefile head/x11-wm/ratpoison/Makefile
Committed. Thanks!
I think PORTREVISION bump was meant for databases/postgresql11-client, not for -server. psql is still linked to libreadline.so.7 and doesn't start after this update: $ readelf -d /usr/local/bin/psql | grep readline 0x0000000000000000 NEEDED Shared library: [libreadline.so.7]
A commit references this bug: Author: sunpoet Date: Wed Apr 10 20:29:18 UTC 2019 New revision: 498593 URL: https://svnweb.freebsd.org/changeset/ports/498593 Log: Bump PORTREVISION for devel/readline shlib change PR: 236156 Notified by: Dmitri Goutnik <dg@syrec.org> Changes: head/databases/postgresql11-client/Makefile
(In reply to Dmitri Goutnik from comment #10) Thanks. It should be fixed in r498593.
Re-open by user request (via IRC), using default postgresql95{-client}, reporting that it and other postgresql versions have not had their PORTREVISION's bumped. The full list of (readline) dependents should be swept again, at least for postgresql ports.
Specifically, the list of ports to bump was clearly obtained by grepping the port makefiles, which is an inadequate approach given the existence of the .include directive. The actual dependencies should have been checked instead.
CC pgsql@ for a headsup/feedback (See comment 10, comment 13)
At least for postgresql95, the -server port appears to override the PORTREVISION in -client, that is to say that PORTREVISION in postgresql95-client (at least) appears to be ignored.
A commit references this bug: Author: crees Date: Thu Apr 25 18:34:45 UTC 2019 New revision: 500060 URL: https://svnweb.freebsd.org/changeset/ports/500060 Log: Don't overwrite PORTREVISION from the slave, following readline update. I've had to bump revision for several slaves here, but most will not be rebuilt, except the -client slaves. Apologies for anyone having to rebuild -clients unnecessarily, but it's not a heavy task- better safe than sorry. PR: ports/236156 Reported by: Andrew Dunstan (PostgreSQL), koobs, Dmitri Goutnik Changes: head/databases/postgresql10-client/Makefile head/databases/postgresql10-contrib/Makefile head/databases/postgresql10-pgtcl/Makefile head/databases/postgresql10-server/Makefile head/databases/postgresql11-client/Makefile head/databases/postgresql94-server/Makefile head/databases/postgresql95-client/Makefile head/databases/postgresql95-server/Makefile head/databases/postgresql96-client/Makefile head/databases/postgresql96-contrib/Makefile head/databases/postgresql96-docs/Makefile head/databases/postgresql96-pgtcl/Makefile head/databases/postgresql96-pltcl/Makefile head/databases/postgresql96-server/Makefile
Committed, thanks for the headsup. Finding dependencies is easily and correctly done by grepping INDEX, but that misses out OPTIONS. Grepping Makefiles matches OPTIONS, but can break slaves as we see here. If you have a slave port that has OPTIONS, you're basically stuffed as far as I can see :)
only databases/postgresql??-client links with libreadline AFAIK. They are covered by the latset commit.