Bug 911540
| Summary: | ncurses C++ dynamic library missing | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tim Niemueller <tim> | ||||
| Component: | ncurses | Assignee: | Miroslav Lichvar <mlichvar> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 18 | CC: | dickey, mlichvar | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2013-03-18 17:52:49 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Tim Niemueller
2013-02-15 09:58:41 UTC
Created attachment 705670 [details]
Patch to build using libtool for C++ shared library
This patch applies with git am.
Thanks for the patch. We can also ask the upstream maintainer if there are plans to build shared ncurses++ also in the non-libtool build. It's been neglected (a) since rules for building C++ shared libraries differ for various compilers and (b) there aren't a lot of users of the C++ binding. (It's likely that libtool only works reliablyfor g++, which in this context isn't an argument about (a) - and making g++ work with shared libraries in the normal build is doable - will add it to my to-do list. What's the problem about just using the libtool-based build in Fedora? It works just fine, I'm using it... I'm not against using it, but I think it would be nice if the default build created shared ncurses++ too, perhaps more applications will be written for it if it's included in more distributions. I'd like to keep the flags for parallel build though. Ok, I see and I agree. On Ubuntu they also only build the static library, but at least it's in the -dev package and comes with the pkg-config file (that this file is missing and deleted in the spec actually bothers me the most). Plan B could be to re-introduce the ncurses-cpp-devel package that brings the required files and pulls in the static libraries. Though I'd still prefer a dynamic lib it would at least bring back the pkg-config file without always pulling in the static lib. Static libraries are strongly discouraged in Fedora and I'd like to eventually see the -static subpackage dropped. http://fedoraproject.org/wiki/Packaging:Guidelines#Packaging_Static_Libraries_2 I think building and including the shared library is the correct approach. I added a configure option last weekend to address this; there's some followup changes which will be in this week's patch. Thanks. The shared C++ libs are now packaged in ncurses-5.9-9.20130316.fc19. There was a small problem with RPATH being set in the C++ libs. Thomas, should LDFLAGS in the c++ Makefile.in include LOCAL_LDFLAGS? no - I overlooked it. That is needed for the demo program, but should not be used for the library. I'll work on this, to separate the two uses. ncurses-5.9-10.20130413.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/ncurses-5.9-10.20130413.fc18 ncurses-5.9-10.20130413.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/ncurses-5.9-10.20130413.fc17 ncurses-5.9-10.20130413.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report. ncurses-5.9-10.20130413.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. |