[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#70880] [PATCH v2 6/8] gnu: Add prusa-libbgcode.
From: |
Artyom V. Poptsov |
Subject: |
[bug#70880] [PATCH v2 6/8] gnu: Add prusa-libbgcode. |
Date: |
Sat, 13 Jul 2024 12:41:48 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
> Sorry for missing that in my first review: there should be an
> explanatory comment here mentioning which commit to use and why (e.g.:
> "Use the latest commit since there are no proper releases nor tags.")
> [...]
> Nitpick: when there are few inputs (< 5), it's OK to list them inline if
> it fits the 80 columns rule, like:
>
> (propagated-inputs (list zlib boost heatshrink))
>
> I think this is what 'guix style' would do.
> [...]
> It looks like this package does not provide the Python bindings, so it
> may be best to drop that part of the last sentence mentioning their
> existence, as it would be misleading for Guix users.
Fixed.
- avp
--
Artyom "avp" Poptsov <poptsov.artyom@gmail.com>
Home page: https://memory-heap.org/~avp/
CADR Hackerspace co-founder: https://cadrspace.ru/
GPG: D0C2 EAC1 3310 822D 98DE B57C E9C5 A2D9 0898 A02F
signature.asc
Description: PGP signature