https://bugzilla.redhat.com/show_bug.cgi?id=1190055 Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #4 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> --- (In reply to Lubomir Rintel from comment #3) > > - Please add an appdata file > > [https://fedoraproject.org/wiki/Packaging:AppData]. > > I suggest you use appstream-util validate (w/o -relax) while developing it, > > but leave validate-relax in the spec file. > > Seems like it's SHOULD now. Given massif-visualizer targets hackers who are > likely able to install the tool in a different way than GNOME software I > don't believe it's worth patching the appdata file now. > > I've contacted upstream and asked them to add the appdata file, offering > help if they need any. Could we do without the appdata file now and assume > upstream is going to include it in a later version anyway? Makes sense. > SPEC: http://v3.sk/~lkundrak/SPECS/massif-visualizer.spec > SRPM: http://v3.sk/~lkundrak/SRPMS/massif-visualizer-0.4.0-3.fc21.src.rpm fedora-review has nothing interesting to say. The only thing is mime datebase snippets: they are different than those on http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#mimeinfo. Is this on purpose? Rpmlint ------- Checking: massif-visualizer-0.4.0-3.fc22.x86_64.rpm massif-visualizer-0.4.0-3.fc22.src.rpm massif-visualizer.x86_64: W: spelling-error Summary(en_US) profiler -> profile, profiles, profiled massif-visualizer.x86_64: W: spelling-error %description -l en_US pid -> peed, dip, pud massif-visualizer.x86_64: W: spelling-error %description -l en_US Gzip -> Zip, G zip, Grip massif-visualizer.x86_64: W: no-manual-page-for-binary massif-visualizer massif-visualizer.src: W: spelling-error Summary(en_US) profiler -> profile, profiles, profiled massif-visualizer.src: W: spelling-error %description -l en_US pid -> peed, dip, pud massif-visualizer.src: W: spelling-error %description -l en_US Gzip -> Zip, G zip, Grip 2 packages and 0 specfiles checked; 0 errors, 7 warnings. Package is APPROVED. -- 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