Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 38133 - error in povray 3.5c ebuild
Summary: error in povray 3.5c ebuild
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Luca Barbato
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-13 20:50 UTC by Jayson
Modified: 2004-01-15 09:12 UTC (History)
0 users

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


Attachments
new fixed ebuild (povray-3.50c.ebuild,3.84 KB, text/plain)
2004-01-13 20:53 UTC, Jayson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jayson 2004-01-13 20:50:12 UTC
There's an error in the povray 3.5c ebuild that causes it to be compiled without X display support, regardless if USE="X" is set or not. Here is the relevant part starting on line 47:

        use X && myconf="${myconf} --with-x" \
                || myconf="${myconf} --without-x"\
                && CFLAGS="${CFLAGS} -DX_DISPLAY_MISSING"

The last line will always be used due to precedence of operators ("and" comes before "or"), so -DX_DISPLAY_MISSING will always be set whether or not USE="X" is set. 

This means the render preview won't work, which is handy to have in modeling programs. 

steps to reproduce:
1. emerge sync
2. emerge \>=povray-3.5
3. start kpovmodeler (from kdegraphics)
4. click view->render
5. nothing is displayed in the render window

steps to fix:
1. modify povray-3.5c.ebuild 
2. emerge povray

Here's a diff which fixes the ebuild. It just removes the && from the last line since it isn't necessary to use && between variable settings anyway. 

--- povray-3.50c.ebuild.orig   2003-12-31 20:01:10.000000000 -0600
+++ povray-3.50c.ebuild     2004-01-13 22:20:10.000000000 -0600
@@ -46,7 +46,7 @@

        use X && myconf="${myconf} --with-x" \
                || myconf="${myconf} --without-x"\
-               && CFLAGS="${CFLAGS} -DX_DISPLAY_MISSING"
+                  CFLAGS="${CFLAGS} -DX_DISPLAY_MISSING"

        econf ${myconf} || die
Comment 1 Jayson 2004-01-13 20:53:06 UTC
Created attachment 23777 [details]
new fixed ebuild
Comment 2 Luca Barbato gentoo-dev 2004-01-15 09:12:39 UTC
Thank you, changed in portage