New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

travis: add build for "configure --disable-verbose" #3144

Closed
wants to merge 7 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@bagder
Member

bagder commented Oct 16, 2018

No description provided.

@bagder

This comment has been minimized.

Member

bagder commented Oct 17, 2018

Interestingly, Travis gets an error that I didn't (or missed):

In file included from file.c:55:0:
sendf.h:35:15: error: anonymous variadic macros were introduced in C99 [-Werror=variadic-macros]
 #define infof(...)  Curl_nop_stmt
               ^
@bagder

This comment has been minimized.

Member

bagder commented Oct 17, 2018

Disabling verbose really needs vararg macro support (to "remove" the infof() calls) which C89 didn't offer, so I'm inclined to just switch off the warning for this build.

bagder added some commits Oct 17, 2018

@MarcelRaad

Why do the 5XX tests fail so often on AppVeyor… :-(

@bagder

This comment has been minimized.

Member

bagder commented Oct 18, 2018

Very annoying yeah 👎

@bagder bagder closed this in ad547fc Oct 18, 2018

@bagder bagder deleted the bagder/travis-disable-verbose branch Oct 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment