Skip to content

v3.12.x: Ensure that elixir escript does not read from stdin #10131

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

Merged

Conversation

lukebakken
Copy link
Collaborator

This change ensures that you do not have to redirect stdin from /dev/null to use rabbitmqctl and related utilities in a while / read shell loop.

References:

(cherry picked from commit 761d592)

@lukebakken lukebakken self-assigned this Dec 13, 2023
@lukebakken lukebakken force-pushed the lukebakken/figure-out-rabbitmqctl-stdin-v3.12.x branch from 555de91 to 278c5e3 Compare December 13, 2023 19:00
@michaelklishin michaelklishin changed the title Ensure that elixir escript does not read from stdin v3.12.x: Ensure that elixir escript does not read from stdin Dec 13, 2023
@lukebakken lukebakken marked this pull request as ready for review December 14, 2023 04:36
@lukebakken lukebakken force-pushed the lukebakken/figure-out-rabbitmqctl-stdin-v3.12.x branch from 278c5e3 to 8a2d344 Compare December 14, 2023 04:37
This change ensures that you do not have to redirect `stdin` from
`/dev/null` to use `rabbitmqctl` and related utilities in a `while` /
`read` shell loop.

References:
* https://github.com/lukebakken/vesc-1073/blob/main/delete.bash#L24-L32
* rabbitmq/support-tools#38

(cherry picked from commit 761d592)
@lukebakken lukebakken force-pushed the lukebakken/figure-out-rabbitmqctl-stdin-v3.12.x branch from 8a2d344 to 86c4ae3 Compare December 14, 2023 14:14
@michaelklishin michaelklishin added this to the 3.12.11 milestone Dec 14, 2023
@michaelklishin michaelklishin merged commit 676df61 into v3.12.x Dec 14, 2023
@michaelklishin michaelklishin deleted the lukebakken/figure-out-rabbitmqctl-stdin-v3.12.x branch December 14, 2023 18:26
lukebakken added a commit that referenced this pull request Dec 30, 2023
lukebakken added a commit that referenced this pull request Dec 30, 2023
lukebakken added a commit that referenced this pull request Dec 30, 2023
lukebakken added a commit that referenced this pull request Dec 31, 2023
lukebakken added a commit that referenced this pull request Dec 31, 2023
mergify bot pushed a commit that referenced this pull request Jan 1, 2024
Follow-up to #10131

(cherry picked from commit 3f33dfa)
lukebakken added a commit that referenced this pull request Jan 2, 2024
lukebakken added a commit that referenced this pull request Jan 2, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`
lukebakken added a commit that referenced this pull request Jan 2, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`
mergify bot pushed a commit that referenced this pull request Jan 3, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`

(cherry picked from commit 7da7d4e)
michaelklishin pushed a commit that referenced this pull request Jan 31, 2024
michaelklishin pushed a commit that referenced this pull request Jan 31, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`
michaelklishin pushed a commit that referenced this pull request Feb 1, 2024
michaelklishin pushed a commit that referenced this pull request Feb 1, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`
michaelklishin pushed a commit that referenced this pull request Feb 29, 2024
michaelklishin pushed a commit that referenced this pull request Feb 29, 2024
Follow-up to:
* #10131
* #10257

The following `rabbitmqctl` commands may require `stdin` input:
* `add_user`
* `authenticate_user`
* `change_password`
* `decode`
* `encode`
* `eval`
* `hash_password`

The following `rabbitmq-diagnostics` commands may require `stdin` input:
* `observer`
* `remote_shell`
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.

2 participants