You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Alex Herbert <al...@gmail.com> on 2019/11/20 14:47:59 UTC

[VOTE] Release Apache Commons Skin 4.2 based on RC2

We have fixed a few bugs and added one enhancement since Apache Commons 
Skin 4.1 was released, so I would like to release Apache Commons Skin 4.2.

The Git tag commons-skin-4.2-RC2 commit for this RC is 
5b820663afd0535a9a9d089a8a4ecc794873e9d7 which you can browse here:
https://gitbox.apache.org/repos/asf?p=commons-skin.git;a=commit;h=5b820663afd0535a9a9d089a8a4ecc794873e9d7
You may checkout this tag using:
     git clone https://gitbox.apache.org/repos/asf/commons-skin.git 
--branch commons-skin-4.2-RC2 commons-skin-4.2-RC2


Maven artifacts are here:

https://repository.apache.org/content/repositories/orgapachecommons-1479/org/apache/commons/commons-skin/4.2/


Note: There are no binary artifacts for this component.


I have tested this with 'mvn clean install' using:

Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 
2018-10-24T19:41:47+01:00)
Maven home: /usr/local/apache-maven-3.6.0
Java version: 1.8.0_222, vendor: Private Build, runtime: 
/usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "4.4.0-166-generic", arch: "amd64", family: 
"unix"


*****************
NOTE: Please read the 'Validating a release candidate' section below to 
test the candidate since there are no class files in commons-skin. It 
contains templates for rendering the site for other commons components.
*****************


Details of changes since 4.1 are in the release notes and reproduced here:

New features:
o   Optionally render bare <div class="source"><pre> tags using 
prettyprint.
     The functionality is enabled using custom properties to be added to 
the site.xml.

Fixed Bugs:
o   Fix processing of custom <footer> element from commons-parent site.xml.
o   Update processing of <head> section to handle injected XHTML.
o   Fixed background on navigation menu collapsed icon from white to 
transparent.

Removed:
o   Remove unused css files. These are not used in the current skin and
     are from previous skin versions.


Note:

No stylesheet changes have been made in this release. It fixes 
functionality in commons-skin-4.1 that no longer works with the 
maven-site-plugin specified in the most recent commons-parent.


There is no site for this component.



KEYS:
https://www.apache.org/dist/commons/KEYS

Please review the release candidate and vote.
This vote will close no sooner that 72 hours from now.

   [ ] +1 Release these artifacts
   [ ] +0 OK, but...
   [ ] -0 OK, but really should fix...
   [ ] -1 I oppose this release because...

Thank you,

Alex Herbert,
Release Manager (using key BC87A3FD0A54480F0BADBEBD21939FF0CA2A6567)


Validating a release candidate
==============================

These guidelines are NOT complete.

Requirements: Git, Java, Maven.

You can validate a release from a release candidate (RC) tag as follows.

1) Clone and checkout the RC tag

git clone https://gitbox.apache.org/repos/asf/commons-skin.git --branch 
commons-skin-4.2-RC2 commons-skin-4.2-RC2
cd commons-skin-4.2-RC2

2) Check Apache licenses

mvn apache-rat:check

3) Check binary compatibility

SKIPPED - not applicable

4) Build and install the package

mvn install

You can record the Maven and Java version produced by -V in your VOTE 
reply.
To gather OS information from a command line:
Windows: ver
Linux: uname -a

5) Test the skin:

You can test Apache Commons Skin by adding the following to your 
site.xml site descriptor for the maven-site-plugin:

   <skin>
     <groupId>org.apache.commons</groupId>
     <artifactId>commons-skin</artifactId>
     <version>4.2</version>
   </skin>

   <body>
     <!-- Custom <head> tag with injected XHTML is supported. -->
     <head>
       <![CDATA[<script type="text/javascript" id="MathJax-script" async 
src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-mml-chtml.js">
       </script>]]>
     </head>
     ...
   </body>



   <custom>
     <!-- Escaping with CDATA is not required. -->
     <footer>
       <div class="center">
       Apache Commons, Apache @project.name@, Apache, the Apache feather 
