[Bug 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #7 from Michael Osipov  ---
What do you think about the erratic behavior of APR in regards of the address
family when this feature is enabled or disabled? Our fault or APR's?

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #6 from Mark Thomas  ---
Yes, I think that check is safe to remove.

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #5 from Michael Osipov  ---
Since all tests pass, this check for threading support can be dropped sooner or
later? The threading support is required for OpenSSL < 1.1.0 only.

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Mark Thomas  ---
Fixed in:
- master for 10.0.0-M5 onwards
- 9.0.x for 9.0.35 onwards

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #3 from Mark Thomas  ---
Ah. That explains it. I can reproduce this now. I've fixed the hang on failure.
I'll look at the failure next.

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #2 from Michael Osipov  ---
(In reply to Mark Thomas from comment #1)
> I'm struggling to recreate this.
> 
> I have APR 1.7.x compiled with --disable-threads
> I have Tomcat-Native at 57da160 (just before the fix for BZ 63671)
> I have tomcat master (current HEAD)
> 
> The tests fail because of this test:
> https://github.com/apache/tomcat/blob/master/java/org/apache/tomcat/jni/
> Library.java#L251
> 
> which has been present for 14+ years.
> 
> How did you manage to run the test with APR threading disabled?

My bad, I forgot to mention this. I have removed the if clause to see whether
Tomcat really requires this. It obviously don't. Only OpenSSL < 1.1.0 requires
the threading support.

Please remove that check and try again.

-- 
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 64311] org.apache.tomcat.jni.TestSocketServerAnyLocalAddress locks entire testbed run under certain conditions

2020-04-07 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=64311

--- Comment #1 from Mark Thomas  ---
I'm struggling to recreate this.

I have APR 1.7.x compiled with --disable-threads
I have Tomcat-Native at 57da160 (just before the fix for BZ 63671)
I have tomcat master (current HEAD)

The tests fail because of this test:
https://github.com/apache/tomcat/blob/master/java/org/apache/tomcat/jni/Library.java#L251

which has been present for 14+ years.

How did you manage to run the test with APR threading disabled?

-- 
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