1. Michael Bayer
  2. sqlalchemy
  3. Issues


Issue #3009 wontfix

tarball contains only minified versions of javascript libraries

Piotr Ożarowski
created an issue


Could you please add sources (i.e. not only minified versions) for below files in the tarball?

  • doc/_static/jquery.js
  • doc/_static/underscore.js

I rebuild docs in Debian anyway, but we distribute original tarball as well and hence need all the sources (I can rebuild the tarball to remove all generated docs, but I'd prefer to keep the original tarball)

Comments (15)

  1. Michael Bayer repo owner

    This has come up before but it seems like it wasnt a SQLAlchemy ticket.

    1. searchindex.js is not source code, it's a data file. nobody wrote this file.

    2. jquery.js/underscore.js come directly from Sphinx. Isn't there a debian package for Sphinx? and if so, why isn't this a Sphinx issue? I'd rather not have to start re-implementing sphinx's source code in my own package.

  2. Piotr Ożarowski reporter

    I know all of these files are generated by Sphinx and I didn't see a problem with that, but I guess I will have to repackage the tarball and remove these files to make everyone happy.

  3. Piotr Ożarowski reporter

    minified versions of libraries are considered non-free in Debian (and AFAICT also f.e. in RedHat/Fedora) so not minifying (at least by default, or adding a switch to disable it) in sphinx-build would be great

  4. Michael Bayer repo owner

    based on the discussion over there it's looking pretty doubtful SQLA's .tar.gz can be packaged as-is into a debian package. It might be helpful if there were some tool that folks could use to package all of my tar.gzs, which are all in the identical format (mako, alembic, dogpile, sqla, etc.) into debian-acceptable packages.

    If something changes over there and there's action we can take here, then please reopen.

  5. Piotr Ożarowski reporter

    perfect solution for us is a tarball with sources only, but if you want to include pregenerated docs as well, it's acceptable

    in other words: please use minifiedjs=False once it's available

  6. Log in to comment