-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make check fails with Alpine Linux #2
Comments
Do you know which version of libconfuse that you're using? See https://github.com/fhunleth/fwup#building. It really sounds like the libconfuse bug. |
@fhunleth current version of confuse is 2.7 on Alpine. I take it it's a more recent revisions that fixes the bug? Should I build from their repository? |
@systmkor I just checked https://github.com/martinh/libconfuse, and it still doesn't look like there's a new release tag (over a year since the fix was integrated). I've submitted an issue libconfuse/libconfuse#23 in hopes that the maintainer will make an official release. For now, you'll need to build from their repository. |
@fhunleth Thanks 👍 I tried compiling from the github libconfuse head commit however it fails (i.e. the searchpath test results in a segmentation fault) when running |
@systmkor Ugh. Are you running ./configure like at http://git.alpinelinux.org/cgit/aports/tree/main/confuse/APKBUILD? (Sorry - I'm just guessing right now since I haven't used Alpine Linux.) |
@fhunleth tl;dr yes. I didn't check the configuration options since I didn't want to break other programs that depended upon that package. 👏 your google-fu 😄 I assume I need to change the ./confugre options. Also the libconfuse |
Closing old issue. Hopefully fwup works on Alpine now, but if not, I would assume it to be a new issue since libconfuse is at 3.0 upstream now. |
Compiling and running the latest commit in master branch.
System Info
Log
The text was updated successfully, but these errors were encountered: