Date: Tue, 8 Jul 2025 03:18:23 +0200 From: "Herbert J. Skuhra" <herbert@gojira.at> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: Dima Panov <fluffy@freebsd.org>, Cy Schubert <cy@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 7e35117eb07f - main - Makefile: Hook MIT KRB5 into the build Message-ID: <aGxx371-HV384A0r@mail.bsd4all.net> In-Reply-To: <20250707230446.F279E3A1@slippy.cwsent.com> References: <202506160251.55G2pwx4063231@gitrepo.freebsd.org> <cc9222b0-8745-48cf-988c-4206287a7a7b@FreeBSD.org> <20250620073050.7f03f74e@slippy> <ea078103-ece6-4f00-af38-ec5688bef7db@FreeBSD.org> <aGwv06rGlgW8zmJI@mail.bsd4all.net> <20250707230446.F279E3A1@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Jul 07, 2025 at 04:04:46PM -0700, Cy Schubert wrote: > In message <aGwv06rGlgW8zmJI@mail.bsd4all.net>, "Herbert J. Skuhra" writes: > > On Mon, Jul 07, 2025 at 10:36:31PM +0300, Dima Panov wrote: > > > Hello! > > > > > > > > > BTW, it looks like symlink for libkadm5clnt.so is broken and produced a bad > > file > > > $ ls -la libkadm5clnt > > > lrwxr-xr-x 1 root wheel 13 7 Jul 20:40 libkadm5clnt -> kadm5clnt_mit > > > > > > instead of assumed libkadm5clnt.so -> kadm5clnt_mit.so > > > > > > > > > Please review: > > > > > > diff --git a/krb5/lib/kadm5clnt/Makefile b/krb5/lib/kadm5clnt/Makefile > > > index e377f95f5b6e..be09dd3e210e 100644 > > > --- a/krb5/lib/kadm5clnt/Makefile > > > +++ b/krb5/lib/kadm5clnt/Makefile > > > @@ -88,7 +88,7 @@ ${CHPASS_UTIL_STRINGS_ERR_C}: ${CHPASS_UTIL_STRINGS_ERR} > > > rm -f et-c-${.PREFIX}.et et-c-${.PREFIX}.c > > > > > > afterinstall: > > > - ${INSTALL_LIBSYMLINK} ${SHLIB} ${DESTDIR}${LIBDIR}/libkadm5clnt > > > + ${INSTALL_LIBSYMLINK} ${SHLIB_NAME} ${DESTDIR}${LIBDIR}/libkadm5cln > > t.so > > > > > > .include <bsd.lib.mk> > > > > This produces a valid symlink that 'make delete-old' wants to > > remove again: > > > > lrwxr-xr-x 1 root wheel 23 Jul 7 22:26 /usr/lib/libkadm5clnt.so -> > > libkadm5clnt_mit.so.121 > > > > # make delete-old > > >>> Removing old files (only deletes safe to delete libs) > > remove /usr/lib/libkadm5clnt.so? > > Thanks Dima and Herbert for reporting this. It's been fixed. Cy, I am sorry for not being more precise. Dima's change, which hasn't been commited yet, fixes the real issue of creating a bad symlink. I just wanted to add that his fix also needs removal of usr/lib/libkadm5clnt.so in tools/build/mk/OptionalObsoleteFiles.inc. Maybe usr/lib/libkadm5clnt should be added to OptionalObsoleteFiles.inc?
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?aGxx371-HV384A0r>