{math}[foss/2017b] Ipopt 3.12.9#5800
Conversation
| configopts += ' --with-asl-lib=-lasl --with-asl-incdir=$EBROOTAMPLMINMP/include/asl' | ||
|
|
||
| sanity_check_paths = { | ||
| 'files': ['bin/ipopt', 'lib/libipopt.so'], |
There was a problem hiding this comment.
Please don't hardcode .so, use SHLIB_EXT instead please (.so is specific to Linux, we want to leave the door open for macOS too where a .so is a .dylib)
Maybe also check for libipoptamplinterface.so?
'files': ['bin/ipopt', 'lib/libipopt.%s' % SHLIB_EXT, 'lib/libipoptamplinterface.%s' % SHLIB_EXT],|
|
||
| sanity_check_paths = { | ||
| 'files': ['bin/ipopt', 'lib/libipopt.so'], | ||
| 'dirs': ['include/coin'], |
| separate_build_dir = True | ||
|
|
||
| sanity_check_paths = { | ||
| 'files': ['lib/libasl.a', 'lib/libmp.a'], |
There was a problem hiding this comment.
Why not also check for lib/libbenchmark.a?
I would include checking for specific binaries here too:
sanity_check_paths = {
'files': ['bin/tableproxy64', 'lib/libasl.a', 'lib/libbenchmark.a', 'lib/libmp.a'],
'dirs': ['include/asl', 'include/benchmark', 'include/mp', 'share'],
}|
Test report by @boegel |
|
Test report by @boegel |
|
Going in, thanks @smoors! |
No description provided.