Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 351404 - app-emulation/virtualbox-4.0.0-r1 needs sys-libs/pam
Summary: app-emulation/virtualbox-4.0.0-r1 needs sys-libs/pam
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-12 02:46 UTC by Ben Kohler
Modified: 2011-07-24 00:28 UTC (History)
3 users (show)

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

Attachments
build log (build.log,139.77 KB, text/plain)
2011-01-12 02:47 UTC, Ben Kohler 		Details
Patch to virtualbox-4.0.12.ebuild to make pam optional (virtualbox-4.0.12.patch,1.14 KB, patch)
2011-07-23 23:28 UTC, Bruno 		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 Ben Kohler gentoo-dev 2011-01-12 02:46:23 UTC 
I'm installing app-emulation/virtualbox-4.0.0-r1 on a machine that has USE="-pam" set globally, so sys-libs/pam is not installed.  The vbox build seems to fail due to a missing pam header.

Reproducible: Always

Steps to Reproduce:
1.  "emerge =virtualbox-4.0.0-r1" on a machine without pam

Actual Results:  
/var/tmp/portage/app-emulation/virtualbox-4.0.0-r1/work/VirtualBox-4.0.0_OSE/src/VBox/HostServices/auth/pam/VBoxAuthPAM.c:77:31: fatal error: security/pam_appl.h: No such file or directory

Expected Results:  
successful build
Comment 1 Ben Kohler gentoo-dev 2011-01-12 02:47:18 UTC 
Created attachment 259595 [details]
build log
Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2011-01-12 19:58:48 UTC 
+  12 Jan 2011; Lars Wendler <polynomial-c@gentoo.org>
+  virtualbox-4.0.0-r1.ebuild:
+  Added sys-libs/pam to build dependencies. Thanks to Ben Kohler who reported
+  this in bug #351404.
Comment 3 Bruno 2011-07-23 23:28:06 UTC 
Created attachment 280781 [details, diff]
Patch to virtualbox-4.0.12.ebuild to make pam optional

An alternative to mandatory dependency on PAM is to make it use-based and disabled PAM authentication from virtualbox is PAM is not wanted.

The attached patch makes PAM optional for virtualbox-4.0.12.
Comment 4 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2011-07-24 00:28:04 UTC 
+  24 Jul 2011; Lars Wendler <polynomial-c@gentoo.org> virtualbox-4.0.10.ebuild,
+  virtualbox-4.0.12.ebuild, virtualbox-4.1.0.ebuild:
+  Made pam support optional (bug #351404).

Thanks for the patch but please next time create a new bug report. Posting such things in a bug closed as fixed can sometimes be overseen by us package maintainers.