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.

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

Reply via email to