HalosGhost / pbpst

A command-line libcurl C client for pb deployments
GNU General Public License v2.0
44 stars 9 forks source link

Compile errors on use of “fprintf(stderr,…” #43

Closed HalosGhost closed 8 years ago

HalosGhost commented 8 years ago

5dcd648 leads to a fair number of warnings (which we treat as errors) on the use of stderr because it is a recursive macro. We use it liberally for error reporting. We could either use a lot of diagnostic #pragmas to ignore these warnings, or we could create a simple error-reporting function and only ignore its definition. The latter is, of course, simpler, and would actually allow us to save some space too. However, I am not convinced it will be universally better. Either way, this is a high priority as it has broken the build for clean clones.

HalosGhost commented 8 years ago

Hmm, I actually cannot reproduce this on my own machine. I will see if I cannot reproduce this on my other machine. If so, then I'll fix it; if not, I'll close this.

Confirmed; not sure why this is not happening on my main machine.

HalosGhost commented 8 years ago

via: