Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

draft: optional binds #1605

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

draft: optional binds #1605

wants to merge 1 commit into from

Conversation

j-ogas
Copy link
Contributor

@j-ogas j-ogas commented Mar 20, 2023

No description provided.

@j-ogas j-ogas requested a review from reidpr March 20, 2023 18:37
Comment on lines +133 to +137
:code:`--bind-path={PATH[:PREFIX}`
Add file from path to to ch-run optional default run-time binds.

* Path to host file. Optionally bind at prefix if set; otherwise,
:code:`/usr/local/lib`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure this (i.e., arbitrary default bind-mounts) can be implemented, but I'm not sure offhand how to do it. If there's not an immediate use case, maybe leave this for future work?

Comment on lines +127 to +128
Optional bind selection: :code:`--bind-FOO`
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these will want to be spelled --enable-bind-FOO, because --enable gets special treatment in Autotools.

Comment on lines +124 to +125
Some dependencies can be specified as follows. Note only some of these support
:code:`--with-FOO=no`, as listed.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did this move?

By default, :code:`ch-run` binds the minimal number mount points needed to run.
You can add more files with:

:code:`--bind-path={PATH[:PREFIX}`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
:code:`--bind-path={PATH[:PREFIX}`
:code:`--bind-path={HOST_PATH[:IMAGE_PATH]}`

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants