svn commit: r1834283 - in /tomcat/tc8.5.x/trunk: build.properties.default res/maven/mvn.properties.default webapps/docs/changelog.xml

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 07:30:01 2018
New Revision: 1834283

URL: http://svn.apache.org/viewvc?rev=1834283&view=rev
Log:
Increment version number for next dev cycle

Modified:
tomcat/tc8.5.x/trunk/build.properties.default
tomcat/tc8.5.x/trunk/res/maven/mvn.properties.default
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc8.5.x/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/build.properties.default?rev=1834283&r1=1834282&r2=1834283&view=diff
==
--- tomcat/tc8.5.x/trunk/build.properties.default (original)
+++ tomcat/tc8.5.x/trunk/build.properties.default Mon Jun 25 07:30:01 2018
@@ -25,7 +25,7 @@
 # - Version Control Flags -
 version.major=8
 version.minor=5
-version.build=32
+version.build=33
 version.patch=0
 version.suffix=-dev
 

Modified: tomcat/tc8.5.x/trunk/res/maven/mvn.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/res/maven/mvn.properties.default?rev=1834283&r1=1834282&r2=1834283&view=diff
==
--- tomcat/tc8.5.x/trunk/res/maven/mvn.properties.default (original)
+++ tomcat/tc8.5.x/trunk/res/maven/mvn.properties.default Mon Jun 25 07:30:01 
2018
@@ -39,7 +39,7 @@ maven.asf.release.repo.url=https://repos
 maven.asf.release.repo.repositoryId=apache.releases.https
 
 # Release version info
-maven.asf.release.deploy.version=8.5.32
+maven.asf.release.deploy.version=8.5.33
 
 #Where do we load the libraries from
 tomcat.lib.path=../../output/build/lib

Modified: tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml?rev=1834283&r1=1834282&r2=1834283&view=diff
==
--- tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml Mon Jun 25 07:30:01 2018
@@ -54,7 +54,7 @@
 
   
 
-
+
   
 
   



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



svn commit: r1834293 - in /tomcat/trunk/test/javax/websocket: TestContainerProviderPerformance.java TesterContainerProviderPerformance.java

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 10:09:29 2018
New Revision: 1834293

URL: http://svn.apache.org/viewvc?rev=1834293&view=rev
Log:
Rename to exclude from tests as this is a support class not a test case (it has 
not assertions).
It exists to explore performance differences if 
ContainerProvider.getWebSocketContainer() is refactored. 

Added:
tomcat/trunk/test/javax/websocket/TesterContainerProviderPerformance.java
  - copied, changed from r1834292, 
tomcat/trunk/test/javax/websocket/TestContainerProviderPerformance.java
Removed:
tomcat/trunk/test/javax/websocket/TestContainerProviderPerformance.java

Copied: 
tomcat/trunk/test/javax/websocket/TesterContainerProviderPerformance.java (from 
r1834292, 
tomcat/trunk/test/javax/websocket/TestContainerProviderPerformance.java)
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/javax/websocket/TesterContainerProviderPerformance.java?p2=tomcat/trunk/test/javax/websocket/TesterContainerProviderPerformance.java&p1=tomcat/trunk/test/javax/websocket/TestContainerProviderPerformance.java&r1=1834292&r2=1834293&rev=1834293&view=diff
==
--- tomcat/trunk/test/javax/websocket/TestContainerProviderPerformance.java 
(original)
+++ tomcat/trunk/test/javax/websocket/TesterContainerProviderPerformance.java 
Mon Jun 25 10:09:29 2018
@@ -23,7 +23,7 @@ import org.junit.Test;
 
 import org.apache.tomcat.unittest.TesterThreadedPerformance;
 
