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

CONFIGURE: Improve detection of xmlrpc_c flags #600

Closed
wants to merge 1 commit into from

Conversation

lslebodn
Copy link
Contributor

The pkg-config files for xmlrpc_c libraries are shipped just
in fedora/rhel due to downstream patch. Debian does not have
pkg-config files for xmlrpc_c. Therefore we need to fallback to older
method of detection XMLRPC_*FLAGS which was reverted
by the commit 1e0143c

The pkg-config files for xmlrpc_c libraries are shipped just
in fedora/rhel due to downstream patch. Debian does not have
pkg-config files for xmlrpc_c. Therefore we need to fallback to older
method of detection XMLRPC_*FLAGS which was reverted
by the commit 1e0143c
@lslebodn
Copy link
Contributor Author

@tjaalton It should simplify you work on debian

@abbra
Copy link
Contributor

abbra commented Mar 15, 2017

LGTM. Falling back to a standard check is fine.

@MartinBasti
Copy link
Contributor

Could you add ticket to commit message please https://fedorahosted.org/freeipa/ticket/6418

@tjaalton
Copy link
Contributor

yes, I just added a comment on the old PR, I need this on Debian, so +1 from me

@MartinBasti
Copy link
Contributor

Commit message amended before pushed

@MartinBasti MartinBasti added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Mar 15, 2017
@MartinBasti
Copy link
Contributor

master:

  • 2a4f7f2 CONFIGURE: Improve detection of xmlrpc_c flags

@lslebodn
Copy link
Contributor Author

lslebodn commented Mar 15, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
4 participants