[
https://issues.jboss.org/browse/JBIDE-24682?page=com.atlassian.jira.plugi...
]
Rob Stryker commented on JBIDE-24682:
-------------------------------------
Escalating this issue. [~odockal] Really good catch. Thanks for being so on-the-ball here.
Output of 'minishift version' was (and is) expected to follow a consistent format
at the bare minimum. Previous expectations (and current, until changed) are a stream
suitable to fed into a Properties object.
Any change from this expected output is an API breakage as far as we see it.
However, what's worse is that the current output is 100% inconsistent:
minishift v1.2.0+001371f
CDK Version:3.0.0-13072017-1
Notice how the CDK line includes a colon, and can be fed into properties format, but the
minishift line has no colon, and breaks our reader. Any attempt to parse this manually
will fail, unless I'm coding a parser specifically geared for today's output and
nothing else.
I've pinged [~hardy.ferentschik] to see where and how to escalate this for a prompt
solution.
Reflect changes in minishift's version command
----------------------------------------------
Key: JBIDE-24682
URL:
https://issues.jboss.org/browse/JBIDE-24682
Project: Tools (JBoss Tools)
Issue Type: Bug
Components: cdk
Affects Versions: 4.5.0.Final
Reporter: Ondrej Dockal
Assignee: Rob Stryker
Priority: Blocker
Fix For: 4.5.0.Final
Due to changes in upstream project the 'minishift version' command returns
different lines.
CDK result from
http://cdk-builds.usersys.redhat.com/builds/weekly/19-Jun-2017.cdk-3.0.0-... build:
{code}
Minishift version: 1.0.0+4f8cb6d
CDK Version: 3.0.0-2
{code}
vs.
CDK result from nightly
http://cdk-builds.usersys.redhat.com/builds/nightly/13-Jul-2017
build:
{code}
minishift v1.2.0+001371f
CDK Version:3.0.0-13072017-1
{code}
And this change results in error while creating CDK 3 server adapter with newer CDK
nightly build as it ends with "Unknown error while checking minishift version"
as we expect the string to contains "Minishift Version".
We should get in touch with cdk team to check whether they plan to change the output
'CDK Version' to something else as well, in case we are going to distinguish
between minishift and cdk. If we accept in CDK 3 server adapter both cdk and minishift
binary maybe we should create a CDK jira that would ask for changing output from
'minishift version' from 'CDK Version: X.X.X' to 'cdk vX.X.X'. I
do not know if we plan to do so, it is just an idea...
--
This message was sent by Atlassian JIRA
(v7.2.3#72005)