logo, and the Apache Commons project logos are trademarks of The Apache 
Software Foundation.
       All other marks mentioned may be trademarks or registered 
trademarks of their respective owners.
       </div>
     </footer>

     <!-- Custom properties controlling the commons-skin template. -->
     <commonsSkin>
       <!--
         Convert preformatted source section tags to prettyprint:
           <div class="source"><pre> to <div class="source"><pre 
class="prettyprint">
       -->
<prettyPrintSourcePreTags>false</prettyPrintSourcePreTags>

       <!-- Add the "linenums" class to the prettyprint enabled source 
tags -->
<prettyPrintLineNumbersEnabled>false</prettyPrintLineNumbersEnabled>
     </commonsSkin>
   </custom>


Please check that:

1. The site looks like it does with commons-skin-4.1. CRITICAL!

2. The head meta data contains the injected MathJax script (as described 
above). Try injecting other XHTML to verify this functionality adds the 
XHTML to the rendered pages.

3. The footer contains the Apache copyright information (requires the 
custom <footer> element described above; it should not be escaped with 
CDATA tags). The duplicate test 'Apache Apache' should not appear when 
describing the project.

4. The collapsed menu icons DO NOT have the white background around the 
small grey arrow.


Note: Most commons sites do not require the custom properties to 
configure prettyprint in their documentation since they are written 
using the XDOC format.

Any XDOC format can include code snippets using for example:

<source class=<prettyprint">
String value = "coloured";
</source>

This functionality was supported by commons-skin-4.1. Check it still 
works if you use code snippets with prettyprint.

APT format documents can include code snippets using for example:

+--------------------------+
String value = "coloured";
+--------------------------+

This can now be rendered using prettyprint with the custom properties in 
the site.xml.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Alex Herbert <al...@gmail.com>.
On 25/11/2019 10:39, Gilles Sadowski wrote:
> Le lun. 25 nov. 2019 à 11:17, Alex Herbert <al...@gmail.com> a écrit :
>>
>> On 25/11/2019 10:02, Gilles Sadowski wrote:
>>> Hello.
>>>
>>> Le sam. 23 nov. 2019 à 09:49, Alex Herbert <al...@gmail.com> a écrit :
>>>> Thanks Bruno.
>>>>
>>>> Can I get a review from anyone else?
>>> I've just updated
>>>       http://commons.apache.org/proper/commons-geometry/
>>> using your instructions.
>>> Looks fine.
>> I assume you can now get MathJax to work on the geometry site.
> There is no MathJax comment yet.
>
>> Is that a +1?
> Sure.
>
> The site is not fully functional (same issue as for [RNG] with the logo in
> modules pages) but everything which I used to click seems to work.

OK. I will get the commons-skin release out and then the changes to 
commons-parent to support the RNG site. Then it should be relatively 
easy to port the same changes for the multi-module set-up to geometry, 
statistics and numbers.

I think there may be a way to support the relative links in the header 
for each module using the top level site.xml. But I have yet to try. The 
fix for RNG was to use absolute links and let the maven site plugin 
resolve them. However commons-skin should support another solution.

>
>>> Regards,
>>> Gilles
>>>
>>>> [...]
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Gilles Sadowski <gi...@gmail.com>.
Le lun. 25 nov. 2019 à 11:17, Alex Herbert <al...@gmail.com> a écrit :
>
>
> On 25/11/2019 10:02, Gilles Sadowski wrote:
> > Hello.
> >
> > Le sam. 23 nov. 2019 à 09:49, Alex Herbert <al...@gmail.com> a écrit :
> >> Thanks Bruno.
> >>
> >> Can I get a review from anyone else?
> > I've just updated
> >      http://commons.apache.org/proper/commons-geometry/
> > using your instructions.
> > Looks fine.
>
> I assume you can now get MathJax to work on the geometry site.

