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

Try to fix Opensuse certificate errors. #4075

Merged
merged 4 commits into from
Jul 19, 2023

Conversation

fruffy-bfn
Copy link
Contributor

@fruffy-bfn fruffy-bfn commented Jul 19, 2023

The OpenSuse repository keeps failing to download the p4lang debian packages. Maybe updating the deprecated key installation technique can help?

@rst0git I can not quite figure out how to fix this. Do maybe know how to fix the key installation issue?

I believe the OpenSuse repository is malformed? It does not have a dists folder. I managed to work around that, but I still run into key issues.

@fruffy-bfn fruffy-bfn force-pushed the certificate_errors branch 7 times, most recently from 6325fb9 to 1b07319 Compare July 19, 2023 16:56
@rst0git
Copy link
Member

rst0git commented Jul 19, 2023

There are updated installation instructions at
https://software.opensuse.org//download.html?project=home%3Ap4lang&package=p4lang-p4c

echo "deb http://download.opensuse.org/repositories/home:/p4lang/xUbuntu_${DISTRIB_RELEASE}/ /" | sudo tee /etc/apt/sources.list.d/home:p4lang.list
curl -fsSL https://download.opensuse.org/repositories/home:p4lang/xUbuntu_${DISTRIB_RELEASE}/Release.key | gpg --dearmor | sudo tee /etc/apt/trusted.gpg.d/home_p4lang.gpg > /dev/null
sudo apt-get update

@fruffy-bfn fruffy-bfn marked this pull request as ready for review July 19, 2023 19:23
@fruffy-bfn
Copy link
Contributor Author

fruffy-bfn commented Jul 19, 2023

This does seem to work! Thanks for pointing to the right instructions, could not figure out how install the key. Apparently this approach is still deemed insecure because it installs the key globally, but this should not matter for CI...

@rst0git
Copy link
Member

rst0git commented Jul 19, 2023

It might be good to also update the README.md file.

@fruffy fruffy requested a review from rst0git July 19, 2023 20:04
README.md Outdated Show resolved Hide resolved
@fruffy fruffy merged commit 09c82cf into p4lang:main Jul 19, 2023
15 checks passed
@fruffy-bfn fruffy-bfn deleted the certificate_errors branch July 27, 2023 15:50
kjw6855 pushed a commit to kjw6855/p4c that referenced this pull request May 29, 2024
* Try to fix certificate errors.

* Try recommended version.

* Also update README

* lsb-release instead of os-release.
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

Successfully merging this pull request may close these issues.

3 participants