[Bug 665733] Review Request: Coin3 - High-level 3D visualization library

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=665733

--- Comment #4 from Ralf Corsepius <rc040203 at freenet.de> 2011-11-07 22:47:27 EST ---
(In reply to comment #2)
> (In reply to comment #1)
> > 1. Are you planning on building for EL5?
> Well, note the age of this submission (~ 1 year) - It predates this change in
> Fedora - and the history of this package (Coin[12] packages's spec. This
> rpm'specs origins are much older than Fedora - Can now be cleaned up ;)

Done.

> > 3. Ok, in %configure I made quite a few changes. I went ahead and took
> > advantage of all the options that seemed appropriate. Here's my version, keep
> > in mind I removed all the alternative stuff for my personal build:
> > 
> > %configure \
> >         --includedir=%{_includedir}/Coin3 \
> >         --htmldir=%{_datadir}/doc/Coin3 \
> >         --disable-dependency-tracking \
> >         --enable-shared \
> >         --disable-dl-libbzip2 \
> >         --disable-dl-glu \
> >         --disable-dl-zlib \
> >         --disable-dl-freetype \
> >         --disable-dl-fontconfig \
> >         --disable-spidermonkey \
> >         --enable-man \
> >         --enable-html \
> >         --enable-3ds-import \
> >         CPPFLAGS=$(pkg-config --cflags freetype2)
> > 
> > The htmldir environment variable worked but since it offered a --htmldir option

The configure script treats --htmldir equivalently to passing 
htmldir to configure. As htmldir also works for Coin < 3, while --htmldir
doesn't, I'll continue using "htmldir=...".

>>  If this is truly documentation, should it not go in
> > /usr/share/doc/... and not /usr/share/...?
> I don't recall the details - Could be an artefact from Coin1 and Coin2 or a
> side effect of coin-config's habit to hard-code paths.
> To be checked.

It's not an artefact. Coin spans up a documentation system of its own for Coin
and its frontends (SoQt, SoGtk, SoWin, etc.). I.e. it expects a common
documentation root directory to install its frontends' html documentation into.

This doesn't fit into rpm's /usr/share/doc usage.

> > 4. %files:
> > 
> > %doc AUTHORS COPYING README* LICENSE* THANKS FAQ*
> > 
> > - The README* also grabs readme's for windows and mac so change to:
> > 
> > %doc AUTHORS COPYING README README.UNIX LICENSE* THANKS FAQ*
> OK, I'll look into this.
Done. All README.*'s are gone (Comprising README.UNIX, which only contains an
"install howto")

> An updated package to come sometime next week.

Here it is:

 http://corsepiu.fedorapeople.org/packages/Coin3.spec
 http://corsepiu.fedorapeople.org/packages/Coin3-3.1.3-2.fc17.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.