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

Rename dpms output command to power #7083

Merged
merged 1 commit into from
Jun 23, 2022

Conversation

emersion
Copy link
Member

The "dpms" command refers to VESA Display Power Management
Signaling, a deprecated standard. It's superseded by VESA DPM.

Instead of tying out command name to a particular standard, use the
neutral term "power".

Copy link
Contributor

@bl4ckb0ne bl4ckb0ne left a comment

Choose a reason for hiding this comment

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

LGTM beside the small nit, thanks!

The "dpms" command refers to VESA Display Power Management
Signaling, a deprecated standard. It's superseded by VESA DPM.

Instead of tying out command name to a particular standard, use the
neutral term "power".
@bl4ckb0ne bl4ckb0ne merged commit 445bc2a into swaywm:master Jun 23, 2022
@emersion emersion deleted the deprecated-dpms branch June 23, 2022 19:05
@emersion emersion mentioned this pull request Nov 26, 2022
@any1 any1 mentioned this pull request Dec 1, 2022
primeos added a commit to primeos/nixpkgs that referenced this pull request Dec 26, 2022
Release notes:
- Sway: https://github.com/swaywm/sway/releases/tag/1.8
- wlroots: https://gitlab.freedesktop.org/wlroots/wlroots/-/releases/0.16.0

Known user-facing breaking changes:
- Manufacturer names in output identifiers might have changed:
  https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3541#note_1384124
- The output dpms command has been deprecated (superseded by output power):
  swaywm/sway#7083
PedroHLC pushed a commit to PedroHLC/nixpkgs that referenced this pull request Dec 26, 2022
Release notes:
- Sway: https://github.com/swaywm/sway/releases/tag/1.8
- wlroots: https://gitlab.freedesktop.org/wlroots/wlroots/-/releases/0.16.0

Known user-facing breaking changes:
- Manufacturer names in output identifiers might have changed:
  https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3541#note_1384124
- The output dpms command has been deprecated (superseded by output power):
  swaywm/sway#7083
GaetanLepage pushed a commit to GaetanLepage/nixpkgs that referenced this pull request Dec 27, 2022
Release notes:
- Sway: https://github.com/swaywm/sway/releases/tag/1.8
- wlroots: https://gitlab.freedesktop.org/wlroots/wlroots/-/releases/0.16.0

Known user-facing breaking changes:
- Manufacturer names in output identifiers might have changed:
  https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3541#note_1384124
- The output dpms command has been deprecated (superseded by output power):
  swaywm/sway#7083
@@ -119,12 +119,20 @@ must be separated by one space. For example:
Enables or disables the specified output (all outputs are enabled by
default).

As opposed to the _power_ command, the output will loose its current
Copy link
Contributor

Choose a reason for hiding this comment

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

s/loose/lose/ 😉

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch!

#7342

colemickens pushed a commit to colemickens/nixpkgs that referenced this pull request Dec 28, 2022
Release notes:
- Sway: https://github.com/swaywm/sway/releases/tag/1.8
- wlroots: https://gitlab.freedesktop.org/wlroots/wlroots/-/releases/0.16.0

Known user-facing breaking changes:
- Manufacturer names in output identifiers might have changed:
  https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3541#note_1384124
- The output dpms command has been deprecated (superseded by output power):
  swaywm/sway#7083
manner82 pushed a commit to manner82/sxmo-utils that referenced this pull request Jan 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants