Bug 189937 - Review Request: unifdef
Review Request: unifdef
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: David Cantrell
Fedora Extras Quality Assurance
:
Depends On:
Blocks: FC-ACCEPT 190362
  Show dependency treegraph
 
Reported: 2006-04-25 17:58 EDT by David Woodhouse
Modified: 2013-01-09 20:23 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2006-04-26 16:16:21 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)

  None (edit)
Description David Woodhouse 2006-04-25 17:58:01 EDT
Spec URL: http://david.woodhou.se/unifdef.spec
SRPM URL: http://david.woodhou.se/unifdef-1.171-1.src.rpm
Description: 
Unifdef is useful for removing ifdefed lines from a file while otherwise
leaving the file alone. Unifdef acts on #ifdef, #ifndef, #else, and #endif
lines, and it knows only enough about C and C++ to know when one of these
is inactive because it is inside a comment, or a single or double quote.
Comment 1 David Woodhouse 2006-04-25 18:10:04 EDT
The plan is to use this when building the kernel package, which is why it's
proposed for Core, not Extras.
Comment 2 Jesse Keating 2006-04-25 18:10:46 EDT
I'll review this.
Comment 3 Jesse Keating 2006-04-26 10:31:36 EDT
Bad: 
- Source should be a URL to the source
- build root should be
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

Good:
- rpmlint is silent.
- builds in mock for i386 / x86_64

Question:  How usable is this for kernel building right now?  There is some
pushback to not have it in Core until it is actually usable/needed to build the
kernel and that Extras would be a good place to put it until such time as it is
needed.  Your thoughts?
Comment 4 David Woodhouse 2006-04-26 10:58:19 EDT
The source was obtained from FreeBSD's cvsweb. You can have a CVS pserver URL,
or you can have the name of the tarball I prepared. I prefer the latter.

Fixed buildroot in 1.171-2


Comment 5 David Woodhouse 2006-04-26 11:01:39 EDT
I committed a change to the rawhide kernel this morning which uses the new 'make
headers_install' rules to generate kernel headers from the kernel tree. It's
conditional, and turned off right now because it BuildRequires unifdef, which we
don't have yet.
Comment 6 Jesse Keating 2006-04-26 11:11:20 EDT
Ah, good point about the source issue.  Everything else looks good now, passes
review.  Please follow the next steps on the PackageListProcess.
Comment 7 David Woodhouse 2006-04-26 16:16:21 EDT
Built in rawhide. Thanks.

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