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

Bind mount does not provide the required permissions for the installer #29

Open
davidchisnall opened this issue Nov 9, 2023 · 15 comments

Comments

@davidchisnall
Copy link

The installer needs to be able to chown files in the working directory. This is not possible with a bind mount from the host. Copying files into the container in the first step of docker.sh avoids this problem.

@riccardilorenzo
Copy link

Hi! I'm facing this same issue. Can you better explain how to solve this issue please?

@davidchisnall
Copy link
Author

I will try to write up what I did, but unfortunately I worked with a download (as recommended in the README) not a git clone, so it's going to take a while to figure out what I changed.

I ended up running the installer in graphical mode, but that also required fixing things because bind mounting the UNIX domain socket didn't work and I needed to use TCP.

For the Arty A7, the shipped version of xvcd didn't work either, but I have managed to patch that to make it work.

@DaryaGolovko
Copy link

Hi, I have the same problem. Can you please write how you fixed it?

@NelsonDane
Copy link

@DaryaGolovko If you take a look at my fork, I solved this issue by extracting and taking ownership of the files in an internal docker directory before moving them to the mountpoint

@tachsin
Copy link

tachsin commented Dec 7, 2023

@DaryaGolovko If you take a look at my fork, I solved this issue by extracting and taking ownership of the files in an internal docker directory before moving them to the mountpoint

That solved my problem as well !!

@NelsonDane
Copy link

Awesome, I created a PR with my changes

@VladBara
Copy link

OMG, it works. Be blessed !!!
This fix should be added to the main branch and also mentioned in the description.

@mohay3069
Copy link

@DaryaGolovko If you take a look at my fork, I solved this issue by extracting and taking ownership of the files in an internal docker directory before moving them to the mountpoint

doesnt worK:

errors:
1)ERROR - There was an error executing: [/bin/sh, -c, echo $TMPDIR]

2)ProcessBuilder.start() or Runtime.exec() in a java application
Copyright (c) 1986-2022 Xilinx, Inc. All rights reserved.
Copyright (c) 2022-2024 Advanced Micro Devices, Inc. All rights reserved.
INFO - User has accepted the EULAs.
ERROR - There is not enough disk space on /home/user/Xilinx to install.

Generating App icon
./install.sh:73: no matches found: Xilinx/Vivado/*/doc/images/vivado_logo.png
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
icon.iconset:Failed to generate ICNS.
mv: rename icon.icns to Launch_Vivado.app/Contents/Resources/icon.icns: No such file or directory

@NelsonDane
Copy link

NelsonDane commented Jan 17, 2024

It says you don't have enough disk space @mohay3069

@mohay3069
Copy link

It says you don't have enough disk space @mohay3069
INFO - User has accepted the EULAs.
ERROR - There is not enough disk space on /home/user/Xilinx to install.

Generating App icon
./install.sh:73: no matches found: Xilinx/Vivado/*/doc/images/vivado_logo.png
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
Warning: not a valid file - skipping
icon.iconset:Failed to generate ICNS.
mv: rename icon.icns to Launch_Vivado.app/Contents/Resources/icon.icns: No such file or directory
i have 100 gibabytes of space how much more do i need?

@mohay3069
Copy link

It says you don't have enough disk space @mohay3069

is this it?

is this it:{
"builder": {
"gc": {
"defaultKeepStorage": "20GB",
"enabled": true
}
},
"experimental": false
}

@ichi4096
Copy link
Owner

Isn't all data not within a bind mount lost anyway? How could you do an install to a directory in the container that is not a bind mount and have it be permanent?

@davidchisnall
Copy link
Author

No, a bind mount is a host directory that is mounted into the container. The scripts install into that directory. It is not ephemeral.

@ichi4096
Copy link
Owner

Exactly, so you still need to install into the bind mound for the installation to be non-volatile.

@sapertuz
Copy link

I pulled @NelsonDane 's for but I still ge the same error... I already set the settings in docker to include both the /tmp and git repo folders... I get:

Drag and drop the installation binary into this terminal window and press Enter: 
path-to-bin/FPGAs_AdaptiveSoCs_Unified_2023.2_1013_2256_Lin64.bin 
Launching Docker container and installation script
docker: Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /tmp/.X11-unix.
See 'docker run --help'.
Generating App icon
./install.sh:73: no matches found: Xilinx/Vivado/*/doc/images/vivado_logo.png
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
Warning:  not a valid file - skipping
icon.iconset:Failed to generate ICNS.
mv: rename icon.icns to Launch_Vivado.app/Contents/Resources/icon.icns: No such file or directory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants