Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

scripts: Add bash tab completion script #563

Merged
merged 1 commit into from
Sep 15, 2017

Conversation

jodh-intel
Copy link
Contributor

Add tab completion script for bash(1) to make working with the
runtime "standalone" more comfortable and efficient.

Fixes #562.

Signed-off-by: James O. D. Hunt [email protected]

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

Can't say I read all the details, or tried it - a little something wacky for a Friday eh?
lgtm :-)

@@ -250,6 +250,7 @@ func createRuntimeApp(args []string) error {
app.Flags = runtimeFlags
app.Commands = runtimeCommands
app.Before = runtimeBeforeSubcommands
app.EnableBashCompletion = true
Copy link
Contributor

Choose a reason for hiding this comment

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

Add tab completion script for bash(1) to make working with the
runtime "standalone" more comfortable and efficient.

Fixes clearcontainers#562.

Signed-off-by: James O. D. Hunt <[email protected]>
@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage increased (+0.002%) to 96.833% when pulling 10730ba on jodh-intel:add-bash-completion into bc51f37 on clearcontainers:master.

@jodh-intel jodh-intel merged commit fdc8864 into clearcontainers:master Sep 15, 2017
mcastelino pushed a commit to mcastelino/runtime that referenced this pull request Dec 6, 2018
When starting a kata container, if both initrd and rootfs
are provided in the configuration file then the error message
presented is "cannot specify an image and an initrd in
configuration file" which might be a bit confusing. This
fix makes the error message more explicit.

Fixes:  clearcontainers#563

Signed-off-by: Nitesh Konkar [email protected]
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.

3 participants