[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!
From: |
Pedro Lopez-Cabanillas |
Subject: |
Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released! |
Date: |
Thu, 4 Aug 2011 19:38:20 +0200 |
User-agent: |
KMail/1.13.5 (Linux/2.6.34.10-0.2-desktop; KDE/4.4.4; i686; ; ) |
On Thursday 04 August 2011, Orcan Ogetbil wrote:
> On Thu, Aug 4, 2011 at 11:27 AM, Pedro Lopez-Cabanillas wrote:
> > On Thursday 04 August 2011, Orcan Ogetbil wrote:
> >> On Thu, Aug 4, 2011 at 3:54 AM, David Henningsson wrote:
> >> > In short, version 1.1.4 of FluidSynth brings us:
> >> >
> >> > * Several improvements and fixes to the CMake build system,
> >> > especially for Mac OS X
> >> > * Several bug fixes to the engine, notably quite a few which could
> >> > cause FluidSynth to sound bad in some cases.
> >> > * API additions, that make it possible to
> >> > - load MIDI files from memory
> >> > - to inspect/modify MIDI events as they are being played from a
> >> > MIDI file
> >> > - to change channels between melodic and drum mode
> >> > - and to silence all notes in one command.
> >> > * Improvements to the CoreAudio driver
> >> >
> >> > A more detailed list of changes is available at
> >> > https://sourceforge.net/apps/trac/fluidsynth/wiki/ChangeLog1_1_4
> >> >
> >> > Looking back at the changes, I'm feeling a little proud today. One of the
> >> > reasons for feeling proud, is that we have more than one new contributor:
> >> > people who have taken the step to actually work with the code, improve
> >> > it,
> >> > and create non-trivial patches with new functionality in it. That is a
> >> > good
> >> > foundation for the future and I hope all contributors feel welcomed and
> >> > properly attributed.
> >> >
> >> > Also, the improvements/fixes to the build system is worth a special note
> >> > (thanks Pedro!), I was surprised to see so many of them! And finally,
> >> > we've
> >> > managed to close several long standing bugs, which makes me feel that
> >> > this
> >> > release will be stable and enjoyable for a lot of people, with less
> >> > annoyances than ever. (Thanks to the jOrgan folks for the test run!)
> >> >
> >> > Now, enjoy the release, and last but not least - make sure you make some
> >> > really great music with it! :-)
> >> >
> >> > On behalf of the FluidSynth developer team,
> >> > David Henningsson
> >> >
> >>
> >> Hi all, thanks for releasing 1.1.4. Sorry I did not have time to test
> >> the prerelease tarball you made the other day. I was planning to do
> >> this today.
> >>
> >> Anyhow, I saw in the CMakeLists.txt, a couple lines such as
> >> ${CMAKE_INSTALL_PREFIX}/${LIB_INSTALL_DIR}${LIB_SUFFIX} )
> >>
> >> Unfortunately this is wrong. The variable ${LIB_INSTALL_DIR} is
> >> already expected to have the lib suffix in it, i.e. it is set to
> >> /usr/lib64 on multilib 64bit systems. This is the cmake standard we
> >> have with hundreds of packages in Fedora.
> >
> > This may be the standard in Fedora, but it is not a CMake standard at all.
> > You can ask in the CMake mailing list, or search for similar questions in
> > the archives, for instance:
> > http://www.cmake.org/pipermail/cmake/2011-March/043180.html
> >
> > I've adopted the variable name "LIB_INSTALL_DIR" because i'm used to it in
> > my KDE projects, but it is not a standard outside KDE. The FluidSynth's
> > LIB_INSTALL_DIR is a constant defined in cmake_admin/DefaultDirs.cmake,
> > with the value "lib". The other variable LIB_SUFFIX is initialized in the
> > main CMakeLists.txt with the value "64" or empty. You can override the
> > initialization value of these variables in the command line, if you want:
> >
> > $ cmake .. -DLIB_SUFFIX=""
> >
> > This can be easily added to the RPM spec file too. I find this method much
> > more comfortable than applying patches.
> >
>
> Hi Pedro,
>
> Sure the behavior can be overridden that way. However, when building
> RPMs, there are cmake macros we use that pass all the standard flags
> to all packages that use cmake. I am sure other RPM based
> distributions, or even DEB ones use some sort of standardization in
> cmake flags too.
Here are the options currently used in Debian to build the FluidSynth package:
http://anonscm.debian.org/gitweb/?p=pkg-multimedia/fluidsynth.git;a=blob;f=debian/rules
( they set LIB_SUFFIX='' )
> I really do believe that what I claimed is a (maybe unwritten) cmake
> standard,
Google is your friend. Show me evidences.
> since we do not have to do such overrides you suggested for
> any other package (we have a lot of them).
Your LIB_INSTALL_DIR variable is defined in the "%cmake" RPM macro:
http://pkgs.fedoraproject.org/gitweb/?p=cmake.git;a=blob;f=macros.cmake
> I also adopt this
> convention in my projects where I am the upstream. Of course,
> fluidsynth can be a special case, but does it really have to be?
You can propose/contribute a better way to do this, but please take into
account all other Linux distros and not only Fedora. Some of them are already
using cmake to build FluidSynth.
Arch linux:
http://projects.archlinux.org/svntogit/packages.git/tree/fluidsynth/trunk/PKGBUILD
Gentoo:
http://gentoo-portage.com/AJAX/Ebuild/119045/View
Mandriva:
http://svn.mandriva.com/cgi-bin/viewvc.cgi/packages/cooker/fluidsynth/current/SPECS/fluidsynth.spec?view=markup
Mageia:
http://svnweb.mageia.org/packages/cauldron/fluidsynth/current/SPECS/fluidsynth.spec?view=markup
Mandriva and Mageia have also a %cmake RPM macro, not conflicting with
FluidSynth's buildsystem.
openSUSE and SlackWare are still using the auto-tools based build system.
Regards,
Pedro
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, (continued)
Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Orcan Ogetbil, 2011/08/04
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, James Ong, 2011/08/04
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Pedro Lopez-Cabanillas, 2011/08/04
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Orcan Ogetbil, 2011/08/04
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Orcan Ogetbil, 2011/08/05
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Pedro Lopez-Cabanillas, 2011/08/05
- Re: [fluid-dev] [ANNOUNCE] FluidSynth 1.1.4 released!, Orcan Ogetbil, 2011/08/05