[Bug 226422] Merge Review: slrn

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: slrn


https://bugzilla.redhat.com/show_bug.cgi?id=226422





------- Additional Comments From panemade at gmail.com  2008-01-29 07:27 EST -------
(In reply to comment #5)
> Ok, converting the changelog and renaming the logrotate file is easy.
 Ok.

> 
> As for the permissions, I'm not really sure.
> 
> - /var/spool/slrnpull needs to be writable by news group.
  Good to keep it then.

> 
> - /usr/bin/slrnpull doesn't seem to need the setgid (it's supposed to be
> executed under user news). 755 should be enough and it will also allow users to
> run slrnpull.

  I leave this upto you. 

> 
> - /var/spool/slrnpull/out.going needs to be writable by users allowed to post
> articles which is now everyone. The directory needs to have setgid news to allow
> slrnpull to remove the files.
  Good to keep it then.

When you update CVS, do comment here for review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.