Bug 192982 - Error in spec file -- wrong condition in `find' command
Summary: Error in spec file -- wrong condition in `find' command
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dave Jones
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-05-24 16:35 UTC by Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail}
Modified: 2015-01-04 22:27 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-25 19:10:10 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
this simply adds the missing parentheses as described in the `Additional info' section (422 bytes, patch)
2006-05-24 16:45 UTC, Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail}
no flags Details | Diff

Description Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail} 2006-05-24 16:35:47 UTC
Description of problem:

There's a `find' command call in the spec file, which takes care of deleting the
"garbage" left after applying the patches. This commands is supposed to delete
files matching the shell patterns `*.orig' or `*~'. This means files matching
the first *or* the second pattern. The way the condition is written, the `or'
became to `and', which obviously means no files get removed as these patterns
are mutually exclusive.


Version-Release number of selected component (if applicable):

2.6.16-1.2208_FC6 (kernel-2.6.16-1.2208_FC6)


How reproducible:

always


Steps to Reproduce:

1. Run e. g. `rpmbuild --target i686 -bp kernel-2.6.spec'.
2. Look for files matching `*.orig' -- you'll find several ones.

  
Actual results:

There's a lot of "garbage" (ie. files with the `.orig' suffix) left under the
build tree.


Expected results:

This "garbage" should have been deleted at the end of prep stage.


Additional info:

The format of `find' command has changed compared to CentOS (which I believe
corresponds to some older FC). Someone forgot to add parentheses around the `or'
part.

This bug may have existed for some time. Don't want to look into this furter, I
believe filing a bug against the latest devel SRPM is sufficient.

Patch will be provided immediately...

Comment 1 Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail} 2006-05-24 16:45:24 UTC
Created attachment 129951 [details]
this simply adds the missing parentheses as described in the `Additional info' section

Comment 2 Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail} 2006-05-24 16:56:36 UTC
In fact there's still a little garbage left after this fix: Under the build
tree, a  file `kernel-2.6.16/xen/common/sched_sedf.c.orig' still survives, but I
think this is because the `xen' subdirectory is not taken care of. So,
obviously, further cleanup might have to be implemented.


Comment 3 Dave Jones 2006-05-25 19:10:10 UTC
thanks, fixed in cvs, should be in tomorrows build.


Comment 4 Jiri TRAVNICEK, alias JITR {temporarily not reading bugmail} 2006-05-28 22:39:46 UTC
Was glad to help. :-)

In fact, I noticed this small bug while analysing kernel SRPMs for CentOS/RHEL,
for which I also included latest FC SRPM.


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