Compiling under Solaris 10

Bug #245173 reported by Marius Konitzer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Joshua Blocher

Bug Description

I tried to compile inkscape (svn head) under Solaris 10 using gcc-4.2. I had to patch a few lines only (see attachment) and what can I say -- it worked like a charm!

Regarding is{nan,inf,finite,normal} we seem to be in the same situation as under Solaris 8 here and (assuming continuity) probably under Solaris 9, too.

What I don't understand is, why some stuff is fixed for gcc-3.2 only; I had to remove this condition to make it work for me. This issue merely looks OS-specific, so one would expect it to be quite independent from the used gcc version, or am I wrong?

Tags: build solaris
Revision history for this message
Marius Konitzer (m-konitzer) wrote :
Revision history for this message
Joshua Blocher (verbalshadow) wrote :

This Patch is svn as of revision 19126.

Changed in inkscape:
assignee: nobody → verbalshadow
importance: Undecided → Low
status: New → Fix Committed
Changed in inkscape:
status: Fix Committed → Fix Released
Revision history for this message
Sam Sirlin (samuel-w-sirlin) wrote :

From the comments it looks like it was fixed, but it
doesn't seem fixed as of 0.47...

Revision history for this message
Marius Konitzer (m-konitzer) wrote :

It was until someone awkwardly broke it again in rev 19127. So please re-patch src/2geom/isnan.h.

Looks like we needlessly have to keep on patching till 0.48 is released. *sigh*

Revision history for this message
su_v (suv-lp) wrote :

re-opening - can someone confirm the patch in comment #1 for 'src/2geom/isnan.h' still works, without causing issues in 2geom or on other platforms?

Changed in inkscape:
status: Fix Released → Confirmed
Revision history for this message
Bryce Harrington (bryce) wrote :

The patch from comment #1 no longer works, but really only the first chunk is required; the remainder of that patch is already applied on trunk.

The regression is probably because the original fix was applied to inkscape's codebase and not to upstream 2geom. I've re-applied the fix to trunk, but am making the same mistake of not upstreaming it...

Changed in inkscape:
status: Confirmed → Fix Committed
su_v (suv-lp)
Changed in inkscape:
milestone: none → 0.91
Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.