Bug 1627084
Summary: | Review Request: mutter328 - Window and compositing manager based on Clutter | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Fabio Valentini <decathorpe> |
Component: | Package Review | Assignee: | Neal Gompa <ngompa13> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | igor.raits, ngompa13, package-review |
Target Milestone: | --- | Flags: | ngompa13:
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: | 2018-09-10 21:26:28 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
Fabio Valentini
2018-09-10 12:03:44 UTC
Taking this review. Review notes: [x] Package is functionally identical to mutter [x] Package builds and is installable [x] Package follows guidelines for compatibility packages PACKAGE APPROVED. Thanks so much for the quick review! I requested a repository with master and f29 branches: https://pagure.io/releng/fedora-scm-requests/issue/8084 https://pagure.io/releng/fedora-scm-requests/issue/8085 (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/mutter328 Built on fedora 29 and rawhide. https://koji.fedoraproject.org/koji/buildinfo?buildID=1143527 https://koji.fedoraproject.org/koji/buildinfo?buildID=1143530 why is it called 328 and not 3.28? Because I think periods in package names look weird and wrong. According to [0], including the periods is a SHOULD, not a MUST: "The package version, which SHOULD include the periods present in the original version." [0]: https://fedoraproject.org/wiki/Packaging:Naming?rd=Packaging:NamingGuidelines#Multiple_packages_with_the_same_base_name |