[CoinUtils-tickets] [COIN-OR Common Utilities] #10: Check for defined(__sparc), CoinSignal.hpp(73) excludes Solaris on Intel; build failure

[COIN-OR Common Utilities]

#10: Check for defined(__sparc), CoinSignal.hpp(73) excludes Solaris on Intel;
build failure
------------------------+---------------------------------------------------
 Reporter:  lou         |       Owner:  somebody
     Type:  defect      |      Status:  new     
 Priority:  major       |   Milestone:          
Component:  component1  |     Version:          
 Keywords:              |  
------------------------+---------------------------------------------------
 At line 73 of {{{CoinSignal.hpp}}}, there's a guard,
 {{{
 #if defined(__sparc) && defined(__sun)
 }}}
 This excludes Solaris on Intel hardware. Hence no definition of
 {{{CoinSighandler_t}}}, and the build reaches the default case at the end
 of the file. At this point, --pedantic-errors causes a build failure
 because #warning is a GCC extension.

 Proposed change would be to reduce the guard to
 {{{
 #if defined(__sun)
 }}}
 Does anyone know if the test for {{{defined(__sparc)}}} was added to solve
 a specific problem?

 Problem reported by Dan Chen (re. BCP build failure), David Bremner (re.
 SYMPHONY build failure).

-- 
Ticket URL: <https://projects.coin-or.org/CoinUtils/ticket/10>
COIN-OR Common Utilities <http://projects.coin-or.org/CoinUtils>
Common data structures and linear algebra functions for COIN-OR projects