You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2018/11/01 04:49:34 UTC

[kudu-CR] [docs] WARN AS ERROR requires doxygen >= 1.8.11

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11832 )

Change subject: [docs] WARN_AS_ERROR requires doxygen >= 1.8.11
......................................................................


Patch Set 4:

> So this means you can't run make_site.sh on el7 easily, right? What
 > distros do our release managers use when doing a release? Will this
 > be inconvenient for them?

That's a good point.  After some consideration I realized that I don't want to add restriction on the platform our RM use just because of this.  It seems a better approach would leaving the version requirement as is, so it will automatically kick in when running with a newer version of the doxygen tool.

The presence of WARN_AS_ERROR works with older versions of doxygen, but the tool emits additional warnings about unknown configuration parameter.  Maybe, I'll just make the presence of that parameter dependent on the doxygen's version.


-- 
To view, visit http://gerrit.cloudera.org:8080/11832
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7f2cfc0a9765ee0641f1cf0138ce92e54d18ce96
Gerrit-Change-Number: 11832
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <wd...@gmail.com>
Gerrit-Comment-Date: Thu, 01 Nov 2018 04:49:34 +0000
Gerrit-HasComments: No