From: cpebenito@tresys.com (Christopher J. PeBenito)
Date: Tue, 26 Jun 2012 10:04:13 -0400
Subject: [refpolicy] [PATCH v3 1/3] Support read/append/manage functions
 for various httpd content
In-Reply-To: <1340719037.12652.6.camel@x220.mydomain.internal>
References: <20120624110736.GA995@siphos.be> <20120624110826.GB995@siphos.be>
	<4FE9B7AD.6010707@tresys.com>
	<1340719037.12652.6.camel@x220.mydomain.internal>
Message-ID: <4FE9C15D.6050405@tresys.com>
To: refpolicy@oss.tresys.com
List-Id: refpolicy.oss.tresys.com

On 06/26/12 09:57, Dominick Grift wrote:
> 
>>> +########################################
>>> +## <summary>
>>> +##	Append to all appendable web content files.
>>> +## </summary>
>>> +## <param name="domain">
>>> +##	<summary>
>>> +##	Domain allowed access.
>>> +##	</summary>
>>> +## </param>
>>> +## <rolecap/>
>>> +#
>>> +interface(`apache_append_all_ra_content_files',`
>>
>> Inconsistent interface naming.  I'm ok w/o the _files.
>>
> 
> SwifT , i was the one who suggesting this change. I'm sorry about that.
> 
> I personally prefer this and do not agree with the inconsistency comment
> but i guess others do not agree with me.
> 
> I guess next time i suggest a similar change feel free to ignore it.

Look closely at the naming of all the additions.  Thats what I meant by inconsistency.  They should have the same pattern.

-- 
Chris PeBenito
Tresys Technology, LLC
www.tresys.com | oss.tresys.com