Skip to content
This repository has been archived by the owner on Jun 11, 2021. It is now read-only.

Fixes #1061 with VM name selection #1161

Closed
wants to merge 1 commit into from

Conversation

FrenchBen
Copy link
Contributor

Signed-off-by: French Ben me+git@frenchben.com

@FrenchBen FrenchBen force-pushed the 1061-VM-name branch 2 times, most recently from 97f6c1c to 8188248 Compare October 30, 2015 23:33
@FrenchBen
Copy link
Contributor Author

Tried to QA with Windows but currently getting some errors from virtualbox. Will wait for new toolbox to be pushed out

@jmorganca
Copy link
Contributor

Taking a look!

@FrenchBen
Copy link
Contributor Author

FYI, it looks like the branch needs to be updated prior to final push

Signed-off-by: French Ben <me+git@frenchben.com>
@FrenchBen FrenchBen changed the title Fixes 1061 with VM name selection Fixes #1061 with VM name selection Nov 24, 2015
@FrenchBen FrenchBen closed this Dec 10, 2015
@FrenchBen FrenchBen deleted the 1061-VM-name branch December 10, 2015 23:49
@pmario
Copy link

pmario commented Mar 13, 2016

@FrenchBen Did this PR quietly die?

@FrenchBen
Copy link
Contributor Author

@pmario it did due to machine management. The VM selection was fine, but for the most part, there was no mounting of folders and misc actions were broken.

@pmario
Copy link

pmario commented Mar 16, 2016

Yea. I found out already, that the whole system is heavily connected to vbox. From a programmers point of view. Way to heavy. ...

IMO kitematic needs a plugin mechanism, where vbox handling is a plugin too.

This would give us (the community with different requirements) the possibility to remove the vbox battery and replace it.

At the moment I'm investigating to create shared folders with hyper-v and using container tech and not vbox tech.

@FrenchBen
Copy link
Contributor Author

I agree with that approach, unfortunately the current heavy vbox integration makes it difficult.
We've considered re-writing the app completely with the plugin in mind, but there's a lot to do to make that happen.
I'm open to suggestions if you see an easy way to get that working?

@FrenchBen FrenchBen restored the 1061-VM-name branch June 20, 2017 17:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants