https://bz.apache.org/bugzilla/show_bug.cgi?id=57707

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|REOPENED                    |RESOLVED

--- Comment #11 from Mark Thomas <ma...@apache.org> ---
Yes, the error message could be better but I also don't think it is
unreasonable to expect folks to look at BUILDING.txt if they run into a problem
building Tomcat. It is in the root of the src distro and the svn repo which are
the first places I'd expect people to look for docs if they were having issues.

Testing for the presence of a binary on the path (for a given OS) and then
failing the build if it is not found is a on-trivial addition to what is
already a large, complex build file. There is a balance to strike between
between catching all failure modes and a simpler build file.

My instinct was to close this as WONTFIX but I wanted a better idea of how much
code was required. Having written the patch and found it was ~20 lines I was on
the fence whether to apply it or not. It isn't small, but neither is it huge
and it is fairly simple so I decided, on balance, to apply it. If I have known
at the start it was ~20 lines I probably would have followed by instincts and
closed this as WONTFIX.

The patch has been applied to trunk and I'll back-port it to 8.0.x but I don't
intend to back-port it to 7.0.x. If will probably get back-ported when/if code
signing is back-ported to 7.0.x.

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

Reply via email to