Issue677

Title use separate Makefile/environment variables for 32- and 64-bit CPLEX library
Priority feature Status resolved
Superseder Nosy List florian, jendrik, malte
Assigned To florian Keywords
Optional summary
See issue504

Created on 2016-10-11.22:17:49 by florian, last changed by malte.

Summary
See issue504
Messages
msg5713 (view) Author: malte Date: 2016-10-12.01:17:21
Confirmation: it works nicely on the two machines I tested. As discussed off the
tracker, for the future I would suggest *only* having library environment
variables that are specific to a certain bit width. I think this would eliminate
a potential source of errors and would simplify the documentation because there
would be fewer different variations to document. But this is a lower priority
item for me than this issue here was.
msg5712 (view) Author: florian Date: 2016-10-12.00:23:32
Merged and updated the documentation.
msg5711 (view) Author: malte Date: 2016-10-12.00:08:43
Patch looks good to me. Feel free to merge, I'll try to test it as soon as
possible. Once it is merged, can you also update the LP build instructions page?
msg5710 (view) Author: florian Date: 2016-10-12.00:01:09
I created a pull request on Bitbucket:
https://bitbucket.org/FlorianPommerening/downward-issues/pull-requests/25

The change duplicates some code from FindOSI.cmake but CMake find-scripts should be 
self-contained, so I didn't want to factor this out.

My 32 bit and 64 bit version of CPLEX are installed in the same directory, but it 
looks like its working correctly from some debug output I introduced for testing 
locally.
msg5709 (view) Author: florian Date: 2016-10-11.22:17:48
We introduced separate variables for different build environments for COIN/OSI but 
not for CPLEX in issue504. Now, we also want to add them for CPLEX.
History
Date User Action Args
2016-10-12 01:17:21maltesetmessages: + msg5713
2016-10-12 00:23:32floriansetstatus: chatting -> resolved
messages: + msg5712
2016-10-12 00:08:43maltesetmessages: + msg5711
2016-10-12 00:01:09floriansetmessages: + msg5710
2016-10-11 22:17:49floriancreate