209025 – sys-apps/hal fails on an unconfigured kernel (again)

Bug 209025 - sys-apps/hal fails on an unconfigured kernel (again)

Summary: sys-apps/hal fails on an unconfigured kernel (again)

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	High blocker
Assignee:	Project Gentopia

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2008-02-05 19:25 UTC by Chris Gianelloni (RETIRED)
Modified:	2008-02-08 20:10 UTC (History)
CC List:	3 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
hal.patch (hal.patch,2.43 KB, patch) 2008-02-05 19:26 UTC, Chris Gianelloni (RETIRED)	Details \| Diff
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 Chris Gianelloni (RETIRED) gentoo-dev

2008-02-05 19:25:52 UTC

OK, guys.  Once again, hal is holding up a release build because someone has gone and made hal fail on an unconfigured kernel.

I'm attaching a patch to fix this problem.

Please include this patch immediately.  It causes no ill effects for users and fixes this issue for the release.  (Marking as blocker since it affects the release moving forward...)

Thanks,

Comment 1 Chris Gianelloni (RETIRED) gentoo-dev

2008-02-05 19:26:38 UTC

Created attachment 142743 [details, diff]
hal.patch

This patch makes the hal ebuild actually complete without a configured kernel sources.

Comment 2 Chris Gianelloni (RETIRED) gentoo-dev

2008-02-05 19:30:08 UTC

By the way, if there's no objections, I can go ahead and commit this and make sure to put it on all the versions in the tree.

Comment 3 Chris Gianelloni (RETIRED) gentoo-dev

2008-02-08 20:10:07 UTC

OK, I modified the patch a little, so it's more concise and slightly cleaner.  I've tested it on hal-0.5.9.1-r3 and hal-0.5.10, so I think that we're good to go on this.  There have been no complaints, so I'm going to go ahead and commit this to the tree, so we'll have it for the release.