[POSIX] incorrect included header files in src/color-profile.cpp for open(2)

[Milan Juřík]

src/color-profile.cpp includes sys/fcntl.h but it should include the whole set of headers for syscall open (fix build on Solaris)

[Jon A. Cruz (jon-joncruz)]

Looks like an OS difference, and one that would warrant some autoconf and #ifdef conditioning.

[su_v (suv-lp)]

Duplicate of, or covered by Bug #740969 in Inkscape: “build problems on Oracle "Solaris 10" X86_64”?
<https://bugs.launchpad.net/inkscape/+bug/740969>

[su_v (suv-lp)]

See also:
CompilingSunSolaris - Inkscape Wiki:
<http://wiki.inkscape.org/wiki/index.php/CompilingSunSolaris>

[Milan Juřík (milan-jurik)]

It is not Solaris specific issue, look at open(2) manpage in Linux (e.g. ,http://linux.die.net/man/2/open), including just sys/fcntl.h is not enough, it works on some Linux distros only because of luck. It is not about some ifdef, it is about POSIX.

[John Spencer (maillist-launchpad)]

this is really just an oversight of the developer who mistakenly added the wrong declaration, which happened to work on his system (glibc).

this really doesnt compile anywhere except on glibc.
and it is completely trivial to fix since glibc also has the right header

see the posix specs here:
http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/fcntl.h.html

affected distro: sabotage linux, using musl libc [1]
[1] http://www.musl-libc.org

[John Spencer (maillist-launchpad)]

btw, the proposed patch above is overkill
it is sufficient to only change sys/fcntl.h to fcntl.h.
there are no other missing symbols.

i can tell this for sure since musl doesnt expose any superfluous symbols, unlike glibc which has the habit of including other headers (e.g. stdio.h there pulls in stdlib.h which is a namespace violation)

[jazzynico (jazzynico)]

> btw, the proposed patch above is overkill
> it is sufficient to only change sys/fcntl.h to fcntl.h

Change tested on Windows XP and Debian testing, Inkscape trunk revision 12007.
The application compiles and runs with no problem on both systems.

If it's ok on OS X, I guess it's safe to commit it.

[su_v (suv-lp)]

> If it's ok on OS X, I guess it's safe to commit it.

Seems to compile and run ok on Mac OS X 10.5.8 (Xcode 3.1.4) and OS X 10.7 (Xcode 4.3.2) - I can't test on the most recent version OS X 10.8 (with Xcode 4.5.2) though.

On 10.5 and 10.7, the main content of '/usr/include/fcntl.h' is to include <sys/fcntl.h>:

$ tail -2 /usr/include/fcntl.h
 */
#include <sys/fcntl.h>

[jazzynico (jazzynico)]

Fix committed in the trunk, revision 12009.
 Milan and John, Thanks!

[su_v (suv-lp)]

Fix backported to lp:inkscape/0.48.x in revision 9997.