590106 – sys-auth/munge: Prefix support

Bug 590106 - sys-auth/munge: Prefix support

Summary: sys-auth/munge: Prefix support

Status:	RESOLVED FIXED

Alias:	None

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

Importance:	Normal normal (vote)
Deadline:	2016-08-07
Assignee:	Gentoo Cluster Team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	prefix-gx86
	Show dependency tree

Reported:	2016-07-31 03:46 UTC by Benda Xu
Modified:	2016-08-02 09:45 UTC (History)
CC List:	1 user (show)

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

Attachments
munge-prefix.patch (munge-prefix.patch,1.49 KB, patch) 2016-07-31 03:46 UTC, Benda Xu	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 Benda Xu gentoo-dev

2016-07-31 03:46:04 UTC

Created attachment 442034 [details, diff]
munge-prefix.patch

Hi, this patch allows munge to be run on Prefix.  Does it look good to be committed?  Thanks!

Comment 1 Justin Bronder (RETIRED) gentoo-dev

2016-08-01 12:53:08 UTC

-	diropts -o munge -g munge -m700
+	[[ EUID == 0 ]] && diropts -o munge -g munge -m700

That should be ${EUID} right?

Otherwise, looks good to me, feel free to commit.

Comment 2 Benda Xu gentoo-dev

2016-08-02 09:45:49 UTC

(In reply to Justin Bronder from comment #1)
> -	diropts -o munge -g munge -m700
> +	[[ EUID == 0 ]] && diropts -o munge -g munge -m700
> 
> That should be ${EUID} right?

Oh yeah, sorry it's my mistake.
 
> Otherwise, looks good to me, feel free to commit.

Thanks, landed.

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=882586d1c9845709e6c