Bug 235501
Summary: | Review Request: jsdoc - Produces javadoc-style documentation from JavaScript sourcefiles | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Matěj Cepl <mcepl> |
Component: | Package Review | Assignee: | Xavier Lamien <lxtnow> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | mcepl |
Target Milestone: | --- | Flags: | lxtnow:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 1.10.2-4.fc7 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-07-16 16:56:08 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
Matěj Cepl
2007-04-06 12:01:56 UTC
Currently there is a known problem with this package -- missing license. I am trying to get upstream to make some changes in that. The license issue resolved -- it is GPL. Updated SRPM URL: http://www.ceplovi.cz/matej/progs/rpms/jsdoc-1.10.2-2.src.rpm (of course, URL of the .spec file is the same) ping ? This package has some delay review, so i will Well, [ OK ] Mock built on x86_64 (F-7 and devel) [ FAILED ] Package is named according to the Package Naming Guidelines. [ FAILED ] Spec file name must match the base package %{name}. [ OK ] Package meets the Packaging Guidelines. [ OK ] Buildroot is correct (nothing to do). [ OK ] Package is licensed with an open-source compatible license. [ ? ] License field in the package spec file matches the actual license. [ ? ] License type: GPL [ FAILED ] License text is included in the package or If (and only if)the license is containing the text of the license(s) for the package is included in %doc. [ OK ] Spec file is legible and written in American English. [ OK ] Package has not binaries files and set to noarch package. [ OK ] BuildRequires aren't redundant (unneeded for this package). [ OK ] The spec file handles locales properly. [ OK ] Package must own all directories that it creates. [ OK ] Package requires other packages for directories it uses. [ Ok ] Package does not contain duplicates in %files. [ FAILED ] Permissions on files are set properly. [ OK ] Package has a %clean section. [ Ok ] Package consistently uses macros. [ OK ] Package contains code, or permissable content. [ OK ] Package does not contain any libtool archives (.la). [ FAILED ] Package does not own files or directories owned by other packages. == Rpmlint output: == [ OK ] From SRPM: silent. [ FAILED ] From RPM: E: jsdoc script-without-shebang /usr/share/jsdoc/index.tmpl E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JSDoc.pm E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JSDoc/xmi.tmpl E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JSDoc/XML.pm E: jsdoc script-without-shebang /usr/share/jsdoc/allclasses-noframe.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/overview-frame.tmpl E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JavaScript/Syntax/HTML.pm E: jsdoc script-without-shebang /usr/share/jsdoc/allclasses-frame.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/overview-summary.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/main.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/help-doc.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/overview-tree.tmpl E: jsdoc wrong-script-end-of-line-encoding /usr/share/jsdoc/overview-tree.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/stylesheet.css E: jsdoc script-without-shebang /usr/share/jsdoc/constant-values.tmpl E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JSDoc/xml.tmpl E: jsdoc script-without-shebang /usr/share/jsdoc/index-all.tmpl E: jsdoc script-without-shebang /usr/lib/perl5/vendor_perl/5.8.8/JSDoc/XMI.pm === Issues === * Your package name SHOULD match the base package name which is JSDoc (with upperCase letters). So move your package name from jsdoc to JSDoc. Also your spec file name. * The license text seem not added in package and not included in other text file which are included in the package. * From %install stage: Permission on files actually not correctly set for script files (see rpmlint output above). Also keep time stamps on docs files. * From %files section Your installed directory "%{perl_vendorlib}"is not good, cause your package own other directories owned by orther installed packages which already owned them. So your installed path should looks like this: %{perl_vendorlib}/JSDoc %{perl_vendorlib}/JavaScript instead of %{perl_vendorlib}/* * rpmlint: If your files plan/are exacutable scripts, add shebang to the script otherwise remove executable from these files. .pm and .tpml files no need to be executable. =================================== Should be fixed. New packages are at http://www.ceplovi.cz/matej/progs/rpms/, specifically SRPM is http://www.ceplovi.cz/matej/progs/rpms/JSDoc-1.10.2-3.fc7.src.rpm and SPEC file http://www.ceplovi.cz/matej/progs/rpms/JSDoc.spec All right, all issues above have been fixed. Also silent of rpmlint output on both, srpm and rpm. ============ * APPROVED * ============ New Package CVS Request ======================= Package Name: JSDoc Short Description: Produces javadoc-style documentation from JavaScript sourcefiles Owners: mcepl Branches: FC-6 F-7 EL-4 EL-5 InitialCC: cvs done. JSDoc-1.10.2-4.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report. JSDoc-1.10.2-4.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. |