From: Stuart Cassoff Subject: Re: subversion weirdness To: Stuart Henderson Cc: Stefan Sperling , OpenBSD Ports Date: Wed, 14 Jan 2026 10:12:15 -0500 Like this? Remove incorrect and unneeded UPDATE_PLIST_ARGS= -I MODPY_WANTLIB that was breaking update-plist. diff -Nurp /usr/ports/devel/subversion/Makefile ./Makefile --- /usr/ports/devel/subversion/Makefile Tue Jul 22 08:45:05 2025 +++ ./Makefile Wed Jan 14 10:06:24 2026 @@ -184,7 +184,6 @@ CONFIGURE_ARGS+=--without-gnome-keyring TEST_DEPENDS=${MODPY_RUN_DEPENDS} TEST_DEPENDS+= ${FULLPKGNAME-gnome-keyring}:${BUILD_PKGPATH},-gnome-keyring SUBST_VARS+= LOCALBASE SO_VERSION MODRUBY_BINREV -UPDATE_PLIST_ARGS= -I MODPY_WANTLIB pre-patch: @ln -s ${MODPY_BIN} ${WRKDIR}/bin/python Return Ruby subpackage to the unflavored build. Removed in r1.198 due to build failure with clang 15. Ok now with clang 19. diff -Nurp /usr/ports/devel/subversion/Makefile ./Makefile --- /usr/ports/devel/subversion/Makefile Wed Jan 14 10:06:24 2026 +++ ./Makefile Wed Jan 14 10:08:17 2026 @@ -43,7 +43,7 @@ EXTRACT_SUFX= .tar.bz2 PSEUDO_FLAVORS= no_bindings no_ap2 no_perl no_python no_ruby no_gnome-keyring FLAVORS= maintainer_mode -FLAVOR?= no_ruby +FLAVOR?= .if ${FLAVOR:Mno_bindings} FLAVOR += no_perl no_python no_ruby .endif @@ -55,7 +55,7 @@ WANTLIB= expat iconv intl lz4 m pthread z BUILD_DEPENDS= devel/gettext,-tools \ devel/py3c -MULTI_PACKAGES = -main -ap2 -perl -python -gnome-keyring +MULTI_PACKAGES = -main -ap2 -perl -python -ruby -gnome-keyring MAINSPEC = ${PKGNAME}:devel/subversion,-main LIB_DEPENDS-main = ${LIB_DEPENDS} \ On Wed, 14 Jan 2026, Stuart Henderson wrote: > On 2026/01/14 09:15, Stuart Cassoff wrote: > > Ruby bindings are a bit stale (lots of deprecation warnings) > > but otherwise build and test ok on i386 and amd64. > > > > If the port were left as-is, would -ruby need to > > be conditionally added to MULTI_PACKAGES? > > > > > > Stu > > > > > > > > Return Ruby subpackage to the unflavored build. > > Removed in r1.198 due to build failure with clang 15. > > Ok now with clang 19. > > > > Remove incorrect and unneeded > > UPDATE_PLIST_ARGS= -I MODPY_WANTLIB > > that was breaking update-plist. > > > > if readding them: > > > > > diff -Nurp /usr/ports/devel/subversion/Makefile ./Makefile > > --- /usr/ports/devel/subversion/Makefile Tue Jul 22 08:45:05 2025 > > +++ ./Makefile Tue Jan 13 08:35:45 2026 > > @@ -43,7 +43,7 @@ EXTRACT_SUFX= .tar.bz2 > > > > PSEUDO_FLAVORS= no_bindings no_ap2 no_perl no_python no_ruby no_gnome-keyring > > FLAVORS= maintainer_mode > > -FLAVOR?= no_ruby > > +FLAVOR?= > > .if ${FLAVOR:Mno_bindings} > > FLAVOR += no_perl no_python no_ruby > > .endif > > @@ -55,7 +55,7 @@ WANTLIB= expat iconv intl lz4 m pthread z > > BUILD_DEPENDS= devel/gettext,-tools \ > > devel/py3c > > > > -MULTI_PACKAGES = -main -ap2 -perl -python -gnome-keyring > > +MULTI_PACKAGES = -main -ap2 -perl -python -ruby -gnome-keyring > > that would be one commit > > > MAINSPEC = ${PKGNAME}:devel/subversion,-main > > LIB_DEPENDS-main = ${LIB_DEPENDS} \ > > @@ -184,7 +184,6 @@ CONFIGURE_ARGS+=--without-gnome-keyring > > TEST_DEPENDS=${MODPY_RUN_DEPENDS} > > TEST_DEPENDS+= ${FULLPKGNAME-gnome-keyring}:${BUILD_PKGPATH},-gnome-keyring > > SUBST_VARS+= LOCALBASE SO_VERSION MODRUBY_BINREV > > -UPDATE_PLIST_ARGS= -I MODPY_WANTLIB > > and that is unrelated to ruby so would be a separate commit > > > pre-patch: > > @ln -s ${MODPY_BIN} ${WRKDIR}/bin/python > > > > > > On Sat, 10 Jan 2026, Stuart Henderson wrote: > > > > > that should have been -i not -I, but it's not doing anything useful anyway, so just drop it. > > > > > > update-plist(1) > > > > > > --ᅵ > > > ᅵ Sent from a phone, apologies for poor formatting. > > > > > > > > > On 10 January 2026 06:15:01 Stuart Cassoff wrote: > > > > > > Removing UPDATE_PLIST_ARGS unbreaks update-plist. > > > > > > "UPDATE_PLIST_ARGS= -I MODPY_WANTLIB" > > > > > > I can't see why this is causing breakage. > > > > > > Ultimately unrelated to ruby. > > > > > > > > > On Wed, 7 Jan 2026, Stuart Henderson wrote: > > > > > > On 2026/01/07 09:30, Stuart Cassoff wrote: > > > $ make update-plist > > > ===> ᅵUpdating plist for subversion-1.14.5 > > > Installing /usr/ports/mystuff/devel/subversion/pkg/svnserve.rc as /usr/perts/pobj/subversion-1.14.5/fake-amd64/etc/rc.d/svnserve > > > Usage: update-plist [-FfmnOrvx] [-C dir] [-c comment] [-E ext] [-e ext] [-H hints] > > > ᅵ[-i var] [-I ignored] [-j jobs] [-L logfile] [-P pkgdir] > > > ᅵ[-S var] [-s var] [-V var] [-w suffix] [-X path] > > > ᅵ-- pkg_create_args ... > > > > > > > > > I think it's due to how the ruby bindings were knocked out (after > > > failing to build with newer clang). > > > > > > > > > > > > > > > > >