Ticket #952 (closed defect: wontfix)

Opened 21 months ago

Last modified 9 months ago

Sphinx make duplicates image files

Reported by: wildintellect Owned by: live-demo@…
Priority: normal Milestone: OSGeoLive7.9
Component: LiveDVD Keywords: sphinx, 7.0
Cc:

Description

Sphinx appears to be duplicating each image for each translation. So we end up with 8 copies of every screenshot, for ~400MB too much.

Look in /var/www/_images/ on a recent 6.0 beta iso.

Change History

  Changed 21 months ago by wildintellect

related to ticket:950

  Changed 21 months ago by hamish

  • keywords sphinx, 6.0 added; sphinx removed
  • priority changed from normal to blocker

  Changed 21 months ago by hamish

see  http://osdir.com/ml/sphinx-dev/2012-02/msg00006.html

Jake wrote:

It seems to be a problem with sphinx not being willing to overwrite an
image file, even when the new file is identical to the one being
overwritten. Currently, we have a hack in place which, on each build,
selectively removes the files that are causing the problem. This is
not an ideal solution: any help with this would be appreciated! Thank
you
Jake Vanderplas
scikit-learn dev team

Hamish

(argh, trac keeps logging me out)

  Changed 21 months ago by hamish

here's a better thread view:

 http://groups.google.com/group/sphinx-dev/browse_thread/thread/289358d8f506649b

and the relevant ticket. it was supposed to be fixed in sphinx 1.0.8, but we are using 1.3.3. ?!

 https://bitbucket.org/birkenfeld/sphinx/issue/704

(perhaps the python patch there is worth trying)

Hamish

follow-up: ↓ 7   Changed 21 months ago by hamish

The nightly build on Adhoc does not have this bug (it's using 1.0.8+dfsg-2~bpo60+1 from Deb/Squeeze?), but I do notice this line at the end of  http://adhoc.osgeo.osuosl.org/livedvd/logs/nightly_build.err

WARNING: html_static_path entry './svn/doc/_static' does not exist

maybe relevant?

Hamish

  Changed 21 months ago by hamish

work-around now implemented in install_main_docs.sh, /var/www/_images/ is down to 46mb and ~/gisvm/doc/_build/ is make cleaned away..

it's ugly though, duplicate files are replaced with symlinks to the originals. better to fix it in sphinx's python code. alternatively I added some commented out code to 'sed -i' the filenames in the .html docs dirs to match the base image name, if anyone wants to try to get that working (still a little more to do before it works).

Hamish

in reply to: ↑ 5   Changed 21 months ago by hamish

Replying to hamish:

The nightly build on Adhoc

(I've now made it the hourly build by the way)

Hamish

  Changed 21 months ago by kalxas

  • priority changed from blocker to normal

In latest build8234 the _images folder is 45MB. No doc duplicates.

I will keep this around until it is confirmed in beta4

  Changed 21 months ago by hamish

the bug should be kept open until it is fixed, the current work-around just alleviates the symptoms, it doesn't actually address the real problem. it's no longer a critical problem though. It also confirms that either the hardlinking by fslint is failing for the iso build, or that the remastersys max size calc is failing to account for hardlinks not taking up any (significant) extra space. If it is the latter, perhaps we can hack it to not trigger the max size check and just confirm it is not too big by hand.

thanks, Hamish

follow-up: ↓ 11   Changed 21 months ago by kalxas

  • milestone changed from OSGeoLive6.0RC1 to OSGeoLive6.5

this does not affect current beta5. I am leaving it open as requested but moving it to next version

in reply to: ↑ 10   Changed 21 months ago by hamish

Replying to kalxas:

this does not affect current beta5.

actually it very much does, but since we've mitigated the problem with an ugly hack it's no longer a blocker.

Hamish

  Changed 21 months ago by camerons

  • keywords 6.5 added; 6.0 removed

  Changed 15 months ago by kalxas

  • keywords 7.0 added; 6.5 removed
  • milestone changed from OSGeoLive6.5 to OSGeoLive7.0

no problem with 6.5 release...

  Changed 11 months ago by kalxas

As long as we stay on LTS, I don't see sphinx getting an update. The bug is resolved upstream.

  Changed 9 months ago by hamish

  • status changed from new to closed
  • resolution set to wontfix
  • milestone changed from OSGeoLive7.0 to OSGeoLive7.5

7.0 is keeping with 12.04, hoping it is fixed in some future version. reopen if that doesn't happen after we upgrade sphinx.

Note: See TracTickets for help on using tickets.