Skip to content

use -f to cf install-plugin#1941

Merged
chgeo merged 1 commit into
mainfrom
qmacro-patch-2
Jul 2, 2025
Merged

use -f to cf install-plugin#1941
chgeo merged 1 commit into
mainfrom
qmacro-patch-2

Conversation

@qmacro

@qmacro qmacro commented Jun 30, 2025

Copy link
Copy Markdown
Collaborator

If you copy/paste these three shell commands the effect won't be what the user expects, as without the -f option the install-plugin command will stop and prompt the user like this:

; cf add-plugin-repo CF-Community https://plugins.cloudfoundry.org
https://plugins.cloudfoundry.org already registered as CF-Community
# /work/scratch/debugtest
; cf install-plugin multiapps
Searching CF-Community for plugin multiapps...
Plugin multiapps 3.5.0 found in: CF-Community
Attention: Plugins are binaries written by potentially untrusted authors.
Install and use plugins at your own risk.
Do you want to install the plugin multiapps? [yN]: cf install-plugin html5-plugin

i.e. the 3rd line cf install-plugin html5-plugin will be presented as the value at the "yN" prompt instead of being a separate command.

Using -f will bypass the prompt. I would say this is fine, it's what the user wants anyway.

If you copy/paste these three shell commands the effect won't be what the user expects, as without the `-f` option the `install-plugin` command will stop and prompt the user like this:

```shell
; cf add-plugin-repo CF-Community https://plugins.cloudfoundry.org
https://plugins.cloudfoundry.org already registered as CF-Community
# /work/scratch/debugtest
; cf install-plugin multiapps
Searching CF-Community for plugin multiapps...
Plugin multiapps 3.5.0 found in: CF-Community
Attention: Plugins are binaries written by potentially untrusted authors.
Install and use plugins at your own risk.
Do you want to install the plugin multiapps? [yN]: cf install-plugin html5-plugin
```

i.e. the 3rd line `cf install-plugin html5-plugin` will be presented as the value at the "yN" prompt instead of being a separate command.
@qmacro qmacro requested a review from renejeglinsky as a code owner June 30, 2025 08:19
@renejeglinsky renejeglinsky requested a review from chgeo July 1, 2025 07:08
@chgeo chgeo enabled auto-merge July 2, 2025 08:07
@chgeo chgeo added this pull request to the merge queue Jul 2, 2025
Merged via the queue into main with commit de0dc15 Jul 2, 2025
4 checks passed
@chgeo chgeo deleted the qmacro-patch-2 branch July 2, 2025 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants