Bug 241079
| Summary: | Review Request: R-DynDoc-1.14.0 - Functions for dynamic documents | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Pierre-Yves <pingoufc4> |
| Component: | Package Review | Assignee: | Jason Tibbitts <j> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | splinux |
| Target Milestone: | --- | Flags: | j:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | 1.14.0-5.fc7 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-10-24 07:04:58 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: | 241082 | ||
|
Description
Pierre-Yves
2007-05-23 22:19:22 UTC
Based on the R packaging guidelines, there are the new files : SPEC: http://pingoured.dyndns.org/public/RPM/R-Dyndoc/R-DynDoc.spec SRPM: http://pingoured.dyndns.org/public/RPM/R-Dyndoc/R-DynDoc-1.14.0-3.fc6.src.rpm Thanks This doesn't build: No Package Found for R >= 2.5.1-2. There are the corrected files: SPEC: http://pingoured.dyndns.org/public/RPM/R-Dyndoc/ SRPM: http://pingoured.dyndns.org/public/RPM/R-Dyndoc/R-DynDoc-1.14.0-4.fc6.src.rpm Changed the license tag SPEC http://www.pingoured.fr/public/RPM/R-Dyndoc/R-DynDoc.spec SRPM http://www.pingoured.fr/public/RPM/R-Dyndoc/R-DynDoc-1.14.0-5.fc6.src.rpm Looks good to me and follows the R packaging guidelines closely. rpmlint is silent except for the expected one-line-command-in-* complaints. * source files match upstream: 0e95de3c2ea047e7c46655bf2c4751895ede720b179195cff0831d12146a71ba DynDoc_1.14.0.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * build root is OK. * license field matches the actual license. * license is open source-compatible. * license text not included upstream. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (development, x86_64). * package installs properly * rpmlint is silent. * final provides and requires are sane: R-DynDoc = 1.14.0-5.fc8 = /bin/sh R * %check is present and all tests pass. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (R package registration) * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. APPROVED New Package CVS Request ======================= Package Name: R-DynDoc Short Description: Functions for dynamic documents Owners: PierreYvesChibon Branches: FC-6 F-7 InitialCC: Cvsextras Commits: yes cvs done. R-DynDoc-1.14.0-5.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update R-DynDoc' Any reason not to close this ticket? Only the fact that this package is waiting to be pushed as stable for nearly two weeks... R-DynDoc-1.14.0-5.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. |