Dave,<div><br></div><div>This looks good to me for now (until we support vcproj, if ever). I&#39;ll try and commit a variant of this with a warning to the console probably.</div><div><br></div><div>Thanks,</div><div>Jonathan<br>
<br><div class="gmail_quote">On Thu, May 14, 2009 at 12:35 PM, David Mitchell <span dir="ltr">&lt;<a href="mailto:dmitchell@logos.com">dmitchell@logos.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Does anyone have any feedback on this? Should I be submitting it elsewhere?<br>
<br>
--Dave<br>
<div><div></div><div class="h5"><br>
<br>
On 5/12/09 4:39 PM, &quot;David Mitchell&quot; &lt;<a href="mailto:dmitchell@logos.com">dmitchell@logos.com</a>&gt; wrote:<br>
<br>
&gt; Below is a small patch for xbuild that causes vcproj files to be ignored.<br>
&gt; Unless I&#39;m missing out on some huge secret, vcproj files aren&#39;t actually valid<br>
&gt; msbuild project files anyway, so ignoring them should do no harm (and will do<br>
&gt; a lot of good in the case of solutions that contain both csproj and vcproj<br>
&gt; files--provided that the csproj files have no dependencies on the vcproj<br>
&gt; files).<br>
&gt;<br>
&gt; I would appreciate it very much if someone would review/apply this patch. It<br>
&gt; is blocking my organization&#39;s adoption of xbuild as our build tool on<br>
&gt; Unix-based systems.<br>
&gt;<br>
&gt; I&#39;m providing this patch under the same license as xbuild itself.<br>
&gt;<br>
&gt; --Dave<br>
&gt;<br>
&gt; Index: SolutionParser.cs<br>
&gt; ===================================================================<br>
&gt; --- SolutionParser.cs    (revision 134016)<br>
&gt; +++ SolutionParser.cs    (working copy)<br>
&gt; @@ -100,6 +100,12 @@<br>
&gt;<br>
&gt;              Match m = projectRegex.Match (line);<br>
&gt;              while (m.Success) {<br>
&gt; +                if (m.Groups[3].Value.EndsWith(&quot;.vcproj&quot;)) {<br>
&gt; +                    // Ignore C++ projects<br>
&gt; +                    m = m.NextMatch ();<br>
&gt; +                    continue;<br>
&gt; +                }<br>
&gt; +<br>
&gt;                  ProjectInfo projectInfo = new ProjectInfo (m.Groups[2].Value,<br>
&gt; m.Groups[3].Value);<br>
&gt;                  if (String.Compare (m.Groups [1].Value, solutionFolderGuid,<br>
&gt;                          StringComparison.InvariantCultureIgnoreCase) == 0) {<br>
<br>
_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
</div></div></blockquote></div><br></div>