Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docbuilder changes sagetex.rst #14199

Closed
jdemeyer opened this issue Feb 27, 2013 · 13 comments
Closed

Docbuilder changes sagetex.rst #14199

jdemeyer opened this issue Feb 27, 2013 · 13 comments

Comments

@jdemeyer
Copy link

Install sage-5.8.beta2 and build the documentation. Now

$ touch devel/sage/doc/en/reference/misc/sagetex.rst
$ make doc

and you will see the following change appear:

diff --git a/doc/en/reference/misc/sagetex.rst b/doc/en/reference/misc/sagetex.rst
--- a/doc/en/reference/misc/sagetex.rst
+++ b/doc/en/reference/misc/sagetex.rst
@@ -1,11 +1,16 @@
-Installing and using SageTeX
-============================
+.. nodoctest

-SageTeX is a system for embedding computations and plots from Sage into
-LaTeX documents. It is included by default with Sage, so if you have
-installed Sage, you already have SageTeX. However, to get it to work,
-you need to make TeX aware of SageTeX. Instructions for that are in the
-"Make SageTeX known to TeX" section of the `Sage installation guide
-<http://sagemath.org/doc/installation/index.html>`_ (`this link
-<../../installation/index.html>`_ should take you to a local copy of the
-installation guide).
+.. _sagetex:
+
+MISSING TITLE
+=============
+
+.. This file has been autogenerated.
+
+
+.. automodule:: sagetex
+   :members:
+   :undoc-members:
+   :show-inheritance:
+
+

If this an autogenerated file, it should not be tracked by Mercurial. And why did all content get removed?

Making the file sagetex.rst unwritable shows where the file is written:

Traceback (most recent call last):
  File "/Users/buildbot/sage-5.8.beta2/devel/sage/doc/common/builder.py", line 45, in build_ref_doc
    getattr(ReferenceSubBuilder(doc, lang), format)(*args, **kwds)
  File "/Users/buildbot/sage-5.8.beta2/devel/sage/doc/common/builder.py", line 662, in _wrapper
    self.write_auto_rest_file(module_name.replace(os.path.sep, '.'))
  File "/Users/buildbot/sage-5.8.beta2/devel/sage/doc/common/builder.py", line 924, in write_auto_rest_file
    outfile = open(filename, 'w')
IOError: [Errno 13] Permission denied: '/Users/buildbot/sage-5.8.beta2/devel/sage/doc/en/reference/misc/sagetex.rst'

CC: @jhpalmieri

Component: documentation

Author: John Palmieri

Reviewer: Jeroen Demeyer

Merged: sage-5.8.beta4

Issue created by migration from https://trac.sagemath.org/ticket/14199

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jhpalmieri
Copy link
Member

comment:5

I think I may have seen something like this a long time ago, in the early stages of testing #6495, but then I thought it had to do with maybe not deleting the old doc output or something like that. Was this a build from scratch or an upgrade?

I think that the only OS X 10.6 system that I have access to is bsd.math.washington.edu, and I can't reproduce this there, at least after building the docs a few times with a freshly built Sage 5.8.beta2.

@jdemeyer
Copy link
Author

jdemeyer commented Mar 1, 2013

comment:6

Replying to @jhpalmieri:

Was this a build from scratch or an upgrade?

I don't remember, it was on a buildbot machine (bsd.math), but I guess it was a build from scratch. On bsd.math look for /Users/buildbot/sage-5.8.beta2-14199.tar.bz2 which is a build with this problem.

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jhpalmieri
Copy link
Member

comment:9

Okay, I think I've figured it out: the problem is that "sagetex.rst" starts with "sage". See the attached patch.

@jhpalmieri
Copy link
Member

Author: John Palmieri

@jdemeyer jdemeyer changed the title Docbuilder and auto-generated files Docbuilder changes sagetex.rst Mar 6, 2013
@jdemeyer
Copy link
Author

jdemeyer commented Mar 6, 2013

comment:11

Maybe you should document better what's the purpose of the condition

if name.startswith('sage' + os.sep):

@jhpalmieri
Copy link
Member

comment:12

Attachment: trac_14199.patch.gz

I added a comment to try to address your suggestion.

@jdemeyer
Copy link
Author

jdemeyer commented Mar 7, 2013

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link
Author

jdemeyer commented Mar 7, 2013

Merged: sage-5.8.beta4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants