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@xxxxxxxxxx> 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review