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

Update audio.md #1005

Merged
merged 6 commits into from
May 26, 2022
Merged

Update audio.md #1005

merged 6 commits into from
May 26, 2022

Conversation

thomas-zimmerman
Copy link
Contributor

  • Add sets of commands to restart PipeWire
  • Add commands to reinstall default PipeWire packages
  • Update commands to reinstall default PulseAudio packages
  • Add commands for PipeWire status and pw-top

@jacobgkau jacobgkau requested review from a team and removed request for jacobgkau May 5, 2022 23:12
content/audio.md Outdated Show resolved Hide resolved
content/audio.md Outdated Show resolved Hide resolved
content/audio.md Outdated Show resolved Hide resolved
content/audio.md Outdated Show resolved Hide resolved
content/audio.md Outdated Show resolved Hide resolved
systemctl --user status wireplumber
```

If you would like to monitor PipeWire, run:
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure what this command is showing me, having just run it for the first time. A more specific description would be helpful.

@ahoneybun
Copy link
Member

I've fixed most of the comments but some I'd like to address at a later date so that we can get the PipeWire changes in as soon as possible.

@ahoneybun ahoneybun requested review from jacobgkau and ahoneybun and removed request for storyaddict and ctsdownloads May 25, 2022 19:09
Copy link
Member

@ahoneybun ahoneybun left a comment

Choose a reason for hiding this comment

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

LGTM as a base for an update to this article. Provides helpful commands for PipeWire now instead of just commands for PulseAudio which do not work in 22.04 LTS+

@jacobgkau jacobgkau requested review from a team and removed request for jacobgkau May 25, 2022 20:19
content/audio.md Outdated
```

When PulseAudio starts again (which it should do automatically), it will create new configuration files.
This set of commands first restarts the sound daemon and removes the user's configuration for PulseAudio. If still using PulseAudio as a server, restarts the PulseAudio server that will create new default audio configuration files.
Copy link
Member

Choose a reason for hiding this comment

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

This is not a complete sentence. There is no subject in the clause after the comma.

sudo apt install --reinstall libpulse0 libpulsedsp pulseaudio pulseaudio-module-bluetooth pulseaudio-utils
```

This command will reinstall the PipeWire packages:
Copy link
Member

Choose a reason for hiding this comment

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

Why did you include the in the second one of these sentences but not the first one?

@ahoneybun ahoneybun merged commit 458a940 into master May 26, 2022
@ahoneybun ahoneybun deleted the update-audio.md branch May 26, 2022 15:55
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.

None yet

3 participants