On 1/21/16, Stephan Beal <sgb...@googlemail.com> wrote: > Hiho, > > since this is pending review... > > http://www.fossil-scm.org/index.html/info/06fd798bdcb9da50 > > FYI: that won't compile properly on some systems because C does not allow > an empty compilation unit, which is what that amounts to when fuse is > disabled. (i had a similar proble > Personally i prefer the existing (trunk) behaviour over eliding the command > altogether.
I tried to merge it yesterday, but it gave problems. In particular I got "command not found" when I ran "fossil fuse" rather than "not implemented in this build", which I do not think is an improvement. So I left that proposal on its branch. I wasn't aware of the empty translation unit problem, but that seems like yet another reason to not merge this change. > > i remember seeing that when integrating json: > > [stephan@host:~/cvs/fossil/fossil/src]$ gcc -UFOSSIL_ENABLE_JSON -c > cson_amalgamation.c -o x > [stephan@host:~/cvs/fossil/fossil/src]$ l x > -rw-r--r-- 1 stephan stephan 941 Jan 21 13:50 x > > hmmm, okay, that works. But add -pedantic... > > [stephan@host:~/cvs/fossil/fossil/src]$ gcc -pedantic -UFOSSIL_ENABLE_JSON > -c cson_amalgamation.c -o x > cson_amalgamation.c:5706:0: warning: ISO C forbids an empty translation > unit [-Wpedantic] > #endif /* FOSSIL_ENABLE_JSON */ > ^ > > > -- > ----- stephan beal > http://wanderinghorse.net/home/stephan/ > http://gplus.to/sgbeal > "Freedom is sloppy. But since tyranny's the only guaranteed byproduct of > those who insist on a perfect world, freedom will have to do." -- Bigby > Wolf > -- D. Richard Hipp d...@sqlite.org _______________________________________________ fossil-dev mailing list fossil-dev@mailinglists.sqlite.org http://mailinglists.sqlite.org/cgi-bin/mailman/listinfo/fossil-dev
