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

Adding support for host-shared folders using virtio-9p #97

Open
wants to merge 14 commits into
base: master
from

Conversation

Projects
None yet
7 participants
@stv0g

stv0g commented Apr 3, 2016

Please consider this PR a WiP.

@jceel has implemented support for host-shared directories / mounts like requested in #45 and #70.

There are a few points we still need to tackle.
And the best way to do this is by getting some feedback. Especially from @mist64 :-)

  • Are you fine with introducing new dependencies like jceel's lib9p?
  • How do you want to track dependencies:
    • git submodule?
    • pkg-config
    • ??
  • Allow building without virtio-9p support
  • lib9p is not based on on Plan 9 lib9p nor libixp. We should clarify that in the README of jceel's lib9p.
@bms

This comment has been minimized.

Show comment
Hide comment
@bms

bms Feb 24, 2016

Please consider porting to FreeBSD.

bms commented on 353ac80 Feb 24, 2016

Please consider porting to FreeBSD.

This comment has been minimized.

Show comment
Hide comment
@jceel

jceel Feb 24, 2016

Owner

@bms: I already did that, please take a look at my FreeBSD repo fork (https://github.com/jceel/freebsd) and and announcement e-mail (https://lists.freebsd.org/pipermail/freebsd-hackers/2016-February/049068.html)

Owner

jceel replied Feb 24, 2016

@bms: I already did that, please take a look at my FreeBSD repo fork (https://github.com/jceel/freebsd) and and announcement e-mail (https://lists.freebsd.org/pipermail/freebsd-hackers/2016-February/049068.html)

@c4milo

This comment has been minimized.

Show comment
Hide comment
@c4milo

c4milo Apr 9, 2016

Contributor

I'm all for getting lib9p introduced into xhyve. Any thoughts on this @xez?

Contributor

c4milo commented Apr 9, 2016

I'm all for getting lib9p introduced into xhyve. Any thoughts on this @xez?

@stv0g

This comment has been minimized.

Show comment
Hide comment
@stv0g

stv0g Apr 11, 2016

@c4milo What do you mean by 'introduced'?

Using a git-submodule? Or by copy&paste the code?
Or even without all?

stv0g commented Apr 11, 2016

@c4milo What do you mean by 'introduced'?

Using a git-submodule? Or by copy&paste the code?
Or even without all?

@c4milo

This comment has been minimized.

Show comment
Hide comment
@c4milo

c4milo Apr 11, 2016

Contributor

Whatever method is more convenient to update from upstream.

On Mon, Apr 11, 2016 at 2:55 PM Steffen Vogel notifications@github.com
wrote:

@c4milo https://github.com/c4milo What do you mean by 'introduced'?

Using a git-submodule? Or by copy&paste the code?
Or even without all?


You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#97 (comment)

Contributor

c4milo commented Apr 11, 2016

Whatever method is more convenient to update from upstream.

On Mon, Apr 11, 2016 at 2:55 PM Steffen Vogel notifications@github.com
wrote:

@c4milo https://github.com/c4milo What do you mean by 'introduced'?

Using a git-submodule? Or by copy&paste the code?
Or even without all?


You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#97 (comment)

@foxx

This comment has been minimized.

Show comment
Hide comment
@foxx

foxx Sep 7, 2016

Any update on this?

foxx commented Sep 7, 2016

Any update on this?

@stv0g

This comment has been minimized.

Show comment
Hide comment
@stv0g

stv0g Sep 16, 2016

I lost the overview which repo is now the official one.
I don't know where its merged or not.

stv0g commented Sep 16, 2016

I lost the overview which repo is now the official one.
I don't know where its merged or not.

@bms

This definitely needs to be exercised :-) next time I'm around a Mac, I may try this directly with a Linux guest.

@smiren

This comment has been minimized.

Show comment
Hide comment
@smiren

smiren Oct 12, 2016

What news?

smiren commented Oct 12, 2016

What news?

@mfojtik

This comment has been minimized.

Show comment
Hide comment
@mfojtik

mfojtik Sep 18, 2017

Seems like this needs a rebase as this won't build on latest OSX :-(

mfojtik commented Sep 18, 2017

Seems like this needs a rebase as this won't build on latest OSX :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment