Opened 7 years ago

Closed 6 years ago

#12781 closed defect (fixed)

code inspection appends doc dirs to sys.path

Reported by: burcin Owned by: jason
Priority: minor Milestone: sage-5.10
Component: misc Keywords:
Cc: Merged in: sage-5.10.beta1
Authors: Volker Braun Reviewers: Burcin Erocal
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by vbraun)

~/sage/sage-5.0.beta5 $ ./sage
----------------------------------------------------------------------
| Sage Version 5.0.beta5, Release Date: 2012-02-22                   |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
**********************************************************************
*                                                                    *
* Warning: this is a prerelease version, and it may be unstable.     *
*                                                                    *
**********************************************************************
sage: len(sys.path)
28
sage: x??
sage: len(sys.path)
33
sage: x??          
sage: len(sys.path)
37
sage: x??          
sage: len(sys.path)
41
sage: x??          
sage: len(sys.path)
45
sage: sys.path[41:]
['/home/burcin/sage/sage-5.0.beta5/devel/sage/doc', '/home/burcin/sage/sage-5.0.beta5/devel/sage/doc', '/home/burcin/sage/sage-5.0.beta5/devel/sage/doc', '/home/burcin/sage/sage-5.0.beta5/devel/sage/doc']

Apply trac_12781_remove_cruft.patch

Attachments (1)

trac_12781_remove_cruft.patch (1.1 KB) - added by vbraun 6 years ago.
Initial patch

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by vbraun

This is done by the sagenb code, I sent a pull request:

https://github.com/sagemath/sagenb/pull/156

Changed 6 years ago by vbraun

Initial patch

comment:2 Changed 6 years ago by vbraun

  • Authors set to Volker Braun
  • Status changed from new to needs_review

I also happened upon this gem that looks like it changes sys.path but actually is never called

comment:3 Changed 6 years ago by vbraun

  • Description modified (diff)

comment:4 Changed 6 years ago by burcin

  • Reviewers set to Burcin Erocal
  • Status changed from needs_review to positive_review

Looks good to me.

Thanks for tracking down the sys.path problem in sagenb.

comment:5 Changed 6 years ago by jdemeyer

  • Merged in set to sage-5.10.beta1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.