On 9/14/09 3:33 PM, "Stephen Smalley" <sds@xxxxxxxxxxxxx> wrote: > On Mon, 2009-09-14 at 15:07 -0400, Joshua Brindle wrote: >> On 09/14/2009 09:22 AM, pjnuzzi wrote: >>> Add support for multiple target OSes to libsepol and checkpolicy, where >>> <snip> >>> Solaris (e.g. "SolFlask") will be introduced later using this new target >>> mechanism if needed. The same support can easily be extended for other >>> OSes. > >> This is a fairly large patch, it might take a while to get through it. >> Splitting it up would have been helpful. > > Sorry - based on Chad's earlier comment in response to another patch > that a complete single patch against the selinux tree was fine, and > given that this does form a single logical change against a single > repository, I recommended keeping it as a single patch. > Sorry if I created confusion. I was just meaning splitting on component (libsepol, checkpolicy, etc.) was unnecessary. Splitting into logical pieces is still preferred, and I have a hard time believing that a patch this large could not be split into logical pieces. Thanks, Chad -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
