Bug 226481 - Merge Review: tcpdump
Merge Review: tcpdump
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: manuel wolfshant
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 16:09 EST by Nobody's working on this, feel free to take it
Modified: 2009-01-20 07:29 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2009-01-20 07:29:18 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
wolfy: fedora‑review+

Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 16:09:07 EST
Fedora Merge Review: tcpdump

Initial Owner: mlichvar@redhat.com
Comment 1 Daniel Kopeček 2007-02-26 09:51:50 EST
(!!) MUST: rpmlint output:
**** Review message:
E: tcpdump tag-not-utf8 %changelog
E: tcpdump non-utf8-spec-file tcpdump.spec
   - The spec file must use UTF-8 encoding

W: tcpdump macro-in-%changelog post
   - Line: 316 - disable /etc/init.d requirement and fix %post scripts in arpwatch
     Should be "%%post"

W: tcpdump macro-in-%changelog attr
   - Line: 493 - uses a buildroot and %attr
     Should be "%%attr"

Comment 2 Miroslav Lichvar 2007-03-15 08:22:19 EDT
Thanks, should be fixed in tcpdump-3.9.5-3.fc7.
Comment 3 manuel wolfshant 2009-01-10 17:50:36 EST
Just done a full check on tcpdump-14:3.9.8-6
The following aesthetic change will make rpmlint happier. Otherwise the package is just fine, no need to keep it on the list of merge reviews

< iconv -f iso-8859-1 -t utf-8 CREDITS > CREDITS.UTF-8

I can do the update in CVS myself if you want.

Comment 4 Miroslav Lichvar 2009-01-13 07:25:56 EST
Thanks for the review. I'll add the conversion in the next update.
Comment 5 manuel wolfshant 2009-01-13 09:25:41 EST
make sure to close this bug when you are done, please.

Note You need to log in before you can comment on or make changes to this bug.