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

Jump to bottom

Building with Docker #1985

Closed
hgdagon opened this issue Dec 14, 2020 · 4 comments
Closed

Building with Docker #1985

hgdagon opened this issue Dec 14, 2020 · 4 comments

Comments

@hgdagon
Copy link
Contributor

hgdagon commented Dec 14, 2020

OK, I'm not familiar with Docker at all (don't kill me). In fact, before reading up on it today, I thought it was like flatpak/snap.

That said, it seems there's not a lot that can go wrong. So, I installed docker desktop, tested it on docker/getting-started. Now, when I'm running this:

docker run --rm -t -v D:\msys64\home\George\mgba\src mgba/windows:w64

I get:

mkdir: cannot create directory 'build-win64': Permission denied

Same for w32. I tried substituting \ for /, but that didn't do anything.
Starting an issue here, since I'm not sure if it's an issue with Docker or the images (or me).
@endrift
Copy link
Member

endrift commented Dec 14, 2020

Docker for Windows is very unhappy when you use a volume outside of the home directory. Further, you've got the syntax for the volume flag wrong. It's supposed to be the host location on the left of the colon and the container location on the right. Further, the container location must be the same as what's in the REAME for it to work.

@endrift
Copy link
Member

endrift commented Dec 14, 2020

image

You can see in this image of the Docker Desktop GUI that I have D:\msys32\home\Vi\mgba mounted to /home/mgba/src. You'll need to do something similar with your container.

@hgdagon
Copy link
Contributor Author

hgdagon commented Dec 14, 2020

Further, you've got the syntax for the volume flag wrong. It's supposed to be the host location on the left of the colon and the container location on the right.

So it is an issue with my eyes. I was a bit confused with that volume. It's running now. Reeeally slowly, but running.

@endrift
Copy link
Member

endrift commented Dec 14, 2020

You can run it with -e MAKEFLAGS=-j4 to build with four cores instead of one, etc, though you have to put those arguments before the name of the image (in this case mgba/windows:w64)

@endrift endrift closed this as completed Dec 14, 2020
tommai78101 added a commit to tommai78101/mgba that referenced this issue Jul 1, 2022
@tommai78101
Updated link to Github issue mgba-emu#1985. Also, clarified the issue…
5676bf9 
… pertaining to Docker to be only for Windows, and have instructed those who aren't using Windows to move on.
tommai78101 added a commit to tommai78101/mgba that referenced this issue Jul 1, 2022
@tommai78101
Reverted the headers. Updated link to Github issue mgba-emu#1985. Als…
3eaec64 
…o, clarified the issue pertaining to Docker to be only for Windows, and have instructed those who aren't using Windows to move on.
@tommai78101 tommai78101 mentioned this issue Jul 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@endrift @hgdagon