Ticket #7606 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

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

Reported by: mvngu Owned by: mvngu
Priority: major Milestone: sage-4.3
Component: documentation Keywords:
Cc: ncohen Work issues:
Report Upstream: N/A Reviewers: Mike Hansen
Authors: Minh Van Nguyen Merged in: sage-4.3.rc0
Dependencies: Stopgaps:

Description

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.

Attachments

trac_7606-manifest.patch Download (698 bytes) - added by mvngu 3 years ago.
based on Sage 4.3.alpha1

Change History

comment:1 Changed 3 years ago by ncohen

  • Cc ncohen added

Changed 3 years ago by mvngu

based on Sage 4.3.alpha1

comment:2 Changed 3 years ago by mvngu

  • Status changed from new to needs_review
  • Authors set to Minh Van Nguyen

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 <version-number>" should also pick up the two image files.

comment:3 Changed 3 years ago by mhansen

  • Status changed from needs_review to closed
  • Reviewers set to Mike Hansen
  • Resolution set to fixed
  • Merged in set to sage-4.3.rc0

Looks good to me.

Note: See TracTickets for help on using tickets.