On Wed, Oct 18, 2023, at 4:56 PM, Dan Langille wrote: > I don't see a reply to my question. > > The patch is wrong. The build still fails. > > I call again: bring back regression testing for users. Host a cdash instance. > We have bugs going 5+ months undetected. > > This is the correct patch (pasted, so errors may be with tabs) > > --- src/filed/bxattr_freebsd.c.orig 2023-10-18 20:41:39 UTC > > +++ src/filed/bxattr_freebsd.c > > @@ -103,7 +103,7 @@ bRC_BXATTR BXATTR_FreeBSD::os_backup_xattr (JCR *jcr, > > case bRC_BXATTR_skip: > > case bRC_BXATTR_cont: > > /* no xattr available, so skip rest of it */ > > - rc = bRC_XACL_ok; > > + rc = bRC_BXATTR_ok; > > continue; > > default: > > goto bail_out; >
There are additional errors: Full build at https://services.unixathome.org/poudriere/data/132amd64-dvl/2023-10-18_21h00m53s/logs/errors/bacula13-client-13.0.3.log I don't know why OSX is involved here. Cleaning up ==> Plugin directory src/plugins/fd ==> Plugin directory src/plugins/dir ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src/lib ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src/findlib ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src/filed ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src/console ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/manpages ==>Entering directory /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 23: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 71: Invalid line type make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 74: Invalid line type make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 237: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 241: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 241: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 242: warning: duplicate script for target "openssl-.tar.gz/" ignored make[3]: "Makefile" line 239: warning: using previous script for "openssl-.tar.gz/" defined here make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 242: warning: duplicate script for target "&&" ignored make[3]: "Makefile" line 239: warning: using previous script for "&&" defined here make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 242: warning: duplicate script for target "print" ignored make[3]: "Makefile" line 239: warning: using previous script for "print" defined here make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 244: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 244: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 249: Unknown modifier "//www.openssl.org/source/ 2>/dev/null| perl -ne '/openssl-(${OPENSSL_VERSION_BASE}[a-z]+" make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 250: warning: duplicate script for target "openssl-.tar.gz/" ignored make[3]: "Makefile" line 239: warning: using previous script for "openssl-.tar.gz/" defined here make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 250: warning: duplicate script for target "&&" ignored make[3]: "Makefile" line 239: warning: using previous script for "&&" defined here make[3]: "/wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx/Makefile" line 250: warning: duplicate script for target "print" ignored make[3]: "Makefile" line 239: warning: using previous script for "print" defined here make[3]: Fatal errors encountered -- cannot continue make[3]: stopped in /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms/osx *** Error code 1 Stop. make[2]: stopped in /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/platforms *** Error code 1 Stop. make[1]: stopped in /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3 > > On Wed, May 31, 2023, at 7:49 AM, Dan Langille wrote: >> NOTE: the patch in git uses bRC_XATTR_ok >> >> The patch I have uses bRC_BXATTR_ok (note the extra B in my version). My >> patch is below. >> >> This patch builds. The patch in git does not. Please advise. >> >> [11:45 pkg01 dan ~/ports/head] % cat >> sysutils/bacula13-server/files/patch-src_filed_bxattr_freebsd.c >> diff --git a/bacula/src/filed/bxattr_freebsd.c >> b/bacula/src/filed/bxattr_freebsd.c >> index >> b83331fc578ebc6caf74ff1bf5e67e49329394be..ed2efad02cf2b4f1d1437b25962d28a4d326b873 >> 100644 >> --- src/filed/bxattr_freebsd.c >> +++ src/filed/bxattr_freebsd.c >> @@ -103,7 +103,7 @@ bRC_BXATTR BXATTR_FreeBSD::os_backup_xattr (JCR *jcr, >> FF_PKT *ff_pkt){ >> case bRC_BXATTR_skip: >> case bRC_BXATTR_cont: >> /* no xattr available, so skip rest of it */ >> - rc = bRC_XACL_ok; >> + rc = bRC_BXATTR_ok; >> continue; >> default: >> goto bail_out; >> >> — >> Dan Langille >> http://langille.org/ >> >> >> >> >>> On May 26, 2023, at 1:54 PM, Dan Langille <d...@langille.org> wrote: >>> >>> For those looking for the commit: >>> https://gitlab.bacula.org/bacula-community-edition/bacula-community/-/commit/c0e27eca4f29dfcc000b2156094b867e995a2f64 >>> >>> Is that the right change? >>> >>> If I use bRC_XATTR_ok instead, it does compile, but I don't know if that's >>> the right value. >>> >>> I ask because. >>> >>> bxattr_freebsd.c:106:18: error: use of undeclared identifier >>> 'bRC_XATTR_ok'; did you mean 'bRC_BXATTR_ok'? >>> rc = bRC_XATTR_ok; >>> ^~~~~~~~~~~~ >>> bRC_BXATTR_ok >>> ./bxattr.h:51:4: note: 'bRC_BXATTR_ok' declared here >>> bRC_BXATTR_ok = 0, // success >>> ^ >>> 1 error generated. >>> *** [bxattr_freebsd.o] Error code 1 >>> >>> make[2]: stopped in >>> /wrkdirs/usr/ports/sysutils/bacula13-client/work/bacula-13.0.3/src/filed >>> 1 error >>> >>> >>> — >>> Dan Langille >>> http://langille.org/ >>> >>> >>> >>> >>> >>>> On May 26, 2023, at 9:39 AM, Eric Bollengier via Bacula-devel >>>> <bacula-devel@lists.sourceforge.net> wrote: >>>> >>>> Hello Dan, >>>> >>>> >>>> Thanks for the report, I have pushed a fix. >>>> >>>> Best Regards, >>>> >>>> Eric >>>> >>>> On 26.05.23 14:18, Dan Langille wrote: >>>>> Hello, >>>>> >>>>> When building bacula13-client-13.0.3 on FreeBSD, I encountered this error. >>>>> >>>>> Compiling bacl.c >>>>> Compiling bxattr.c >>>>> --- bxattr_freebsd.o --- >>>>> bxattr_freebsd.c:106:18: error: use of undeclared identifier >>>>> 'bRC_XACL_ok'; did you mean 'bRC_BXATTR_ok'? >>>>> rc = bRC_XACL_ok; >>>>> ^~~~~~~~~~~ >>>>> bRC_BXATTR_ok >>>>> ./bxattr.h:51:4: note: 'bRC_BXATTR_ok' declared here >>>>> bRC_BXATTR_ok = 0, // success >>>>> ^ >>>>> 1 error generated. >>>>> >>>>> >>>>> Full log output at >>>>> https://services.unixathome.org/poudriere/data/131amd64-dvl-testing/2023-05-26_12h07m40s/logs/errors/bacula13-client-13.0.3.log >>>>> >>>>> Thank you. >>>> >>>> >>>> _______________________________________________ >>>> Bacula-devel mailing list >>>> Bacula-devel@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/bacula-devel >>> >>> _______________________________________________ >>> Bacula-devel mailing list >>> Bacula-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/bacula-devel >> >> _______________________________________________ >> Bacula-devel mailing list >> Bacula-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/bacula-devel >> > > -- > Dan Langille > d...@langille.org > > > > _______________________________________________ > Bacula-devel mailing list > Bacula-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/bacula-devel > -- Dan Langille d...@langille.org
_______________________________________________ Bacula-devel mailing list Bacula-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bacula-devel
