[Bug 902086] Review request: Elasticsearch

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

 



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



--- Comment #72 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
- Use %license for the license file

- Consider adding (from sigar):
# Use the same directory of the main package for subpackage licence and docs
%global _docdir_fmt %{name}

- %description needs to be extended to say what this package does.

- Consider using %autosetup

- Can the tests be enabled? If this package is as brittle as it is rumored to
be, tests would be beneficial.

I'd very much prefer to review the whole thing, including the .service file if
one is to be added.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]