Re: svn commit: r1800614 - in /tomcat/trunk: build.xml java/javax/xml/ res/checkstyle/javax-import-control.xml webapps/docs/changelog.xml

2017-07-03 Thread Rémy Maucherat 
On Mon, Jul 3, 2017 at 9:45 AM, Mark Thomas  wrote:

> On 03/07/17 15:26, Rémy Maucherat wrote:
> > On Mon, Jul 3, 2017 at 4:48 AM,  wrote:
> >
> >> Author: markt
> >> Date: Mon Jul  3 09:48:01 2017
> >> New Revision: 1800614
> >>
> >> URL: http://svn.apache.org/viewvc?rev=1800614=rev
> >> Log:
> >> Restore the local definition of the web service annotations since the
> JRE
> >> provided versions are deprecated and Java 9 does not provide them by
> >> default.
> >>
> >
> > Ok, so I suppose you like this solution better than conditionally adding
> > the command line option then ?
>
> Yes.
>
> Given that:
> - they were added in Java 8;
> - Java 9 disables them by default; and
> - they will be removed from some (TBD) future Java version.
>
> We are going to have to add them back at some point and doing it now
> looked to be the simple, future proof solution.
>
> Hmm, ok, that's better.

Rémy

Re: svn commit: r1800614 - in /tomcat/trunk: build.xml java/javax/xml/ res/checkstyle/javax-import-control.xml webapps/docs/changelog.xml

2017-07-03 Thread Mark Thomas 
On 03/07/17 15:26, Rémy Maucherat wrote:
> On Mon, Jul 3, 2017 at 4:48 AM,  wrote:
> 
>> Author: markt
>> Date: Mon Jul  3 09:48:01 2017
>> New Revision: 1800614
>>
>> URL: http://svn.apache.org/viewvc?rev=1800614=rev
>> Log:
>> Restore the local definition of the web service annotations since the JRE
>> provided versions are deprecated and Java 9 does not provide them by
>> default.
>>
> 
> Ok, so I suppose you like this solution better than conditionally adding
> the command line option then ?

Yes.

Given that:
- they were added in Java 8;
- Java 9 disables them by default; and
- they will be removed from some (TBD) future Java version.

We are going to have to add them back at some point and doing it now
looked to be the simple, future proof solution.

Mark


> 
> Rémy
> 
>>
>> Added:
>> tomcat/trunk/java/javax/xml/
>>   - copied from r1783190, tomcat/trunk/java/javax/xml/
>> Modified:
>> tomcat/trunk/build.xml
>> tomcat/trunk/res/checkstyle/javax-import-control.xml
>> tomcat/trunk/webapps/docs/changelog.xml
>>
>> Modified: tomcat/trunk/build.xml
>> URL: http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=
>> 1800614=1800613=1800614=diff
>> 
>> ==
>> --- tomcat/trunk/build.xml (original)
>> +++ tomcat/trunk/build.xml Mon Jul  3 09:48:01 2017
>> @@ -293,6 +293,7 @@
>>  
>>  
>>  
>> +
>>
>>
>>
>>
>> Modified: tomcat/trunk/res/checkstyle/javax-import-control.xml
>> URL: http://svn.apache.org/viewvc/tomcat/trunk/res/checkstyle/
>> javax-import-control.xml?rev=1800614=1800613=1800614=diff
>> 
>> ==
>> --- tomcat/trunk/res/checkstyle/javax-import-control.xml (original)
>> +++ tomcat/trunk/res/checkstyle/javax-import-control.xml Mon Jul  3
>> 09:48:01 2017
>> @@ -68,4 +68,7 @@
>>
>>  
>>
>> +  
>> +
>> +  
>>  
>> \ No newline at end of file
>>
>> Modified: tomcat/trunk/webapps/docs/changelog.xml
>> URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/
>> changelog.xml?rev=1800614=1800613=1800614=diff
>> 
>> ==
>> --- tomcat/trunk/webapps/docs/changelog.xml (original)
>> +++ tomcat/trunk/webapps/docs/changelog.xml Mon Jul  3 09:48:01 2017
>> @@ -143,6 +143,15 @@
>>
>>  
>>
>> +  
>> +
>> +  
>> +Restore the local definition of the web service annotations since
>> the
>> +JRE provided versions are deprecated and Java 9 does not provide
>> them by
>> +default. (markt)
>> +  
>> +
>> +  
>>  
>>  
>>
>>
>>
>>
>> -
>> 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

