search

aicenter / ShoDi

0 stars 0 forks source link

Fix the doxygen dependency #2

Closed F-I-D-O closed 4 years ago

F-I-D-O commented 4 years ago
  1. Find out whether doxygen is a standard dependency or not
  2. If it is, add doxygen install to the tutorial, if not, remove the dependency
neumannjan commented 4 years ago

I don't know what it means if something is a "standard dependency".

Currently, the doc build target (the documentation) is only available if Doxygen is present in the system. For building the library and/or the executable, Doxygen is currently not required.

The information that Doxygen is optional was present in README.md already. Information about additional build targets was added in 5dfb9df.

Since Doxygen isn't actually needed for building the library itself, I think it makes sense for it to be optional.

F-I-D-O commented 4 years ago

Ok, we can leave it as it is