There is no MathJax comment yet.

>
> Is that a +1?

Sure.

The site is not fully functional (same issue as for [RNG] with the logo in
modules pages) but everything which I used to click seems to work.

> >
> > Regards,
> > Gilles
> >
> >> [...]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Alex Herbert <al...@gmail.com>.
On 25/11/2019 10:02, Gilles Sadowski wrote:
> Hello.
>
> Le sam. 23 nov. 2019 à 09:49, Alex Herbert <al...@gmail.com> a écrit :
>> Thanks Bruno.
>>
>> Can I get a review from anyone else?
> I've just updated
>      http://commons.apache.org/proper/commons-geometry/
> using your instructions.
> Looks fine.

I assume you can now get MathJax to work on the geometry site.

Is that a +1?

>
> Regards,
> Gilles
>
>> [...]
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Gilles Sadowski <gi...@gmail.com>.
Hello.

Le sam. 23 nov. 2019 à 09:49, Alex Herbert <al...@gmail.com> a écrit :
>
> Thanks Bruno.
>
> Can I get a review from anyone else?

I've just updated
    http://commons.apache.org/proper/commons-geometry/
using your instructions.
Looks fine.

Regards,
Gilles

> [...]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Alex Herbert <al...@gmail.com>.
Thanks Bruno.

Can I get a review from anyone else?

I was also wondering if this should have been done under a [Lazy][Vote] as this is an internal component. If so it would pass by lazy consensus.

Alex


