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

[Noetic] Dependencies_noetic docker build crashing #2178

Open
pawanw17 opened this issue Jul 16, 2023 · 5 comments
Open

[Noetic] Dependencies_noetic docker build crashing #2178

pawanw17 opened this issue Jul 16, 2023 · 5 comments

Comments

@pawanw17
Copy link
Collaborator

pawanw17 commented Jul 16, 2023

Failing at this line

wget --no-check-certificate https://sourceforge.net/projects/turbovnc/files/${TURBOVNC_VERSION}/turbovnc_${TURBOVNC_VERSION}_amd64.deb && \

Download link no longer working

EDIT: This mirror is working https://sourceforge.net/projects/turbovnc/files/3.0/turbovnc_3.0_amd64.deb/download?use_mirror=versaweb

@Bhardwaj-Himanshu
Copy link
Contributor

Hi @pawanw17 , replacing it with a particular version would cause us to replace it every single time a new version is released!

But the general link as mentioned by you above is failing, could we replace it with something else in a longer run?And sticking to the 3.0 for now.

@pawanw17
Copy link
Collaborator Author

@Bhardwaj-Himanshu the version we use has been 3.0 for a while, and hasn't been upgraded to the latest version (I don't think we require an upgrade), I suggest to have .deb file inside the repository, the way it is done for Humble RADI, you can make a PR for this...

@Bhardwaj-Himanshu
Copy link
Contributor

Bhardwaj-Himanshu commented Jul 22, 2023

I have created a pull request, should we update the line after as well?

apt-get update && apt-get install -y --no-install-recommends ./turbovnc_${TURBOVNC_VERSION}_amd64.deb && \
rm turbovnc_${TURBOVNC_VERSION}_amd64.deb && \

and if not, could you tell me on why?
@pawanw17

@pawanw17
Copy link
Collaborator Author

Hi @Bhardwaj-Himanshu I meant to have the deb file locally saved in the gpu folder and change the install instructions in the Dockerfile making them similar to how Humble dockerfile installs turbovnc,

@pawanw17 pawanw17 changed the title Dependencies_noetic docker build crashing [Noetic] Dependencies_noetic docker build crashing Sep 8, 2023
@Tanishq30052002
Copy link

Hi @pawanw17 ,
It seems the issue is still there, since currently also using that same link in the master branch. So should it need a correction?

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

3 participants