[GitHub] [maven-surefire] kriegaex edited a comment on pull request #343: SUREFIRE-1881 Adds additional debug log and fork connection timeout

[GitBox <gi...@apache.org>]

kriegaex edited a comment on pull request #343:
URL: https://github.com/apache/maven-surefire/pull/343#issuecomment-810952849


   > @kriegaex
   > This code is right!
   
   What the heck, @Tibor17? Again, you did not read my comment carefully or misinterpret it. I was not commenting the code in this PR at all but **my own** last changes in my reproducer project.
   
   As for this PR, even I as a hobby programmer can see at first glance that the PR is work in progress and in its current state not meant to be merged but just to produce some insights into what might be going wrong. It is a debugging tool, nothing more! @reinhapa IMO only created a PR from his fork because it is easy for you to check out and comment on. So you mentioning that it makes build time out is not helpful, because it is not a surprise either. We are both just trying to help you debug this, because you seem to be unwilling to do it alone. Sorry for trying to support you in this imperfect way, as you seem to know everything better (even if you don't) and seem to know how to solve the problem (even if you don't) or else put the blame on my sample project or this PR which both just try to reproduce the problem.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org