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 <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 <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

Reply via email to