Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: kadu - Gadu-Gadu client for online messaging https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199192 ------- Additional Comments From mr.ecik@xxxxxxxxx 2006-07-23 09:08 EST ------- First, thank for review! > o American English: OK (but there is a trivial typo: s/Develpoments/ Development/ > in %description devel and missing "by" at some "made by" in other > descriptions) Fixed. > - sources match upstream: not OK > http://www.kadu.net/download/snapshots/kadu-20060716.tar.bz2 doesn't exist > anymore Fixed; the address was changed. > comments: > ========= > > - build should include debugging symbols, e.g. the buildlog has: > > Compile with debug symbols: no > Please compile with debug symbols and let rpm automagically extract them out > into the debuginfo package. Fixed. > - permissions of shared libs should be executable to allow for debuginfo > extraction Fixed. > > - "Requires: libsndfile" is not neccessary (unless it needs something from > %{_bindir}/sndfile-* at runtime Require deleted. > - "mixed-use-of-spaces-and-tabs": Use emacs to tabbify/untabbify the specfile Now, it's OK. > - "%setup -D -T -n kadu" is missing "-q" Fixed. > - "unstripped-binary-or-object": Fix by making the modules executable either > with chmod at the end of %install of with %attr/%defattr Fixed. > - self-defined macros: In general I'm not against self-defined macros to make > packages more flexible or legible. In this case some macros make the specfile > harder to read than neccessary for example _kadudir being %{_datadir}/kadu. > I wouldn't call this a blocker, but I would recommend to unwrap most of these > macros to increase legibility. No, I won't unwrap these macros. This is still SVN pre-release, some things may change in future and macros make me easier to change path. > Please install rpmlint on your system and use it on the packages while > developing the specfile. I've already installed it, but most of your errors wasn't showed for me by rpmlint, do you use any arguments to it? I'll give new addresses for SRPM and SPEC in next comment. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review