Bug 154385

Summary:	build: clean up /boot/efi error messages on rpm removal
Product:	[openSUSE] SUSE Linux 10.1	Reporter:	Greg Edwards <edwardsg>
Component:	Other	Assignee:	Michael Schröder <mls>
Status:	RESOLVED FIXED	QA Contact:	E-mail List <qa-bugs>
Severity:	Normal
Priority:	P5 - None	CC:	gp, sbahling, susedev
Version:	Beta 5
Target Milestone:	---
Hardware:	All
OS:	Other
See Also:	http://bugworks.engr.sgi.com/query.cgi/949800
Whiteboard:
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 ignore failed stat on rpm removal fixed patch header

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.