OpenIDC / liboauth2

OAuth 2.x and OpenID Connect C library
Apache License 2.0
105 stars 26 forks source link

apache: use include directory from APXS #20

Closed abbra closed 3 years ago

abbra commented 3 years ago

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

abbra commented 3 years ago

@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 commented 3 years ago

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 commented 3 years ago

@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 commented 3 years ago

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.