Skip to content
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

fix(lvm): clean up whitespace in messages #2672

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

Conversation

teigland
Copy link
Contributor

The lvm module has been printing messages with whitespace damage:

Scanning devices sda sdb for LVM logical volumes vg/root
vg/swap

Remove the trailing space after the final device name, and remove the newlines between LV names. The message above becomes:

Scanning devices sda sdb for LVM logical volumes vg/root vg/swap

Checklist

  • [ x] I have tested it locally

@github-actions github-actions bot added modules Issue tracker for all modules lvm Issues related to the lvm module labels Aug 27, 2024
@jozzsi
Copy link

jozzsi commented Aug 27, 2024

@teigland Fedora is using https://github.com/dracut-ng/dracut-ng . Would you consider uploading it there. Thanks !

@aafeijoo-suse
Copy link
Member

@teigland Fedora is using https://github.com/dracut-ng/dracut-ng . Would you consider uploading it there. Thanks !

This has to be a bot, there is no other explanation.

The lvm module has been printing messages with whitespace damage:

  Scanning devices sda sdb  for LVM logical volumes vg/root
  vg/swap

Remove the trailing space after the final device name, and remove the
newlines between LV names.  The message above becomes:

  Scanning devices sda sdb for LVM logical volumes vg/root vg/swap

(The same change is made to the VG activation message.)
@aafeijoo-suse
Copy link
Member

This has to be a bot, there is no other explanation.

Oh, now we know what it is, and it's worse than a bot... though I'm not surprised (facepalm)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lvm Issues related to the lvm module modules Issue tracker for all modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants