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/install Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open

Add a New Comment(Rich 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 Files:
   
   
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
  • -email is unavailable- added by pinotree (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

     

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-04-22 pinotree Attached File- => Added 003_playexr_gcc42.diff, #15505
    2008-04-22 pinotree Attached File- => Added 002_gcc43.diff, #15495

    Back to the top


    Powered by Savane 3.3