[Bug 1207847] Review Request: auto - A collection of source code generators for Java

https://bugzilla.redhat.com/show_bug.cgi?id=1207847 Raphael Groner <projects.rg(a)smart.ms&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |projects.rg(a)smart.ms --- Comment #2 from Raphael Groner <projects.rg(a)smart.ms&gt; --- This is an informal review of your spec file without any required modification. all is SHOULD only. * Please use a more specific name to avoid any confusion. Auto is too generically though it is the upstream name. Suggestion for a descriptive name: auto-code-generators https://fedoraproject.org/wiki/Packaging:NamingGuidelines * Use the %{url} macro to shorten the links. URL: https://github.com/google/%{name} Source0: %{url}/archive/auto-value-%{version}.tar.gz * Use the %{name} macro consistently: pleae replace all occurances of auto (or the final package name we chose) with the %{name} macro. * Be more specific in the summaries of the subpackages. Given one word only seems to be too few for a descriptive summary. If in doubt, you could use %{summary} then for the %description text to have the text there, too. * Are there no dependencies between the subpackages? Why is there a common subpackage? -- 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