Opened 9 years ago

Closed 7 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 9 years ago.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by nthiery

comment:1 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:2 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:3 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:4 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:5 Changed 7 years ago by nthiery

  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_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 7 years ago by nthiery

  • Status changed from needs_review to positive_review

comment:7 Changed 7 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.