You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by Mark Thomas <ma...@apache.org> on 2019/12/01 21:28:57 UTC

Re: Initial set of patches for Jakarta EE 9

On 30/11/2019 21:11, Igal Sapir wrote:
> Mark,
> 
> On 11/28/2019 11:46 AM, Mark Thomas wrote:
>> Hi all,
>>
>> I have pushed an initial set of patches for Jakarta EE 9 here:
>> https://github.com/markt-asf/tomcat/tree/jakarta
>>
>> The current status is:
>> - All the packages changing in Jakarta EE 9 have been renamed
>> - Any associated constants, service loader files etc. have also been
>>    renamed
>> - It builds
>> - The unit tests pass (excluding those that depend on JSTL - more on
>>    that below)
>> - A basic smoke test passes
>>
>> Please try and build it, take it for a spin and report back on this
>> thread if you find any issues.
> 
> First issue I noticed when trying to build on Windows:
> 
> compile:
>     [javac] Compiling 1727 source files to
> E:\Workspace\test\tomcat-jakarta\output\classes
>     [javac]
> E:\Workspace\test\tomcat-jakarta\java\org\apache\tomcat\util\http\RequestUtil.java:21:
> error: package javax.servlet.http does not exist
>     [javac] import javax.servlet.http.HttpServletRequest;
>     [javac]                          ^
>     [javac]
> E:\Workspace\test\tomcat-jakarta\java\org\apache\tomcat\util\http\RequestUtil.java:122:
> error: cannot find symbol
>     [javac]     public static boolean isSameOrigin(HttpServletRequest
> request, String origin) {
>     [javac]                                        ^
>     [javac]   symbol:   class HttpServletRequest
>     [javac]   location: class RequestUtil
>     [javac] Note: Some input files use or override a deprecated API.
>     [javac] Note: Recompile with -Xlint:deprecation for details.
>     [javac] 2 errors
> 
> BUILD FAILED
> E:\Workspace\test\tomcat-jakarta\build.xml:706: Compile failed; see the
> compiler error output for details.

Sorry. My error. When I rebased the branch I forgot to check if any
additional changes were required. I've fixed this now. Note I used a
force push to try and keep to the one commit per package I used originally.

Should be OK now.

Mark

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


Re: Initial set of patches for Jakarta EE 9

Posted by Igal Sapir <is...@apache.org>.
On 12/1/2019 1:28 PM, Mark Thomas wrote:
> On 30/11/2019 21:11, Igal Sapir wrote:
>
> <snip/>
>> First issue I noticed when trying to build on Windows:
>>
>> compile:
>>      [javac] Compiling 1727 source files to
>> E:\Workspace\test\tomcat-jakarta\output\classes
>>      [javac]
>> E:\Workspace\test\tomcat-jakarta\java\org\apache\tomcat\util\http\RequestUtil.java:21:
>> error: package javax.servlet.http does not exist
>>      [javac] import javax.servlet.http.HttpServletRequest;
>>      [javac]                          ^
>>      [javac]
>> E:\Workspace\test\tomcat-jakarta\java\org\apache\tomcat\util\http\RequestUtil.java:122:
>> error: cannot find symbol
>>      [javac]     public static boolean isSameOrigin(HttpServletRequest
>> request, String origin) {
>>      [javac]                                        ^
>>      [javac]   symbol:   class HttpServletRequest
>>      [javac]   location: class RequestUtil
>>      [javac] Note: Some input files use or override a deprecated API.
>>      [javac] Note: Recompile with -Xlint:deprecation for details.
>>      [javac] 2 errors
>>
>> BUILD FAILED
>> E:\Workspace\test\tomcat-jakarta\build.xml:706: Compile failed; see the
>> compiler error output for details.
> Sorry. My error. When I rebased the branch I forgot to check if any
> additional changes were required. I've fixed this now. Note I used a
> force push to try and keep to the one commit per package I used originally.
>
> Should be OK now.

This one built fine but got some failing test cases on a busy Windows 
machine, so possibly false positives:

    [concat] Testsuites with failed tests:
    [concat] 
TEST-org.apache.catalina.authenticator.TestFormAuthenticator.APR.txt
    [concat] TEST-org.apache.catalina.core.TestAsyncContextImpl.NIO2.txt
    [concat] TEST-org.apache.coyote.http2.TestStreamProcessor.APR.txt
    [concat] TEST-org.apache.coyote.http2.TestStreamProcessor.NIO.txt
    [concat] TEST-org.apache.coyote.http2.TestStreamProcessor.NIO2.txt
    [concat] TEST-org.apache.jasper.runtime.TestJspRuntimeLibrary.APR.txt
    [concat] TEST-org.apache.jasper.runtime.TestJspRuntimeLibrary.NIO.txt
    [concat] TEST-org.apache.jasper.runtime.TestJspRuntimeLibrary.NIO2.txt
    [concat] 
TEST-org.apache.tomcat.websocket.TestWebSocketFrameClientSSL.APR.txt

Igal




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