bugOpenEXR - Bugs: bug #23012, Compile error with g++ 4.3

 
 

bug #23012: Compile error with g++ 4.3

Submitted by:  Pino Toscano <pinotree>
Submitted on:  Tue 22 Apr 2008 06:52:51 AM UTC  
 
Category: build/installSeverity: 3 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 22 Apr 2008 06:52:51 AM UTC, original submission:

Hi,

currently exrdisplay does not compile with g++ 4.3, as the declaration of the getenv() function is missing:

<<<<<
g++-4.3 -DHAVE_CONFIG_H -I. -I. -I../config -I/usr/include/freetype2 -D_THREAD_SAFE -D_REENTRANT -I.. -DHAVE_CTL_INTERPRETER=0 -pthread -I/usr/include/OpenEXR -pipe -g -Wall -O2 -c -o applyCtl.o applyCtl.cpp
applyCtl.cpp: In function ‘float displayVideoGamma()’:
applyCtl.cpp:86: error: ‘getenv’ was not declared in this scope
applyCtl.cpp: In function ‘Imf::Chromaticities<unnamed>::displayChromaticities()’:
applyCtl.cpp:125: error: ‘getenv’ was not declared in this scope

>>>>>


Attached there is a patch for exrdisplay/applyCtl.cpp that solves the problem, just including <cstdlib> is enough.

Please also note that a similar issue could be also in playexr/ctlToLut.cpp, and basically the same patch could be applied there too. (I miss the dependancies for compiling the CTL stuff, so I was not able to verify.)

Pino Toscano <pinotree>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #15505:  003_playexr_gcc42.diff added by pinotree (405B - text/x-diff - possible (untested though) patch for playexr/ctlToLut.cpp)
file #15495:  002_gcc43.diff added by pinotree (411B - text/x-diff - build patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by pinotree (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 22 Apr 2008 02:49:57 PM UTCpinotreeAttached File-=>Added 003_playexr_gcc42.diff, #15505
    Tue 22 Apr 2008 06:52:52 AM UTCpinotreeAttached File-=>Added 002_gcc43.diff, #15495

    Back to the top


    Powered by Savane 3.1-cleanup