Summary: | net-misc/curl-7.27.0-r3 CURL_SSL=nss USE=metalink - tool_metalink.c:56:4: error: #error "Can't compile METALINK support without a crypto library." | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Michał Bartoszkiewicz <mbartoszkiewicz> |
Component: | Current packages | Assignee: | Anthony Basile <blueness> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | gregkh, xdudka00, yamadharma |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://sourceforge.net/tracker/?func=detail&aid=3578163&group_id=976&atid=100976 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build log
config log (xzipped) |
Description
Michał Bartoszkiewicz
2012-09-03 17:35:05 UTC
Attach full build log (and - most likely - config.log too). Created attachment 322930 [details]
build log
Created attachment 322932 [details]
config log (xzipped)
I checked with curl-7.28.0 and this issue is still there. Looks like its time to do some debugging and submit patches upstream. Actually, it seem it's simply that tool_metalink.c doesn't have an option to be built against NSS. (In reply to comment #5) > Actually, it seem it's simply that tool_metalink.c doesn't have an option to > be built against NSS. Okay I finally gave this issue some attention. You are clearly right, that code is just not there. Ideally one should patch the build system, ie configure.ac, to not allow both options together. Something like if [[ "x$OPT_NSS" == "xyes" && "x$OPT_LIBMETALINK" == "xyes" ]]; then AC_MSG_ERROR([cannot enable both nss and metalink]) fi This would go upstream, but they prefer doing AC_MSG_RESULT([no ($version)]) AC_MSG_NOTICE([libmetalink library defective or too old]) want_metalink="no" which is a problem for Gentoo since it introduces automagic. We don't want our use flags turned on or off behind the users back. Instead, I simple added that logic to the ebuild. I'll open a bug upstream and link to this one to let them know the issue and they can decide how to proceed. For the records, here's the upstream bug report: https://sourceforge.net/tracker/?func=detail&aid=3578163&group_id=976&atid=100976 there's a patch on the upstream bug report http://thread.gmane.org/gmane.comp.web.curl.library/37248 that let's metalink use NSS, which should fix this. (In reply to comment #8) > there's a patch on the upstream bug report > http://thread.gmane.org/gmane.comp.web.curl.library/37248 > that let's metalink use NSS, which should fix this. Thanks for this. I'm certain Dudka's patch is correct. Patch 2 of 2 is the real guts of it and adds the "missing" code. Still, I think we can wait until the next release to include this feature. fix included in curl 7.28.1 :) (In reply to comment #10) > fix included in curl 7.28.1 :) please test. the ebuild is in the tree |