<div dir="ltr"><div style>Okay, then this is fine for a push.</div><div style><br></div>Combining the two into a common templates directory is undesirable to me.  I don&#39;t want remediation to get in the way of delivering security guidance and checking content, so I&#39;d prefer to keep them separate if possible.  Using common CSV files is pretty good already with regard to maintaining linkage.  SSG will be ready for remediation content when the checks function properly and there is a well-defined process for contribution for those who know bash that minimizes their need to interact with the raw SCAP content.  This would include instructions for how to add new fix scripts, conduct unit tests (to include parameters), and instructions for using oscap to generate fixes.  These things are not there yet.<div>
<br></div><div><br><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 1, 2013 at 8:09 PM, Shawn Wells <span dir="ltr">&lt;<a href="mailto:shawn@redhat.com" target="_blank">shawn@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 6/1/13 7:01 PM, Jeffrey Blank wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
NACK.  This is mostly just a copy of a file from another directory.<br>
</blockquote></div>
What are thoughts on creating a combined bash+OVAL RHEL6/templates directory? If things can have template OVAL, they should e able to have template bash as well. Might as well put everything together.<div class="im"><br>

<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Why is there code to handle packages?<br>
<br>
</blockquote></div>
Not sure why I didn&#39;t see that. Deleted:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+        if packagename:<br>
+            filestring = filestring.replace(&quot;<u></u>PACKAGENAME&quot;, packagename)<br>
+        else:<br>
+            filestring = re.sub(&quot;\n\s*&lt;criteria.*&gt;\n\s*<u></u>&lt;extend_definition.*/&gt;&quot;, &quot;&quot;, filestring)<br>
+            filestring = re.sub(&quot;\s*&lt;/criteria&gt;\n\s*&lt;/<u></u>criteria&gt;&quot;, &quot;\n    &lt;/criteria&gt;&quot;, filestring)<br>
</blockquote><div class="HOEnZb"><div class="h5">
<br>
<br>
<br>
______________________________<u></u>_________________<br>
scap-security-guide mailing list<br>
<a href="mailto:scap-security-guide@lists.fedorahosted.org" target="_blank">scap-security-guide@lists.<u></u>fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/scap-security-guide" target="_blank">https://lists.fedorahosted.<u></u>org/mailman/listinfo/scap-<u></u>security-guide</a><br>
</div></div></blockquote></div><br></div>