search

openv / vcontrold

:fire: vcontrold Daemon for control and logging of Viessmann® type heating devices
https://github.com/openv/openv/wiki
GNU General Public License v3.0
101 stars 54 forks source link

Build options #12

Closed l3u closed 6 years ago

l3u commented 6 years ago

This makes building of the vsim and vclient helper programs optional (vclient is built by default, vsim not – this seems to be some tool only for development, isn't it?)

Also, compilation of the manpages can be switched on and off now (so that somebody having the Docutils installed but not wanting the manpages won't get them).

Can be used as follows:

cmake -DVSIM=ON -DMANPAGES=OFF ..

would e. g. trigger the vsim program to be compiled along with vcontrold and vclient, but not the manpages.

l3u commented 6 years ago

What's that AppVeyor stuff btw? Tells me something about MSBuild and Visual Studio – we're on Linux here, aren't we?!

speters commented 6 years ago

Sorry for the Appveyor confusion (now off). I registered for it to make it build for Cygwin , but I didn't realise that it was enabled be default without having it configured properly. I spent quite a while to figure out how to cross-compile for Cygwin on a Linux host (and failed), then I realised that Appveyor has Cygwin.

My short-term plans are to supply auto-generated binary builds for often-requested platforms Linux/x86, Linux/armhf (Raspi), Cygwin via the deployment possibilites of Travis CI and the likes.

l3u commented 6 years ago

So … what do you think about this?

speters commented 6 years ago

Yes, might be a good thing too keep vsim out of a normal build. For vclient, is don't see any advantages, as it's currently a cheap build without new dependencies.

hmueller01 commented 6 years ago

I do not see a real advantage. The build is so fast and the code is so small that it is not really worth it. But just my opinion.

hmueller01 commented 6 years ago

One should add these switches to INSTALL.md (or somewhere else) that it is documented ...

speters commented 6 years ago

Good point, is documented now.