Bug 878734
| Summary: | Review Request: xs-release-6-1 - XS repository configuration | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | kparmar4 |
| Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
| Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | anto.trande, christoph.wickert, misc, notting, package-review |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2012-11-23 11:29:12 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
kparmar4
2012-11-21 04:57:04 UTC
mmm... I already see some errors among first lines (release, source*, build section above all). Maybe you should review packaging guidelines: http://fedoraproject.org/wiki/Packaging:Guidelines Antonio, would you please be so kind as to outline the errors instead of making a very vague statement? (In reply to comment #2) > Antonio, would you please be so kind as to outline the errors instead of > making a very vague statement? Sure. :) I mistaken to define them "errors". This package should provide a repository configuration for yum, so it should be named xs-release, ok (I don't know if there are specific instructions in these cases). Release should be 1%{?dist} (first on Fedora) https://fedoraproject.org/wiki/Packaging:NamingGuidelines?rd=Packaging/NamingGuidelines#Release_Tag Source* should be an URL or at least say where come from. https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL 'Build' section ... Probably there is nothing to build but maybe he should write something to restate it. :) 'changelog' is outdated. Hi Antonio, Thank you for pointing out that. I would gladly edit the spec file to correct the mistakes and resubmit it for review again. I should point that I got this file from OLPC webpage provided to me by my instructors. This package cotains on repository configurations, which used by OLPC. Note that repository configuration for non-Fedora repositories isn't allowed in Fedora itself. The package certainly can be reviewed for adherence to other Fedora guidelines, and packaging correctness, but it can't be added to Fedora. After discussing with reporter on irc, I close this one, do a review of the 2nd one so he see what it look like, but the package will not be approved per comment #5 Second one being #879568 |