Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.maint.python > #17231

Re: review for datasette/0.65.2-1

From Mahangu Weerasinghe <mahangu@gmail.com>
Newsgroups linux.debian.maint.python
Subject Re: review for datasette/0.65.2-1
Date 2025-12-05 08:10 +0100
Message-ID <LYy6Z-vc8-5@gated-at.bofh.it> (permalink)
References <LWqGB-gBQ3-1@gated-at.bofh.it> <LWS9P-gWOP-1@gated-at.bofh.it> <LXg2t-hcIe-9@gated-at.bofh.it> <LY7q9-cQ5-1@gated-at.bofh.it> <LYljs-mii-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

Hi Jeroen,

Thanks for catching these and for the housekeeping on d/rules!

> table.js in missing-sources

Removed. Added lintian override with comment noting it's a false positive
(long lines, not minified).

> codemirror-5.57.0.min.css missing from d/missing-sources

Added codemirror-5.57.0.css from the 5.57.0 tag [0]. Updated README
accordingly.

> stack limit for all 32-bit archs

Noted on extending to sh4, m68k, etc. Will keep that in mind for the future!

Best,
Mahangu

[0] https://github.com/codemirror/codemirror5/blob/5.57.0/lib/codemirror.css


On Thu, Dec 4, 2025 at 10:58 PM Jeroen Ploemen <jcfp@debian.org> wrote:

> hi Mahangu,
>
> was doing some last minute checks before upload when I noticed these:
>
> * for datasette/static/table.js, the file in missing-sources seems to
>   be identical to the one in the orig tarball - neither is minified,
>   it just has a long line that triggers the lintian hit. So there's
>   no need for this to make an appearance in missing-sources.
>
> * datasette/static/codemirror-5.57.0.min.css on the other hand is
>   actually minified, but not present in d/missing-sources. I guess
>   lintian only looks for certain types of files, and css isn't among
>   them. Anyway, the original appears to live at [1] (and probably
>   somewhere in a codemirror tarball as well).
>
>
> I also changed the stack limit to cover all 32 bit archs, since
> there's a few ports that are also 32 bit (sh4, mk68, etc.), and did
> some housekeeping to trim down long lines in d/rules and remove
> overrides made redundant by other changes.
>
>
> [1]
> https://github.com/codemirror/codemirror5/blob/master/lib/codemirror.css
>

Back to linux.debian.maint.python | Previous | NextPrevious in thread | Find similar


Thread

review for datasette/0.65.2-1 Jeroen Ploemen <jcfp@debian.org> - 2025-11-29 11:50 +0100
  Re: review for datasette/0.65.2-1 Mahangu Weerasinghe <mahangu@gmail.com> - 2025-11-30 17:10 +0100
    Re: review for datasette/0.65.2-1 Jeroen Ploemen <jcfp@debian.org> - 2025-12-01 18:40 +0100
      Re: review for datasette/0.65.2-1 Mahangu Weerasinghe <mahangu@gmail.com> - 2025-12-04 03:40 +0100
        Re: review for datasette/0.65.2-1 Jeroen Ploemen <jcfp@debian.org> - 2025-12-04 18:30 +0100
          Re: review for datasette/0.65.2-1 Mahangu Weerasinghe <mahangu@gmail.com> - 2025-12-05 08:10 +0100

csiph-web