Hi Ted,

I see that you are active merging e2fsprogs patches upstream and
uploading e2fsprogs to Debian. Thank you. Can I ask you to also review
this patch by Johannes?

On Thu, Sep 05, 2024 at 05:47:14AM +0200, Johannes Schauer Marin Rodrigues 
wrote:
> From: Johannes Schauer Marin Rodrigues <jo...@mister-muffin.de>
> Date: Tue, 3 Sep 2024 13:15:33 +0200
> Subject: [PATCH] misc/create_inode_libarchive.c: decouple --without-libarchive
>  and HAVE_ARCHIVE_H
> 
> To aid bootstrapping, it would be useful to be able to build e2fsprogs
> without archive.h as otherwise there is a build dependency loop with
> libarchive. If archive.h is not present, add the missing forward
> declarations and preprocessor definitions and typedefs. Since this
> allows building e2fsprogs with libarchive support even without the
> archive.h header present on the system, we cannot check HAVE_ARCHIVE_H
> anymore to decide whether to build with libarchive support or not. So if
> --without-libarchive is passed to ./configure, CONFIG_DISABLE_LIBARCHIVE
> gets set and later checked to decide about libarchive support.
> 
> Closes: #1078693

Thanks for considering. Including this patch is necessary to facilitate
architecture cross bootstrap in Debian as we otherwise get into a
dependency loop with libarchive. Regular builds (those that do not pass
the nocheck build profile) remain unaffected.

Helmut

Reply via email to