-public class TestContainerProviderPerformance {
+public class TesterContainerProviderPerformance {
 
 @Test
 public void testGetWebSocketContainer() throws Exception {



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



svn commit: r1834295 - /tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 10:24:31 2018
New Revision: 1834295

URL: http://svn.apache.org/viewvc?rev=1834295&view=rev
Log:
Improve debug messages when test fails

Modified:
tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java

Modified: tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java?rev=1834295&r1=1834294&r2=1834295&view=diff
==
--- tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java (original)
+++ tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java Mon Jun 25 
10:24:31 2018
@@ -108,9 +108,9 @@ public class TestSocketServer extends Ab
 while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
 }
 long wait = System.nanoTime() - start;
-Assert.assertFalse("Socket.timeoutSet failed (<1s) [" + wait + "]",
+Assert.assertFalse("Socket.timeoutSet failed (<1s) [" + wait + "] +-[" 
+ ERROR_MARGIN + "]",
 wait < 10 - ERROR_MARGIN);
-Assert.assertFalse("Socket.timeoutSet failed (>2s) [" + wait + "]",
+Assert.assertFalse("Socket.timeoutSet failed (>2s) [" + wait + "] +-[" 
+ ERROR_MARGIN + "]",
 wait > 20 + ERROR_MARGIN);
 
 client.countDown();
@@ -137,8 +137,8 @@ public class TestSocketServer extends Ab
 while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
 }
 long wait = System.nanoTime() - start;
-Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>2ms)",
-wait > 200 + ERROR_MARGIN);
+Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>2ms) [" + wait +
+"] +-[" + ERROR_MARGIN + "]", wait > 200 + ERROR_MARGIN);
 
 client.countDown();
 client.join();
@@ -162,8 +162,8 @@ public class TestSocketServer extends Ab
 while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
 }
 long wait = System.nanoTime() - start;
-Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>1ms)",
-wait > 100 + ERROR_MARGIN);
+Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>1ms) [" + wait +
+"] +-[" + ERROR_MARGIN + "]", wait > 100 + ERROR_MARGIN);
 
 /* Configure for blocking */
 Socket.optSet(clientSocket, Socket.APR_SO_NONBLOCK, 0);
@@ -172,8 +172,8 @@ public class TestSocketServer extends Ab
 while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
 }
 wait = System.nanoTime() - start;
-Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK false 
failed",
-wait < 100 - ERROR_MARGIN);
+Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK false 
failed (<1ms) [" +
+wait + "] +-[" + ERROR_MARGIN + "]", wait < 100 - 
ERROR_MARGIN);
 
 client.countDown();
 client.join();
@@ -195,8 +195,8 @@ public class TestSocketServer extends Ab
 }
 long wait = System.nanoTime() - start;
 Assert.assertTrue("Timeout failed", ok);
-Assert.assertFalse("non_blocking accept Socket.APR_SO_NONBLOCK failed 
([" + wait + "]>1ms)",
-wait > 100 + ERROR_MARGIN);
+Assert.assertFalse("non_blocking accept Socket.APR_SO_NONBLOCK failed 
(>1ms) [" + wait +
+"] +-[" + ERROR_MARGIN + "]", wait > 100 + ERROR_MARGIN);
 }
 
 



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



Re: [VOTE] Release Apache Tomcat 9.0.10

2018-06-25 Thread Mark Thomas
On 22/06/18 10:47, Konstantin Kolinko wrote:



> 2. Unit tests (Java 8u172 32-bit x All connectors): OK, but two issues
> 1). full run running serially was 5 hours (297 minutes),
> and 1 hour of that was wasted by a single test, running for 20 minutes
> * 3 connectors:
> javax.websocket.TestContainerProviderPerformance

I've renamed that test Tester... as it does not contain any assertions.



> 2) org.apache.tomcat.jni.TestSocketServer fails for all connectors.
>[concat] Testsuites with failed tests:
>[concat] TEST-org.apache.tomcat.jni.TestSocketServer.APR.txt
>[concat] TEST-org.apache.tomcat.jni.TestSocketServer.NIO.txt
>[concat] TEST-org.apache.tomcat.jni.TestSocketServer.NIO2.txt
> 
> It is not a showstopper. It is JNI internals test, and failures are
> apparently caused by timer accuracy. Some test have already been
> corrected to be more tolerant, but apparently not all of them.

I've improved the debug logging so we can see how far out the error
margin is.

Thanks for the feedback.

Mark

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



Re: [VOTE] Release Apache Tomcat 9.0.10

