Opened 10 years ago

Closed 8 years ago

#14091 closed defect (fixed)

Fix intersphinx links to the reference manual from other documents

Reported by: nthiery Owned by: mvngu
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: documentation Keywords: intersphinx, thematic tutorials
Cc: sage-combinat Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

#9128 enabled links to the reference manual from other documents like the thematic tutorials. This import feature is currently broken. Until this is fixed, the updated thematic tutorial index from #14090 will look lame.

The issue to be due to a wrong initialization order for the sphinx app. At least the attached patch, which I am using for now for the Sage-Combinat tutorials, seems to work around it. Probably the sphinx internals were change in the mean time. #13057 may have had an influence too.

Note: I am not suggesting to merge or review the currently attached patch. It's just there to point where the problem seem to be.

Attachments (1)

intersphinx-workaround-nt.patch (851 bytes) - added by nthiery 10 years ago.

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by nthiery

comment:1 Changed 9 years ago by jdemeyer

Milestone: sage-5.11sage-5.12

comment:2 Changed 9 years ago by vbraun_spam

Milestone: sage-6.1sage-6.2

comment:3 Changed 9 years ago by vbraun_spam

Milestone: sage-6.2sage-6.3

comment:4 Changed 8 years ago by vbraun_spam

Milestone: sage-6.3sage-6.4

comment:5 Changed 8 years ago by nthiery

Milestone: sage-6.4sage-duplicate/invalid/wontfix
Status: newneeds_review

This seems to be fixed now. See e.g. the crosslink "Introduction to combinatorics in Sage" which points to the reference manual in:

http://www.sagemath.org/doc/thematic_tutorials/index.html

I thus mark this ticket as duplicate. If anyone has an idea of when this was fixed, it would be good to add a ref.

comment:6 Changed 8 years ago by nthiery

Status: needs_reviewpositive_review

comment:7 Changed 8 years ago by vbraun

Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.