>>>>> On Thu, 12 Jun 2025 09:45:20 -0400, Dan Langille said:
>
> On Thu, Jun 12, 2025, at 8:19 AM, Martin Simmons wrote:
> >>>>>> On Wed, 11 Jun 2025 12:38:27 -0400, Dan Langille said:
> >>
> >> On Wed, Jun 11, 2025, at 10:45 AM, Martin Simmons wrote:
> >> >>>>>> On Tue, 10 Jun 2025 20:23:40 -0400, Dan Langille said:
> >> >>
> >> >> I expected to see std mentioned...
> >> >>
> >> >> I tried building with --with-zstd in the build options - it did not
> >> >> change the above output.
> >> >>
> >> >> i.e
> >> >> --CONFIGURE_ARGS--
> >> >> --disable-conio --enable-batch-insert --enable-smartalloc
> >> >> --sysconfdir=/usr/local/etc/bacula --with-baseport=9101
> >> >> --with-db-name=bacula --with-db-user=bacula
> >> >> --with-dump-email=root@localhost --with-job-email=root@localhost
> >> >> --with-logdir=/var/log --with-plugindir=/usr/local/lib
> >> >> --with-readline=/usr/local --with-sbin-perm=755
> >> >> --with-scriptdir=/usr/local/share/bacula --with-tcp-wrappers=/usr/lib
> >> >> --with-working-dir=/var/db/bacula --with-zstd --with-fd-group=wheel
> >> >> --with-fd-user=root --enable-client-only --enable-nls
> >> >> --prefix=/usr/local ${_LATE_CONFIGURE_ARGS}
> >> >
> >> > It is a bug in the configure script.
> >> >
> >> > As a workaround, try installing the curl package as a build-time
> >> > dependency
> >> > and add --with-curl=/usr/local to the configure args. AFAICS, this will
> >> > not
> >> > change anything in the built binaries because Bacula doesn't actually use
> >> > curl, but it will work around the bug.
> >>
> >> I have been able to reproduce the bug reported in the ticket. I've been
> >> able to get compressin with GZIP and LTO, but not ZSTD.
> >>
> >> This is the FileSet I'm testing with:
> >>
> >> FileSet {
> >> Name = "basic backup with compression"
> >> Include {
> >> Options {
> >> signature=MD5
> >> compression=ZSTD
> >> }
> >> File = /boot
> >> File = /etc
> >> ....
> >>
> >> The build time changes I made were:
> >>
> >> [15:36 pkg01 dvl
> >> /usr/local/poudriere/ports/default/sysutils/bacula15-server] % git diff
> >> diff --git a/sysutils/bacula15-server/Makefile
> >> b/sysutils/bacula15-server/Makefile
> >> index 5ab79d4d4852..bad3f52db132 100644
> >> --- a/sysutils/bacula15-server/Makefile
> >> +++ b/sysutils/bacula15-server/Makefile
> >> @@ -1,6 +1,6 @@
> >> PORTNAME= bacula
> >> DISTVERSION= 15.0.2
> >> -PORTREVISION= 6
> >> +PORTREVISION= 6a
> >> CATEGORIES?= sysutils
> >> MASTER_SITES= SF/bacula/bacula/${DISTVERSION}
> >> PKGNAMEPREFIX?= #
> >> @@ -78,7 +78,8 @@ CONFIGURE_ARGS+= --disable-conio \
> >>
> >> .if defined(WITH_CLIENT_ONLY)
> >> CONFIGURE_ARGS+= --with-fd-group=wheel \
> >> - --with-fd-user=root
> >> + --with-fd-user=root \
> >> + --with-curl=/usr/local
> >> .else
> >> CONFIGURE_ARGS+= --with-dir-group=${BACULA_GROUP} \
> >> --with-dir-user=${BACULA_USER} \
> >> [15:48 pkg01 dvl
> >> /usr/local/poudriere/ports/default/sysutils/bacula15-server] %
> >>
> >> In my initial testing, I did not explicitly add ftp/curl as a run time
> >> dependency of the FreeBSD package. curl was already present on the host
> >> for other dependencies. With
> >> the failed results for "compression=ZSTD", I went back and built the
> >> package with the
> >> dependency - same results: no compression.
> >
> > If I remember correctly, poudriere builds in a jail so it will not see the
> > curl installed on the host.
> >
> > It looks like the underlying bug has was fixed in git revision
> > 358e838a99ad407e439a149a7dc9b472c63db2f8 (and
> > 8d9669fb26bd8bce30e4c48a522d9943c110b5c2 for the autoconf/configure.in).
> > The
> > critical parts are the two new "unset error" lines so you could make a patch
> > from that.
>
> Thank you. I grabbed the two patches from:
>
> https://gitlab.bacula.org/bacula-community-edition/bacula-community/-/commit/358e838a
> https://gitlab.bacula.org/bacula-community-edition/bacula-community/-/commit/8d9669fb26bd8bce30e4c48a522d9943c110b5c2
>
> I added them to my copy of the FreeBSD port and rebuilt the package. By that,
> I mean the bacula-client package, not the server (i.e. only bacula-fd on the
> client has been updated in my testing).
>
> I'm assuming that only the client needs to be updated here, not that server.
> Am I correct there?
bextract also uses it.
> So far, testing with the above patches included is not compressing for
> 'compression=ZSTD'.
>
> For testing, I swapped back to 'compression=GZIP - it compressed.
Is the poudriere build succeeding?
What is the output of ldd now?
If it doesn't show zstd then what is the output of the poudriere build?
_______________________________________________
Bacula-users mailing list
Bacula-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bacula-users
