Bug 233991
| Summary: | net/radsecproxy: Fix MASTER_SITES, Fails to build with OpenSSL 1.1.1 | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Stephen Wall <stephen.wall> | ||||||||||
| Component: | Individual Port(s) | Assignee: | freebsd-ports-bugs (Nobody) <ports-bugs> | ||||||||||
| Status: | Closed Overcome By Events | ||||||||||||
| Severity: | Affects Some People | CC: | FreeBSD, m.tsatsenko, rakuco, sa.inbox | ||||||||||
| Priority: | --- | Keywords: | needs-patch, needs-qa | ||||||||||
| Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(m.tsatsenko) koobs: merge-quarterly? |
||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 231931 | ||||||||||||
| Attachments: |
|
||||||||||||
Looks like [1] tarballs are being distributed via GitHub releases [1] https://radsecproxy.github.io/ It seems so. Patch to fix is trivial. However right now I am working on update 1.6.8=>1.7.2 which also fixes build on 12.0. Almost done already (In reply to m.tsatsenko from comment #2) * Separate bugfixes from version/feature updates where possible. If the quarterly version is also affected by the issue reported here, the fix needs to be merged there too. If you can provide the patch for the MASTER_SITES update, I can take care of the commit/merge. I did make an attempt at modifying the port to download from github, but couldn't quite get things working. The tarball from github that I was able to download did not have any auto-tool generated files in it, and extracted to a different directory name. I don't know github well enough to know if there's a different version of the tarball available somewhere that matches the released version. Created attachment 200414 [details]
the patch
- Switch to github. No functional changes
QA:
portlint OK
GH_ACCOUNT defaults to PORTNAME, so is unnecessary . Configure fails: ===> Configuring for radsecproxy-1.6.8 env: ./configure: No such file or directory The GitHub sources are likely not unprocessed, and likely require USES=autoreconf or similar to pre-process. Could you please update the patch accordingly after investigating, ensuring that the patch is completely tested (portlint and poudriere at least) prior to updating it as there may be further issues. Created attachment 200620 [details]
radsecproxy-1.6.8.log
After adding USES=autoreconf:
1) The build uses -Werror by default which fails due to non-errors, but also...
2) Fails to build with OpenSSL 1.1.1 on 12.0
Well. I have managed to make it build on 11.2 and pass QA with no errors. Getting it work on 12.0 and making portlint happy requires an update to recent version. In addition I was not able to make man pages build. - Switch to github - Remove man pages as they are totally broken in this version - Mark broken on 12.x QA: poudriere 112amd64 - OK portlint - no errors I have an update to 1.7.2 ready which fixes issues with portlint, man pages and building on 12.x Created attachment 202240 [details]
updated patch
radsecproxy 1.8.0 released https://github.com/radsecproxy/radsecproxy/releases/tag/1.8.0 Appreciate if somebody will have time to check if it is possible to make new version work on FreeBSD 12.x with OpenSSL 1.1.1 Probably already fixed in bug #240382 Closing per comment #12. If there are any issues that still need to be solved, please file a new bug with a patch based on the latest version in the ports tree. |
Created attachment 200087 [details] Output from make fetch The URL that radsecproxy tries to download from is invalid, and the FreeBSD distcache package does not exist.