Bug 226481 - Merge Review: tcpdump
Summary: Merge Review: tcpdump
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
(Show other bugs)
Version: rawhide
Hardware: All Linux
medium
medium
Target Milestone: ---
Assignee: manuel wolfshant
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Keywords:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-01-31 21:09 UTC by Nobody's working on this, feel free to take it
Modified: 2009-01-20 12:29 UTC (History)
3 users (show)

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


Attachments (Terms of Use)

Description Nobody's working on this, feel free to take it 2007-01-31 21:09:07 UTC
Fedora Merge Review: tcpdump

http://cvs.fedora.redhat.com/viewcvs/devel/tcpdump/
Initial Owner: mlichvar@redhat.com

Comment 1 Daniel Kopeček 2007-02-26 14:51:50 UTC
(!!) 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 12:22:19 UTC
Thanks, should be fixed in tcpdump-3.9.5-3.fc7.

Comment 3 manuel wolfshant 2009-01-10 22:50:36 UTC
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
< touch -r CREDITS CREDITS.UTF-8
< mv CREDITS.UTF-8 CREDITS

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

APPROVED

Comment 4 Miroslav Lichvar 2009-01-13 12:25:56 UTC
Thanks for the review. I'll add the conversion in the next update.

Comment 5 manuel wolfshant 2009-01-13 14:25:41 UTC
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.