Bug 453761

Summary:	FreeRADIUS %post should not include chown -R
Product:	[Fedora] Fedora	Reporter:	Jos Vos <jos>
Component:	freeradius	Assignee:	John Dennis <jdennis>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	9	CC:	jdennis
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	freeradius-2.0.5-2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-07-30 14:18:17 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jos Vos 2008-07-02 13:02:22 UTC

Description of problem:
Shouldn't the recursive chown in %post, conflicting with the specified
permissions in %files (!), be replaced by proper %attr settings in %files or
otherwise be omitted at all?  AFAIK using chown in %post sections only can be
justified in very rare cases and this is not such a case, I think.

Version-Release number of selected component (if applicable):
2.0.5-1.fc9

Comment 1 John Dennis 2008-07-02 14:24:59 UTC

Thank you for your good eyes Jos, I absolutely agree. The upstream spec file for
2.x had this and I did a fair amount of cut-n-paste from that spec file when I
brought our version up to 2.x. Apparently I missed this operation in %post when
I was doing some cut-n-paste. I will fix it immediately.

Comment 2 John Dennis 2008-07-30 14:18:17 UTC

fixed in freeradius-2.0.5-2

Comment 3 Fedora Update System 2008-07-30 14:56:00 UTC

freeradius-2.0.5-2.fc9 has been submitted as an update for Fedora 9

Comment 4 Fedora Update System 2008-08-01 01:46:24 UTC

freeradius-2.0.5-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.