Invited contributions


#21

i haven't other OS to test. I don't know how to modify build.sh for osx and linux.

no issue, actually I'm not sure really we should be installing doxygen anyway... it not necessary for building etc, it could be considered an 'optional install', so I think its just a matter of checking to make sure that IF doxygen is installed that, the ant task finds it, I can check that for linux/mac OS X and update accordingly

@thetechnobear do you suggest to merge the PR as-is, or create a separate branch to merge it into, and merge that into master when complete?

I dont think its a big issue to merge it, the ant task has failifexecutionfails=false, so its not going to kill the build, if doxygen is not installed, just not produce the output.

the only thing, Im not sure of, without running it... is the affect this has had on existing documentation !?

Id be keen to ensure that from the Axoloti UI you can still pull up the existing help file(s), particularly if doxygen is not installed on the build machine.

IF doxygen is required to get the existing help files, then this will mean updating travis (probably not difficult, but just another step to be done before release)

so.... in short, Id say lets merge it into new branch, and then tomorrow, I can take a look check the existing stuff is not affected on OSX/Linux and then i can easily merge it into the main branch if its fine.

btw... I think its great getting doxygen in place top job @paco :thumbsup:


#22

Many thanks, @thetechnobear
I think this is just a starting point for me. Next, i'll add other pages to documentation, but then i have to implement a WDF tube emulator component to learn how simulate an audio filter circuit.