> On 22 Nov 2019, at 03:24, Bruno P. Kinoshita <ki...@apache.org> wrote:
> 
>    [x] +1 Release these artifacts
> 
> Couldn't validate using the skin on a site, sorry.
> Build passing from tag, with mvn clean install, on
> 
> 
> Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-18T06:33:14+12:00)
> Maven home: /opt/apache-maven-3.5.4
> Java version: 1.8.0_222, vendor: Private Build, runtime: /usr/lib/jvm/java-8-openjdk-amd64/jre
> Default locale: en_NZ, platform encoding: UTF-8
> OS name: "linux", version: "4.15.0-70-generic", arch: "amd64", family: "unix"
> 
> And
> 
> Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-18T06:33:14+12:00)
> Maven home: /opt/apache-maven-3.5.4
> Java version: 11.0.2, vendor: Oracle Corporation, runtime: /home/kinow/Development/java/jdk-11.0.2
> Default locale: en_NZ, platform encoding: UTF-8
> OS name: "linux", version: "4.15.0-70-generic", arch: "amd64", family: "unix"
> 
> Checked signatures of the Maven artefacts and found no issues.
> ThanksBruno
> 
>    On Thursday, 21 November 2019, 3:48:13 am NZDT, Alex Herbert <al...@gmail.com> wrote:  
> 
> We have fixed a few bugs and added one enhancement since Apache Commons 
> Skin 4.1 was released, so I would like to release Apache Commons Skin 4.2.
> 
> The Git tag commons-skin-4.2-RC2 commit for this RC is 
> 5b820663afd0535a9a9d089a8a4ecc794873e9d7 which you can browse here:
> https://gitbox.apache.org/repos/asf?p=commons-skin.git;a=commit;h=5b820663afd0535a9a9d089a8a4ecc794873e9d7
> You may checkout this tag using:
>     git clone https://gitbox.apache.org/repos/asf/commons-skin.git 
> --branch commons-skin-4.2-RC2 commons-skin-4.2-RC2
> 
> 
> Maven artifacts are here:
> 
> https://repository.apache.org/content/repositories/orgapachecommons-1479/org/apache/commons/commons-skin/4.2/
> 
> 
> Note: There are no binary artifacts for this component.
> 
> 
> I have tested this with 'mvn clean install' using:
> 
> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 
> 2018-10-24T19:41:47+01:00)
> Maven home: /usr/local/apache-maven-3.6.0
> Java version: 1.8.0_222, vendor: Private Build, runtime: 
> /usr/lib/jvm/java-8-openjdk-amd64/jre
> Default locale: en_GB, platform encoding: UTF-8
> OS name: "linux", version: "4.4.0-166-generic", arch: "amd64", family: 
> "unix"
> 
> 
> *****************
> NOTE: Please read the 'Validating a release candidate' section below to 
> test the candidate since there are no class files in commons-skin. It 
> contains templates for rendering the site for other commons components.
> *****************
> 
> 
> Details of changes since 4.1 are in the release notes and reproduced here:
> 
> New features:
> o   Optionally render bare <div class="source"><pre> tags using 
> prettyprint.
>     The functionality is enabled using custom properties to be added to 
> the site.xml.
> 
> Fixed Bugs:
> o   Fix processing of custom <footer> element from commons-parent site.xml.
> o   Update processing of <head> section to handle injected XHTML.
> o   Fixed background on navigation menu collapsed icon from white to 
> transparent.
> 
> Removed:
> o   Remove unused css files. These are not used in the current skin and
>     are from previous skin versions.
> 
> 
> Note:
> 
> No stylesheet changes have been made in this release. It fixes 
> functionality in commons-skin-4.1 that no longer works with the 
> maven-site-plugin specified in the most recent commons-parent.
> 
> 
> There is no site for this component.
> 
> 
> 
> KEYS:
> https://www.apache.org/dist/commons/KEYS
> 
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now.
> 
>   [ ] +1 Release these artifacts
>   [ ] +0 OK, but...
>   [ ] -0 OK, but really should fix...
>   [ ] -1 I oppose this release because...
> 
> Thank you,
> 
> Alex Herbert,
> Release Manager (using key BC87A3FD0A54480F0BADBEBD21939FF0CA2A6567)
> 
> 
> Validating a release candidate
> ==============================
> 
> These guidelines are NOT complete.
> 
> Requirements: Git, Java, Maven.
> 
> You can validate a release from a release candidate (RC) tag as follows.
> 
> 1) Clone and checkout the RC tag
> 
> git clone https://gitbox.apache.org/repos/asf/commons-skin.git --branch 
> commons-skin-4.2-RC2 commons-skin-4.2-RC2
> cd commons-skin-4.2-RC2
> 
> 2) Check Apache licenses
> 
> mvn apache-rat:check
> 
> 3) Check binary compatibility
> 
> SKIPPED - not applicable
> 
> 4) Build and install the package
> 
> mvn install
> 
> You can record the Maven and Java version produced by -V in your VOTE 
> reply.
> To gather OS information from a command line:
> Windows: ver
> Linux: uname -a
> 
> 5) Test the skin:
> 
> You can test Apache Commons Skin by adding the following to your 
> site.xml site descriptor for the maven-site-plugin:
> 
>   <skin>
>     <groupId>org.apache.commons</groupId>
>     <artifactId>commons-skin</artifactId>
>     <version>4.2</version>
>   </skin>
> 
>   <body>
>     <!-- Custom <head> tag with injected XHTML is supported. -->
>     <head>
>       <![CDATA[<script type="text/javascript" id="MathJax-script" async 
> src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-mml-chtml.js">
>       </script>]]>
>     </head>
>     ...
>   </body>
> 
> 
> 
>   <custom>
>     <!-- Escaping with CDATA is not required. -->
>     <footer>
>       <div class="center">
>       Apache Commons, Apache @project.name@, Apache, the Apache feather 
> logo, and the Apache Commons project logos are trademarks of The Apache 
> Software Foundation.
>       All other marks mentioned may be trademarks or registered 
> trademarks of their respective owners.
>       </div>
>     </footer>
> 
>     <!-- Custom properties controlling the commons-skin template. -->
>     <commonsSkin>
>       <!--
>         Convert preformatted source section tags to prettyprint:
>           <div class="source"><pre> to <div class="source"><pre 
> class="prettyprint">
>       -->
> <prettyPrintSourcePreTags>false</prettyPrintSourcePreTags>
> 
>       <!-- Add the "linenums" class to the prettyprint enabled source 
> tags -->
> <prettyPrintLineNumbersEnabled>false</prettyPrintLineNumbersEnabled>
>     </commonsSkin>
>   </custom>
> 
> 
> Please check that:
> 
> 1. The site looks like it does with commons-skin-4.1. CRITICAL!
> 
> 2. The head meta data contains the injected MathJax script (as described 
> above). Try injecting other XHTML to verify this functionality adds the 
> XHTML to the rendered pages.
> 
> 3. The footer contains the Apache copyright information (requires the 
> custom <footer> element described above; it should not be escaped with 
> CDATA tags). The duplicate test 'Apache Apache' should not appear when 
> describing the project.
> 
> 4. The collapsed menu icons DO NOT have the white background around the 
> small grey arrow.
> 
> 
> Note: Most commons sites do not require the custom properties to 
> configure prettyprint in their documentation since they are written 
> using the XDOC format.
> 
> Any XDOC format can include code snippets using for example:
> 
> <source class=<prettyprint">
> String value = "coloured";
> </source>
> 
> This functionality was supported by commons-skin-4.1. Check it still 
> works if you use code snippets with prettyprint.
> 
> APT format documents can include code snippets using for example:
> 
> +--------------------------+
> String value = "coloured";
> +--------------------------+
> 
> This can now be rendered using prettyprint with the custom properties in 
> the site.xml.
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by "Bruno P. Kinoshita" <ki...@apache.org>.
    [x] +1 Release these artifacts

