[CoinUtils] Build instruction on Coin Utils main page do not appear to work correctly

[Max Arai]

Hi Ted,

Your repair works perfectly.  Thank you for the help.

Max

On Apr 9, 2012, at 1:05 PM, Ted Ralphs wrote:

> I committed a fix. The missing functions were in fact defined in CoinIndexedVector.hpp (when NDEBUG is defined), but as empty inline functions. For some reason, those symbols don't appear in the library in that case. By defining them in CoinIndexedVector.cpp, everything goes through fine. Can you test?
> 
> Cheers,
> 
> Ted
> 
> 
> On Mon, Apr 9, 2012 at 11:55 AM, Ted Ralphs <ted at lehigh.edu> wrote:
> Ah, I was able to replicate what you're seeing now. I will look into it.
> 
> Ted
> 
> 
> On Mon, Apr 9, 2012 at 12:18 AM, Ted Ralphs <ted at lehigh.edu> wrote:
> Hi Max,
> 
> As it so happens, I just got a brand new iMac on Friday and was able to build CoinUtils (as well as all the rest of the projects in CoinAll) without any problems. The functions your reference *should* be undefined when not building a version of the code for debugging. Can you give a few more details so we track down the problem. What compilers are you using, do you have pkg-config installed, and whatever else you can think of that would be relevant. Unfortunately, there is no real "standard" setup in OSX.
> 
> Ted
> 
> On Sun, Apr 8, 2012 at 9:32 PM, Max Arai <maxarai at verizon.net> wrote:
> The page https://projects.coin-or.org/CoinUtils says
> 
> svn co https://projects.coin-or.org/svn/CoinUtils/trunk coin-CoinUtils
> cd coin-CoinUtils
> ./configure -C
> make
> make test
> make install
> Under OSX 10.7.3 (probably others but I did not test), the configure command needs to be
> 
> 	./configure --enable-debug -C
> 
> Otherwise, with the sources as of 8 April 2012 (SVN-1527 for CoinIndexedVector.cpp), the NDEBUG flag causes the symbols CoinIndexedVector::checkClear and CoinIndexedVector::checkClean to be undefined.  
> 
> I think lines 315 and 388 should be made blank to eliminate an unnecessary #ifndef NDEBUG/#endif pair
> 
> Please accept my apologies if this is not an appropriate forum for this comment.
> 
> Thanks
> Max
> 
> 
> _______________________________________________
> CoinUtils mailing list
> CoinUtils at list.coin-or.org
> http://list.coin-or.org/mailman/listinfo/coinutils
> 
> 
> 
> 
> -- 
> Dr. Ted Ralphs
> Associate Professor, Lehigh University
> (610) 628-1280
> ted 'at' lehigh 'dot' edu
> coral.ie.lehigh.edu/~ted
> 
> 
> 
> 
> -- 
> Dr. Ted Ralphs
> Associate Professor, Lehigh University
> (610) 628-1280
> ted 'at' lehigh 'dot' edu
> coral.ie.lehigh.edu/~ted
> 
> 
> 
> 
> -- 
> Dr. Ted Ralphs
> Associate Professor, Lehigh University
> (610) 628-1280
> ted 'at' lehigh 'dot' edu
> coral.ie.lehigh.edu/~ted
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.coin-or.org/pipermail/coinutils/attachments/20120409/50aa8890/attachment.html>