We're talking about removing whitespace, not obfuscating (e.g. base64-encoding) code. Removing whitespace is NOT "obfuscation". That suggestion is ludicrous.<div><br></div><div>Let us be quite clear on what we're discussing here.</div>
<div><br></div><div>This is "WP CSS Coding Standard compliant" CSS:</div><div><br></div><blockquote class="webkit-indent-blockquote" style="margin: 0 0 0 40px; border: none; padding: 0px;"><div><span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif; font-size: 12px; line-height: 22px; "><pre style="font-variant: normal !important; text-align: left !important; margin-top: 0px; margin-right: 0px; margin-bottom: 22px; margin-left: 0px; padding-top: 11px; padding-right: 11px; padding-bottom: 11px; padding-left: 11px; font-family: Consolas, Monaco, 'Courier New', Courier, monospace; background-image: initial; background-attachment: initial; background-origin: initial; background-clip: initial; background-color: rgb(240, 240, 240); border-top-width: 1px; border-right-width: 1px; border-bottom-width: 1px; border-left-width: 1px; border-top-style: solid; border-right-style: solid; border-bottom-style: solid; border-left-style: solid; border-top-color: rgb(218, 218, 218); border-right-color: rgb(218, 218, 218); border-bottom-color: rgb(218, 218, 218); border-left-color: rgb(218, 218, 218); font-size: 12px; line-height: 1.3em; overflow-x: auto; overflow-y: auto; color: rgb(0, 0, 0); font-weight: inherit; white-space: pre-wrap; word-wrap: break-word; background-position: initial initial; background-repeat: initial initial; ">
#selector-1,
#selector-2,
#selector-3 {
background: #fff;
color: #000;
}</pre></span></div></blockquote><div>And this is the "minified" version of that same CSS:</div><div><br></div><blockquote class="webkit-indent-blockquote" style="margin: 0 0 0 40px; border: none; padding: 0px;">
<div><span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif; font-size: 12px; line-height: 22px; "><pre style="font-variant: normal !important; text-align: left !important; margin-top: 0px; margin-right: 0px; margin-bottom: 22px; margin-left: 0px; padding-top: 11px; padding-right: 11px; padding-bottom: 11px; padding-left: 11px; font-family: Consolas, Monaco, 'Courier New', Courier, monospace; background-image: initial; background-attachment: initial; background-origin: initial; background-clip: initial; background-color: rgb(240, 240, 240); border-top-width: 1px; border-right-width: 1px; border-bottom-width: 1px; border-left-width: 1px; border-top-style: solid; border-right-style: solid; border-bottom-style: solid; border-left-style: solid; border-top-color: rgb(218, 218, 218); border-right-color: rgb(218, 218, 218); border-bottom-color: rgb(218, 218, 218); border-left-color: rgb(218, 218, 218); font-size: 12px; line-height: 1.3em; overflow-x: auto; overflow-y: auto; color: rgb(0, 0, 0); font-weight: inherit; white-space: pre-wrap; word-wrap: break-word; background-position: initial initial; background-repeat: initial initial; ">
#selector-1, #selector-2, #selector-3 { background: #fff; color: #000; }</pre></span></div></blockquote><div>Both are equally valid under the GPL. There is no GPL-compliance issue here, period.</div><div><br></div><div>Chip<br>
<br><div class="gmail_quote">On Fri, Feb 18, 2011 at 8:26 AM, Austin Matzko <span dir="ltr"><<a href="mailto:austin@pressedcode.com">austin@pressedcode.com</a>></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 Fri, Feb 18, 2011 at 8:03 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
> Come now, let us be reasonable.<br>
> The GPL requirement to provide code in the "preferred form of work for<br>
> making modifications" regards file formats, not the quality of the written<br>
> code. "Human readable" != "human parsable". The former is a function of the<br>
> data format; the latter is a function of the developer's coding ability and<br>
> adherence to standards. The GPL is concerned with the former, not the<br>
> latter.<br>
<br>
</div>Your interpretation does not match up with that of the Free Software<br>
Foundation, the authors of the GPL (and "principal organizational<br>
sponsor of the GNU Operating System"), who write, "Therefore,<br>
accessibility of source code is a necessary condition for free<br>
software. Obfuscated “source code” is not real source code and does<br>
not count as source code."<br>
<br>
<<a href="http://www.gnu.org/philosophy/free-sw.html" target="_blank">http://www.gnu.org/philosophy/free-sw.html</a>><br>
<div class="im"><br>
> Both the "minified" (style.css) and "WP CSS Coding Standards compliant"<br>
> (style.full.css) versions of the file are type text/CSS; thus, both are<br>
> *exactly the same format*. The "preferred form of work for making<br>
> modifications" to CSS files is text/css. Thus, as far as the GPL is<br>
> concerned, both meet this criterion.<br>
<br>
</div>Obfuscated code doesn't count as source, no matter the file type.<br>
That's why, for example, you can't claim obfuscated PHP or JavaScript<br>
files as source, even in a PHP text file (as opposed to an<br>
executable).<br>
<br>
For further reference, here's another prominent GPL project that<br>
spells out exactly what CSS source means:<br>
<<a href="http://drupal.org/licensing/faq/" target="_blank">http://drupal.org/licensing/faq/</a>><br>
<br>
"The GPL requires that I distribute the "source code" of my files.<br>
What does that mean for a web application?<br>
...<br>
"For CSS code, the CSS file itself, without any compression or<br>
obfuscation, is its own source code."<br>
<div><div></div><div class="h5">_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</div></div></blockquote></div><br></div>