Summary: | app-crypt/mit-krb5 includes --as-needed in the "krb5-config --libs" output | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Mike Auty (RETIRED) <ikelos> |
Component: | Current packages | Assignee: | Gentoo Kerberos Maintainers <kerberos> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | anakin.skyw, hanno, patrizio.bassi |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 231273 | ||
Attachments: |
wireshark-0.99.6-asneeded.patch
wireshark-0.99.6.ebuild.patch correct wireshark-0.99.6.ebuild.patch krb5-1.5.3-noldflags-config.patch |
Description
Mike Auty (RETIRED)
2007-07-08 23:03:18 UTC
Seem to have narrowed this down to krb5-config, which when called with --libs presents the following: plasma ~ # krb5-config --libs -L/usr/lib -Wl,-rpath -Wl,/usr/lib -Wl,--as-needed -lkrb5 -lk5crypto -lcom_err -lresolv -ldl That means that even if LDFLAGS="" set wireshark will still fail to build. This looks more like a kerberos failure going on somewhere, I just haven't determined exactly where yet. I'm reassigning to the kerberos crew for some help... So sadly, even without the mit-krb5 issue, it still doesn't compile properly with --as-needed. Luckily hanno opened bug 184798 for the original issue, whilst I'm going to keep this bug running for the mit-krb5 issue... *** Bug 184798 has been marked as a duplicate of this bug. *** Let's have wireshark in Summary. In other case I've already lost this bug report once... Reported upstream. Created attachment 124422 [details, diff]
wireshark-0.99.6-asneeded.patch
modification to epan/Makefile.am, include ${top_builddir}/wiretap/libwiretap.la in
deps and libadd..
Created attachment 124424 [details, diff]
wireshark-0.99.6.ebuild.patch
modifications to ebuild..
needs to be tested by others than me. builds fine here with asneeded.
Created attachment 124426 [details, diff]
correct wireshark-0.99.6.ebuild.patch
meh..
I still think this might be two issues, given that mit-krb5 probably shouldn't include the --as-needed flag to all underlying compilations, simply because it was compiled with it? I'm not sure I'd have marked the other bug as a duplicate of this one... patch works, can you commit to portage please? (In reply to comment #9) > I still think this might be two issues, given that mit-krb5 probably shouldn't > include the --as-needed flag to all underlying compilations, simply because it > was compiled with it? I'm not sure I'd have marked the other bug as a > duplicate of this one... > Leaving this bug open for this. (In reply to comment #10) > patch works, can you commit to portage please? > Done. pva, committed it because of multiple requests, poke me if you have better solution. Samuli, that's Ok now. The patch should work and additional issues armin76 is a problem with wireshark-0.99.5 too, which is stable, thus no new problems should appear. Leaving this bug for kerberos LDFLAGS problem. As needed problem is fixed already. Removing blocker of 129413. Updated summary to reflect state of affairs. ping :) I'm not entirely sure what you all want the kerberos team to do with this. If anyone has a patch for me, please attach. i don't have the patch but probably at the end of ebuild, when stripping symbols for istance, you can strip out some $LDFLAGS *I* don't want anything, wireshark now builds and I couldn't care less if kerberos config includes custom ldflags in it's --libs output. Removing CC. so close it... Closing with the comment that if a user has custom LDFLAGS, that's their business, I'm afraid. Guys, please. This bug shouldn't be closed, it's a bug and it's still present. Every other program under the sun manages not to include --as-needed in their pkg-config type output. If we're closing bugs because of as-needed in LDFLAGS then there's 19 open sitting in a year and a half old tracker (bug 129413) that should probably be marked INVALID too. I understand this bug may not be moving anywhere and I understand drac may not want to be CCed anymore, which is fine, but please just leave the bug sit there, don't invent reasons to close it. Patrizio, the ping spam wasn't particularly helpful, and even less helpful was your request to close somebody else's bug. Drac may not have been inclined to help, but the bug still exists. What would be more helpful is finding out if someone's contacted upstream about this? I have the feeling pva's comment 5 was regarding wireshark and not mit-krb5. Could someone from the Kerberos group let me know if this has been reported upstream yet, and if so, what their bug number is? Either way, I'm going to re-open this, even if it just sits here. It hasn't been fixed to my knowledge, and it's not invalid just because of as-needed... Well, I have not contacted upstream about it, and I'm not particularly inclined to. Mike, if you wish to, please go right ahead. Created attachment 132681 [details, diff]
krb5-1.5.3-noldflags-config.patch
Well, should you ever decide to contact upstream, here's a patch for it. It's a little ugly, but then so is their krb5-config script (as attested by a comment in the script itself). I couldn't find a bug tracker, and I'm getting the general vibe (seeing as you're the only member of the herd) that kerberos related packages are generally not fun to look after, so I doubt I'll contact them myself...
I also found during my investigations that neon and wxGTK also have similar problems, but given the enthusiasm this bug has been met with, I'm really not sure I'll bother pestering the other package maintainers. Do what you will with the bug now, there is at least a patch if this ever causes people problems again...
i totally agree with you. unfortunatly sometimes i "ping" with the intent to spam and reacivate some bugs that seems forgotten. i don't want to criticize or blame, absolutly. just ping and wake up. for the "close" issue, i said this because i saw a @gentoo.org dev asking for closing. i mean..i hate pending bugs. tomorrow i'll test the patch. *** Bug 184798 has been marked as a duplicate of this bug. *** This bug blocks the version bump for samba-3.2.x since compilation fails due to --as-needed being at the wrong place. Removed LDFLAGS from krb5-config "--libs" output for the latest unstable release. I've only patched there to see if we produce a regression. g, mueli p.S.: Feel free to reopen bug if problem persists! |