2018-06-25 Thread Mark Thomas
On 20/06/18 20:45, Mark Thomas wrote:
> The proposed 9.0.10 release is:
> [ ] Broken - do not release
> [X] Stable - go ahead and release as 9.0.10

Unit tests pass for NIO, NIO2 and APR/native on Windows, Linux and OSX.

Mark

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



[VOTE][RESULT] Release Apache Tomcat 9.0.10

2018-06-25 Thread Mark Thomas
The following votes were cast:
binding:
+1: remm, huxing, kkolinko, markt

non-binding:
+1: rmannibucau

The vote therefore passes. Thanks to everyone who contributed to this
release.

Mark


On 20/06/18 20:45, Mark Thomas wrote:
> The proposed Apache Tomcat 9.0.10 release is now available for voting.
> 
> Note:
> 9.0.9 was tagged and uploaded when BZ 62476 was created and the decision
> was taken to re-tag to pick up that fix.
> 
> The major changes compared to the 9.0.8 release are:
> 
> - Add the RemoteCIDRFilter and RemoteCIDRValve that can be used to
>   allow/deny requests based on IPv4 and/or IPv6 client address where the
>   IP ranges are defined using CIDR notation.
>   Based on a patch by Francis Galiegue.
> 
> - Use NIO2 API for websockets writes.
> 
> - Update the packaged version of the Tomcat Native Library to 1.2.17 to
>   pick up the latest Windows binaries built with APR 1.6.3 and OpenSSL
>   1.0.2o.
> 
> - Correct a regression in the Host validation by removing the
>   requirement that the final component of a FQDN must be alphabetic.
> 
> Along with lots of other bug fixes and improvements.
> 
> For full details, see the changelog:
> http://svn.apache.org/repos/asf/tomcat/trunk/webapps/docs/changelog.xml
> 
> It can be obtained from:
> https://dist.apache.org/repos/dist/dev/tomcat/tomcat-9/v9.0.10/
> The Maven staging repo is:
> https://repository.apache.org/content/repositories/orgapachetomcat-1186/
> The svn tag is:
> http://svn.apache.org/repos/asf/tomcat/tags/TOMCAT_9_0_10/
> 
> The proposed 9.0.10 release is:
> [ ] Broken - do not release
> [ ] Stable - go ahead and release as 9.0.10
> 
> -
> 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: [VOTE] Release Apache Tomcat 8.5.32

2018-06-25 Thread Mark Thomas
On 20/06/18 21:27, Mark Thomas wrote:

> The proposed 8.5.32 release is:
> [ ] Broken - do not release
> [X] Stable - go ahead and release as 8.5.32

Unit tests pass for NIO, NIO2 and APR/native on Windows, Linux and OSX.

Mark

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



[VOTE][RESULT] Release Apache Tomcat 8.5.32

2018-06-25 Thread Mark Thomas
The following votes were cast:

Binding:
+1: remm, kkolinko, huxing, markt

The vote therefore passes.

Thanks to everyone who contributed to this release.

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



svn commit: r1834305 - /tomcat/trunk/build.properties.default

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 11:20:46 2018
New Revision: 1834305

URL: http://svn.apache.org/viewvc?rev=1834305&view=rev
Log:
Remove unused property

Modified:
tomcat/trunk/build.properties.default

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1834305&r1=1834304&r2=1834305&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Mon Jun 25 11:20:46 2018
@@ -201,7 +201,6 @@ checkstyle.jar=${checkstyle.home}/checks
 cobertura.version=2.1.1
 cobertura.home=${base.path}/cobertura-${cobertura.version}
 cobertura.jar=${cobertura.home}/cobertura-${cobertura.version}.jar
-cobertura.lib=${cobertura.home}/lib
 cobertura.loc=${base-sf.loc}/cobertura/cobertura-2.1.1-bin.tar.gz
 
 # - Findbugs -



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



svn commit: r1834306 - /tomcat/trunk/build.properties.default

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 11:24:08 2018
New Revision: 1834306

URL: http://svn.apache.org/viewvc?rev=1834306&view=rev
Log:
Consistent ordering:
version, home, jar, location

