Bug 198253

Summary: squid: don't chgrp another pkg's files/directory:
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: squidAssignee: Martin Stransky <stransky>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideKeywords: Patch
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-08-04 12:09:22 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:
Bug Depends On: 198251    
Bug Blocks:    
Attachments:
Description Flags
An expected changes for the spec file none

Description Rex Dieter 2006-07-10 16:27:07 UTC 
squid currently contains the scriptlet:
%triggerin -- samba-common
chgrp squid /var/cache/samba/winbindd_privileged > /dev/null 2>& 1 || true

I can understand the idea behind it, but it is way wrong to be changing (another
pkg's!) file/dir ownership this way.

I've outlined a proposal to avoid the need for such hackery, in bug #198251 ,
samba: make /var/cache/samba/windind_privledged group owned/writable.  Once
implemented in samba packaging, the same thing can be accomplished here by
instead by making the squid user (and/or group) be a member of the "samba" group.
Comment 1 Dmitry Butskoy 2006-08-03 15:47:10 UTC 
Created attachment 133562 [details]
An expected changes for the spec file

It also tries to fallback to the current "ugly" way if group"winbind" still not
exists.
Comment 2 Dmitry Butskoy 2006-08-03 15:48:30 UTC 
It would be better to implement this before FC6 time...
Comment 3 Martin Stransky 2006-08-04 12:09:22 UTC 
fixed in rawhide...