Bug 165303 - Review Request: OpenSceneGraph
Summary: Review Request: OpenSceneGraph
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michael Schwendt
QA Contact: David Lawrence
URL: http://www.openscenegraph.org
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-08-07 11:02 UTC by Ralf Corsepius
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-08-31 16:11:34 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ralf Corsepius 2005-08-07 11:02:33 UTC
Spec Name or Url: ftp://packman.iu-bremen.de/fedora/SRPMS/OpenSceneGraph.spec
SRPM Name or Url: ftp://packman.iu-bremen.de/fedora/SRPMS/OpenSceneGraph-0.9.9-1.src.rpm

Description:
The OpenSceneGraph is an OpenSource, cross platform graphics toolkit for the
development of high performance graphics applications such as flight
simulators, games, virtual reality and scientific visualization.
Based around the concept of a SceneGraph, it provides an object oriented
framework on top of OpenGL freeing the developer from implementing and
optimizing low level graphics calls, and provides many additional utilities
for rapid development of graphics applications.

Warning: Due to possibilities to test building on x86_64 or ppc, and due to oddities of this package's configuration, I do not expect this package to build on x86_64 or ppc.

Comment 1 Colin Charles 2005-08-08 00:32:55 UTC
+ It built on ppc32 just fine.

- What is OSLGPL as a license?

(this is by no means a complete review though the spec looks clean, it builds on
PPC, rpmlint throws up a few errors that don't look too important)

Comment 2 Ralf Corsepius 2005-08-08 06:17:19 UTC
(In reply to comment #1)
> + It built on ppc32 just fine.
Good, thanks. x86_64 probably is more troublesome :(

> - What is OSLGPL as a license?
A typo - This should have been OSGPL (OpenSceneGraph Public License).

It's their "LGPL + special exceptions" variant. Cf. OpenSceneGraph/LICENSE.txt
and Producer/LICENSE.txt.

Comment 3 Ralf Corsepius 2005-08-09 15:32:56 UTC
Update with minor fixes applied:
Spec Name or Url: ftp://packman.iu-bremen.de/fedora/SRPMS/OpenSceneGraph.spec
SRPM Name or Url:
ftp://packman.iu-bremen.de/fedora/SRPMS/OpenSceneGraph-0.9.9-2.src.rpm

Comment 4 Michael Schwendt 2005-08-26 19:06:41 UTC
Not much to find:

* weak one-word description and summary for OpenThreads* packages
  and Producer* packages

* lots of C++ files which don't build with default RPM optflags,
  but just -O optimisation

* x86_64 X11 lib64->lib substitution is questionable
  in openscenegraph_0.9.9-5.diff.gz, probably needs to be reverted
  in the revdeb patch, too

* hardcoded ${exec_prefix}/lib in pkgconfig files most likely
  will cause install-time problems on x86_64

* Producer-devel pkgconfig file requires "openscenegraph"
  => I think that dependency is wrong

* OpenSceneGraph-devel pkgconfig requires "openthreads"
  => package is missing "Requires: OpenThreads-devel"

* OpenSceneGraph-examples package contains questionable empty directory
  /usr/share/OpenSceneGraph

* random examples run-time test:

$ osgforest
Warning: could not find file "Images/tree0.rgba"
Warning: font file "fonts/arial.ttf" not found.


Note You need to log in before you can comment on or make changes to this bug.