load-extended-executes fails when py-install-directory is not set

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

Bug Description

If `py-install-directory' is not set, the current logic looks for "/python-extended-executes.el" when it should be "python-extended-executes.el". The attached patch (against rev 836) is a primitive remedy :)

Revision history for this message
Felix Geller (fgeller) wrote :
Revision history for this message
Andreas Roehler (a-roehler) wrote : Re: [Bug 932418] [NEW] load-extended-executes fails when py-install-directory is not set

Am 15.02.2012 00:12, schrieb Felix Geller:
> Public bug reported:
>
> If `py-install-directory' is not set, the current logic looks for
> "/python-extended-executes.el" when it should be "python-extended-
> executes.el". The attached patch (against rev 836) is a primitive remedy
> :)
>

Hi Felix,

thanks pointing at the matter.

AFAIU there is no way to avoid giving just one name where to
install/load from.

Tried in the past several workarounds same kind you sent, however, it
turned out never stable.

It might help if you can tell the reasons, why `py-install-directory'
was not set.

A solution would be rather providing a workaround giving
`py-install-directory' a default value, then fixing the loading of
single files.

OTOH probably it's better to get an `py-install-directory'-not-set-error
than loading from uncertain guesses.

So far, thanks again,

Andreas

Revision history for this message
Andreas Roehler (a-roehler) wrote :

code is merged into python-mode.el, so this issue is fixed as a side-effect :)

Changed in python-mode:
status: New → Fix Committed
Revision history for this message
Felix Geller (fgeller) wrote :

Different approach now -- use different implementation of py-guess-py-install-dir (uses locate-library) and use it if py-install-dir is not set.

The recent version (837) still exhibited the save problem for me. The patch is against r837.

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

Hi Felix,

applied, thanks.
Please re-open this report should be something left.

Andreas

Changed in python-mode:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Andreas Roehler (a-roehler)
milestone: none → 6.0.5
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

Bug attachments

Remote bug watches

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