Modified:
tomcat/trunk/build.properties.default

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1834306&r1=1834305&r2=1834306&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Mon Jun 25 11:24:08 2018
@@ -106,14 +106,14 @@ base-maven.loc=https://repo.maven.apache
 # - Webservices - JAX RPC -
 jaxrpc-lib.version=1.1-rc4
 jaxrpc-lib.home=${base.path}/jaxrpc-${jaxrpc-lib.version}
-jaxrpc-lib.loc=${base-maven.loc}/geronimo-spec/geronimo-spec-jaxrpc/${jaxrpc-lib.version}/geronimo-spec-jaxrpc-${jaxrpc-lib.version}.jar
 
jaxrpc-lib.jar=${jaxrpc-lib.home}/geronimo-spec-jaxrpc-${jaxrpc-lib.version}.jar
+jaxrpc-lib.loc=${base-maven.loc}/geronimo-spec/geronimo-spec-jaxrpc/${jaxrpc-lib.version}/geronimo-spec-jaxrpc-${jaxrpc-lib.version}.jar
 
 # - Webservices - WSDL4J -
 wsdl4j-lib.version=1.6.2
 wsdl4j-lib.home=${base.path}/wsdl4j-${wsdl4j-lib.version}
-wsdl4j-lib.loc=${base-maven.loc}/wsdl4j/wsdl4j/${wsdl4j-lib.version}/wsdl4j-${wsdl4j-lib.version}.jar
 wsdl4j-lib.jar=${wsdl4j-lib.home}/wsdl4j-${wsdl4j-lib.version}.jar
+wsdl4j-lib.loc=${base-maven.loc}/wsdl4j/wsdl4j/${wsdl4j-lib.version}/wsdl4j-${wsdl4j-lib.version}.jar
 
 # - Eclipse JDT, version 4.7 or later -#
 # See https://wiki.apache.org/tomcat/JDTCoreBatchCompiler before updating
@@ -176,26 +176,26 @@ hamcrest.loc=${base-maven.loc}/org/hamcr
 # - EasyMock, version 3.2 or later -
 easymock.version=3.2
 easymock.home=${base.path}/easymock-${easymock.version}
-easymock.loc=${base-sf.loc}/easymock/easymock-${easymock.version}.zip
 easymock.jar=${easymock.home}/easymock-${easymock.version}.jar
+easymock.loc=${base-sf.loc}/easymock/easymock-${easymock.version}.zip
 
 # - cglib, used by EasyMock, version 2.2 or later -
 cglib.version=2.2.3
 cglib.home=${base.path}/cglib-${cglib.version}
-cglib.loc=${base-sf.loc}/cglib/cglib-nodep-${cglib.version}.jar
 cglib.jar=${cglib.home}/cglib-nodep-${cglib.version}.jar
+cglib.loc=${base-sf.loc}/cglib/cglib-nodep-${cglib.version}.jar
 
 # - objenesis, used by EasyMock, version 1.2 or later -
 objenesis.version=1.2
 objenesis.home=${base.path}/objenesis-${objenesis.version}
-objenesis.loc=https://bintray.com/easymock/distributions/download_file?file_path=objenesis-${objenesis.version}-bin.zip
 objenesis.jar=${objenesis.home}/objenesis-${objenesis.version}.jar
+objenesis.loc=https://bintray.com/easymock/distributions/download_file?file_path=objenesis-${objenesis.version}-bin.zip
 
 # - Checkstyle, version 6.16 or later -
 checkstyle.version=6.17
 checkstyle.home=${base.path}/checkstyle-${checkstyle.version}
-checkstyle.loc=${base-sf.loc}/checkstyle/checkstyle/${checkstyle.version}/checkstyle-${checkstyle.version}-all.jar
 checkstyle.jar=${checkstyle.home}/checkstyle-${checkstyle.version}-all.jar
+checkstyle.loc=${base-sf.loc}/checkstyle/checkstyle/${checkstyle.version}/checkstyle-${checkstyle.version}-all.jar
 
 # - Cobertura code coverage tool -
 cobertura.version=2.1.1
