Bug 1623300
Summary: | Review Request: R-tweenr - Interpolate Data for Smooth Animations | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Elliott Sales de Andrade <quantum.analyst> |
Component: | Package Review | Assignee: | Iñaki Ucar <i.ucar86> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | i.ucar86, package-review |
Target Milestone: | --- | Flags: | i.ucar86:
fedora-review+
|
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2018-09-07 15:24:37 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: |
Description
Elliott Sales de Andrade
2018-08-29 01:07:49 UTC
koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=29357688 Just one minor issue/question before approving: what happens with headers under src/? The R packaging guidelines do not specify what to do with them. In principle, they should be placed in a -devel subpackage, but anyway they are useless for other R packages if they are not under inst/include/. - No rpmlint errors, some spelling warnings (false positives). - Requires, BuildRequires ok. - Compatible licenses, breakdown specified. - Files/dirs ok. They aren't installed, so no need for the -devel subpackage. They are installed in the -debugsource subpackage, and the fedora-review tool complains about this. This is why I'm asking. Those are just for debugging; they don't need to be split up. rpmlint is probably complaining about the executable bit that I forgot to remove. Here's an update with that corrected: Spec URL: https://qulogic.fedorapeople.org/R-tweenr.spec SRPM URL: https://qulogic.fedorapeople.org/R-tweenr-0.1.5-2.fc28.src.rpm Ok! Package approved. (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/R-tweenr R-tweenr-0.1.5-2.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2018-9354d688bf R-tweenr-0.1.5-2.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2018-089d519f63 R-tweenr-0.1.5-2.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2018-9061c50408 R-tweenr-0.1.5-2.fc27 has been pushed to the Fedora 27 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-2018-9354d688bf R-tweenr-0.1.5-2.fc28 has been pushed to the Fedora 28 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-2018-089d519f63 R-tweenr-0.1.5-2.fc29 has been pushed to the Fedora 29 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-2018-9061c50408 R-tweenr-0.1.5-2.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. R-tweenr-0.1.5-2.fc28 has been pushed to the Fedora 28 stable repository. If problems still persist, please make note of it in this bug report. R-tweenr-0.1.5-2.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report. |