On 16.09.2013 11:24, Peter Stuge wrote:
> I'll move the conditionals to Makefile.am and we'll have to find
> solutions for the other build systems that we would like to support.
no, thats also not OK. Please lets keep it the way it is: we compile all
files, and define macros which control which crypto backend gets used.
>> We need to have the #ifdefs in the source files to
>> make it easier for people to build "all files" in different build systems
>> with less trouble...
>
> When are "all files" actually needed?
>
> It makes absolutely no sense to me to build source files that are
> *known* not to be needed.
it is the easy way to do things: while all preprocessors deal equally
good with #ifdef in the source files the different make tools do not in
the makefiles.
Also it doesnt hurt much if the compiler has to compile a file which is
'empty' due to define conditionals which are not met.
Gün.
_______________________________________________
libssh2-devel http://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-devel
Received on 2013-09-16