search

NOAA-EMC / NCEPLIBS-landsfcutil

Utility routines useful for initializing land-surface states in NCEP models.
Other
1 stars 5 forks source link

remove pfunit and ENABLE_TESTS option, tests will always be enabled #15

Open edwardhartnett opened 3 years ago

edwardhartnett commented 3 years ago

I build like this:

cmake -DCMAKE_PREFIX_PATH=/home/ed/NCEPLIBS/NCEPLIBS-landsfcutil ..
make VERBOSE=1

and it works. But when I try to run tests:

(base) ed@mikado:~/NCEPLIBS-landsfcutil/b$ make tests
make: *** No rule to make target 'tests'.  Stop.
(base) ed@mikado:~/NCEPLIBS-landsfcutil/b$ make test
make: *** No rule to make target 'test'.  Stop.
aerorahul commented 3 years ago

One needs to pass the -DENABLE_TESTS=ON flag to CMake. Testing requires pFUNIT, so you will need to have that available on your system.

edwardhartnett commented 3 years ago

Yes, I finally figured that out. ;-)

However, we have decided to do away with pfunit for exactly this sort of reason, so then we can also eliminate the ENABLE_TESTS option, and always enable tests, just like most other projects do.