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