@@ -215,16 +215,16 @@ findbugs.loc=${base-sf.loc}/project/find
 # - CDDL Licensed   -
 saaj-api.version=1.3.5
 saaj-api.home=${base.path}/saaj-api-${saaj-api.version}
-saaj-api.loc=${base-maven.loc}/javax/xml/soap/saaj-api/${saaj-api.version}/saaj-api-${saaj-api.version}.jar
 saaj-api.jar=${saaj-api.home}/saaj-api-${saaj-api.version}.jar
+saaj-api.loc=${base-maven.loc}/javax/xml/soap/saaj-api/${saaj-api.version}/saaj-api-${saaj-api.version}.jar
 
 # - bnd & bndlib, version 4.0.0 or later  -
 # - provides OSGI metadata for JARs   -
 bnd.version=4.0.0
 bnd.home=${base.path}/bnd-${bnd.version}
-bnd.loc=${base-maven.loc}/biz/aQute/bnd/biz.aQute.bnd/${bnd.version}/biz.aQute.bnd-${bnd.version}.jar
 bnd.jar=${bnd.home}/biz.aQute.bnd-${bnd.version}.jar
+bnd.loc=${base-maven.loc}/biz/aQute/bnd/biz.aQute.bnd/${bnd.version}/biz.aQute.bnd-${bnd.version}.jar
 
 bndlib.home=${base.path}/bndlib-${bnd.version}
-bndlib.loc=${base-maven.loc}/biz/aQute/bnd/biz.aQute.bndlib/${bnd.version}/biz.aQute.bndlib-${bnd.version}.jar
 bndlib.jar=${bndlib.home}/biz.aQute.bndlib-${bnd.version}.jar
+bndlib.loc=${base-maven.loc}/biz/aQute/bnd/biz.aQute.bndlib/${bnd.version}/biz.aQute.bndlib-${bnd.version}.jar



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



svn commit: r1834307 - /tomcat/trunk/build.properties.default

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 11:24:47 2018
New Revision: 1834307

URL: http://svn.apache.org/viewvc?rev=1834307&view=rev
Log:
Simplify / make consistent

Modified:
tomcat/trunk/build.properties.default

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1834307&r1=1834306&r2=1834307&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Mon Jun 25 11:24:47 2018
@@ -206,8 +206,7 @@ cobertura.loc=${base-sf.loc}/cobertura/c
 # - Findbugs -
 findbugs.version=3.0.1
 findbugs.home=${base.path}/findbugs-${findbugs.version}
-findbugs.lib=${findbugs.home}/lib
-findbugs.jar=${findbugs.lib}/findbugs-ant.jar
+findbugs.jar=${findbugs.home}/lib/findbugs-ant.jar
 
findbugs.loc=${base-sf.loc}/project/findbugs/findbugs/${findbugs.version}/findbugs-${findbugs.version}.tar.gz
 
 # - SAAJ API, used by Code Signing for releases -



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



svn commit: r1834310 - /tomcat/trunk/build.xml

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 12:05:08 2018
New Revision: 1834310

URL: http://svn.apache.org/viewvc?rev=1834310&view=rev
Log:
Fix indent

Modified:
tomcat/trunk/build.xml

Modified: tomcat/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=1834310&r1=1834309&r2=1834310&view=diff
==
--- tomcat/trunk/build.xml (original)
+++ tomcat/trunk/build.xml Mon Jun 25 12:05:08 2018
@@ -1557,12 +1557,12 @@
   
 
 
-
-  
-  
-
-  
-
+  
+
+
+  
+
+  
 
   
 



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



[Bug 62488] New: Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

Bug ID: 62488
   Summary: Obtain dependencies from Maven Central where possible
   Product: Tomcat 9
   Version: unspecified
  Hardware: PC
OS: Linux
Status: NEW
  Severity: normal
  Priority: P2
 Component: Packaging
  Assignee: dev@tomcat.apache.org
  Reporter: ma...@apache.org
  Target Milestone: -

Talking with michaelo at the ApacheCon EU Roadshow, he mentioned that the way
Tomcat pulls dependencies from multiple locations makes it harder to build from
source when located behind a strict firewall.

