Bug 339391
Summary: | ardour package includes duplicates of system libraries | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tim Mayberry <mojofunk> |
Component: | ardour | Assignee: | Anthony Green <green> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | hdegoede |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-10-19 09:14:41 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Tim Mayberry
2007-10-19 09:02:25 UTC
Normally I would agree with you, but here is what ardour upstream has to say on this: (from http://ardour.org/building) SYSLIBS USE AT YOUR OWN RISK: CANCELS ALL SUPPORT FROM ARDOUR AUTHORS. Use existing system versions of various libraries instead of internal ones (yes|no) So doing things the normal way would very much go against what upstream advices, which is not something we want todo IMHO. This was discussed during the original ardour review, and ardour was given an exception... https://bugzilla.redhat.com/show_bug.cgi?id=189315#c6 (In reply to comment #1) > Normally I would agree with you, but here is what ardour upstream has to say on > this: (from http://ardour.org/building) > > > SYSLIBS > USE AT YOUR OWN RISK: CANCELS ALL SUPPORT FROM ARDOUR AUTHORS. Use existing > system versions of various libraries instead of internal ones (yes|no) > > So doing things the normal way would very much go against what upstream advices, > which is not something we want todo IMHO. > If it is this statement that is discouraging you from keeping with the packaging policy then I'll try to get it changed as SYSLIBS was added for distributions(originally Debian) with a similar policy about duplicating system libraries so that they didn't have to patch the build system. The SYSLIBS functionality has been maintained for several years by myself and another Ardour developer and it should be straight forward to use without any patching etc. The build option also existed for the 0.99 versions, but as Anthony pointed out in bug 189315 none of those "system libraries" were packaged for fedora at the time but that is not the case for Ardour 2. (In reply to comment #3) > (In reply to comment #1) > > Normally I would agree with you, but here is what ardour upstream has to say on > > this: (from http://ardour.org/building) > > > > > > SYSLIBS > > USE AT YOUR OWN RISK: CANCELS ALL SUPPORT FROM ARDOUR AUTHORS. Use existing > > system versions of various libraries instead of internal ones (yes|no) > > > > So doing things the normal way would very much go against what upstream advices, > > which is not something we want todo IMHO. > > > > If it is this statement that is discouraging you from keeping with the packaging > policy then I'll try to get it changed as SYSLIBS was added for > distributions(originally Debian) with a similar policy about duplicating system > libraries so that they didn't have to patch the build system. The SYSLIBS > functionality has been maintained for several years by myself and another Ardour > developer and it should be straight forward to use without any patching etc. > > The build option also existed for the 0.99 versions, but as Anthony pointed out > in bug 189315 none of those "system libraries" were packaged for fedora at the > time but that is not the case for Ardour 2. Okay, if I understand you correctly, then you are one of the upstream authors, and you are saying that you and the other upstream authors are ok with distributions using then system versions of the libraries? Does this also mean that the included copies of the libraries are pristine copies? That would make this a whole different ballgame. (In reply to comment #4) > Okay, if I understand you correctly, then you are one of the upstream authors, > and you are saying that you and the other upstream authors are ok with > distributions using then system versions of the libraries? Yes, I am one of the authors. It is my understanding that the SYSLIBS support "clause" was not intended to apply to people packaging ardour and that we also supported the ardour packages on debian when they used it. I'll try to get a more authoritative answer in a few days when the main developer gets back from a trip. > Does this also mean that the included copies of the libraries are pristine copies? The source code is unmodified, there are additional files to support building and installing the libraries into private locations etc. (In reply to comment #5) > (In reply to comment #4) > > > Okay, if I understand you correctly, then you are one of the upstream authors, > > and you are saying that you and the other upstream authors are ok with > > distributions using then system versions of the libraries? > > Yes, I am one of the authors. It is my understanding that the SYSLIBS support > "clause" was not intended to apply to people packaging ardour and that we also > supported the ardour packages on debian when they used it. I'll try to get a > more authoritative answer in a few days when the main developer gets back from a > trip. > Sounds good, I'll wait for that and if there is agreement upstream that this clause does not apply to Fedora I will happily enable syslibs support. > > Does this also mean that the included copies of the libraries are pristine copies? > > The source code is unmodified, there are additional files to support building > and installing the libraries into private locations etc. > Sounds good too. Tim, I guess this has slipped under your radar, any chance you could talk to the other ardour developers about their stance on us using system libs instead of included ones? It would be nice to switch to system libs if OK-ed by upstream. (In reply to comment #7) > Tim, > > I guess this has slipped under your radar, any chance you could talk to the > other ardour developers about their stance on us using system libs instead of > included ones? It would be nice to switch to system libs if OK-ed by upstream. Sure, this did come up on IRC a recently but I didn't participate in the conversation as I'm in a very different timezone to most of the other ardour devs. The main reason given by the main developer(Paul Davis) for pulling all the C++ library dependencies into the ardour source tree was because of changes in the C++ ABI. As I understand it the problem being that people were compiling ardour with a version of g++/libstdc++ with a different ABI version than that used to compile the C++ library dependencies which caused many issues. So the solution(good or bad) was to add the libraries to the source tree and build them with the rest of ardour to ensure they are all built with the same compiler/CXXABI etc. I'll try to talk to the other developers in the next few days but feel free to ask about this issue on the ardour-dev mailing list if you have time. (In reply to comment #8) > The main reason given by the main developer(Paul Davis) for pulling all the C++ > library dependencies into the ardour source tree was because of changes in the > C++ ABI. As I understand it the problem being that people were compiling ardour > with a version of g++/libstdc++ with a different ABI version than that used to > compile the C++ library dependencies which caused many issues. > If that was the main reason, then it would be fine to use the system versions in Fedora as all Fedora packages and libs are built with the same c++ ABI. Since the libs included with ardour fail to build with gcc-4.3, and this has long been fixed in the system versions of these libs. We are now switching to using system libs, considering what has been explained here, I assume upstream won't have any big problems with this. So ardour-2.4-1 which should hit rawhide soon will be built with system-libs. |