[Code Review] Custom PAL fixes

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[Code Review] Custom PAL fixes

Jimmy Schementi-3
Just checked in some small fixes to make the Importer use a custom Platform Adaptation Layer. I also started to tweak clr.CompileModules a bit, but backed those changes out, so just ignore those.

Take a look at the recent commits to https://github.com/jschementi/iron. If there are no objections, I'll push them to main.

By the way, is it about time to split this list into a development and users list, or you all don't mind seeing these type of mails?

~Jimmy

_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Dino Viehland

LGTM.

 

From: [hidden email] [mailto:[hidden email]] On Behalf Of Jimmy Schementi
Sent: Thursday, March 31, 2011 6:06 PM
To: Discussion of IronPython
Subject: [IronPython] [Code Review] Custom PAL fixes

 

Just checked in some small fixes to make the Importer use a custom Platform Adaptation Layer. I also started to tweak clr.CompileModules a bit, but backed those changes out, so just ignore those.

 

Take a look at the recent commits to https://github.com/jschementi/iron. If there are no objections, I'll push them to main.

 

By the way, is it about time to split this list into a development and users list, or you all don't mind seeing these type of mails?


~Jimmy


_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

haniti grk

Please continue using this mailing list. Knowing the innards of IronPython does not hurt at all.

Besides the traffic here is not too much.
On Mar 31, 2011 8:16 PM, "Dino Viehland" <[hidden email]> wrote:
>
> LGTM.
>
>  
>
> From: [hidden email] [mailto:[hidden email]] On Behalf Of Jimmy Schementi
> Sent: Thursday, March 31, 2011 6:06 PM
> To: Discussion of IronPython
> Subject: [IronPython] [Code Review] Custom PAL fixes
>
>  
>
> Just checked in some small fixes to make the Importer use a custom Platform Adaptation Layer. I also started to tweak clr.CompileModules a bit, but backed those changes out, so just ignore those.
>
>  
>
> Take a look at the recent commits to https://github.com/jschementi/iron. If there are no objections, I'll push them to main.
>
>  
>
> By the way, is it about time to split this list into a development and users list, or you all don't mind seeing these type of mails?
>
>
> ~Jimmy
>
>
> _______________________________________________
> Users mailing list
> [hidden email]
> http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
>


_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Bruce Bromberek
In reply to this post by Jimmy Schementi-3
As a non developer user I like all of the detailed discussions. It
helps me understand what is going on behind the scenes. And gives me
better C# search terms.

On 3/31/11, Jimmy Schementi <[hidden email]> wrote:

> Just checked in some small fixes to make the Importer use a custom Platform
> Adaptation Layer. I also started to tweak clr.CompileModules a bit, but
> backed those changes out, so just ignore those.
>
> Take a look at the recent commits to https://github.com/jschementi/iron. If
> there are no objections, I'll push them to main.
>
> By the way, is it about time to split this list into a development and users
> list, or you all don't mind seeing these type of mails?
>
> ~Jimmy
>

--
Sent from my mobile device
_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Dave Wald
In reply to this post by Jimmy Schementi-3
I don't mind a bit.  I like having a sense of what's going on.

On 3/31/2011 8:05 PM, Jimmy Schementi wrote:
Just checked in some small fixes to make the Importer use a custom Platform Adaptation Layer. I also started to tweak clr.CompileModules a bit, but backed those changes out, so just ignore those.

Take a look at the recent commits to https://github.com/jschementi/iron. If there are no objections, I'll push them to main.

By the way, is it about time to split this list into a development and users list, or you all don't mind seeing these type of mails?

~Jimmy
_______________________________________________ Users mailing list [hidden email] http://lists.ironpython.com/listinfo.cgi/users-ironpython.com


_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Doug Blank
In reply to this post by Jimmy Schementi-3
On Thu, Mar 31, 2011 at 9:05 PM, Jimmy Schementi <[hidden email]> wrote:
> By the way, is it about time to split this list into a development and users
> list, or you all don't mind seeing these type of mails?

Many of the people subscribed want to get any insight into the DLR
languages as they can. But I have noticed that there are some
user-type questions that get no responses here.

It is easy to join two mailing lists, and if splitting would allow
user-questions to get more attention, then that would be a good thing.

-Doug

> ~Jimmy
_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Jeff Hardy-4
In reply to this post by Jimmy Schementi-3
On Thu, Mar 31, 2011 at 7:05 PM, Jimmy Schementi <[hidden email]> wrote:
> Just checked in some small fixes to make the Importer use a custom Platform
> Adaptation Layer. I also started to tweak clr.CompileModules a bit, but
> backed those changes out, so just ignore those.
> Take a look at the recent commits to https://github.com/jschementi/iron. If
> there are no objections, I'll push them to main.

If Dino's happy, that's good enough for me.

Should these changes be in the 2.7 branch as well, or just master
(3.0)? It doesn't look like there are any API changes, so applying it
to 2.7 should be fine.

- Jeff
_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
Reply | Threaded
Open this post in threaded view
|

Re: [Code Review] Custom PAL fixes

Markus Schaber-6
Hi, Jimmy,

> Von: Jeff Hardy
> On Thu, Mar 31, 2011 at 7:05 PM, Jimmy Schementi <[hidden email]>
> wrote:
> > Just checked in some small fixes to make the Importer use a custom
> > Platform Adaptation Layer. I also started to tweak
clr.CompileModules
> > a bit, but backed those changes out, so just ignore those.
> > Take a look at the recent commits to
> > https://github.com/jschementi/iron. If there are no objections, I'll
> push them to main.
>
> If Dino's happy, that's good enough for me.
>
> Should these changes be in the 2.7 branch as well, or just master
(3.0)?
> It doesn't look like there are any API changes, so applying it to 2.7
> should be fine.

I'd like to see those changes and fixes in version 2.7, for all those
people who cannot yet migrate to .NET 4.0.

Thanks,
Markus
_______________________________________________
Users mailing list
[hidden email]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com