Bug 351404

Summary:	app-emulation/virtualbox-4.0.0-r1 needs sys-libs/pam
Product:	Gentoo Linux	Reporter:	Ben Kohler <bkohler>
Component:	Current packages	Assignee:	Lars Wendler (Polynomial-C) (RETIRED) <polynomial-c>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bonbons, patrick, swapon
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build log Patch to virtualbox-4.0.12.ebuild to make pam optional

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.