Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 231962 - media-libs/openinventor-2.1.5.10-r2 will not build on sparc: machine.h does not know about sparc/linux systems
Summary: media-libs/openinventor-2.1.5.10-r2 will not build on sparc: machine.h does ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: Sparc Linux
: Normal minor
Assignee: Gentoo Graphics Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 230716
  Show dependency tree
 
Reported: 2008-07-16 13:26 UTC by Ferris McCormick (RETIRED)
Modified: 2008-08-10 14:37 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ferris McCormick (RETIRED) gentoo-dev 2008-07-16 13:26:31 UTC
sci-physics/geant-4.9.1_p02 wants to be able to get an openinventor with USE=openinventor.  However, on sparc we must package.use.mask this option because openinventor will not build.  It will not build because its architecture file (machine.h) does not know about sparc linux (or any big-endian linux for that matter).  With the following little patch, openinventor will build and install on sparc, and the demos all seem good.  I do not know if everything is correct, however, because this change only matters in a couple classes (SoInput, SoOutput), and I don't know if the demos hit them.  No attachment because the test patch changes only one line.
===============================================
polylepis inventor # diff -u lib/database/include/machine.h- lib/database/include/machine.h
--- lib/database/include/machine.h-     2008-07-16 12:54:21.000000000 +0000
+++ lib/database/include/machine.h      2008-07-16 12:54:34.000000000 +0000
@@ -106,7 +106,7 @@
  * SUN/BSD machine dependent setup
  */
 
-#if sun
+#if sun || sparc
 
 #define MACHINE_WORD_FORMAT    DGL_BIG_ENDIAN
 #define MACHINE_FLOAT_FORMAT   DGL_BIG_IEEE
================================================
Comment 1 Markus Meier gentoo-dev 2008-08-10 09:57:12 UTC
feel free to apply this patch, thanks.
Comment 2 Stefaan De Roeck (RETIRED) gentoo-dev 2008-08-10 12:10:55 UTC
I have put the patch in the repository, so it should build now.  Have fun!
Comment 3 Ferris McCormick (RETIRED) gentoo-dev 2008-08-10 14:37:53 UTC
Thanks, Stefaan, it builds now as expected..  I've keyworded media-libs/openinventor-2.1.5.10-r2 as ~sparc and removed the package.use.mask for openinventor on sci-physics/geant.  I intend to mark openinventor stable about the 8th of September unless someone needs it sooner (the patch has been available since the 16th of July).