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

remove ubuntu universe repo #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

whyevenquestion1t
Copy link

@whyevenquestion1t whyevenquestion1t commented Jul 21, 2023

what

Removed: RUN apt-get install -y software-properties-common && \ add-apt-repository "deb http://archive.ubuntu.com/ubuntu bionic universe" && \ gpg --keyserver hkp://keyserver.ubuntu.com:80 --recv 3B4FE6ACC0B21F32 && \ gpg --export --armor 3B4FE6ACC0B21F32 | apt-key add - && \ apt-get update && \ apt-get install -y golang-petname in Dockerfile

why

This script is redundant and causing an error (I might be wrong)

references

Please see the issue I have submitted (it describes an error in details). I think I should have submitted the issue in this repo, sorry about that.

@whyevenquestion1t whyevenquestion1t requested review from a team as code owners July 21, 2023 20:43
@osterman osterman requested review from max-lobur and removed request for jamengual and florian0410 July 31, 2023 18:48
@max-lobur max-lobur added the patch A minor, backward compatible change label Sep 12, 2023
@max-lobur max-lobur closed this Sep 12, 2023
@max-lobur max-lobur reopened this Sep 12, 2023
@max-lobur
Copy link
Contributor

Closed/reopened to trigger builds

@max-lobur
Copy link
Contributor

I believe using multi-arch build for this would be the correct solution. Currently it builds only for x86_64 as I see from a log.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch A minor, backward compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants