-
Notifications
You must be signed in to change notification settings - Fork 3
docs: Document versions in README.md #130
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,10 +6,10 @@ | |
|
|
||
| ```mermaid | ||
| graph RL | ||
| V1[gap-docker:4.13.1-bare] --> T | ||
| V2[gap-docker:4.13.1-full] --> T | ||
| V3[gap-docker:4.13.1-slim] --> T | ||
| V4[gap-docker:4.13.1-buildfull] --> T | ||
| V1[gap-docker:4.15.1-bare] --> T | ||
| V2[gap-docker:4.15.1-full] --> T | ||
| V3[gap-docker:4.15.1-slim] --> T | ||
| V4[gap-docker:4.15.1-buildfull] --> T | ||
| T --> V | ||
| V --> G | ||
|
|
||
|
|
@@ -18,18 +18,36 @@ | |
| T@{ shape: processes, label: "bare/slim/buildfull/full"} | ||
| ``` | ||
|
|
||
| We have the following variants[^1]: | ||
|
|
||
| * **`bare`** - Core compiled from source with only the `PackageManager` package installed. Required packages are loaded via `InstallRequiredPackages()` — no optional packages are compiled. This is the _smallest image_, suitable for minimal or custom setups where you want full control over which packages to add. | ||
|
|
||
| * **`slim`** - Includes additional runtime libraries needed by optional packages. Suitable for users who want to _selectively compile_ only the packages they need. | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Still not clear to me which packages are included with this build: this seems to imply that all deposited packages are included, if so, then this should be written explicitly. If not, then what packages are included?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Well, none.
Its complied gap-system with additional packages,
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we are using "package" to mean two different things :) I mean which "GAP packages" are included? Would it be possible to include which GAP packages are included in the README.md?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @james-d-mitchell I'm wondering if including may be a good idea.
It obviously may very between versions of GAP.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You could write for slim: "all deposited GAP packages (i.e. those distributed in the release archives) are included, but those requiring compilation have not been compiled. Some/all? system packages required to compile GAP packages are also installed in the container." Better that than listing every package which would require more maintenance than is perhaps reasonable, and would quickly get out of date. |
||
|
|
||
| * **`full`** - Contains _all packages compiled_ via `BuildPackages.sh --parallel`. It is the largest image and the most comprehensive variant. Recommended for most users. | ||
limakzi marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| * **`jupyter-gap`** - A separate image based on the `full` variant. Adds `JupyterLab` and the `gap-kernel`. Exposes port `8888` and launches notebook as the default command. | ||
|
|
||
| Besides that we have: | ||
|
|
||
| * **`buildfull`** - An _internal build stage_ used to compile all packages for the `full` image. Referenced in the build pipeline but not intended for direct use. | ||
|
|
||
| --- | ||
|
|
||
| ## Usage | ||
|
|
||
| * Run a container: | ||
|
|
||
| ``` | ||
| docker run -it ghcr.io/gap-system/gap:4.13.1-full | ||
| docker run -it ghcr.io/gap-system/gap:4.15.1-full | ||
| ``` | ||
|
|
||
| ``` | ||
| docker run -it ghcr.io/gap-system/gap:4.14.0-full | ||
| docker run -it ghcr.io/gap-system/gap:4.15.0-full | ||
| ``` | ||
|
|
||
| ``` | ||
| docker run -it ghcr.io/gap-system/gap:4.13.1-bare | ||
| docker run -it ghcr.io/gap-system/gap:4.14.0-bare | ||
| ``` | ||
|
|
||
| ``` | ||
|
|
@@ -45,3 +63,5 @@ Although it was completely rewritten, this repository consists of various ideas | |
|
|
||
| [1]: https://github.com/james-d-mitchell/gap-docker-minimal | ||
| [2]: https://github.com/stertooy/gda-image | ||
|
|
||
| [^1]: All variants are based on `Ubuntu 22.04` LTS. | ||
Uh oh!
There was an error while loading. Please reload this page.