[Bug 1430364] Review Request: argbash - Bash argument parsing code generator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1430364

Matthew Smith <asymptotically508@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |asymptotically508@xxxxxxxxx



--- Comment #2 from Matthew Smith <asymptotically508@xxxxxxxxx> ---
Just to start with, this is an informal review and I am not a packager.

>srpm/argbash.spec:48: E: hardcoded-library-path in %{_prefix}/lib/argbash/

I think that %{_prefix}/lib/argbash/ should be changed to %{_libdir}/argbash

>argbash.noarch: W: no-manual-page-for-binary argbash
>argbash.noarch: W: no-manual-page-for-binary argbash-1to2
>argbash.noarch: W: no-manual-page-for-binary argbash-init

Additionally there are no manpages, however I don't think that this is a big
issue because of the built in help.

Other than that looks good to me :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux