Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1440406 (thermald)
Summary: | Review Request: thermald - Thermal Management daemon | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Björn 'besser82' Esser <besser82> |
Component: | Package Review | Assignee: | Jens Lody <fedora> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora, package-review, srinivas.pandruvada |
Target Milestone: | --- | Flags: | fedora:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-04-16 14:38:41 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 177841 |
Description
Björn 'besser82' Esser
2017-04-08 15:15:15 UTC
Updated package: Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=18858019 Urls: Spec URL: https://pagure.io/besser82/package-review/raw/master/f/thermald.spec SRPM URL: https://pagure.io/besser82/package-review/raw/master/f/thermald-1.6-0.2.fc27.src.rpm Updated package: Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=18861986 Urls: Spec URL: https://pagure.io/besser82/package-review/raw/master/f/thermald.spec SRPM URL: https://pagure.io/besser82/package-review/raw/master/f/thermald-1.6-0.3.fc27.src.rpm Looks good so far. There are two or three questions: Shouldn't the versioned Requires only have a single equal-sign ? What about the dbus-files ? The service-file should be under /usr/, but no other package installs the *.conf-file there, they all keep (or install) it under /etc. I did not find anything in the guidelines about that. Shouldn't the service be enabled as default ? And therefore get a preset-file ? I think it might be expected by the normal user to work out of the box with default settings. It should be covered by https://fedoraproject.org/wiki/Packaging:DefaultServices#Locally_running_services . Jens (In reply to Jens Lody from comment #3) > Looks good so far. > There are two or three questions: > > Shouldn't the versioned Requires only have a single equal-sign ? Well, it works with a single or a double equal-sign (tested from el5 to recent Fedora). I simply prefer the doubled one, because - from perspective of a hacker - it looks more obvious like a comparison than an assignment. > What about the dbus-files ? > The service-file should be under /usr/, but no other package installs the > *.conf-file there, they all keep (or install) it under /etc. I did not find > anything in the guidelines about that. There is nothing noted in the guidelines, but there is a common rule of thumb saying: User editable / customizable config goes in /etc, system-presets go to /usr/share. > Shouldn't the service be enabled as default ? > And therefore get a preset-file ? > I think it might be expected by the normal user to work out of the box with > default settings. > It should be covered by > https://fedoraproject.org/wiki/Packaging: > DefaultServices#Locally_running_services . Just requested a systemd-preset for it: https://bugzilla.redhat.com/show_bug.cgi?id=1440479 Updated package: Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=18908688 Urls: Spec URL: https://pagure.io/besser82/package-review/raw/master/f/thermald.spec SRPM URL: https://pagure.io/besser82/package-review/raw/master/f/thermald-1.6-0.4.fc27.src.rpm Package is approved. I personally would like to see it in Fedora, no matter if it has a systemd-preset or not. So feel free to add it and probably add the preset later (if the request will be accepted). Jens Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/thermald thermald-1.6-3.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-ab248cfca4 thermald-1.6-3.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-9ab3f22a16 thermald-1.6-3.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-a4b9c07b2c thermald-1.6-3.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-b883f07f79 thermald-1.6-3.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-9ab3f22a16 thermald-1.6-3.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-a4b9c07b2c thermald-1.6-3.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-ab248cfca4 thermald-1.6-3.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2017-b883f07f79 thermald-1.6-3.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report. thermald-1.6-3.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report. thermald-1.6-3.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report. thermald-1.6-3.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. *** Bug 974714 has been marked as a duplicate of this bug. *** |