[VOTE] Release Apache Commons Skin 4.2 based on RC2

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[VOTE] Release Apache Commons Skin 4.2 based on RC2

Alex Herbert
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: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Bruno P. Kinoshita-2
    [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 <[hidden email]> 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: [hidden email]
For additional commands, e-mail: [hidden email]

 
Reply | Threaded
Open this post in threaded view
|

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

Alex Herbert
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 <[hidden email]> 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 <[hidden email]> 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: [hidden email]
> For additional commands, e-mail: [hidden email]
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Gilles Sadowski-2
Hello.

Le sam. 23 nov. 2019 à 09:49, Alex Herbert <[hidden email]> 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: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Alex Herbert

On 25/11/2019 10:02, Gilles Sadowski wrote:

> Hello.
>
> Le sam. 23 nov. 2019 à 09:49, Alex Herbert <[hidden email]> 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: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Gilles Sadowski-2
Le lun. 25 nov. 2019 à 11:17, Alex Herbert <[hidden email]> a écrit :

>
>
> On 25/11/2019 10:02, Gilles Sadowski wrote:
> > Hello.
> >
> > Le sam. 23 nov. 2019 à 09:49, Alex Herbert <[hidden email]> 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: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Alex Herbert

On 25/11/2019 10:39, Gilles Sadowski wrote:

> Le lun. 25 nov. 2019 à 11:17, Alex Herbert <[hidden email]> a écrit :
>>
>> On 25/11/2019 10:02, Gilles Sadowski wrote:
>>> Hello.
>>>
>>> Le sam. 23 nov. 2019 à 09:49, Alex Herbert <[hidden email]> 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: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

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

Alex Herbert
In reply to this post by Alex Herbert
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: [hidden email]
For additional commands, e-mail: [hidden email]