Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 500164 - media-libs/alure-1.2[fluidsynth] - .../work/alure-1.2/src/codec_fluidsynth.cpp:333:25: error: ‘close’ was not declared in this scope
Summary: media-libs/alure-1.2[fluidsynth] - .../work/alure-1.2/src/codec_fluidsynth.cp...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-03 10:06 UTC by Dominique Michel
Modified: 2017-07-02 21:01 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (emerge.info,16.75 KB, text/plain)
2014-02-03 10:09 UTC, Dominique Michel
Details
environment file (environment,94.39 KB, text/plain)
2014-02-03 10:11 UTC, Dominique Michel
Details
build.log (build.log,20.65 KB, text/plain)
2014-02-03 10:12 UTC, Dominique Michel
Details
alure-1.2-include-unistd.patch (alure-1.2-include-unistd.patch,246 bytes, patch)
2015-07-02 13:31 UTC, Joshua Clayton
Details | Diff
alure-1.2-r1.ebuild (alure-1.2-r1.ebuild,1.32 KB, text/plain)
2015-07-02 13:32 UTC, Joshua Clayton
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dominique Michel 2014-02-03 10:06:20 UTC
alure fail to compile with:

[ 81%] Building CXX object CMakeFiles/alure.dir/src/codec_fluidsynth.o
/usr/bin/x86_64-pc-linux-gnu-g++  -DALURE_BUILD_LIBRARY -DDYNLOAD=1 -DHAVE_CONFIG_H -D_GNU_SOURCE=1  -DNDEBUG -march=amdfam10 -mcx16 -msahf -mpopcnt -mabm --param l1-cache-size=32 --param l1-cache-line-size=64 --param l2-cache-size=512 -mtune=amdfam10 -O2 -pipe  -fPIC -I/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/include -I/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2_build -I/usr/include/AL    -Wextra -Wall -funswitch-loops -fvisibility=hidden -pthread -o CMakeFiles/alure.dir/src/codec_fluidsynth.o -c /var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/src/codec_fluidsynth.cpp
/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/src/codec_fluidsynth.cpp: In member function ‘virtual bool fluidStream::SetPatchset(const char*)’:
/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/src/codec_fluidsynth.cpp:333:25: error: ‘close’ was not declared in this scope
                 close(fd);
                         ^
In file included from /var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/src/codec_fluidsynth.cpp:26:0:
/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/include/main.h: At global scope:
/var/tmp/portage/media-libs/alure-1.2/work/alure-1.2/include/main.h:61:19: warning: ‘BigEndian’ defined but not used [-Wunused-variable]
 static const bool BigEndian = !LittleEndian;
                   ^
CMakeFiles/alure.dir/build.make:333: recipe for target 'CMakeFiles/alure.dir/src/codec_fluidsynth.o' failed
make[2]: *** [CMakeFiles/alure.dir/src/codec_fluidsynth.o] Error 1


Reproducible: Always

Steps to Reproduce:
1. emerge alure
2.
3.



# emerge -pqv '=media-libs/alure-1.2::gentoo'
[ebuild  N    ] media-libs/alure-1.2  USE="examples flac fluidsynth mp3 sndfile vorbis -dumb -static-libs"
Comment 1 Dominique Michel 2014-02-03 10:09:37 UTC
Created attachment 369422 [details]
emerge --info
Comment 2 Dominique Michel 2014-02-03 10:11:25 UTC
Created attachment 369424 [details]
environment file
Comment 3 Dominique Michel 2014-02-03 10:12:30 UTC
Created attachment 369426 [details]
build.log
Comment 4 Dominique Michel 2014-02-09 12:05:51 UTC
On the git, the last commit is from 2013-07-10.
http://repo.or.cz/w/alure.git

It compile fine here. So maybe the easiest way to solve that bug is to make tarball from the git.
Comment 5 Joshua Clayton 2015-07-02 13:28:53 UTC
I get this same failure. The fix is simple. just need to #include <unistd.h> in that file
Comment 6 Joshua Clayton 2015-07-02 13:31:24 UTC
Created attachment 406070 [details, diff]
alure-1.2-include-unistd.patch

patch codec_fluidsynth.cpp to compile on nonwindows platforms
Comment 7 Joshua Clayton 2015-07-02 13:32:56 UTC
Created attachment 406072 [details]
alure-1.2-r1.ebuild

Ebuild with alure-1.2-include-unistd.patch
Comment 8 Oleh 2016-05-05 14:16:01 UTC
any progress in here. header include patch fixes compilation.
Comment 9 Oleh 2016-05-17 08:45:30 UTC
ping?
Comment 10 Michael Palimaka (kensington) gentoo-dev 2016-06-05 16:34:34 UTC
Thanks, fixed in git.

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=937a0fe10289f67201f7384ef57b0a833e9a1cea
Comment 11 James L. Hammons 2017-07-02 20:54:07 UTC
Unfortunately, this fix was undone in the latest -r1 version of the ebuild: instead of PATCHES the patch is put in a line called PATCJES. Fixing that line (PATCJES -> PATCHES) fixes the ebuild.
Comment 12 David Seifert gentoo-dev 2017-07-02 21:01:22 UTC
(In reply to James L. Hammons from comment #11)
> Unfortunately, this fix was undone in the latest -r1 version of the ebuild:
> instead of PATCHES the patch is put in a line called PATCJES. Fixing that
> line (PATCJES -> PATCHES) fixes the ebuild.

commit 03a9cede605f6a7c3e8b731b149d9cc71e13a0a4
Author: David Seifert <soap@gentoo.org>
Date:   Sun Jul 2 14:00:01 2017 -0700

    media-libs/alure: Fix spelling typo 'PATCHES'
    
    Bug: https://bugs.gentoo.org/show_bug.cgi?id=500164