[Bug 225660] Merge Review: crash

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=225660





--- Comment #8 from manuel wolfshant <wolfy at nobugconsulting.ro>  2009-03-07 21:58:32 EDT ---
It would have been nice if a mention of the existence of a new version in the
CVS + a link to the new spec and src.rpm would have been provided here. Back
where I come from this is called courtesy.

Current issues, as of version crash-4_0-8_7_2_fc11
1. 
$ rpmlint *rpm
crash.src: E: summary-too-long Kernel analysis utility for live systems,
netdump, diskdump, kdump, LKCD or mcore dumpfiles
crash.x86_64: W: spurious-executable-perm /usr/share/doc/crash-4.0/COPYING
crash.x86_64: E: summary-too-long Kernel analysis utility for live systems,
netdump, diskdump, kdump, LKCD or mcore dumpfiles
crash-devel.x86_64: W: no-documentation
crash-devel.x86_64: W: summary-not-capitalized kernel crash analysis utility
for live systems, netdump, diskdump, kdump, LKCD or mcore dumpfiles
crash-devel.x86_64: E: summary-too-long kernel crash analysis utility for live
systems, netdump, diskdump, kdump, LKCD or mcore dumpfiles
3 packages and 0 specfiles checked; 3 errors, 3 warnings.
The no documentation for -devel is ignorable. The error messages are easy
fixable by shortening the summary lines.

2. Source should be "http://people.redhat.com/anderson/crash-4.0-7.7.tar.gz"
(full URL to the source should be provided, not just a filename)

3. ppc is excluded from the list of architectures. However, ppc64 is included
(and koji builds happily for it) so I have to ask: is excluding ppc really
intended or just an oversight?

4. The package uses its private copies of several programs (gdb, binutils,
readline) without providing any reason for that. At least a note should be
included in the spec, explaining the reason. And I think that an exception must
be requested from the FPC.

5. The package ignores the compiler flags which are mandatory according to the
guidelines. Once again, no reason is provided. On top of that, the flags are
not consistently used (gdb gets " -c -DHAVE_CONFIG_H -g -O2 -I. -I./../include 
-W -Wall -Wtraditional -pedantic", readline gets " -c -DHAVE_CONFIG_H   -I. -I.
-DRL_LIBRARY_VERSION='"4.3"' -g -O2",  while crash itself is built using  "gcc
-c -g -O2    -I. -I. -I./config -DLOCALEDIR="\"/usr/local/share/locale\""
-DCRASH_MERGE -DHAVE_CONFIG_H -I./../include/opcode -I./../readline/.. -I../bfd
-I./../bfd -I./../include -I../intl -I./../intl  -DMI_OUT=1 -DTUI=1 -Wimplicit
-Wreturn-type -Wcomment -Wtrigraphs -Wformat -Wparentheses -Wpointer-arith
-Wuninitialized -Wformat-nonliteral -Wunused-label -Wunused-function "

6. Parallel make
(https://fedoraproject.org/wiki/Packaging/Guidelines#Parallel_make ) is not
used. Once again, no reason provided.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.