Hide Forgot
Fedora Merge Review: unifdef http://cvs.fedora.redhat.com/viewcvs/devel/unifdef/ Initial Owner: dwmw2@redhat.com
APPROVED due to previous reviews in Bug #190362 and Bug #189937 (ASSIGNED to owner, deal with this only after the new buildsys is up)
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." http://www.sunifdef.strudl.org/
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.
Well, since unifdef is largely unmaintained as far as I can tell, shall we drop the package? Would you like to co-maintain sunifdef? :)
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.