Skip to content
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

Upgrade to libconfuse v2.9 to improve backward compatibility #12

Closed
fhunleth opened this issue Dec 2, 2015 · 2 comments
Closed

Upgrade to libconfuse v2.9 to improve backward compatibility #12

fhunleth opened this issue Dec 2, 2015 · 2 comments

Comments

@fhunleth
Copy link
Collaborator

fhunleth commented Dec 2, 2015

The current version of libconfuse throws an error if it encounters a configuration parameter that it doesn't recognize. This is a big problem since it prevents the addition of a parameters if you want a new firmware file to be applied by an older version of fwup. In many cases, the new parameters are not mandatory for applying the firmware so the error is completely unnecessary. Version v2.9 of libconfuse has support for handling unknown arguments, so it will be possible to ignore these new arguments when parsing meta.conf files inside the .fw files. Errors should still be thrown when encountering unknown parameters when creating .fw files.

At this moment, libconfuse v2.9 hasn't been released. See https://github.com/martinh/libconfuse/blob/master/ChangeLog.md for details.

@rmorgans
Copy link

@fhunleth
Copy link
Collaborator Author

libconfuse v3.0 was needed for this to work. fwup now uses the unknown parameter feature if it's available. Older libconfuse version still can be used with the old behavior, but they'll emit a warning when compiled. See 172eed5.

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

No branches or pull requests

2 participants