coin-or-tools / BuildTools

Macros and patches for GNU autotools
https://coin-or-tools.github.io/BuildTools/
Other
3 stars 7 forks source link

Prefix doesn't necessarily exist #127

Closed tkralphs closed 5 years ago

tkralphs commented 5 years ago

There is the following line in coin.m4 here:

full_prefix=`cd $full_prefix ; pwd`

This assumes the prefix exists, which it need not. I don't actually see the purpose of this, since the prefix must be specified as a full path anyway. If the prefix doesn't exist, this just spits out an error and doesn't cause the configure to fail, but it would be nice if this didn't happen.

svigerske commented 5 years ago

I'll just believe you on the "must be specified as a full path" and so removed this line. Thanks.

tkralphs commented 5 years ago

I don't know why/where the requirement is enforced, but here you go:

~/Projects/CoinUtils > mkdir build ~/Projects/CoinUtils > cd build; ../configure --prefix=test configure: error: expected an absolute directory name for --prefix: test