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

Skip to content

Exclude command-not-found from base image. #4

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: dev-container-ubuntu-minimal
Choose a base branch
from
Open

Exclude command-not-found from base image. #4

wants to merge 1 commit into from

Conversation

slowsage
Copy link

On images without python3, apt-get update causes:

Traceback (most recent call last):
  File "/usr/lib/cnf-update-db", line 3, in <module>
    import apt_pkg
ModuleNotFoundError: No module named 'apt_pkg'
```.
This removes the python3 dependency on base image.

@ChristopherHX
Copy link
Owner

Things excluded in the base image should be added back as part of the bigger image.

So my suggestion include them explicitly in the tar command of the python3 image.

@slowsage slowsage force-pushed the cnf branch 2 times, most recently from 0eeac0b to d71d985 Compare September 23, 2025 18:28
@slowsage
Copy link
Author

Makes sense. Please review. Thank you!

ChristopherHX
ChristopherHX reviewed Sep 23, 2025
View reviewed changes
@ChristopherHX
Copy link
Owner

Thanks, looks good now.

Now I want some time to download and test the produced images in https://github.com/slowsage/runner-image-blobs/pkgs/container/runner-images-experiments.

After this is done, I can merge this.

BTW you can delete that package from your account if you wish once this is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChristopherHX ChristopherHX ChristopherHX left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@slowsage @ChristopherHX