From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 71E48A051C; Fri, 17 Jan 2020 18:42:50 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5E15414581; Fri, 17 Jan 2020 18:42:49 +0100 (CET) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 1FB6D378B for ; Fri, 17 Jan 2020 18:42:46 +0100 (CET) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Jan 2020 09:42:20 -0800 X-IronPort-AV: E=Sophos;i="5.70,331,1574150400"; d="scan'208";a="218975482" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.5.70]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 17 Jan 2020 09:42:19 -0800 Date: Fri, 17 Jan 2020 17:42:16 +0000 From: Bruce Richardson To: Aaron Conole Cc: david.marchand@redhat.com, dev@dpdk.org Message-ID: <20200117174216.GA1724@bricha3-MOBL.ger.corp.intel.com> References: <20200117104820.71403-1-bruce.richardson@intel.com> <20200117104820.71403-2-bruce.richardson@intel.com> <20200117132502.GA1714@bricha3-MOBL.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.1 (2019-06-15) Subject: Re: [dpdk-dev] [PATCH 2/2] doc: pass "werror" setting through to doc build X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jan 17, 2020 at 08:34:01AM -0500, Aaron Conole wrote: > Bruce Richardson writes: > > > On Fri, Jan 17, 2020 at 08:16:55AM -0500, Aaron Conole wrote: > >> Bruce Richardson writes: > >> > >> > When werror is set for the build, we should pass that flag through to > >> > sphinx so that it can flag warnings as errors too. > >> > > >> > Signed-off-by: Bruce Richardson > >> > --- > >> > >> I see that this actually works to generate the errors... BUT > >> > >> if we merge this it will break the build. Can you also insert a patch > >> to address the warning so that the series could be merged? > >> > > > > Ok, I didn't have any warnings in my setup, which is why I didn't see any > > problems. I assume that the warnings are showing up in travis? Anywhere > > else? > > I only saw them on Travis. > The error from sphinx in travis looks like a false positive that is fixed in later versions of sphinx. The error I see is: /home/travis/build/bruce-richardson/dpdk/doc/guides/linux_gsg/eal_args.include.rst:: WARNING: document isn't included in any toctree However, that file is an include one that is included in both the linux and freebsd eal parameters docs, and so is not missing though not included in the index. What is the best approach to deal with this, do you think? * rework so it has a toctree entry e.g. by creating a new section for common parameters * other workaround in the code, e.g. rename the file to not end in .rst * can we update sphinx in the travis build to avoid the warning altogether? /Bruce