run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

Bug #878288 reported by Barry Warsaw
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-mode.el
Fix Released
Medium
Andreas Roehler

Bug Description

In current bzr trunk, using Emacs 23.3.1, hit C-c ! to run py-shell. The 2.7.2 shell comes up. At the prompt, type

>>> from datetime import * [RET]

nothing happens, but you get the error in the Summary.

Revision history for this message
Andreas Roehler (a-roehler) wrote : Re: [Bug 878288] [NEW] run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

Am 19.10.2011 17:40, schrieb Barry Warsaw:
> Public bug reported:
>
> In current bzr trunk, using Emacs 23.3.1, hit C-c ! to run py-shell.
> The 2.7.2 shell comes up. At the prompt, type
>
>>>> from datetime import * [RET]
>
> nothing happens, but you get the error in the Summary.
>
> ** Affects: python-mode
> Importance: Undecided
> Status: New
>

shell-directory-tracker is a compiled Lisp function in
`shell.el'.

loading shell.el should do it

Cheers,

Andreas

Changed in python-mode:
assignee: nobody → Andreas Roehler (a-roehler)
milestone: none → 6.0.4
importance: Undecided → Medium
Revision history for this message
Barry Warsaw (barry) wrote :

On Oct 19, 2011, at 05:09 PM, Andreas Roehler wrote:

>shell-directory-tracker is a compiled Lisp function in
>`shell.el'.
>
>loading shell.el should do it

Sure, but why should the *Python* buffer care? It's not going to use shell
commands like cd, pushd, and popd.

Revision history for this message
Andreas Roehler (a-roehler) wrote : Re: [Bug 878288] [NEW] run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

Am 19.10.2011 19:23, schrieb Barry Warsaw:
> On Oct 19, 2011, at 05:09 PM, Andreas Roehler wrote:
>
>> shell-directory-tracker is a compiled Lisp function in
>> `shell.el'.
>>
>> loading shell.el should do it
>
> Sure, but why should the *Python* buffer care? It's not going to use shell
> commands like cd, pushd, and popd.
>

ipython shell does.

which brings some convenience IMHO

think I'll just add a (require 'shell)

(?)

Revision history for this message
Barry Warsaw (barry) wrote : Re: [Bug 878288] [NEW] run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

On Oct 19, 2011, at 06:01 PM, Andreas Roehler wrote:

>ipython shell does.
>
>which brings some convenience IMHO
>
>think I'll just add a (require 'shell)

Maybe just when the ipython shell is used?

After a fresh restart of Emacs, after I open a "normal" shell, the error
message and hang does not seem to happen, since by that time

(featurep 'shell)
-> t

Revision history for this message
Andreas Roehler (a-roehler) wrote : Re: [Bug 878288] [NEW] run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

Am 19.10.2011 20:31, schrieb Barry Warsaw:
> On Oct 19, 2011, at 06:01 PM, Andreas Roehler wrote:
>
>> ipython shell does.
>>
>> which brings some convenience IMHO
>>
>> think I'll just add a (require 'shell)
>
> Maybe just when the ipython shell is used?
>
> After a fresh restart of Emacs, after I open a "normal" shell, the error
> message and hang does not seem to happen, since by that time
>
> (featurep 'shell)
> -> t
>

Hmm, IMO 'shell is not such a exotic feature.
May have a look, but think requiring it should not be a matter anyway.

BTW what do you think implementing "cd" and the other basic shell
commands ipython-shell has for all python shells?

Revision history for this message
Barry Warsaw (barry) wrote : Re: [Bug 878288] [NEW] run-hook-with-args: Symbol's function definition is void: shell-directory-tracker

On Oct 19, 2011, at 07:27 PM, Andreas Roehler wrote:

>Hmm, IMO 'shell is not such a exotic feature.
>May have a look, but think requiring it should not be a matter anyway.

Probably not.

>BTW what do you think implementing "cd" and the other basic shell
>commands ipython-shell has for all python shells?

Hmm, I think I would keep the buffer interface as a thin layer above the
underlying shell.

Changed in python-mode:
status: New → In Progress
Revision history for this message
Andreas Roehler (a-roehler) wrote :

Am 20.10.2011 04:19, schrieb Barry Warsaw:
> On Oct 19, 2011, at 07:27 PM, Andreas Roehler wrote:
>
>> Hmm, IMO 'shell is not such a exotic feature.
>> May have a look, but think requiring it should not be a matter anyway.
>
> Probably not.
>
>> BTW what do you think implementing "cd" and the other basic shell
>> commands ipython-shell has for all python shells?
>
> Hmm, I think I would keep the buffer interface as a thin layer above the
> underlying shell.
>

OK, made a separate report lp:878669 for the latter.

Will avoid the bug with a (require 'shell) for the moment.

Changed in python-mode:
status: In Progress → Fix Committed
Changed in python-mode:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.