https://bugzilla.redhat.com/show_bug.cgi?id=1190055 --- Comment #3 from Lubomir Rintel <lkundrak@xxxxx> --- Thank you (In reply to Zbigniew Jędrzejewski-Szmek from comment #2) > Issues: > ======= > - update-desktop-database is invoked in %post and %postun if package contains > desktop file(s) with a MimeType: entry. > Note: desktop file(s) with MimeType entry in massif-visualizer > See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#desktop- > database Fixed. > - The license is GPLv2+. The sources might be mixed license, but the > resulting binary can only be under GPL, so there's no need to make things > complicated. Fixed. > - 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? SPEC: http://v3.sk/~lkundrak/SPECS/massif-visualizer.spec SRPM: http://v3.sk/~lkundrak/SRPMS/massif-visualizer-0.4.0-3.fc21.src.rpm -- 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