Review: dart-material-color-utilities 0.13.0+ds-1
Package Information
| Description | Algorithms that power the Material Design 3 colour system This package ships the material_color_utilities Dart library as sources so that other Dart packages can depend on it at build time. It picks theme colours from an image, creates tonal palettes and converts colours between ARGB, HCT and other colour spaces, following the Material Design 3 specification. |
|---|---|
| Maintainer | Emfox Zhou <emfox@debian.org> |
| Changed By | Emfox Zhou <emfox@debian.org> |
| Sponsor | emfox@debian.org |
| Distribution | unstable |
| Architecture | all |
| VCS | git: https://salsa.debian.org/dart-team/dart-material-color-utilities.git (browse) |
| Closes | #1151949 |
| Tracker | https://tracker.debian.org/pkg/dart-material-color-utilities |
| Uploaded | 4 hours ago |
New Package Report
.changes
| Version | 0.13.0+ds-1 |
|---|---|
| Changed-By | Emfox Zhou <emfox@debian.org> |
| Architecture | source all |
| Distribution | unstable |
| Date | Sun, 25 Oct 2026 11:15:00 +0800 |
| Source | dart-material-color-utilities |
Changelog
dart-material-color-utilities (0.13.0+ds-1) unstable; urgency=medium
.
* Initial release. (Closes: #1151949)
* Drop the upstream screenshots, which only illustrate the README,
from the source package..dsc
| Section | devel |
|---|---|
| Priority | optional |
| Component | main |
| Package-List | dart-material-color-utilities deb devel optional arch=all |
debian/copyright
Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ Upstream-Name: material_color_utilities Source: https://github.com/material-foundation/material-color-utilities/tree/main/dart Files-Excluded: screenshots Comment: The screenshots directory only holds images used by the upstream README and is dropped from the repacked upstream archive. Files: * Copyright: 2021-2023, Google LLC License: Apache-2.0 Files: debian/* Copyright: 2026, Emfox Zhou <emfox@debian.org> License: Apache-2.0 License: Apache-2.0 On Debian systems, the complete text of the Apache License 2.0 can be found in "/usr/share/common-licenses/Apache-2.0".
DFSG Assistant LLM REVIEW 0 blocker, 0 important, 1 minor
This is an LLM based review using a Debian Inference based workflow. It was not written by a member of the DFSG team and has not been checked by one. It is not a decision on the package: it may be wrong, and it may have missed things.
Reviewed 3 hours ago.
Report
DFSG NEW review for dart-material-color-utilities version 0.13.0+ds-1
Summary
- Licensing is DFSG-compliant and
debian/copyrightis accurate: Yes - Packaging is of good quality: Yes
- Issues: 0 blocker, 0 important, 1 minor
- debian/watch: version 5
- debian/copyright: machine-readable
- Scan before the review: nothing found
Licensing
The entire source tree is Apache-2.0, authored by Google LLC. Every .dart file carries an Apache-2.0 header naming Google LLC as holder, and the top-level LICENSE is the full Apache License 2.0 text. debian/copyright is machine-readable DEP-5 and correctly covers every file: a Files: * stanza naming Google LLC under Apache-2.0, an overriding Files: debian/* stanza for the packaging, and a stand-alone License paragraph referring to /usr/share/common-licenses/Apache-2.0. No non-free terms, no files lacking a licence, and no built/minified/precompiled files were found.
Packaging
Clean single-binary source-only packaging of a Dart library. debian/control uses debhelper-compat 14 with dh-sequence-single-binary, Standards-Version 4.7.4 (current), Architecture all with Multi-Arch foreign, and a correct Depends on dart-collection (imported by lib/palettes/tonal_palette.dart and lib/score/score.dart; present in the archive). rules is the standard dh $@ plus a sensible override stamping installed directories with SOURCE_DATE_EPOCH. The changelog closes the ITP, the install file places sources under /usr/share/dart/packages, and there is a meaningful offline autopkgtest smoke test exercising HCT round-trip and scheme generation.
Issues
1. [minor] [packaging] The repack suffix is not removed before versions are compared
- Files:
debian/watch - Basis: found by the review tooling's own checks
- Evidence: uscan compares
0.13.0+ds+dswith upstream's0.13.1(status:newer package available) - Problem: The packaged version carries a repack suffix that
debian/watchdoes not remove before comparing, so uscan compares it as it is with upstream's version and never finds them equal. - Fix: Remove the suffix for the comparison, with
Dversion-Mangle: auto(version 5) ordversionmangle=auto(version 4).
Verified
- Prescan (
dfsg-author-check) reported no missing copyright holders: only Google LLC and Emfox Zhou (debian/*) are declared, and every holder found in the tree is named. - All 64 scanned files (lib/, test/, example/, plus root config files) are Apache-2.0 with Google LLC as the sole upstream holder; sampled every copyright-bearing file in lib/, test/ and example/ via grep and confirmed the Apache-2.0 header.
- The full Apache License 2.0 text is shipped at LICENSE and referenced from /usr/share/common-licenses/Apache-2.0 in debian/copyright, so no full text needs inlining.
- The
screenshotsdirectory named inFiles-Excludedis absent from the tree (repacked+dsarchive); no images, fonts, archives, bytecode, or minified files present. - The smoke test's declared package
dart-collection(1.19.1-2) anddart-sdkare in the archive; collection is imported viapackage:collection. - No patches (no debian/patches directory), no maintainer scripts or *.maintscript, no dirs/links/docs files; nothing further to review there.
- uscan found upstream releases with
debian/watch; a newer upstream version is available (upstream0.13.1, packaged0.13.0+ds), which is not a fault of the package. - Every pointer of
debian/copyrightto /usr/share/common-licenses names a file that is there (Apache-2.0).
Not confirmed
None.
Proposed changes
None.
Uploader's response to the LLM review
The uploader was emailed about the review 3 hours ago.
Review Information
accepted — allocated to awm 0 hours ago, started 0 hours ago, completed 0 hours ago.
Final Comment
Hi,
A small fix for your debian/watch file in the LLM review.
Thanks!