Couldn't validate using the skin on a site, sorry.
Build passing from tag, with mvn clean install, on


Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-18T06:33:14+12:00)
Maven home: /opt/apache-maven-3.5.4
Java version: 1.8.0_222, vendor: Private Build, runtime: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_NZ, platform encoding: UTF-8
OS name: "linux", version: "4.15.0-70-generic", arch: "amd64", family: "unix"

And

Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-18T06:33:14+12:00)
Maven home: /opt/apache-maven-3.5.4
Java version: 11.0.2, vendor: Oracle Corporation, runtime: /home/kinow/Development/java/jdk-11.0.2
Default locale: en_NZ, platform encoding: UTF-8
OS name: "linux", version: "4.15.0-70-generic", arch: "amd64", family: "unix"

Checked signatures of the Maven artefacts and found no issues.
ThanksBruno

    On Thursday, 21 November 2019, 3:48:13 am NZDT, Alex Herbert <al...@gmail.com> wrote:  
 
 We have fixed a few bugs and added one enhancement since Apache Commons 
Skin 4.1 was released, so I would like to release Apache Commons Skin 4.2.

The Git tag commons-skin-4.2-RC2 commit for this RC is 
5b820663afd0535a9a9d089a8a4ecc794873e9d7 which you can browse here:
https://gitbox.apache.org/repos/asf?p=commons-skin.git;a=commit;h=5b820663afd0535a9a9d089a8a4ecc794873e9d7
You may checkout this tag using:
     git clone https://gitbox.apache.org/repos/asf/commons-skin.git 
--branch commons-skin-4.2-RC2 commons-skin-4.2-RC2


Maven artifacts are here:

https://repository.apache.org/content/repositories/orgapachecommons-1479/org/apache/commons/commons-skin/4.2/


Note: There are no binary artifacts for this component.


I have tested this with 'mvn clean install' using:

Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 
2018-10-24T19:41:47+01:00)
Maven home: /usr/local/apache-maven-3.6.0
Java version: 1.8.0_222, vendor: Private Build, runtime: 
/usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_GB, platform encoding: UTF-8
OS name: "linux", version: "4.4.0-166-generic", arch: "amd64", family: 
"unix"


