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

lm_sensors → lm-sensors pkgmove & flag rename #12903

Closed
wants to merge 25 commits into from

Conversation

mgorny
Copy link
Member

@mgorny mgorny commented Sep 11, 2019

This patchset combines pkgmoving lm_sensors → lm-sensors, and a matching USE flag change. Both changes are combined because that makes sed-ing easier ;-).

Rename because upstream calls the package 'lm-sensors'. Furthermore, underscore is reserved for USE_EXPAND, and really shouldn't be used in USE flag names.

I have alternatively considered calling this 'sensors' but I think then it would be confusing on the traditional split between lm-sensors/hddtemp/acpi.

Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Michał Górny <mgorny@gentoo.org>
@mgorny mgorny added do not merge Please DO NOT MERGE this PR. It will not be assigned but it will be scanned by CI. no assignee limit Ignore assignee limit, and ping as many maintainers as it takes. labels Sep 11, 2019
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-09-11 09:37 UTC
Newest commit scanned: e61603b
Status: ✅ good

No issues found

@mgorny mgorny removed the do not merge Please DO NOT MERGE this PR. It will not be assigned but it will be scanned by CI. label Sep 11, 2019
@gentoo-bot
Copy link

Copyright policy change

Please note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article.

Pull Request assignment

Submitter: @mgorny
Areas affected: ebuilds, profiles
Packages affected: app-admin/gkrellm, app-admin/sysstat, app-leechcraft/lc-hotsensors, app-metrics/collectd, dev-python/PySensors...

app-admin/gkrellm: polynomial-c
app-admin/sysstat: @jer-gentoo
app-leechcraft/lc-hotsensors: @0xd34df00d, @gentoo/proxy-maint
app-metrics/collectd: @Whissi
dev-python/PySensors: @grobian, @gentoo/python
kde-plasma/ksysguard: @gentoo/kde
lxqt-base/lxqt-panel: @gentoo/lxqt
mate-extra/mate-sensors-applet: @gentoo/mate
media-libs/mesa: @gentoo/x11
media-plugins/vdr-systeminfo: @gentoo/vdr
net-analyzer/net-snmp: @gentoo/netmon
net-misc/bfgminer: @blueness, @luke-jr
net-wireless/kismet: @ZeroChaos-
sys-apps/i2c-tools: @Zlogene
sys-apps/lm-sensors: @gentoo/proxy-maint (new package)
www-apps/phpsysinfo: @ConiKost, @gentoo/proxy-maint, @gentoo/web-apps
www-misc/monitorix: dwosky[at]pm.me, @gentoo/proxy-maint
x11-misc/i3blocks: @mudler
x11-misc/xsensors: @gentoo/desktop-misc
x11-plugins/wmgtemp: @voyageur
xfce-extra/xfce4-sensors-plugin: @gentoo/xfce

At least one of the listed packages is maintained entirely by non-GitHub developers!

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added new package The PR is adding a new package. need assignment It was impossible to assign the PR correctly. Please assign it manually. no bug found No Bug/Closes found in the commits. labels Sep 11, 2019
@mgorny
Copy link
Member Author

mgorny commented Sep 11, 2019

@Dwosky

@Whissi
Copy link
Contributor

Whissi commented Sep 11, 2019

Why does @gentoo-bot says

sys-apps/lm-sensors: @gentoo/proxy-maint (new package)

lm-sensors is not a proxy-maint package.

And I guess you want to add net-analyzer/nagios-plugin-check_lm_sensors to that list.

@mgorny
Copy link
Member Author

mgorny commented Sep 11, 2019

Why does @gentoo-bot says

sys-apps/lm-sensors: @gentoo/proxy-maint (new package)

lm-sensors is not a proxy-maint package.

Because it doesn't understand package moves.

And I guess you want to add net-analyzer/nagios-plugin-check_lm_sensors to that list.

Surprisingly, that package doesn't depend on lm-sensors, so there's nothing to change there.

@ZeroChaos-
Copy link
Contributor

ACK for kismet. Not sure why this is necessary but I'm not going to argue about it.

Copy link
Member

@a17r a17r left a comment

Choose a reason for hiding this comment

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

re kde-plasma/ksysguard: go for it.

Copy link
Contributor

@Whissi Whissi left a comment

Choose a reason for hiding this comment

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

lgtm

thebitpit pushed a commit to thebitpit/gentoo that referenced this pull request Sep 14, 2019
Closes: gentoo#12903
Signed-off-by: Michał Górny <mgorny@gentoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need assignment It was impossible to assign the PR correctly. Please assign it manually. new package The PR is adding a new package. no assignee limit Ignore assignee limit, and ping as many maintainers as it takes. no bug found No Bug/Closes found in the commits.
Projects
None yet
10 participants