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

images not picked up when making source releases of Sage 4.3.alpha0 and 4.3.alpha1 #7606

Closed
sagetrac-mvngu mannequin opened this issue Dec 5, 2009 · 6 comments
Closed

Comments

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Dec 5, 2009

From this sage-devel thread (see also this thread):

> 10. sage: hg_sage.status()
>     Getting status of modified or unknown files:
>     cd "/home/SimonKing/sandbox/sage-4.3.alpha0/devel/sage" && hg
> status
>     ! doc/fr/a_tour_of_sage/eigen_plot.png
>     ! doc/fr/a_tour_of_sage/sin_plot.png

> Aha! There is a problem with the docs, right? Are these files missing?

You get those two lines with the exclamation marks because the file
MANIFEST.in in Sage 4.3.alpha0 isn't configured to pick up those two
image files. When ticket #7190 (French translation: A Tour of Sage)
[1] was merged in Sage 4.3.alpha0, the file
devel/sage-main/MANIFEST.in wasn't also changed to take into account
the new image files, so these are not picked up when releasing the
alpha0 tarball. A result is that one would not see the image files in
devel/sage-main/doc/fr/a_tour_of_sage. You can fix the missing files
problem as follows:

[mvngu@sage sage-4.3.alpha0-7473-sphinx]$ cd devel/sage-main/
[mvngu@sage sage-main]$ hg st
! doc/fr/a_tour_of_sage/eigen_plot.png
! doc/fr/a_tour_of_sage/sin_plot.png
[mvngu@sage sage-main]$ hg revert -a
reverting doc/fr/a_tour_of_sage/eigen_plot.png
reverting doc/fr/a_tour_of_sage/sin_plot.png
[mvngu@sage sage-main]$ hg st
<no output>

This missing files problem is due to #7190.

CC: @nathanncohen

Component: documentation

Author: Minh Van Nguyen

Reviewer: Mike Hansen

Merged: sage-4.3.rc0

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

@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.3 milestone Dec 5, 2009
@sagetrac-mvngu sagetrac-mvngu mannequin self-assigned this Dec 5, 2009
@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Dec 5, 2009

Attachment: trac_7606-manifest.patch.gz

based on Sage 4.3.alpha1

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Dec 5, 2009

comment:2

Here are some steps to fix the problem with missing image files. On a freshly compiled Sage 4.3.alpha1 or a newly unpacked binary of that version, do a Mercurial revert to recover the deleted images:

[mvngu@sage sage-4.3.alpha1-7606-images]$ cd devel/sage-main/
[mvngu@sage sage-main]$ hg st
! doc/fr/a_tour_of_sage/eigen_plot.png
! doc/fr/a_tour_of_sage/sin_plot.png
[mvngu@sage sage-main]$ hg revert -a
reverting doc/fr/a_tour_of_sage/eigen_plot.png
reverting doc/fr/a_tour_of_sage/sin_plot.png
[mvngu@sage sage-main]$ hg st
<no output>

Next, apply the patch trac_7606-manifest.patch, which configures MANIFEST.in to pick up those two image files when making a source release. Afterwards, making a source tarball with "./sage -sdist " should also pick up the two image files.

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Dec 5, 2009

Author: Minh Van Nguyen

@sagetrac-mvngu sagetrac-mvngu mannequin added the s: needs review label Dec 5, 2009
@mwhansen
Copy link
Contributor

mwhansen commented Dec 6, 2009

Reviewer: Mike Hansen

@mwhansen
Copy link
Contributor

mwhansen commented Dec 6, 2009

Merged: sage-4.3.rc0

@mwhansen
Copy link
Contributor

mwhansen commented Dec 6, 2009

comment:3

Looks good to me.

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

1 participant