*****************
NOTE: Please read the 'Validating a release candidate' section below to 
test the candidate since there are no class files in commons-skin. It 
contains templates for rendering the site for other commons components.
*****************


Details of changes since 4.1 are in the release notes and reproduced here:

New features:
o   Optionally render bare <div class="source"><pre> tags using 
prettyprint.
     The functionality is enabled using custom properties to be added to 
the site.xml.

Fixed Bugs:
o   Fix processing of custom <footer> element from commons-parent site.xml.
o   Update processing of <head> section to handle injected XHTML.
o   Fixed background on navigation menu collapsed icon from white to 
transparent.

Removed:
o   Remove unused css files. These are not used in the current skin and
     are from previous skin versions.


Note:

No stylesheet changes have been made in this release. It fixes 
functionality in commons-skin-4.1 that no longer works with the 
maven-site-plugin specified in the most recent commons-parent.


There is no site for this component.



KEYS:
https://www.apache.org/dist/commons/KEYS

Please review the release candidate and vote.
This vote will close no sooner that 72 hours from now.

   [ ] +1 Release these artifacts
   [ ] +0 OK, but...
   [ ] -0 OK, but really should fix...
   [ ] -1 I oppose this release because...

Thank you,

Alex Herbert,
Release Manager (using key BC87A3FD0A54480F0BADBEBD21939FF0CA2A6567)


Validating a release candidate
==============================

These guidelines are NOT complete.

Requirements: Git, Java, Maven.

You can validate a release from a release candidate (RC) tag as follows.

1) Clone and checkout the RC tag

git clone https://gitbox.apache.org/repos/asf/commons-skin.git --branch 
commons-skin-4.2-RC2 commons-skin-4.2-RC2
cd commons-skin-4.2-RC2

2) Check Apache licenses

mvn apache-rat:check

3) Check binary compatibility

SKIPPED - not applicable

4) Build and install the package

mvn install

You can record the Maven and Java version produced by -V in your VOTE 
reply.
To gather OS information from a command line:
Windows: ver
Linux: uname -a

5) Test the skin:

You can test Apache Commons Skin by adding the following to your 
site.xml site descriptor for the maven-site-plugin:

   <skin>
     <groupId>org.apache.commons</groupId>
     <artifactId>commons-skin</artifactId>
     <version>4.2</version>
   </skin>

   <body>
     <!-- Custom <head> tag with injected XHTML is supported. -->
     <head>
       <![CDATA[<script type="text/javascript" id="MathJax-script" async 
src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-mml-chtml.js">
       </script>]]>
     </head>
     ...
   </body>



   <custom>
     <!-- Escaping with CDATA is not required. -->
     <footer>
       <div class="center">
       Apache Commons, Apache @project.name@, Apache, the Apache feather 
logo, and the Apache Commons project logos are trademarks of The Apache 
Software Foundation.
       All other marks mentioned may be trademarks or registered 
trademarks of their respective owners.
       </div>
     </footer>

     <!-- Custom properties controlling the commons-skin template. -->
     <commonsSkin>
       <!--
         Convert preformatted source section tags to prettyprint:
           <div class="source"><pre> to <div class="source"><pre 
class="prettyprint">
       -->
<prettyPrintSourcePreTags>false</prettyPrintSourcePreTags>

       <!-- Add the "linenums" class to the prettyprint enabled source 
tags -->
<prettyPrintLineNumbersEnabled>false</prettyPrintLineNumbersEnabled>
     </commonsSkin>
   </custom>


Please check that:

1. The site looks like it does with commons-skin-4.1. CRITICAL!

2. The head meta data contains the injected MathJax script (as described 
above). Try injecting other XHTML to verify this functionality adds the 
XHTML to the rendered pages.

