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

apache: use include directory from APXS #20

Merged
merged 1 commit into from Apr 19, 2021

Conversation

abbra
Copy link
Contributor

@abbra abbra commented Apr 19, 2021

On Fedora, when building Apache module support, it is not enough to get
APR CFLAGS, we need also APXS include details. Use APXS tool directly.

Signed-off-by: Alexander Bokovoy abokovoy@redhat.com

On Fedora, when building Apache module support, it is not enough to get
APR CFLAGS, we need also APXS include details. Use APXS tool directly.

Signed-off-by: Alexander Bokovoy <abokovoy@redhat.com>
@zandbelt zandbelt merged commit 68e56f1 into OpenIDC:master Apr 19, 2021
@abbra
Copy link
Contributor Author

abbra commented Apr 19, 2021

@zandbelt thanks. I am working on the spec file and later adding the packages to Fedora. Are you interested in merging the spec file as well?

@zandbelt
Copy link
Member

I'm keeping packaging related files for various distributions separate from the source code. I'd be interested to have a link to it though.

@abbra
Copy link
Contributor Author

abbra commented Apr 19, 2021

@zandbelt my current build is in https://copr.fedorainfracloud.org/coprs/abbra/oauth2-support/build/2139903/. This does not include integration with redis and memcached because tests fail when I run them during the build. Also, memcache and redis support are compiled in into the liboauth2 which makes it harder to use in the environment where both memcache and redis aren't expected to be present.

@abbra
Copy link
Contributor Author

abbra commented Apr 21, 2021

FYI. I've got liboauth2 approved for Fedora and pushed updates to F32-F35 (Rawhide now):
https://bodhi.fedoraproject.org/updates/?search=liboauth2
They should appear in the update-testing repositories for the corresponding releases in couple days.

@oliSeven oliSeven mentioned this pull request Oct 12, 2021
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.

None yet

2 participants