Bug 249929
Summary: | Review Request: dynagen - Dynamips Hypervisor Client | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Nigel Jones <dev> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED CANTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting, pingoufc4 |
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: | 2008-06-03 06:37:15 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: | 246150 | ||
Bug Blocks: |
Description
Nigel Jones
2007-07-28 05:13:52 UTC
Hi This is my first review so I will do my best but I might need help :-) At a first look: you should use install rather than cp to install the files you can change %{_mandir}/man1/dynagen.1.gz by %{name}.1.gz I am surprise by your %{_bindir}/* in the %files section I will have a further look later on. (In reply to comment #1) > Hi > This is my first review so I will do my best but I might need help :-) > At a first look: > you should use install rather than cp to install the files As there is no make, it doesn't _really_ matter, and there isn't really any advantages either way that I'm aware of (although I'll most likely change it later on). > you can change %{_mandir}/man1/dynagen.1.gz by %{name}.1.gz I won't do this, as it breaks the logical layout of the files section, (i.e. by removing all the context), plus it's not consistent with other packages (nothing personal, just a style thing). > I am surprise by your %{_bindir}/* in the %files section As there isn't a make install (where by the contents of bindir may change without warning), and instead I'm explicatly copying the two executable scripts, it's not too much of an issue in comparison to other packages, I will look at changing if there is anything major wrong. > I will have a further look later on. Please look at bug 246150 (dynamips which dynagen depends on) before reviewing this package. First complains here : $ rpmlint ../RPMS/noarch/dynagen-0.9.3-1.fc6.noarch.rpm E: dynagen non-executable-script /usr/lib/python2.4/site-packages/console.py 0644 W: dynagen spurious-executable-perm /usr/share/doc/dynagen-0.9.3/sample_labs/all_config_options.txt W: dynagen wrong-file-end-of-line-encoding /usr/share/doc/dynagen-0.9.3/README.txt E: dynagen non-executable-script /usr/lib/python2.4/site-packages/dynamips_lib.py 0644 E: dynagen wrong-script-end-of-line-encoding /usr/lib/python2.4/site-packages/dynamips_lib.py W: dynagen hidden-file-or-dir /usr/share/doc/dynagen-0.9.3/sample_labs/.DS_Store E: dynagen script-without-shebang /usr/bin/configspec W: dynagen wrong-file-end-of-line-encoding /usr/share/doc/dynagen-0.9.3/COPYING W: dynagen wrong-file-end-of-line-encoding /usr/share/doc/dynagen-0.9.3/docs/tutorial.htm About time I closed this, doesn't meet entry requirements - closing. |