Bug 226511 - Merge Review: unifdef
Summary: Merge Review: unifdef
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: David Woodhouse
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2007-01-31 21:12 UTC by Nobody's working on this, feel free to take it
Modified: 2008-12-28 23:34 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-12-28 23:34:07 UTC
Type: ---
wtogami: fedora-review+

Attachments (Terms of Use)

Description Nobody's working on this, feel free to take it 2007-01-31 21:12:54 UTC
Fedora Merge Review: unifdef

Initial Owner: dwmw2@redhat.com

Comment 1 Warren Togami 2007-02-01 19:07:27 UTC
APPROVED due to previous reviews in Bug #190362 and Bug #189937

(ASSIGNED to owner, deal with this only after the new buildsys is up)

Comment 2 Jonathan Underwood 2007-02-09 10:53:14 UTC
I wonder if you still think there's a need to include an unifdef package when
sunifdef is available currently in Extras (packaged by me) - sunifdef is
effectively upstream now, since unifdef hasn't been maintained for a while. 

"Sunifdef is a commandline tool for eliminating superfluous preprocessor clutter
from C and C++ source files. It is a more powerful successor to the FreeBSD
'unifdef' tool."


Comment 3 David Woodhouse 2007-02-09 11:18:17 UTC
No, I have no particular need for it since the kernel now includes its own copy
of unifdef; it doesn't use the external one any more.

Comment 4 Jonathan Underwood 2007-03-02 15:59:50 UTC
Well, since unifdef is largely unmaintained as far as I can tell, shall we drop
the package? Would you like to co-maintain sunifdef? :)

Comment 6 Robert Scheck 2008-12-28 23:34:07 UTC
David, if you don't care about this package, please orphan it according to
the Fedora Guidelines. Anyway, the package already got reviewed when reading
bug #190362 and bug #189937. Warren set the flag for this review request,
too - closing now.

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