From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Fri, 21 Oct 2016 09:39:28 +0200 Subject: [U-Boot] [PATCH 4/8] travis-ci: Do not make buildman warnings fatal In-Reply-To: <1476991566-2989-5-git-send-email-trini@konsulko.com> References: <1476991566-2989-1-git-send-email-trini@konsulko.com> <1476991566-2989-5-git-send-email-trini@konsulko.com> Message-ID: <5809C630.8030202@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Tom, Am 20.10.2016 um 21:26 schrieb Tom Rini: > We currently will always see a number of warnings due to device tree > issues. These (and other warnings) should not make the build be marked > as failure so catch exit status 129 specifically and return 0 in that > case. > > Cc: Heiko Schocher > Signed-off-by: Tom Rini You mean here this "Warning (unit_address_vs_reg):" warnings? IIRC I posted some month ago a patch, which suppress them ... the patch was rejected, with the plausible reason, that we should fix this issues ... so I started and posted some dts fix patches, also rejected with the reason that this should come from linux syncs ... Hmm.. do we syncs for our DTS files with linux? Nevertheless: we should at least fix this for travis, so: Reviewed-by: Heiko Schocher bye, Heiko > --- > .travis.yml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index 2f1d05d443f5..d93efe02b0b1 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -69,8 +69,15 @@ script: > - if [[ "${TEST_CMD}" != "" ]]; then > ${TEST_CMD}; > fi > + # Exit code 129 means warnings only. > - if [[ "${BUILDMAN}" != "" ]]; then > + set +e; > tools/buildman/buildman ${BUILDMAN}; > + if [[ "$?" == "0" || "$?" == "129" ]]; then > + exit 0; > + else > + exit $?; > + fi > fi > > matrix: > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany