On 11/29/16 Nov 29 -2:19 PM, Mark Evenson wrote:

I'm unsure of the current ASDF work flow for patch submissions, but I guessed that y'all would like [merge requests][57].

Without this patch, Elias' recent work on UIOP/RUN-PROGRAM won't be utilized on any ABCL which is built from development source.

ABCL uses the convention that development versions start appending strings separated via #- characters to the primary value returned by CL:LISP-IMPLEMENTATION-VERSION (e.g. '1.5.0-dev'. Such values cause the UIOP/UTILITY:PARSE-VERSION function to return nil, meaning that this is not a suitable conditional for whether LAUNCH-PROGRAM is invoked.

This patch uses the value of UIOP/OS:IMPLEMENTATION-IDENTIFIER to identify version. Unknown whether this would work on MKCL/ECL, which might simplify the code path here.

Please let me know what I can do to conform to y'alls current style conventions, test coverage, etc. for such a patch submission. This patch is necessary for [my backporting of Elias's work to JDK56][1]

OK, this has been merged. I think later it would be nice to make PARSE-VERSION more permissive, but Faré convinced me it was better to get the ABCL support in before making a more general solution.

Best, r