Bug 123129 – Udev starts evms unconditionally

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Chris Gianelloni (RETIRED) 2006-02-16 19:53:00 UTC]

The udev-start.sh script starts evms unconditionally.  This is bad for any CD media, as it binds itself to any partitions and causes them to be in use.  The attached patch works around the issue by forcing evms_activate to not run if ${CDBOOT} is set.

[Comment 1 Chris Gianelloni (RETIRED) 2006-02-16 19:54:01 UTC]

Created attachment 79982 [details, diff]
udev-start.diff

Here is the patch...  If you have no objections, I would like to apply it.

[Comment 2 Chris Gianelloni (RETIRED) 2006-02-17 06:20:58 UTC]

Bleh... silly me... CDROOT should be CDBOOT...

[Comment 3 Greg Kroah-Hartman (RETIRED) 2006-02-17 10:41:08 UTC]

I don't have a problem if you want to fix this, and do an update with it.

I'll be making a 084 release later today, but 079 should probably be bumped
if you want this to be picked up by those users.

[Comment 4 Chris Gianelloni (RETIRED) 2006-02-17 11:27:02 UTC]

Sicne it only affects LiveCD builds and we've already made our snapshot for 2006.0, there's no need for a bump (unless you really want one) as this will have no effect on users installed systems.  So far, it looks like all udev versions 079+ are using the ${FILESDIR}/udev-start-079.sh, so this will fix all versions from the latest stable to the latest ~arch.

[Comment 5 Greg Kroah-Hartman (RETIRED) 2006-02-17 11:43:27 UTC]

Nope, I don't need a bump :)

So, can I close this?

[Comment 6 Chris Gianelloni (RETIRED) 2006-02-17 14:17:03 UTC]

Sure.  I'm committing it now.

[Comment 7 Greg Kroah-Hartman (RETIRED) 2006-02-18 17:08:03 UTC]

You broke the digests on the udev package when you did this!

[Comment 8 Chris Gianelloni (RETIRED) 2006-02-20 07:10:41 UTC]

WTF... I used repoman!

I apologize for breaking the digests.  I am honestly not sure how it happened considering all that was involved was editing a file and running it through a repoman commit.

[Comment 9 Greg Kroah-Hartman (RETIRED) 2006-02-21 12:16:57 UTC]

fixed now, right?