Re: svn commit: r1800614 - in /tomcat/trunk: build.xml java/javax/xml/ res/checkstyle/javax-import-control.xml webapps/docs/changelog.xml

2017-07-03 Thread Rémy Maucherat 
On Mon, Jul 3, 2017 at 4:48 AM,  wrote:

> Author: markt
> Date: Mon Jul  3 09:48:01 2017
> New Revision: 1800614
>
> URL: http://svn.apache.org/viewvc?rev=1800614=rev
> Log:
> Restore the local definition of the web service annotations since the JRE
> provided versions are deprecated and Java 9 does not provide them by
> default.
>

Ok, so I suppose you like this solution better than conditionally adding
the command line option then ?

Rémy

>
> Added:
> tomcat/trunk/java/javax/xml/
>   - copied from r1783190, tomcat/trunk/java/javax/xml/
> Modified:
> tomcat/trunk/build.xml
> tomcat/trunk/res/checkstyle/javax-import-control.xml
> tomcat/trunk/webapps/docs/changelog.xml
>
> Modified: tomcat/trunk/build.xml
> URL: http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=
> 1800614=1800613=1800614=diff
> 
> ==
> --- tomcat/trunk/build.xml (original)
> +++ tomcat/trunk/build.xml Mon Jul  3 09:48:01 2017
> @@ -293,6 +293,7 @@
>  
>  
>  
> +
>
>
>
>
> Modified: tomcat/trunk/res/checkstyle/javax-import-control.xml
> URL: http://svn.apache.org/viewvc/tomcat/trunk/res/checkstyle/
> javax-import-control.xml?rev=1800614=1800613=1800614=diff
> 
> ==
> --- tomcat/trunk/res/checkstyle/javax-import-control.xml (original)
> +++ tomcat/trunk/res/checkstyle/javax-import-control.xml Mon Jul  3
> 09:48:01 2017
> @@ -68,4 +68,7 @@
>
>  
>
> +  
> +
> +  
>  
> \ No newline at end of file
>
> Modified: tomcat/trunk/webapps/docs/changelog.xml
> URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/
> changelog.xml?rev=1800614=1800613=1800614=diff
> 
> ==
> --- tomcat/trunk/webapps/docs/changelog.xml (original)
> +++ tomcat/trunk/webapps/docs/changelog.xml Mon Jul  3 09:48:01 2017
> @@ -143,6 +143,15 @@
>
>  
>
> +  
> +
> +  
> +Restore the local definition of the web service annotations since
> the
> +JRE provided versions are deprecated and Java 9 does not provide
> them by
> +default. (markt)
> +  
> +
> +  
>  
>  
>
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>

svn commit: r1800614 - in /tomcat/trunk: build.xml java/javax/xml/ res/checkstyle/javax-import-control.xml webapps/docs/changelog.xml

2017-07-03 Thread markt 
Author: markt
Date: Mon Jul  3 09:48:01 2017
New Revision: 1800614

URL: http://svn.apache.org/viewvc?rev=1800614=rev
Log:
Restore the local definition of the web service annotations since the JRE 
provided versions are deprecated and Java 9 does not provide them by default.

Added:
tomcat/trunk/java/javax/xml/
  - copied from r1783190, tomcat/trunk/java/javax/xml/
Modified:
tomcat/trunk/build.xml
tomcat/trunk/res/checkstyle/javax-import-control.xml
tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=1800614=1800613=1800614=diff
==
--- tomcat/trunk/build.xml (original)
+++ tomcat/trunk/build.xml Mon Jul  3 09:48:01 2017
@@ -293,6 +293,7 @@
 
 
 
+
   
 
   

Modified: tomcat/trunk/res/checkstyle/javax-import-control.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/res/checkstyle/javax-import-control.xml?rev=1800614=1800613=1800614=diff
==
--- tomcat/trunk/res/checkstyle/javax-import-control.xml (original)
+++ tomcat/trunk/res/checkstyle/javax-import-control.xml Mon Jul  3 09:48:01 
2017
@@ -68,4 +68,7 @@
   
 
   
+  
+
+  
 
\ No newline at end of file

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1800614=1800613=1800614=diff
==
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Mon Jul  3 09:48:01 2017
@@ -143,6 +143,15 @@
   
 
   
+  
+
+  
+Restore the local definition of the web service annotations since the
+JRE provided versions are deprecated and Java 9 does not provide them 
by
+default. (markt)
+  
+
+  
 
 
   



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org