Install script raising an error. about osmanip HOT 9 CLOSED

It's working properly now. Thanks!

from osmanip.

Hi @myermo yes, Doctest installation is optional. The first output string:

Doctest is not installed, cannot compile the test codes!

tells you that ONLY test codes are not compiled, but source code should be ok until that point.

The next compilation issue is due to the fact that I accidentally deleted a word from the echo command, during the last update. Now it has been restored with commit 8c62805 and should correctly work.

Thanks a lot for reporting this bug.

@all-contributors please add @myermo for review

from osmanip.

@JustWhit3

I've put up a pull request to add @myermo! 🎉

from osmanip.

Hello @JustWhit3. Thanks for the quick fix.

The script is still not compiling:

make: *** No rule to make target `bin/main'. Stop.

from osmanip.

Are you sure to be in the main directory of the repository? What is the exact command you ran? It should be:

./scripts/install.sh

and should be run directly from the repository home directory, not from inside the scripts directory.

from osmanip.

Yes, I am doing exactly that.

from osmanip.

In which operating system are you running?

from osmanip.

Hi,

uname -a
Linux manjaro-PC 5.13.19-2-MANJARO #1 SMP PREEMPT Sun Sep 19 21:31:53 UTC 2021 x86_64 GNU/Linux

ls
CITATION.cff doc img include LICENSE makefile obj README.md scripts src test

./scripts/install.sh
Doctest is not installed, cannot compile the test codes! Compiling only the main code (this is not a problem for the installation)... make: *** No rule to make target "bin/main". Stop. Compilation failed!

from osmanip.

Ok, with commits 186bf5a , f1606a8 and 207385b the bug should have been fixed. I tested it on both Ubuntu and WSL operating systems. I also updated the documentation, but nothing changed for the installation: you can simply update the repository and launch the installation command again.

Let me know if now is ok.

from osmanip.