<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i><br></i></div><div>PR1349: <a href="https://github.com/mono/mono/pull/1349" target="_blank">https://github.com/mono/mono/pull/1349</a> <br><i>This is the machine key work, and needs a small tweak before it can be merged that I will do this week.<br></i></div></div></blockquote><div><br></div><div>I believe the TODO can be removed.   Can you do that?  See comments on pull request.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i></i></div><div>PR1363: <a href="https://github.com/mono/mono/pull/1363" target="_blank">https://github.com/mono/mono/pull/1363</a> <br><i>Another of mine with the MembershipPasswordAttribute<br></i></div></div></blockquote><div><br></div><div>Do you mind resending this?  It can no longer be auto-merged from the UI.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i></i></div><div>PR1365: <a href="https://github.com/mono/mono/pull/1365" target="_blank">https://github.com/mono/mono/pull/1365</a> <br><i>This is Kornel Pal's around the HttpTaskAsyncHandler, and Miguel has said he'll take a look at it.<br></i></div></div></blockquote><div><br></div><div>Manually aded</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i></i></div><div>PR1370: <a href="https://github.com/mono/mono/pull/1370" target="_blank">https://github.com/mono/mono/pull/1370</a> <br><i>Small one implementing a default of the ReadEntityBodyMode<br></i></div></div></blockquote><div><br></div><div>Got this one by hand.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i></i></div><div>PR1371: <a href="https://github.com/mono/mono/pull/1371" target="_blank">https://github.com/mono/mono/pull/1371</a><br></div><div><i>Another small one, implementing the ClientDisconnectedToken<br></i></div></div></blockquote><div><br></div><div>And this one automatically.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><i></i></div><div>PR1372: <a href="https://github.com/mono/mono/pull/1372" target="_blank">https://github.com/mono/mono/pull/1372</a><br></div><div><i>A final small one around the GetBuffer* methods in the httprequest.<br></i></div></div></blockquote><div><br></div><div>I do not like this one.  Is there a reason we can not implement the required functionality instead?</div><div><br></div><div>Miguel</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>There is 1 final small piece that either myself of Chris Carroll will get done this week which is around the AppendTrailing slash and lowercaseUrls properties in RouteBase class.  We just need to put the implementation together.<br><br>Anyway, after applying all of these, my large WebAPI solution not only compiles, but it also runs!<br><br></div><div>If you want to checkout what it looks like with all the patches applied, that would be great, I'd love to have some more information on whether it does work.  I'm sure there will still be bugs, but if it works mostly, then bug fixing is easy (famous last words).<br><br><a href="https://github.com/martinjt/mono/tree/mvc_allfixes" target="_blank">https://github.com/martinjt/mono/tree/mvc_allfixes</a><br><br></div><div>Thanks for everyone's help.<span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Martin</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 20 October 2014 20:42, Martin Thwaites <span dir="ltr"><<a href="mailto:monoforum@my2cents.co.uk" target="_blank">monoforum@my2cents.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Hi Miguel,</p>
<p dir="ltr">The code that I'm referring to here is that of the aspnetwebstack on codeplex.  That is to say that they are not something where you can remove the code and recompile (unless there as a specific mono implementation which is not ideal).  The goal is to have the compiled dlls that are available on nuget work, without tweaking to a person's application.  </p>
<p dir="ltr">I'll have a look and see if I can see where it would be used, but still as you've said on one of my pulls, a half done implementation is better than none.</p>
<p dir="ltr">Having the application throw a missing method exception should not be the recommended approach when we can add the property and default it to false.</p>
<p dir="ltr">Thanks, and please don't think that things won't getting better with my reviews.  I'm learning what you want so I can review better and help reduce the burden on you and your staff.</p><span><font color="#888888">
<p dir="ltr">Martin</p></font></span><div><div>
<div class="gmail_quote">On 20 Oct 2014 20:04, "Miguel de Icaza" <<a href="mailto:miguel@xamarin.com" target="_blank">miguel@xamarin.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">As for the properties, although they should do something to the generated urls, simply adding them should surely be a valid pull?  the issue at the moment is that without them, you get an exception even if it should be false.  I actually think that these are used by other classes when generating urls, not the route collection itself, but I don't know for sure.  Considering that adding them is very low risk, can we not just accept the pull and ask for further work.</p></blockquote><div>Nope, all they do is allow some code to be compiled, and then get the wrong result.</div><div><br></div><div>You might as well remove the dependency of those properties, and see what else breaks on whatever piece of code you are trying to build.</div><div><br></div><div>Miguel</div></div></div></div>
</blockquote></div>
</div></div></blockquote></div><br></div>
</div></div><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>
<br></blockquote></div><br></div></div>