He suggested obtaining references from Maven Central where possible as most
orgs have a mechanism for proxying / caching Maven Central.

Not all dependencies are available from Maven Central (e.g. NSIS is not) but
any progress we can make on this would be helpful.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

Mark Thomas  changed:

   What|Removed |Added

   Severity|normal  |enhancement

--- Comment #1 from Mark Thomas  ---
Whoops. Should have created this as an enhancement request.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #2 from Remy Maucherat  ---
So you simply base all downloads (except one) on
https://repo.maven.apache.org/maven2/ ?
Why not.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #3 from Emmanuel Bourg  ---
What about using the Maven Ant Tasks?

https://maven.apache.org/resolver-ant-tasks/

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #4 from Mark Thomas  ---
(In reply to Emmanuel Bourg from comment #3)
> What about using the Maven Ant Tasks?

I'm on the fence.

For: It would handle dependencies of dependencies

Against: It is another component that would have to be downloaded (manually?)
first.


At the moment, we obtain a number of dependencies from Maven Central. My plan
was to see how many of the others could be easily switched and then see where
we were.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1834354 - in /tomcat/trunk: build.properties.default build.xml

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 18:06:22 2018
New Revision: 1834354

URL: http://svn.apache.org/viewvc?rev=1834354&view=rev
Log:
Switch Objensis source to Maven Central

Modified:
tomcat/trunk/build.properties.default
tomcat/trunk/build.xml

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1834354&r1=1834353&r2=1834354&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Mon Jun 25 18:06:22 2018
@@ -189,7 +189,7 @@ cglib.loc=${base-sf.loc}/cglib/cglib-nod
 objenesis.version=1.2
 objenesis.home=${base.path}/objenesis-${objenesis.version}
 objenesis.jar=${objenesis.home}/objenesis-${objenesis.version}.jar
-objenesis.loc=https://bintray.com/easymock/distributions/download_file?file_path=objenesis-${objenesis.version}-bin.zip
+objenesis.loc=${base-maven.loc}/org/objenesis/objenesis/${objenesis.version}/objenesis-${objenesis.version}.jar
 
 # - Checkstyle, version 6.16 or later -
 checkstyle.version=6.17

Modified: tomcat/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=1834354&r1=1834353&r2=1834354&view=diff
==
--- tomcat/trunk/build.xml (original)
+++ tomcat/trunk/build.xml Mon Jun 25 18:06:22 2018
@@ -2706,10 +2706,10 @@ skip.installer property in build.propert
   
 
 
-
+
   
   
-  
+  
 
 
   



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



svn commit: r1834356 - in /tomcat/trunk: build.properties.default build.xml res/ide-support/eclipse/eclipse.classpath

2018-06-25 Thread markt
Author: markt
Date: Mon Jun 25 18:19:55 2018
New Revision: 1834356

URL: http://svn.apache.org/viewvc?rev=1834356&view=rev
Log:
Switch EasyMock and cglib source to Maven Central
Note that EasyMock 2.2.3 wasn't available from Maven Central so I went back a 
version to 2.2.2. I confirmed that the tests pass.

Modified:
tomcat/trunk/build.properties.default
tomcat/trunk/build.xml
tomcat/trunk/res/ide-support/eclipse/eclipse.classpath

Modified: tomcat/trunk/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.properties.default?rev=1834356&r1=1834355&r2=1834356&view=diff
==
--- tomcat/trunk/build.properties.default (original)
+++ tomcat/trunk/build.properties.default Mon Jun 25 18:19:55 2018
@@ -177,13 +177,13 @@ hamcrest.loc=${base-maven.loc}/org/hamcr
 easymock.version=3.2
 easymock.home=${base.path}/easymock-${easymock.version}
 easymock.jar=${easymock.home}/easymock-${easymock.version}.jar
-easymock.loc=${base-sf.loc}/easymock/easymock-${easymock.version}.zip
+easymock.loc=${base-maven.loc}/org/easymock/easymock/${easymock.version}/easymock-${easymock.version}.jar
 
 # - cglib, used by EasyMock, version 2.2 or later -
-cglib.version=2.2.3
+cglib.version=2.2.2
 cglib.home=${base.path}/cglib-${cglib.version}
 cglib.jar=${cglib.home}/cglib-nodep-${cglib.version}.jar
-cglib.loc=${base-sf.loc}/cglib/cglib-nodep-${cglib.version}.jar
+cglib.loc=${base-maven.loc}/cglib/cglib-nodep/${cglib.version}/cglib-nodep-${cglib.version}.jar
 
 # - objenesis, used by EasyMock, version 1.2 or later -
 objenesis.version=1.2

Modified: tomcat/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=1834356&r1=1834355&r2=1834356&view=diff
==
--- tomcat/trunk/build.xml (original)
+++ tomcat/trunk/build.xml Mon Jun 25 18:19:55 2018
@@ -2694,10 +2694,10 @@ skip.installer property in build.propert
   
 
 
-
+
   
   
-  
+  
 
 
 

Modified: tomcat/trunk/res/ide-support/eclipse/eclipse.classpath
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/res/ide-support/eclipse/eclipse.classpath?rev=1834356&r1=1834355&r2=1834356&view=diff
==
--- tomcat/trunk/res/ide-support/eclipse/eclipse.classpath (original)
+++ tomcat/trunk/res/ide-support/eclipse/eclipse.classpath Mon Jun 25 18:19:55 
2018
@@ -27,7 +27,7 @@
 
 
 
-
+
 
 
 



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



buildbot failure in on tomcat-trunk

2018-06-25 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-trunk while building 
. Full details are available at:
https://ci.apache.org/builders/tomcat-trunk/builds/3397

Buildbot URL: https://ci.apache.org/

Buildslave for this Build: silvanus_ubuntu

Build Reason: The AnyBranchScheduler scheduler named 'on-tomcat-commit' 
triggered this build
Build Source Stamp: [branch tomcat/trunk] 1834356
Blamelist: markt

BUILD FAILED: failed compile_1

Sincerely,
 -The Buildbot




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



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #5 from Mark Thomas  ---
I've done the simple ones (there were three of them). There are now seven
dependencies not obtained from Maven Central.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #6 from Michael Osipov <1983-01...@gmx.net> ---
Mark, thanks for raising!

Let me add some stuff here what I'd like to see/expect: Easily swappable URL of
Maven Central against an internal Nexus instance, providing credentials for it.

At best, we'd use the Resolver Ant Tasks which I can finish and publish to
Central. It requires an Über JAR (either lightweight/URLConnection or with
HttpComponents Client) which I wouldn't commit along with the code, but would
require it to be in Ant's classpath. The best about this, you can define
classpath entries for test and compilation and the tasks do the rest for you.
People could leverage their settings.xml. Using Ant's get task if fine for the
first shot.

This is what I do with those tasks at work to customize the vanilla Tomcat
tarball from Central with some internal and external libraries.

I'd be happy to test a patch.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #7 from Michael Osipov <1983-01...@gmx.net> ---
(In reply to Emmanuel Bourg from comment #3)
> What about using the Maven Ant Tasks?
> 
> https://maven.apache.org/resolver-ant-tasks/

Please don't call them Maven Ant Tasks, those have been deprecated by me long
time ago. The Resolver Ant Tasks have nothing to do with them.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 62488] Obtain dependencies from Maven Central where possible

2018-06-25 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=62488

--- Comment #8 from Emmanuel Bourg  ---
(In reply to Michael Osipov from comment #7)
> Please don't call them Maven Ant Tasks, those have been deprecated by me
> long time ago. The Resolver Ant Tasks have nothing to do with them.

Well, the full name is "Maven Artifact Resolver Ant Tasks" but that's rather
long and boring ;)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: JDK 11 EA 18

2018-06-25 Thread Christopher Schultz
William,

On 6/20/18 9:21 PM, William L. Thomson Jr. wrote:
> Turns out my issues were due to resources ending up in a sub-directory,
> prefixed with java/org, like find out in sources. One aspect I would
> like to see done if possible. Tomcat mixes a lot of stuff into the same
> directories. Its layout and what not started before the normal java
> conventions now such as;
> 
> src/main/java
> src/main/resources
> 
> It would be great if tomcat could move towards such structure. My
> problem was due to grabbing resources that are mixed in with sources.
> Then failing to strip java/ from the path as I copied over the files
> into the jar. Various properties and xml files. I understand some
> properties files need be in the same folders but other stuff like xml.
> But many of those can be merged at build time.
> 
> Packaging hundreds of Java applications. Very few mix in resources with
> source files. Thus my logic for handling such cases was off. Self
> inflicted problem, but would help if project layout was better.
> 
> Utopian would be moving various pieces into sub-directories for the
> various jars. I had to be creative to get my jars to be the same as
> those from upstream. Due to sources all being mixed together. Beyond
> mixing resources and java sources.

I'm confused about what you're asking, here. Do you want Tomcat to
change its own directory layout? Why?

-chris



signature.asc
Description: OpenPGP digital signature


Re: JDK 11 EA 18

2018-06-25 Thread William L. Thomson Jr.
On Mon, 25 Jun 2018 19:40:27 -0400
Christopher Schultz  wrote:

> I'm confused about what you're asking, here. Do you want Tomcat to
> change its own directory layout? 

Ideally yes, like a directory per jar, with 

src/java/main
src/java/resources

Jetty is good example or Netbeans
https://github.com/eclipse/jetty.project
https://github.com/apache/incubator-netbeans

> Why?

Because I am crazy :)