3. The footer contains the Apache copyright information (requires the 
custom <footer> element described above; it should not be escaped with 
CDATA tags). The duplicate test 'Apache Apache' should not appear when 
describing the project.

4. The collapsed menu icons DO NOT have the white background around the 
small grey arrow.


Note: Most commons sites do not require the custom properties to 
configure prettyprint in their documentation since they are written 
using the XDOC format.

Any XDOC format can include code snippets using for example:

<source class=<prettyprint">
String value = "coloured";
</source>

This functionality was supported by commons-skin-4.1. Check it still 
works if you use code snippets with prettyprint.

APT format documents can include code snippets using for example:

+--------------------------+
String value = "coloured";
+--------------------------+

This can now be rendered using prettyprint with the custom properties in 
the site.xml.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

  

[RESULT][VOTE] Release Apache Commons Skin 4.2 based on RC2

Posted by Alex Herbert <al...@gmail.com>.
The votes are as follows:

+1 - Bruno Kinoshita (binding)
+1 - Gilles Sadowski (binding)
+1 - Alex Herbert (binding)

I will perform the release now. Many thanks to the validators.

Alex

On 20/11/2019 14:47, Alex Herbert wrote:
> We have fixed a few bugs and added one enhancement since Apache 
> Commons Skin 4.1 was released, so I would like to release Apache 
> Commons Skin 4.2.
>
> The Git tag commons-skin-4.2-RC2 commit for this RC is 
> 5b820663afd0535a9a9d089a8a4ecc794873e9d7 which you can browse here:
> https://gitbox.apache.org/repos/asf?p=commons-skin.git;a=commit;h=5b820663afd0535a9a9d089a8a4ecc794873e9d7 
>
> You may checkout this tag using:
>     git clone https://gitbox.apache.org/repos/asf/commons-skin.git 
> --branch commons-skin-4.2-RC2 commons-skin-4.2-RC2
>
>
> Maven artifacts are here:
>
> https://repository.apache.org/content/repositories/orgapachecommons-1479/org/apache/commons/commons-skin/4.2/ 
>
>
>
> Note: There are no binary artifacts for this component.
>
>
> I have tested this with 'mvn clean install' using:
>
> Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 
> 2018-10-24T19:41:47+01:00)
> Maven home: /usr/local/apache-maven-3.6.0
> Java version: 1.8.0_222, vendor: Private Build, runtime: 
> /usr/lib/jvm/java-8-openjdk-amd64/jre
> Default locale: en_GB, platform encoding: UTF-8
> OS name: "linux", version: "4.4.0-166-generic", arch: "amd64", family: 
> "unix"
>
>
> *****************
> NOTE: Please read the 'Validating a release candidate' section below 
> to test the candidate since there are no class files in commons-skin. 
> It contains templates for rendering the site for other commons 
> components.
> *****************
>
>
> Details of changes since 4.1 are in the release notes and reproduced 
> here:
>
> New features:
> o   Optionally render bare <div class="source"><pre> tags using 
> prettyprint.
>     The functionality is enabled using custom properties to be added 
> to the site.xml.
>
> Fixed Bugs:
> o   Fix processing of custom <footer> element from commons-parent 
> site.xml.
> o   Update processing of <head> section to handle injected XHTML.
> o   Fixed background on navigation menu collapsed icon from white to 
> transparent.
>
> Removed:
> o   Remove unused css files. These are not used in the current skin and
>     are from previous skin versions.
>
>
> Note:
>
> No stylesheet changes have been made in this release. It fixes 
> functionality in commons-skin-4.1 that no longer works with the 
> maven-site-plugin specified in the most recent commons-parent.
>
>
> There is no site for this component.
>
>
>
> KEYS:
> https://www.apache.org/dist/commons/KEYS
>
> Please review the release candidate and vote.
> This vote will close no sooner that 72 hours from now.
>
>   [ ] +1 Release these artifacts
>   [ ] +0 OK, but...
>   [ ] -0 OK, but really should fix...
>   [ ] -1 I oppose this release because...
>
> Thank you,
>
> Alex Herbert,
> Release Manager (using key BC87A3FD0A54480F0BADBEBD21939FF0CA2A6567)
>
>
> Validating a release candidate
> ==============================
>
> These guidelines are NOT complete.
>
> Requirements: Git, Java, Maven.
>
> You can validate a release from a release candidate (RC) tag as follows.
>
> 1) Clone and checkout the RC tag
>
> git clone https://gitbox.apache.org/repos/asf/commons-skin.git 
> --branch commons-skin-4.2-RC2 commons-skin-4.2-RC2
> cd commons-skin-4.2-RC2
>
> 2) Check Apache licenses
>
> mvn apache-rat:check
>
> 3) Check binary compatibility
>
> SKIPPED - not applicable
>
> 4) Build and install the package
>
> mvn install
>
> You can record the Maven and Java version produced by -V in your VOTE 
> reply.
> To gather OS information from a command line:
> Windows: ver
> Linux: uname -a
>
> 5) Test the skin:
>
> You can test Apache Commons Skin by adding the following to your 
> site.xml site descriptor for the maven-site-plugin:
>
>   <skin>
>     <groupId>org.apache.commons</groupId>
>     <artifactId>commons-skin</artifactId>
>     <version>4.2</version>
>   </skin>
>
>   <body>
>     <!-- Custom <head> tag with injected XHTML is supported. -->
>     <head>
>       <![CDATA[<script type="text/javascript" id="MathJax-script" 
> async src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-mml-chtml.js">
>       </script>]]>
>     </head>
>     ...
>   </body>
>
>
>
>   <custom>
>     <!-- Escaping with CDATA is not required. -->
>     <footer>
>       <div class="center">
>       Apache Commons, Apache @project.name@, Apache, the Apache 
> feather logo, and the Apache Commons project logos are trademarks of 
> The Apache Software Foundation.
>       All other marks mentioned may be trademarks or registered 
> trademarks of their respective owners.
>       </div>
>     </footer>
>
>     <!-- Custom properties controlling the commons-skin template. -->
>     <commonsSkin>
>       <!--
>         Convert preformatted source section tags to prettyprint:
>           <div class="source"><pre> to <div class="source"><pre 
> class="prettyprint">
>       -->
> <prettyPrintSourcePreTags>false</prettyPrintSourcePreTags>
>
>       <!-- Add the "linenums" class to the prettyprint enabled source 
> tags -->
> <prettyPrintLineNumbersEnabled>false</prettyPrintLineNumbersEnabled>
>     </commonsSkin>
>   </custom>
>
>
> Please check that:
>
> 1. The site looks like it does with commons-skin-4.1. CRITICAL!
>
> 2. The head meta data contains the injected MathJax script (as 
> described above). Try injecting other XHTML to verify this 
> functionality adds the XHTML to the rendered pages.
>
> 3. The footer contains the Apache copyright information (requires the 
> custom <footer> element described above; it should not be escaped with 
> CDATA tags). The duplicate test 'Apache Apache' should not appear when 
> describing the project.
>
> 4. The collapsed menu icons DO NOT have the white background around 
> the small grey arrow.
>
>
> Note: Most commons sites do not require the custom properties to 
> configure prettyprint in their documentation since they are written 
> using the XDOC format.
>
> Any XDOC format can include code snippets using for example:
>
> <source class=<prettyprint">
> String value = "coloured";
> </source>
>
> This functionality was supported by commons-skin-4.1. Check it still 
> works if you use code snippets with prettyprint.
>
> APT format documents can include code snippets using for example:
>
> +--------------------------+
> String value = "coloured";
> +--------------------------+
>
> This can now be rendered using prettyprint with the custom properties 
> in the site.xml.
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org