Merge lp://qastaging/~kiorky/ipython/ipython-readline into lp://qastaging/ipython/0.11
- ipython-readline
- Merge into trunk
Status: | Rejected |
---|---|
Rejected by: | Brian Granger |
Proposed branch: | lp://qastaging/~kiorky/ipython/ipython-readline |
Merge into: | lp://qastaging/ipython/0.11 |
Diff against target: |
22 lines 1 file modified
IPython/deathrow/ipy_profile_zope.py (+7/-0) |
To merge this branch: | bzr merge lp://qastaging/~kiorky/ipython/ipython-readline |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brian Granger | Needs Resubmitting | ||
Review via email:
|
Commit message
Description of the change
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
- 1217. By Mathieu PASQUET <email address hidden>
-
oups, stderr and stdin need to be restored as well
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
Could this be turned into an IPython extension that is shipped with zope?
Things that have ended up in deathrow are in two categories:
1) They are old, broken or otherwise decaying.
2) They should be distributed by themselves or with a third party project.
I think ipy_profile_zope.py belongs in the second category.
Thoughts?
Brian
> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
> lp:ipython.
>
> Requested reviews:
> IPython Developers (ipython-dev)
>
>
> see #443982 ;)
> --
> https:/
> You are subscribed to branch lp:ipython.
>
> === modified file 'IPython/
> --- IPython/
> +++ IPython/
> @@ -24,6 +24,8 @@
> import os
> import textwrap
>
> +sys.oldstdin = sys.stdin
> +
> # The import below effectively obsoletes your old-style ipythonrc[.ini],
> # so consider yourself warned!
> # import ipy_defaults
> @@ -314,6 +316,7 @@
> """ % ( ",".join([ x for x in dir(zope_
> x.startswith("_") ] ) ) )
>
>
> + sys.stdin = sys.oldstdin
> ip.user_ns.update( zope_debug.
>
>
>
>
>
--
Brian E. Granger, Ph.D.
Assistant Professor of Physics
Cal Poly State University, San Luis Obispo
<email address hidden>
<email address hidden>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
Brian Granger a écrit :
> Could this be turned into an IPython extension that is shipped with zope?
I don't think so, its place is great there.
>
> Things that have ended up in deathrow are in two categories:
>
> 1) They are old, broken or otherwise decaying.
Maybe it was just unmaintened, but you see, i got it to work nowodays without
much problems with the applied patch, i use it even now :)
> 2) They should be distributed by themselves or with a third party project.
>
> I think ipy_profile_zope.py belongs in the second category.
I never heard about zope related product shipping it.
>
> Thoughts?
>
> Brian
>
>
>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
>
>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
>> lp:ipython.
>>
>> Requested reviews:
>> IPython Developers (ipython-dev)
>>
>>
>> see #443982 ;)
>> --
>> https:/
>> You are subscribed to branch lp:ipython.
>>
>> === modified file 'IPython/
>> --- IPython/
>> +++ IPython/
>> @@ -24,6 +24,8 @@
>> import os
>> import textwrap
>>
>> +sys.oldstdin = sys.stdin
>> +
>> # The import below effectively obsoletes your old-style ipythonrc[.ini],
>> # so consider yourself warned!
>> # import ipy_defaults
>> @@ -314,6 +316,7 @@
>> """ % ( ",".join([ x for x in dir(zope_
>> x.startswith("_") ] ) ) )
>>
>>
>> + sys.stdin = sys.oldstdin
>> ip.user_ns.update( zope_debug.
>>
>>
>>
>>
>>
>
>
--
--
Cordialement,
KiOrKY
GPG Key FingerPrint: 0x1A1194B7681112AF
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
I am going to open up this discussion to the wider ipython dev list.
Can you join in there?
Brian
On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
>
> Brian Granger a écrit :
> > Could this be turned into an IPython extension that is shipped with zope?
>
> I don't think so, its place is great there.
>
> >
> > Things that have ended up in deathrow are in two categories:
> >
> > 1) They are old, broken or otherwise decaying.
>
> Maybe it was just unmaintened, but you see, i got it to work nowodays
> without
> much problems with the applied patch, i use it even now :)
>
> > 2) They should be distributed by themselves or with a third party
> project.
> >
> > I think ipy_profile_zope.py belongs in the second category.
>
> I never heard about zope related product shipping it.
>
> >
> > Thoughts?
> >
> > Brian
> >
> >
> >> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
> >
> >> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
> >> lp:ipython.
> >>
> >> Requested reviews:
> >> IPython Developers (ipython-dev)
> >>
> >>
> >> see #443982 ;)
> >> --
> >>
> https:/
> <
> https:/
> >
> >> You are subscribed to branch lp:ipython.
> >>
> >> === modified file 'IPython/
> >> --- IPython/
> +0000
> >> +++ IPython/
> +0000
> >> @@ -24,6 +24,8 @@
> >> import os
> >> import textwrap
> >>
> >> +sys.oldstdin = sys.stdin
> >> +
> >> # The import below effectively obsoletes your old-style
> ipythonrc[.ini],
> >> # so consider yourself warned!
> >> # import ipy_defaults
> >> @@ -314,6 +316,7 @@
> >> """ % ( ",".join([ x for x in dir(zope_
> >> x.startswith("_") ] ) ) )
> >>
> >>
> >> + sys.stdin = sys.oldstdin
> >> ip.user_ns.update( zope_debug.
> >>
> >>
> >>
> >>
> >>
> >
> >
>
> --
> --
> Cordialement,
> KiOrKY
> GPG Key FingerPrint: 0x1A1194B7681112AF
>
>
>
> https:/
> You are subscribed to branch lp:ipython.
>
--
Brian E. Granger, Ph.D.
Assistant Professor of Physics
Cal Poly State University, San Luis Obispo
<email address hidden>
<email address hidden>
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
sure.
Brian Granger a écrit :
> I am going to open up this discussion to the wider ipython dev list.
> Can you join in there?
>
> Brian
>
> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
>
>> Brian Granger a écrit :
>>> Could this be turned into an IPython extension that is shipped with zope?
>> I don't think so, its place is great there.
>>
>>> Things that have ended up in deathrow are in two categories:
>>>
>>> 1) They are old, broken or otherwise decaying.
>> Maybe it was just unmaintened, but you see, i got it to work nowodays
>> without
>> much problems with the applied patch, i use it even now :)
>>
>>> 2) They should be distributed by themselves or with a third party
>> project.
>>> I think ipy_profile_zope.py belongs in the second category.
>> I never heard about zope related product shipping it.
>>
>>> Thoughts?
>>>
>>> Brian
>>>
>>>
>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
>>>> lp:ipython.
>>>>
>>>> Requested reviews:
>>>> IPython Developers (ipython-dev)
>>>>
>>>>
>>>> see #443982 ;)
>>>> --
>>>>
>> https:/
>> <
>> https:/
>>>> You are subscribed to branch lp:ipython.
>>>>
>>>> === modified file 'IPython/
>>>> --- IPython/
>> +0000
>>>> +++ IPython/
>> +0000
>>>> @@ -24,6 +24,8 @@
>>>> import os
>>>> import textwrap
>>>>
>>>> +sys.oldstdin = sys.stdin
>>>> +
>>>> # The import below effectively obsoletes your old-style
>> ipythonrc[.ini],
>>>> # so consider yourself warned!
>>>> # import ipy_defaults
>>>> @@ -314,6 +316,7 @@
>>>> """ % ( ",".join([ x for x in dir(zope_
>>>> x.startswith("_") ] ) ) )
>>>>
>>>>
>>>> + sys.stdin = sys.oldstdin
>>>> ip.user_ns.update( zope_debug.
>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>> --
>> --
>> Cordialement,
>> KiOrKY
>> GPG Key FingerPrint: 0x1A1194B7681112AF
>>
>>
>>
>> https:/
>> You are subscribed to branch lp:ipython.
>>
>
>
>
--
--
Cordialement,
KiOrKY
GPG Key FingerPrint: 0x1A1194B7681112AF
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
One things I did realize, this is really a profile, and the profile API has
changed
substantially:
Also, are you working off of trunk?
Cheers,
Brian
On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
> sure.
>
> Brian Granger a écrit :
> > I am going to open up this discussion to the wider ipython dev list.
> > Can you join in there?
> >
> > Brian
> >
> > On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
> >
> >> Brian Granger a écrit :
> >>> Could this be turned into an IPython extension that is shipped with
> zope?
> >> I don't think so, its place is great there.
> >>
> >>> Things that have ended up in deathrow are in two categories:
> >>>
> >>> 1) They are old, broken or otherwise decaying.
> >> Maybe it was just unmaintened, but you see, i got it to work nowodays
> >> without
> >> much problems with the applied patch, i use it even now :)
> >>
> >>> 2) They should be distributed by themselves or with a third party
> >> project.
> >>> I think ipy_profile_zope.py belongs in the second category.
> >> I never heard about zope related product shipping it.
> >>
> >>> Thoughts?
> >>>
> >>> Brian
> >>>
> >>>
> >>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
> >>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
> >>>> lp:ipython.
> >>>>
> >>>> Requested reviews:
> >>>> IPython Developers (ipython-dev)
> >>>>
> >>>>
> >>>> see #443982 ;)
> >>>> --
> >>>>
> >>
> https:/
> <
> https:/
> >
> >> <
> >>
> https:/
> >>>> You are subscribed to branch lp:ipython.
> >>>>
> >>>> === modified file 'IPython/
> >>>> --- IPython/
> >> +0000
> >>>> +++ IPython/
> >> +0000
> >>>> @@ -24,6 +24,8 @@
> >>>> import os
> >>>> import textwrap
> >>>>
> >>>> +sys.oldstdin = sys.stdin
> >>>> +
> >>>> # The import below effectively obsoletes your old-style
> >> ipythonrc[.ini],
> >>>> # so consider yourself warned!
> >>>> # import ipy_defaults
> >>>> @@ -314,6 +316,7 @@
> >>>> """ % ( ",".join([ x for x in dir(zope_
> >>>> x.startswith("_") ] ) ) )
> >>>>
> >>>>
> >>>> + sys.stdin = sys.oldstdin
> >>>> ip.user_ns.update( zope_debug.
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>
> >> --
> >> --
> >> Cordialement,
> >> KiOrKY
> >> GPG Key FingerPrint: 0x1A1194B7681112AF
> >>
> >>
> >>
> >>
> https:/
> <
> https:/
> >
> >> You are subscribed to branch lp:ipython.
> >>
> >
> >
> >
>
> --
> --
> Cordialement,
> KiOrKY
> ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
I use it on 0.10 release.
Brian Granger a écrit :
> One things I did realize, this is really a profile, and the profile API has
> changed
> substantially:
>
> http://
>
> Also, are you working off of trunk?
>
> Cheers,
>
> Brian
>
> On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
>
>> sure.
>>
>> Brian Granger a écrit :
>>> I am going to open up this discussion to the wider ipython dev list.
>>> Can you join in there?
>>>
>>> Brian
>>>
>>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
>>>
>>>> Brian Granger a écrit :
>>>>> Could this be turned into an IPython extension that is shipped with
>> zope?
>>>> I don't think so, its place is great there.
>>>>
>>>>> Things that have ended up in deathrow are in two categories:
>>>>>
>>>>> 1) They are old, broken or otherwise decaying.
>>>> Maybe it was just unmaintened, but you see, i got it to work nowodays
>>>> without
>>>> much problems with the applied patch, i use it even now :)
>>>>
>>>>> 2) They should be distributed by themselves or with a third party
>>>> project.
>>>>> I think ipy_profile_zope.py belongs in the second category.
>>>> I never heard about zope related product shipping it.
>>>>
>>>>> Thoughts?
>>>>>
>>>>> Brian
>>>>>
>>>>>
>>>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden> wrote:
>>>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
>>>>>> lp:ipython.
>>>>>>
>>>>>> Requested reviews:
>>>>>> IPython Developers (ipython-dev)
>>>>>>
>>>>>>
>>>>>> see #443982 ;)
>>>>>> --
>>>>>>
>> https:/
>> <
>> https:/
>>>> <
>>>>
>> https:/
>>>>>> You are subscribed to branch lp:ipython.
>>>>>>
>>>>>> === modified file 'IPython/
>>>>>> --- IPython/
>>>> +0000
>>>>>> +++ IPython/
>>>> +0000
>>>>>> @@ -24,6 +24,8 @@
>>>>>> import os
>>>>>> import textwrap
>>>>>>
>>>>>> +sys.oldstdin = sys.stdin
>>>>>> +
>>>>>> # The import below effectively obsoletes your old-style
>>>> ipythonrc[.ini],
>>>>>> # so consider yourself warned!
>>>>>> # import ipy_defaults
>>>>>> @@ -314,6 +316,7 @@
>>>>>> """ % ( ",".join([ x for x in dir(zope_
>>>>>> x.startswith("_") ] ) ) )
>>>>>>
>>>>>>
>>>>>> + sys.stdin = sys.oldstdin
>>>>>> ip.user_ns.update( zope_debug.
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>> --
>>>> --
>>>> Cordialement,
>>>> KiOrKY
>>>> GPG Key FingerPrint: 0x1A1194B7681112AF
>>>>
>>>>
>>>>
>>>>
>> https:/
>> <
>> https:/
>>>> You are subscribed to bran...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you sure
you are working on the 0.10 branch?
Brian
On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
> I use it on 0.10 release.
>
> Brian Granger a écrit :
> > One things I did realize, this is really a profile, and the profile API
> has
> > changed
> > substantially:
> >
> >
> http://
> >
> > Also, are you working off of trunk?
> >
> > Cheers,
> >
> > Brian
> >
> > On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
> >
> >> sure.
> >>
> >> Brian Granger a écrit :
> >>> I am going to open up this discussion to the wider ipython dev list.
> >>> Can you join in there?
> >>>
> >>> Brian
> >>>
> >>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
> >>>
> >>>> Brian Granger a écrit :
> >>>>> Could this be turned into an IPython extension that is shipped with
> >> zope?
> >>>> I don't think so, its place is great there.
> >>>>
> >>>>> Things that have ended up in deathrow are in two categories:
> >>>>>
> >>>>> 1) They are old, broken or otherwise decaying.
> >>>> Maybe it was just unmaintened, but you see, i got it to work nowodays
> >>>> without
> >>>> much problems with the applied patch, i use it even now :)
> >>>>
> >>>>> 2) They should be distributed by themselves or with a third party
> >>>> project.
> >>>>> I think ipy_profile_zope.py belongs in the second category.
> >>>> I never heard about zope related product shipping it.
> >>>>
> >>>>> Thoughts?
> >>>>>
> >>>>> Brian
> >>>>>
> >>>>>
> >>>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden>
> wrote:
> >>>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
> >>>>>> lp:ipython.
> >>>>>>
> >>>>>> Requested reviews:
> >>>>>> IPython Developers (ipython-dev)
> >>>>>>
> >>>>>>
> >>>>>> see #443982 ;)
> >>>>>> --
> >>>>>>
> >>
> https:/
> <
> https:/
> >
> >> <
> >>
> https:/
> >>>> <
> >>>>
> >>
> https:/
> >>>>>> You are subscribed to branch lp:ipython.
> >>>>>>
> >>>>>> === modified file 'IPython/
> >>>>>> --- IPython/
> >>>> +0000
> >>>>>> +++ IPython/
> >>>> +0000
> >>>>>> @@ -24,6 +24,8 @@
> >>>>>> import os
> >>>>>> import textwrap
> >>>>>>
> >>>>>> +sys.oldstdin = sys.stdin
> >>>>>> +
> >>>>>> # The import below effectively obsoletes your old-style
> >>>> ipythonrc[.ini],
> >>>>>> # so consider yourself warned!
> >>>>>> # import ipy_defaults
> >>>>>> @@ -314,6 +316,7 @@
> >>>>>> """ % ( ",".join([ x for x in dir(zope_
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
I have a buildout to recycle and make that ipzope script to work (a thing that
use this extension as entry point, see [1]).
So, i seeked the approriate repository, i found the lp branch and applied my
changes in the deathrow module while, of course, in my release i apply the
changes in Extensions [2].
[1] -
http://
[2] -
http://
Brian Granger a écrit :
> Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you sure
> you are working on the 0.10 branch?
>
> Brian
>
> On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
>
>> I use it on 0.10 release.
>>
>> Brian Granger a écrit :
>>> One things I did realize, this is really a profile, and the profile API
>> has
>>> changed
>>> substantially:
>>>
>>>
>> http://
>>> Also, are you working off of trunk?
>>>
>>> Cheers,
>>>
>>> Brian
>>>
>>> On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
>>>
>>>> sure.
>>>>
>>>> Brian Granger a écrit :
>>>>> I am going to open up this discussion to the wider ipython dev list.
>>>>> Can you join in there?
>>>>>
>>>>> Brian
>>>>>
>>>>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden> wrote:
>>>>>
>>>>>> Brian Granger a écrit :
>>>>>>> Could this be turned into an IPython extension that is shipped with
>>>> zope?
>>>>>> I don't think so, its place is great there.
>>>>>>
>>>>>>> Things that have ended up in deathrow are in two categories:
>>>>>>>
>>>>>>> 1) They are old, broken or otherwise decaying.
>>>>>> Maybe it was just unmaintened, but you see, i got it to work nowodays
>>>>>> without
>>>>>> much problems with the applied patch, i use it even now :)
>>>>>>
>>>>>>> 2) They should be distributed by themselves or with a third party
>>>>>> project.
>>>>>>> I think ipy_profile_zope.py belongs in the second category.
>>>>>> I never heard about zope related product shipping it.
>>>>>>
>>>>>>> Thoughts?
>>>>>>>
>>>>>>> Brian
>>>>>>>
>>>>>>>
>>>>>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden>
>> wrote:
>>>>>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline into
>>>>>>>> lp:ipython.
>>>>>>>>
>>>>>>>> Requested reviews:
>>>>>>>> IPython Developers (ipython-dev)
>>>>>>>>
>>>>>>>>
>>>>>>>> see #443982 ;)
>>>>>>>> --
>>>>>>>>
>> https:/
>> <
>> https:/
>>>> <
>>>>
>> https:/
>>>>>> <
>>>>>>
>> https:/
>>>>>>>> You are subscribe...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
But what branch from launchpad did you use.
Can you do:
bzr info from inside the branch directory?
Brian
On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
> I have a buildout to recycle and make that ipzope script to work (a thing
> that
> use this extension as entry point, see [1]).
>
> So, i seeked the approriate repository, i found the lp branch and applied
> my
> changes in the deathrow module while, of course, in my release i apply the
> changes in Extensions [2].
>
> [1] -
>
> http://
> [2] -
>
> http://
>
> Brian Granger a écrit :
> > Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you
> sure
> > you are working on the 0.10 branch?
> >
> > Brian
> >
> > On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
> >
> >> I use it on 0.10 release.
> >>
> >> Brian Granger a écrit :
> >>> One things I did realize, this is really a profile, and the profile API
> >> has
> >>> changed
> >>> substantially:
> >>>
> >>>
> >>
> http://
> <
> http://
> >
> >>> Also, are you working off of trunk?
> >>>
> >>> Cheers,
> >>>
> >>> Brian
> >>>
> >>> On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
> >>>
> >>>> sure.
> >>>>
> >>>> Brian Granger a écrit :
> >>>>> I am going to open up this discussion to the wider ipython dev list.
> >>>>> Can you join in there?
> >>>>>
> >>>>> Brian
> >>>>>
> >>>>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden>
> wrote:
> >>>>>
> >>>>>> Brian Granger a écrit :
> >>>>>>> Could this be turned into an IPython extension that is shipped with
> >>>> zope?
> >>>>>> I don't think so, its place is great there.
> >>>>>>
> >>>>>>> Things that have ended up in deathrow are in two categories:
> >>>>>>>
> >>>>>>> 1) They are old, broken or otherwise decaying.
> >>>>>> Maybe it was just unmaintened, but you see, i got it to work
> nowodays
> >>>>>> without
> >>>>>> much problems with the applied patch, i use it even now :)
> >>>>>>
> >>>>>>> 2) They should be distributed by themselves or with a third party
> >>>>>> project.
> >>>>>>> I think ipy_profile_zope.py belongs in the second category.
> >>>>>> I never heard about zope related product shipping it.
> >>>>>>
> >>>>>>> Thoughts?
> >>>>>>>
> >>>>>>> Brian
> >>>>>>>
> >>>>>>>
> >>>>>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden>
> >> wrote:
> >>>>>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline
> into
> >>>>>>>> lp:ipython.
> >>>>>>>>
> >>>>>>>> Requested reviews:
> >>>>>>>> IPython Developers (ipython-dev)
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> see #443982 ;...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
The one told in the dev documentation ;) [1] ...
[1] - bzr+ssh:
Brian Granger a écrit :
> But what branch from launchpad did you use.
>
> Can you do:
>
> bzr info from inside the branch directory?
>
> Brian
>
> On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
>
>> I have a buildout to recycle and make that ipzope script to work (a thing
>> that
>> use this extension as entry point, see [1]).
>>
>> So, i seeked the approriate repository, i found the lp branch and applied
>> my
>> changes in the deathrow module while, of course, in my release i apply the
>> changes in Extensions [2].
>>
>> [1] -
>>
>> http://
>> [2] -
>>
>> http://
>>
>> Brian Granger a écrit :
>>> Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you
>> sure
>>> you are working on the 0.10 branch?
>>>
>>> Brian
>>>
>>> On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
>>>
>>>> I use it on 0.10 release.
>>>>
>>>> Brian Granger a écrit :
>>>>> One things I did realize, this is really a profile, and the profile API
>>>> has
>>>>> changed
>>>>> substantially:
>>>>>
>>>>>
>> http://
>> <
>> http://
>>>>> Also, are you working off of trunk?
>>>>>
>>>>> Cheers,
>>>>>
>>>>> Brian
>>>>>
>>>>> On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden> wrote:
>>>>>
>>>>>> sure.
>>>>>>
>>>>>> Brian Granger a écrit :
>>>>>>> I am going to open up this discussion to the wider ipython dev list.
>>>>>>> Can you join in there?
>>>>>>>
>>>>>>> Brian
>>>>>>>
>>>>>>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden>
>> wrote:
>>>>>>>> Brian Granger a écrit :
>>>>>>>>> Could this be turned into an IPython extension that is shipped with
>>>>>> zope?
>>>>>>>> I don't think so, its place is great there.
>>>>>>>>
>>>>>>>>> Things that have ended up in deathrow are in two categories:
>>>>>>>>>
>>>>>>>>> 1) They are old, broken or otherwise decaying.
>>>>>>>> Maybe it was just unmaintened, but you see, i got it to work
>> nowodays
>>>>>>>> without
>>>>>>>> much problems with the applied patch, i use it even now :)
>>>>>>>>
>>>>>>>>> 2) They should be distributed by themselves or with a third party
>>>>>>>> project.
>>>>>>>>> I think ipy_profile_zope.py belongs in the second category.
>>>>>>>> I never heard about zope related product shipping it.
>>>>>>>>
>>>>>>>>> Thoughts?
>>>>>>>>>
>>>>>>>>> Brian
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>> On Mon, Oct 5, 2009 at 1:20 PM, kiorky <email address hidden>
>>>> wrote:
>>>>>>>>>> kiorky has proposed merging lp:~kiorky/ipython/ipython-readline
>...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
Can you do a bzr pull and see what that does...
On Fri, Oct 9, 2009 at 2:48 PM, kiorky <email address hidden> wrote:
> The one told in the dev documentation ;) [1] ...
>
> [1] - bzr+ssh:
>
> Brian Granger a écrit :
> > But what branch from launchpad did you use.
> >
> > Can you do:
> >
> > bzr info from inside the branch directory?
> >
> > Brian
> >
> > On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
> >
> >> I have a buildout to recycle and make that ipzope script to work (a
> thing
> >> that
> >> use this extension as entry point, see [1]).
> >>
> >> So, i seeked the approriate repository, i found the lp branch and
> applied
> >> my
> >> changes in the deathrow module while, of course, in my release i apply
> the
> >> changes in Extensions [2].
> >>
> >> [1] -
> >>
> >>
> http://
> >> [2] -
> >>
> >>
> http://
> >>
> >> Brian Granger a écrit :
> >>> Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you
> >> sure
> >>> you are working on the 0.10 branch?
> >>>
> >>> Brian
> >>>
> >>> On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
> >>>
> >>>> I use it on 0.10 release.
> >>>>
> >>>> Brian Granger a écrit :
> >>>>> One things I did realize, this is really a profile, and the profile
> API
> >>>> has
> >>>>> changed
> >>>>> substantially:
> >>>>>
> >>>>>
> >>
> http://
> <
> http://
> >
> >> <
> >>
> http://
> >>>>> Also, are you working off of trunk?
> >>>>>
> >>>>> Cheers,
> >>>>>
> >>>>> Brian
> >>>>>
> >>>>> On Fri, Oct 9, 2009 at 10:45 AM, kiorky <email address hidden>
> wrote:
> >>>>>
> >>>>>> sure.
> >>>>>>
> >>>>>> Brian Granger a écrit :
> >>>>>>> I am going to open up this discussion to the wider ipython dev
> list.
> >>>>>>> Can you join in there?
> >>>>>>>
> >>>>>>> Brian
> >>>>>>>
> >>>>>>> On Fri, Oct 9, 2009 at 10:30 AM, kiorky <email address hidden>
> >> wrote:
> >>>>>>>> Brian Granger a écrit :
> >>>>>>>>> Could this be turned into an IPython extension that is shipped
> with
> >>>>>> zope?
> >>>>>>>> I don't think so, its place is great there.
> >>>>>>>>
> >>>>>>>>> Things that have ended up in deathrow are in two categories:
> >>>>>>>>>
> >>>>>>>>> 1) They are old, broken or otherwise decaying.
> >>>>>>>> Maybe it was just unmaintened, but you see, i got it to work
> >> nowodays
> >>>>>>>> without
> >>>>>>>> much problems with the applied patch, i use it even now :)...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
(minitage-
/home/kiorky/
Standalone tree (format: unnamed)
Location:
branch root: .
Related branches:
parent branch: bzr+ssh:
Using saved parent location:
bzr+ssh:
No revisions to pull.
(minitage-
~/projects/
~/projects/
~/projects/
~/projects/
Standalone tree (format: unnamed)
Location:
branch root: .
Related branches:
push branch: bzr+ssh:
parent branch: /home/kiorky/
Using saved parent location: /home/kiorky/
No revisions to pull.
(minitage-0.4)~>
Brian Granger a écrit :
> Can you do a bzr pull and see what that does...
>
> On Fri, Oct 9, 2009 at 2:48 PM, kiorky <email address hidden> wrote:
>
>> The one told in the dev documentation ;) [1] ...
>>
>> [1] - bzr+ssh:
>>
>> Brian Granger a écrit :
>>> But what branch from launchpad did you use.
>>>
>>> Can you do:
>>>
>>> bzr info from inside the branch directory?
>>>
>>> Brian
>>>
>>> On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
>>>
>>>> I have a buildout to recycle and make that ipzope script to work (a
>> thing
>>>> that
>>>> use this extension as entry point, see [1]).
>>>>
>>>> So, i seeked the approriate repository, i found the lp branch and
>> applied
>>>> my
>>>> changes in the deathrow module while, of course, in my release i apply
>> the
>>>> changes in Extensions [2].
>>>>
>>>> [1] -
>>>>
>>>>
>> http://
>>>> [2] -
>>>>
>>>>
>> http://
>>>> Brian Granger a écrit :
>>>>> Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you
>>>> sure
>>>>> you are working on the 0.10 branch?
>>>>>
>>>>> Brian
>>>>>
>>>>> On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
>>>>>
>>>>>> I use it on 0.10 release.
>>>>>>
>>>>>> Brian Granger a écrit :
>>>>>>> One things I did realize, this is really a profile, and the profile
>> API
>>>>>> has
>>>>>>> changed
>>>>>>> substantially:
>>>>>>>
>>>>>>>
>> http://
>> <
>> http://
>>>> <
>>>>
>> http://
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
Uhm with trunk, the zope extension can't work as ipython code has a lot "with"
statements, and it is not python2.4 friendly. But Zope2 requires python2.4 :(.
Its also a shame it does not work anymore with python2.4 ... ;(
kiorky a écrit :
> (minitage-
> /home/kiorky/
> Standalone tree (format: unnamed)
> Location:
> branch root: .
>
> Related branches:
> parent branch: bzr+ssh:
> Using saved parent location:
> bzr+ssh:
> No revisions to pull.
> (minitage-
> ~/projects/
> ~/projects/
> ~/projects/
> ~/projects/
> Standalone tree (format: unnamed)
> Location:
> branch root: .
>
> Related branches:
> push branch: bzr+ssh:
> parent branch: /home/kiorky/
> Using saved parent location: /home/kiorky/
> No revisions to pull.
> (minitage-0.4)~>
>
>
> Brian Granger a écrit :
>> Can you do a bzr pull and see what that does...
>>
>> On Fri, Oct 9, 2009 at 2:48 PM, kiorky <email address hidden> wrote:
>>
>>> The one told in the dev documentation ;) [1] ...
>>>
>>> [1] - bzr+ssh:
>>>
>>> Brian Granger a écrit :
>>>> But what branch from launchpad did you use.
>>>>
>>>> Can you do:
>>>>
>>>> bzr info from inside the branch directory?
>>>>
>>>> Brian
>>>>
>>>> On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
>>>>
>>>>> I have a buildout to recycle and make that ipzope script to work (a
>>> thing
>>>>> that
>>>>> use this extension as entry point, see [1]).
>>>>>
>>>>> So, i seeked the approriate repository, i found the lp branch and
>>> applied
>>>>> my
>>>>> changes in the deathrow module while, of course, in my release i apply
>>> the
>>>>> changes in Extensions [2].
>>>>>
>>>>> [1] -
>>>>>
>>>>>
>>> http://
>>>>> [2] -
>>>>>
>>>>>
>>> http://
>>>>> Brian Granger a écrit :
>>>>>> Ahhh, the 0.10 release didn't have the deathrow sub-packages. Are you
>>>>> sure
>>>>>> you are working on the 0.10 branch?
>>>>>>
>>>>>> Brian
>>>>>>
>>>>>> On Fri, Oct 9, 2009 at 11:21 AM, kiorky <email address hidden> wrote:
>>>>>>
>>>>>>> I use it on 0.10 release.
>>>>>>>
>>>>>>> Brian Granger a écrit :
>>>>>>>> One things I did realize, this is really a profile, and the profile
>>> API
>>>>>>> has
>>>>>>>> changed
>>>>>>>> substantially:
>>>>>>>>
>>>>>>>>
>>> http://
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
We made this decision recently for a couple of reasons:
* We are beginning to need some Python 2.5 features like with and absolute
imports.
* We *really* need to move towards Python 3.0 support. For IPython, this
can't happen until we drop *both* 2.4 and 2.5 support (and eventually 2.6
when 2.7 comes out. Dropping 2.4 support is the first step along this
path. IPython is not alone in this - every Python project in existence has
to make this same choice.
We understand that this is a huge pain for projects like Zope2 that can't
drop 2.4 support. Our (unfortunate for you) recommendation is that you use
an older version of IPython, such as 0.10 or 0.9, which both support 2.4.
We have a launchpad branch for 0.10 and we are more than willing to accept
patches on that branch.
Cheers,
Brian
On Sat, Oct 10, 2009 at 1:45 AM, kiorky <email address hidden> wrote:
> Uhm with trunk, the zope extension can't work as ipython code has a lot
> "with"
> statements, and it is not python2.4 friendly. But Zope2 requires python2.4
> :(.
> Its also a shame it does not work anymore with python2.4 ... ;(
>
>
> kiorky a écrit :
> > (minitage-
> > /home/kiorky/
> > Standalone tree (format: unnamed)
> > Location:
> > branch root: .
> >
> > Related branches:
> > parent branch: bzr+ssh://
> bazaar.
> > Using saved parent location:
> > bzr+ssh:
> > No revisions to pull.
> > (minitage-
> pull
> > ~/projects/
> > ~/projects/
> > ~/projects/
> > ~/projects/
> > Standalone tree (format: unnamed)
> > Location:
> > branch root: .
> >
> > Related branches:
> > push branch: bzr+ssh://
> bazaar.
> > parent branch: /home/kiorky/
> > Using saved parent location: /home/kiorky/
> > No revisions to pull.
> > (minitage-0.4)~>
> >
> >
> > Brian Granger a écrit :
> >> Can you do a bzr pull and see what that does...
> >>
> >> On Fri, Oct 9, 2009 at 2:48 PM, kiorky <email address hidden> wrote:
> >>
> >>> The one told in the dev documentation ;) [1] ...
> >>>
> >>> [1] - bzr+ssh:
> <http://
> >>>
> >>> Brian Granger a écrit :
> >>>> But what branch from launchpad did you use.
> >>>>
> >>>> Can you do:
> >>>>
> >>>> bzr info from inside the branch directory?
> >>>>
> >>>> Brian
> >>>>
> >>>> On Fri, Oct 9, 2009 at 1:00 PM, kiorky <email address hidden> wrote:
> >>>>
> >>>>> I have a buildout to recycle and make that ipzope script to work (a
> >>> thing
> >>>>> that...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Brian Granger (ellisonbg) wrote : | # |
I am going to close this. If you want to submit a patch for 0.10 (that would work under python 2.4) that is fine.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
You can just apply this patch, it works on 0.10 and 0.9 as well...
Brian Granger a écrit :
> The proposal to merge lp:~kiorky/ipython/ipython-readline into lp:ipython has been updated.
>
> Status: Needs review => Rejected
--
Cordialement,
KiOrKY
GPG Key FingerPrint: 0x1A1194B7681112AF
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
kiorky (kiorky) wrote : | # |
You can just apply this patch, it works on 0.10 and 0.9 as well...
Preview Diff
1 | === modified file 'IPython/deathrow/ipy_profile_zope.py' |
2 | --- IPython/deathrow/ipy_profile_zope.py 2009-09-18 02:40:20 +0000 |
3 | +++ IPython/deathrow/ipy_profile_zope.py 2009-10-05 20:38:09 +0000 |
4 | @@ -24,6 +24,10 @@ |
5 | import os |
6 | import textwrap |
7 | |
8 | +sys.oldstdin = sys.stdin |
9 | +sys.oldstdout = sys.stdout |
10 | +#sys.oldstderr = sys.stderr |
11 | + |
12 | # The import below effectively obsoletes your old-style ipythonrc[.ini], |
13 | # so consider yourself warned! |
14 | # import ipy_defaults |
15 | @@ -314,6 +318,9 @@ |
16 | """ % ( ",".join([ x for x in dir(zope_debug.utils) if not x.startswith("_") ] ) ) ) |
17 | |
18 | |
19 | + sys.stdin = sys.oldstdin |
20 | + #sys.stderr = sys.oldstderr |
21 | + sys.stdout = sys.oldstdout |
22 | ip.user_ns.update( zope_debug.namespace ) |
23 | |
24 |
see #443982 ;)