Skip to content

fsearch: update to 0.3.1 - #62306

Open
gc-user wants to merge 1 commit into
void-linux:masterfrom
gc-user:fsearch
Open

fsearch: update to 0.3.1#62306
gc-user wants to merge 1 commit into
void-linux:masterfrom
gc-user:fsearch

Conversation

@gc-user

@gc-user gc-user commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: briefly

Local build testing

  • I built this PR locally for my native architecture, (x86_64)
  • I built this PR locally for these architectures (crossbuilds): aarch64, i686 (builds fine locally)

Notes:

Due to build errors I added the packages itstool intltool icu-devel.
Due to cross-build error I added build_helper="qemu".

[Edit]

Added @ihateemoji's solution to solve the 64bit-musl cross-build error.

[/Edit]

@gc-user
gc-user marked this pull request as draft August 31, 2026 18:45
@gc-user

gc-user commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

Not sure why the -musl builds for 64bit archs fail and if this is an upstream issue or can be fixed by modifying the template / adding a patch.

@gc-user
gc-user marked this pull request as ready for review August 31, 2026 19:30
@ihateemoji

ihateemoji commented Sep 1, 2026

Copy link
Copy Markdown
Contributor
if [ "$XBPS_TARGET_LIBC" = "musl" ]; then
	CFLAGS+=" -D_LARGEFILE64_SOURCE"
fi

post_extract() {
    sed -i '/#include <linux\/fanotify.h>/d' \
        src/fsearch_folder_monitor_fanotify.c
}

Append this to your template and see if it solves your issue.

From my quick look, the project uses the off64_t type, which can be exposed on musl with the -D_LARGEFILE64_SOURCE flag, and fsearch_folder_monitor_fanotify.c includes both <linux/fanotify.h> and <sys/fanotify.h>, which is not an issue on glibc, but will not work on musl. I've removed <linux/fanotify.h>, and it seems to build fine without it.

@gc-user

gc-user commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Thank you very much for looking into the PR and the error messages!

Your proposed solution works for me locally, so I'm assuming it does so, too, for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants