Bug 1957929
| Summary: | Review Request: lorax-templates-rhel - RHEL build templates for lorax and livemedia-creator | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Stephen Gallagher <sgallagh> |
| Component: | Package Review | Assignee: | Neal Gompa <ngompa13> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | bcl, jkaluza, 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: | 2022-05-24 12:21:38 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
Stephen Gallagher
2021-05-06 18:59:47 UTC
Taking this review. So there are a couple of issues: > URL: https://github.com/weldr/lorax > [..] > Source0: lorax-templates-rhel-9.0-14.tar.gz The URL doesn't point to the actual project, and the Source0 does not point to a proper upstream SourceURL. We generally want to avoid packages in Fedora like this, since the upstream relationship is nonexistent. Cf. https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/ > rm -rf $RPM_BUILD_ROOT This can be removed, as RPM does this automatically. (In reply to Neal Gompa from comment #3) > So there are a couple of issues: > > > URL: https://github.com/weldr/lorax > > [..] > > Source0: lorax-templates-rhel-9.0-14.tar.gz > > The URL doesn't point to the actual project, and the Source0 does not point > to a proper upstream SourceURL. > This package provides only data files to support the lorax project. The URL is correct. This dist-git repository will be its own upstream. The tarball is generated by a Makefile included with the sources. I have added a comment to this effect into the specfile to avoid confusion in the future. > > rm -rf $RPM_BUILD_ROOT > > This can be removed, as RPM does this automatically. Done. I also updated the version to 10.0 since ELN is currently tracking towards that release. Spec URL: https://copr-be.cloud.fedoraproject.org/results/sgallagh/lorax-templates-rhel/fedora-eln-x86_64/02175846-lorax-templates-rhel/lorax-templates-rhel.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/sgallagh/lorax-templates-rhel/fedora-eln-x86_64/02175846-lorax-templates-rhel/lorax-templates-rhel-10.0-1.el110.src.rpm I strongly suggest that you make it so that the tarball isn't needed in the future. But for now, it looks good to me. PACKAGE APPROVED. (In reply to Neal Gompa from comment #5) > I strongly suggest that you make it so that the tarball isn't needed in the > future. But for now, it looks good to me. > > PACKAGE APPROVED. We discussed this on IRC. The tarball is actually generated from files in the dist-git repository. It's done with a tarball for simplicity, since having 90 files listed in Sources would be unwieldy. (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/lorax-templates-rhel Package is available in repositories, closing. |