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

Fix up xmlsec1-config file #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

imirkin
Copy link
Contributor

@imirkin imirkin commented Aug 5, 2019

It's effectively a pkg-config file, but not exactly. This is required to install dm.xmlsec.binding 1.3.7 without additional changes.

The source file normally contains, e.g.,

prefix="/usr"
package="xmlsec1"
exec_prefix="${prefix}"
exec_prefix_set=no
libdir="/usr/lib64"

And this command fixes up the prefix and libdir. Tested to successfully install the python-saml package (which depends on dm.xmlsec.binding) with this change. Previously there was an error finding xmlsec/crypto.h due to incorrect include path.

I am loathe to add 1000 fixups like this, but I also don't see a substantially better way. Hopefully the number of such non-standard config files globally will be limited given pkg-config's popularity.

It's effectively a pkg-config file, but not exactly. This is required to install dm.xmlsec.binding 1.3.7 without additional changes.
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.

1 participant