Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cli] switch to CLI to launch CHE inside chedir instead of the Che launcher #3826

Merged
merged 1 commit into from
Jan 20, 2017

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Jan 20, 2017

What does this PR do?

Make that chedir uses the CLI to start Che instead of the eclipse che launcher which has been dropped

What issues does this PR fix or reference?

#3666

Previous behavior

Chedir was launching Che using che launcher but as it was not pushed we got old versions of che.

New behavior

Use of the latest cli to start che.

Tests written?

No

Docs updated?

no (internal stuff)

Changelog

Switch to CLI to launch CHE inside chedir instead of the Che launcher

Change-Id: I742383ab47ad34a9d94a4e3ce823e802023062c2
Signed-off-by: Florent BENOIT [email protected]

… which is not anymore maintained

Change-Id: I742383ab47ad34a9d94a4e3ce823e802023062c2
Signed-off-by: Florent BENOIT <[email protected]>
@benoitf benoitf added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jan 20, 2017
@benoitf benoitf added this to the 5.2.0 milestone Jan 20, 2017
@benoitf benoitf self-assigned this Jan 20, 2017
@benoitf benoitf requested a review from slemeur January 20, 2017 10:12
@benoitf benoitf changed the title Switch to CLI to launch CHE inside chedir instead of the Che launcher [cli] switch to CLI to launch CHE inside chedir instead of the Che launcher Jan 20, 2017
@benoitf benoitf requested a review from TylerJewell January 20, 2017 10:36
@benoitf benoitf merged commit 373210f into master Jan 20, 2017
@benoitf benoitf deleted the CHE-3666-b branch January 20, 2017 12:23
riuvshin pushed a commit that referenced this pull request Jan 20, 2017
… which is not anymore maintained (#3826)

Change-Id: I742383ab47ad34a9d94a4e3ce823e802023062c2
Signed-off-by: Florent BENOIT <[email protected]>
@codenvy-ci
Copy link

@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 23, 2017
akervern pushed a commit to akervern/che that referenced this pull request Jan 23, 2017
… which is not anymore maintained (eclipse-che#3826)

Change-Id: I742383ab47ad34a9d94a4e3ce823e802023062c2
Signed-off-by: Florent BENOIT <[email protected]>
@JamesDrummond JamesDrummond mentioned this pull request Feb 6, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
… which is not anymore maintained (eclipse-che#3826)

Change-Id: I742383ab47ad34a9d94a4e3ce823e802023062c2
Signed-off-by: Florent BENOIT <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants