Hi, On Fri, Jun 12, 2009 at 11:59:47AM +0000, riggwelter@opensuse.org wrote:
GNOME:Contrib/gwibber -> openSUSE:Factory:Contrib/gwibber
https://build.opensuse.org/package/show?package=gwibber&project=GNOME:Contrib https://build.opensuse.org/package/show?package=gwibber&project=openSUSE:Factory:Contrib
Description: add gwibber
IMHO following two warnings are worth fixing: gwibber.x86_64: W: script-without-shebang /usr/share/gwibber/ui/themes/darkilouche/theme.html This text file has executable bits set or is located in a path dedicated for executables, but lacks a shebang and cannot thus be executed. If the file is meant to be an executable script, add the shebang, otherwise remove the executable bits or move the file elsewhere. gwibber.src: W: source-or-patch-not-bzipped gwibber-darkilouche.patch A source archive or patch in your package is not bzipped (doesn't have the .bz2 extension). Files bigger than 100k should be bzip2'ed in order to save space. To bzip2 a patch, use bzip2. To bzip2 a source tarball, use bznew Moreover, I think some comment about where the sources came from (not obvious in this case) would be helpful for both review and possible future contributors. Thanks, -- Best regards / s pozdravem Petr Uzel, Packages maintainer --------------------------------------------------------------------- SUSE LINUX, s.r.o. e-mail: puzel@suse.cz Lihovarská 1060/12 http://www.suse.cz 190 00 Prague 9 Czech Republic -- To unsubscribe, e-mail: opensuse-contrib+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-contrib+help@opensuse.org