It makes packaging it easier IMHO. Each directory contains what ends up
in a given jar. Right now they are all mixed together in java,
including resources. Fine if you use the build system. But then its all
or nothing. Cannot really build a jar, or just those needed/wanted.

Tomcat on Gentoo used to use ant and was a monolithic package short of
servlet-api. Which still exists in Gentoo.
https://github.com/gentoo/gentoo/blob/master/www-servers/tomcat/tomcat-9.0.7.ebuild
https://github.com/gentoo/gentoo/blob/master/dev-java/tomcat-servlet-api/tomcat-servlet-api-9.0.7.ebuild
 
I have split it out to be a package per jar. Its working
great, it was just a bit of a pain to find what goes in each jar. If
that changes, I will have to match jar contents again. Make sure
correct classes are going in the correct jar, and resources.

See all the tomcat-* packages here, one per jar
https://github.com/Obsidian-StudiosInc/os-xtoo/tree/master/dev-java

Some are a bit tricky to package I have to remove some files or specify
which to include. It is not really ideal.

Example of matching jar contents
https://github.com/Obsidian-StudiosInc/os-xtoo/blob/master/dev-java/tomcat-jasper/tomcat-jasper-.ebuild#L39

Then I have a meta package, that just pulls in all others to form the
actual tomcat installation. 

