Bug 191792 - [PATCH] www/ap22-mod_auth_cas: mod_auth_cas broken with openssl from ports on 9.x
Summary: [PATCH] www/ap22-mod_auth_cas: mod_auth_cas broken with openssl from ports on...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Many People
Assignee: Olli Hauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-10 20:44 UTC by geoffroy desvernay
Modified: 2014-07-13 22:36 UTC (History)
1 user (show)

See Also:

Attachments
ap22-mod_auth_cas-1.0.9.1.patch (1.61 KB, patch)
2014-07-10 20:44 UTC, geoffroy desvernay 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description geoffroy desvernay 2014-07-10 20:44:15 UTC 
Created attachment 144562 [details]
ap22-mod_auth_cas-1.0.9.1.patch

Unbreak with openssl from ports on 9.x

Port maintainer (apache@FreeBSD.org) is cc'd.

Generated with FreeBSD Port Tools 1.02 (mode: change, diff: SVN)

How-To-Repeat:
* use WITH_OPENSSL_PORT=yes
 * build mod_auth_cas (tested with apache22)
 * Try to activate it and see: mod_auth_cas.so: undefined symbol: CRYPTO_THREADID_get_id_callback

Fix:
* old patch from https://github.com/Jasig/mod_auth_cas/pull/13 but no release upstream sinceâ¦
Comment 1 commit-hook freebsd_committer freebsd_triage 2014-07-13 22:32:17 UTC 
A commit references this bug:

Author: ohauer
Date: Sun Jul 13 22:31:18 UTC 2014
New revision: 361716
URL: http://svnweb.freebsd.org/changeset/ports/361716

Log:
  - add fix from https://github.com/Jasig/mod_auth_cas/pull/13
  - switch to github
  - bump PORTREVISION

  PR:		191792
  Submitted by:	dgeo <dgeo@centrale-marseille.fr>

Changes:
  head/www/mod_auth_cas/Makefile
  head/www/mod_auth_cas/distinfo
  head/www/mod_auth_cas/files/
  head/www/mod_auth_cas/files/patch-mod_auth_cas.c
Comment 2 Olli Hauer freebsd_committer freebsd_triage 2014-07-13 22:36:42 UTC 
Thanks for the PR!
I wasn't aware of the github change, but this will simplify the port.

Fix was committed.