Bug 154385 - build: clean up /boot/efi error messages on rpm removal
Summary: build: clean up /boot/efi error messages on rpm removal
Status: RESOLVED FIXED
Alias: None
Product: SUSE Linux 10.1
Classification: openSUSE
Component: Other (show other bugs)
Version: Beta 5
Hardware: All Other
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Michael Schröder
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-01 15:08 UTC by Greg Edwards
Modified: 2006-04-21 10:51 UTC (History)
3 users (show)

See Also:
Found By: Third Party Developer/Partner
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
create /boot/efi on ia64 during build root setup (787 bytes, patch)
2006-03-01 15:08 UTC, Greg Edwards 		Details | Diff
ignore failed stat on rpm removal (2.20 KB, patch)
2006-04-13 22:02 UTC, Greg Edwards 		Details | Diff
fixed patch header (1.82 KB, patch)
2006-04-13 22:06 UTC, Greg Edwards 		Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Edwards 2006-03-01 15:08:04 UTC 
On ia64, we see lots of error messages trying to stat /boot/efi when the
build script is installing/removing rpms from the build root, e.g.

installing xorg-x11-libs-6.9.0-9
error: failed to stat /boot/efi: No such file or directory
installing xorg-x11-devel-6.9.0-9
error: failed to stat /boot/efi: No such file or directory
installing libstdc++-devel-4.1.0_20060123-5
error: failed to stat /boot/efi: No such file or directory
installing gcc-c++-4.1.0_20060123-5
error: failed to stat /boot/efi: No such file or directory

rpm has a habit of liking to stat the mount points when it installs/removes\
rpms.  Creating the /boot/efi directory inside the build root during
the setup phase takes care of it.

Patch against build-2006.2.8-3 from factory.
Comment 1 Greg Edwards 2006-03-01 15:08:37 UTC 
Created attachment 70814 [details]
create /boot/efi on ia64 during build root setup
Comment 2 Michael Schröder 2006-03-01 15:15:50 UTC 
Creating all the directories in /proc/mounts is IMHO the wrong approach (it's not just /boot/efi, rpm only reports the first error). Instead build should be called with '--ignoresize'.
Comment 3 Greg Edwards 2006-03-01 16:15:57 UTC 
Yeah, I agree.  I didn't know rpm had that option.  Thanks for pointing it out.
Comment 4 Michael Schröder 2006-03-28 12:20:48 UTC 
We now use this option.
Comment 5 Greg Edwards 2006-04-03 20:31:00 UTC 
I'm still seeing this with the latest version in factory (build-2006.3.27-2),
or is there a newer version this is fixed in?
Comment 6 Michael Schröder 2006-04-04 10:58:29 UTC 
Do you see it when packages are installed or when packages are removed? Standard rpm only supports the "--ignoresize" option for package installation, not removal.
Comment 7 Greg Edwards 2006-04-04 13:57:40 UTC 
It's on package removal, for example, reusing the build root for the next
package build.  It cleans out the rpms no longer required.  I'm pretty sure
I tested adding --ignoresize to rpm -e lines and it worked.  I'll test again
this morning.
Comment 8 Greg Edwards 2006-04-04 13:58:31 UTC 
Reopening.
Comment 9 Michael Schröder 2006-04-04 14:07:58 UTC 
Yes, it works 'cause I patched the code. The problem is that it doesn't work with the rpm of the older distributions...
Comment 10 Greg Edwards 2006-04-04 14:10:56 UTC 
Ah, ok.  Thanks.
Comment 11 Greg Edwards 2006-04-13 22:02:56 UTC 
Created attachment 78412 [details]
ignore failed stat on rpm removal

How about this instead of adding --ignoresize to the
rpm -e lines?
Comment 12 Greg Edwards 2006-04-13 22:06:16 UTC 
Created attachment 78413 [details]
fixed patch header
Comment 13 Michael Schröder 2006-04-18 10:28:37 UTC 
Looks sane. I'll apply it.
Comment 15 Michael Schröder 2006-04-21 10:51:41 UTC 
Fixed in STABLE.