Meta package
https://github.com/Obsidian-StudiosInc/os-xtoo/blob/master/www-servers/tomcat/tomcat-.ebuild

The individual packages allow for embedded use and for development etc.
Like say using Tomcat in Netbeans. This way Netbeans has access to all
Tomcat jars, but does not require a server package to be installed.
Even better Netbeans can have its own "Tomcat install", linking in jars
into its own local server space with its own config, etc. Before others
were doing stuff with the server package.

If making an embedded app, you can pull in what jars you need, rather
than building all of tomcat and having init scripts and other stuff
laying around unused.

Just makes it a bit more modular. A different way of shipping Tomcat :)


-- 
William L. Thomson Jr.


pgp0e4GA3mO9H.pgp
Description: OpenPGP digital signature


Re: JDK 11 EA 18

2018-06-25 Thread William L. Thomson Jr.
On Mon, 25 Jun 2018 23:21:47 -0400
"William L. Thomson Jr."  wrote:

> On Mon, 25 Jun 2018 19:40:27 -0400
> Christopher Schultz  wrote:
> 
> > I'm confused about what you're asking, here. Do you want Tomcat to
> > change its own directory layout?   
> 
> Ideally yes, like a directory per jar, with 
> 
> src/java/main
> src/java/resources

Totally messed that up...

src/main/java
src/main/resources

-- 
William L. Thomson Jr.


pgpESPesoUYDT.pgp
Description: OpenPGP digital signature