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

[Ted Ralphs]

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
>>>
>>>
>>>    1. svn co https://projects.coin-or.org/svn/CoinUtils/trunk
>>>     coin-CoinUtils
>>>    2. cd coin-CoinUtils
>>>    3. ./configure -C
>>>    4. make
>>>    5. make test
>>>    6. 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 <http://coral.ie.lehigh.edu/%7Eted>
>>
>>
>
>
> --
> Dr. Ted Ralphs
> Associate Professor, Lehigh University
> (610) 628-1280
> ted 'at' lehigh 'dot' edu
> coral.ie.lehigh.edu/~ted <http://coral.ie.lehigh.edu/%7Eted>
>
>


-- 
Dr. Ted Ralphs
Associate Professor, Lehigh University
(610) 628-1280
ted 'at' lehigh 'dot' edu
coral.ie.lehigh.edu/~ted <http://coral.ie.lehigh.edu/%7Eted>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.coin-or.org/pipermail/coinutils/attachments/20120409/cf9da25b/attachment.html>