Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: add cmake, nfpm to dogfood dockerfile - old #3556

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

Closed
wants to merge 2 commits into from

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Aug 18, 2022

When zstd wasn't found locally, I ran this script to install it. It requires cmake so I added that to the Dockerfile. I also added nfpm since that's required for make build and is used to embed the site.

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Aug 18, 2022

error: invalid tag "codercom/oss-dogfood:jsjoeio/update-dockerfile": invalid reference format
Error: buildx failed with: error: invalid tag "codercom/oss-dogfood:jsjoeio/update-dockerfile": invalid reference format

Hmm..any ideas why this is failing? Maybe the / in my branch name?

@jsjoeio jsjoeio requested a review from a team August 18, 2022 17:08
@johnstcn
Copy link
Member

error: invalid tag "codercom/oss-dogfood:jsjoeio/update-dockerfile": invalid reference format
Error: buildx failed with: error: invalid tag "codercom/oss-dogfood:jsjoeio/update-dockerfile": invalid reference format

Hmm..any ideas why this is failing? Maybe the / in my branch name?

yep! I ran into this yesterday as well

@jsjoeio jsjoeio changed the title feat: add cmake, nfpm to dogfood dockerfile feat: add cmake, nfpm to dogfood dockerfile - old Aug 18, 2022
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Aug 18, 2022

closing in favor of #3558

@jsjoeio jsjoeio closed this Aug 18, 2022
@github-actions github-actions bot deleted the jsjoeio/update-dockerfile branch February 19, 2023 00:12
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.

2 participants