Open joelmccoy opened 2 months ago
Ah dang, nintendo must've got to them. Yeah go ahead and make the PR with mario removed
Actually it looks like the docker image here isn't actually even necessary in this example. It is actually pulling from the public dockerhub registry. I think the dockerhub outage yesterday made me to believe that I needed to build this image locally. Would it be sane to remove this directory completely as this dockerfile is not being used? Or is the preference still to clean up the dockerfile by removing the mario references?
Environment
Device and OS: Macbook Pro M3 App version: N/A Kubernetes distro being used: N/A Other:
Steps to reproduce
Expected result
Dockerfile builds so that you can have the proper docker image locally for the zarf package in the example.
Actual Result
Dockerfile fails because of a bad link (see visual proof)
Visual Proof (screenshots, videos, text, etc)
Severity/Priority
Low
Additional Context
The dos-games zarf example may be broken because of a bad web link in the Dockerfile. It possibly looks like the mario brothers game got removed and there is no web archive link for this particular game.
If you remove both mario links in the dockerfile it builds correctly. If we are ok with axing Mario Brothers from this I can put up a PR.
If I am supposed to get this image from somewhere else, it is not clear in the example and probably should be clarified.