[yoshimi] Re: a bit late but possible CMakeLists.txt patch

  • From: Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Tue, 12 Dec 2017 20:15:51 +0000

On Tue, 12 Dec 2017 23:33:33 +0800
blubee blubeeme <gurenchan@xxxxxxxxx> wrote:

On FreeBSD there are a few issues that we wrote a custom patch for.
1) Install man pages in the correct location
2) Fix a bug in the build on FreeBSD 10.3

----------------------------
Here's the svn diff of the CMakeLists.txt
- Fix libz check
 src.c:4:14: error: use of undeclared identifier 'NULL'
        if (NULL != zzz)
            ^
- Install manpages under ${PREFIX}/man and not ${PREFIX}/share/man

--- src/CMakeLists.txt.orig 2017-11-08 12:25:31 UTC
+++ src/CMakeLists.txt
@@ -156,6 +156,7 @@ endif(NOT LIBC_HAS_ARGP)
set (CMAKE_REQUIRED_LIBRARIES z)
check_c_source_compiles (
    "#include <zlib.h>
+     #include <stdlib.h>
     int main(int argc, char **argv) {
         gzFile zzz  = gzopen(\"/dev/null\", \"rb\");
         if (NULL != zzz)
@@ -605,7 +606,7 @@ install (FILES ${CMAKE_CURRENT_SOURCE_DIR}/../desktop/
install (FILES
${CMAKE_CURRENT_SOURCE_DIR}/../desktop/metainfo/yoshimi.appdata.xml
    DESTINATION ${CMAKE_INSTALL_DATAROOTDIR}/metainfo)
install (FILES ${CMAKE_CURRENT_SOURCE_DIR}/../desktop/yoshimi.1
-    DESTINATION ${CMAKE_INSTALL_DATAROOTDIR}/man/man1)
+    DESTINATION ${CMAKE_INSTALL_PREFIX}/man/man1)

set_directory_properties (PROPERTIES
    ADDITIONAL_MAKE_CLEAN_FILES "${FltkUI_headers}"
----------------------------

Are these changes able to be applied upstream?

Well it looks like we may soon have to make a bugfix release. The debian folks
have found a couple of other issues with some BSD versions, and we seem to
have a misbehaving splash screen on a window manager I've never heard of
before :(

The first of yours is really quite uncontentions, and will go out in the next
commit.

The other one I'm less sure of. Install locations are another cmake issue that
put up quite a fight, so I'm not keen on changing them unless thay can be
proved on all the main Linux distros too. These days I only have debian here.

-- 
Will J Godfrey
http://www.musically.me.uk
Say you have a poem and I have a tune.
Exchange them and we can both have a poem, a tune, and a song.
Yoshimi source code is available from either: 
https://sourceforge.net/projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx

Other related posts:

  1. Home
  2. yoshimi
  3. Archive
  4. 12-2017