On 14/08/2019 18:48, Rainer Jung wrote: > Hi there, > > this commit introduced a BOM at the beginning of the changelog.xml file, > which leads to a checkstyle validation failure during building 9.0.23 > (when validation is actiive). > > The BOM isn't shown in the below mail, but one can see the leading ef bb > bf wit any binary editor. I first thought the failure was due to the > checkstyle update, but it is just the BOM. > > That's no functional problem, but people trying to build 9.0.23 with > validation on will fail and wonder. Not sure whether that warrants a > retag. I can see the BOM in the 8.5.44 file as well.
I'll re-tag both of them. I think I edited the file with notepad rather than vi this time around which is probably how this happened. Apologies for the hassle. Mark > > Regards, > > Rainer > > Am 14.08.2019 um 13:10 schrieb ma...@apache.org: >> This is an automated email from the ASF dual-hosted git repository. >> >> markt pushed a commit to tag 9.0.23 >> in repository https://gitbox.apache.org/repos/asf/tomcat.git >> >> commit bd48c597e3cb280d73d9c9279babcaf75b6879c0 >> Author: Mark Thomas <ma...@apache.org> >> AuthorDate: Wed Aug 14 12:10:06 2019 +0100 >> >> Tag 9.0.23 >> --- >> build.properties.default | 2 +- >> webapps/docs/changelog.xml | 4 ++-- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/build.properties.default b/build.properties.default >> index 14ef092..3146ccb 100644 >> --- a/build.properties.default >> +++ b/build.properties.default >> @@ -27,7 +27,7 @@ version.major=9 >> version.minor=0 >> version.build=23 >> version.patch=0 >> -version.suffix=-dev >> +version.suffix= >> # ----- Build control flags ----- >> # Note enabling validation uses Checkstyle which is LGPL licensed >> diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml >> index 0cbf970..15646db 100644 >> --- a/webapps/docs/changelog.xml >> +++ b/webapps/docs/changelog.xml >> @@ -1,4 +1,4 @@ >> -<?xml version="1.0" encoding="UTF-8"?> >> +<?xml version="1.0" encoding="UTF-8"?> >> <!-- >> Licensed to the Apache Software Foundation (ASF) under one or more >> contributor license agreements. See the NOTICE file distributed with >> @@ -44,7 +44,7 @@ >> They eventually become mixed with the numbered issues (i.e., numbered >> issues do not "pop up" wrt. others). >> --> >> -<section name="Tomcat 9.0.23 (markt)" rtext="in development"> >> +<section name="Tomcat 9.0.23 (markt)"> >> <subsection name="Catalina"> >> <changelog> >> <update> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org