[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#44492] [PATCH 01/52] gnu: Add rust-ruma-identifiers-validation-0.1.
From: |
Leo Prikler |
Subject: |
[bug#44492] [PATCH 01/52] gnu: Add rust-ruma-identifiers-validation-0.1. |
Date: |
Wed, 24 Feb 2021 14:13:29 +0100 |
User-agent: |
Evolution 3.34.2 |
Hello,
Am Mittwoch, den 24.02.2021, 12:46 +0100 schrieb Nicolas Goaziou:
> Hello,
>
> Leo Prikler <leo.prikler@student.tugraz.at> writes:
>
> > * gnu/packages/crates-io.scm (rust-ruma-identifiers-validation-
> > 0.1): New
> > variable.
>
> Thank you! I have a few general comments about the patch set.
>
> Nitpicks: some synopses end with a full stop, and most descriptions
> are
> not full sentences.
Indeed, for most of them I merely copied the output of `guix import
cargo`. I was hoping a reviewer could help me find better synopses and
descriptions, as well as cross-check licenses.
> If you introduce a new version of an existing package, the old
> package
> should inherit from the new one.
I was afraid this would cause rebuilds for the existing package. Was
that fear unfounded?
> More generally, I still think intermediate packages should use
> #:skip-build #t. Building them brings very little information, if
> any:
>
> - A crate failing to build, for various reasons, is still correct as
> an
> input to another crate
> - Even if all intermediate crates have "#:skip-build #t", building
> the
> top-level crate locates accurately any missing Cargo input in the
> dependency graph.
>
> Not using #:skip-build, OTOH, costs a lot of resources and time for
> the
> CI, for users and developers.
I personally disagree. The only reason a crate failing to build is a
"valid input" to another is because that other crate can decide to
completely disregard it, which sounds neither "reliable" nor
"efficient" for a programming language, that prides itself as both.
I will only skip builds for dead crates, i.e. crates I can reasonably
assume to only contain dead code due to their build failures. This
does not seem to cost much when building dependant packages, as I've
found that in order to actually build the crates I have to explicitly
invoke `guix build <crate>`.
Of course, there's still the problem of CI. Long-term, I think we
should find a way for this efficient programming language to reliably
produce reusable build artifacts. Short term, hitting non-leaf
packages, that have cargo-build-system anywhere with a priority of
negative infinity sounds like a better workaround. I want to be able
(as a developer) to explicitly build crates and determine where they
fail.
> Of course, this last remark is not specific to your patch set. I wish
> we
> can converge towards common Rust packaging guidelines.
I believe we should not cowtow to Rust and Cargo, but instead force
them to adhere to our principles; principles of building applications
*and* libraries reproducibly without encoding hashes in a huge ass-lock
file.
Sorry for the off-topic rant. Dealing with Rust for five days straight
has been somewhat damaging to my mental health, and to be very clear,
that is Rust's fault and not the fault of cargo-build-system.
Regards,
Leo
- [bug#44492] [PATCH 29/52] gnu: Add rust-letter-avatar-1., (continued)
- [bug#44492] [PATCH 29/52] gnu: Add rust-letter-avatar-1., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 32/52] gnu: Add rust-loggerv-0.7., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 34/52] gnu: Add rust-lmdb-0.8., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 38/52] gnu: Add rust-aes-0.6., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 44/52] gnu: Add rust-atk-sys-0.10., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 40/52] gnu: Add rust-hmac-0.10., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 46/52] gnu: Add rust-gtk-sys-0.10., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 47/52] gnu: Add rust-gtk-0.9., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 50/52] gnu: Add rust-sourceview4-for-fractal., Leo Prikler, 2021/02/24
- [bug#44492] [PATCH 01/52] gnu: Add rust-ruma-identifiers-validation-0.1., Nicolas Goaziou, 2021/02/24
- [bug#44492] [PATCH 01/52] gnu: Add rust-ruma-identifiers-validation-0.1.,
Leo Prikler <=
- [bug#44492] [PATCH v2 03/52] gnu: Add rust-ruma-identifiers-0.17., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 09/52] gnu: rust-either-1: Update to 1.6.1., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 00/52] gnu: Add fractal., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 02/52] gnu: Add rust-ruma-identifiers-macros-0.17., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 01/52] gnu: Add rust-ruma-identifiers-validation-0.1., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 08/52] gnu: Add rust-fragile-1., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 05/52] gnu: Add rust-twoway-0.2., Leo Prikler, 2021/02/28
- [bug#44492] [PATCH v2 06/52] gnu: Add rust-unicode-categories-0.1., Leo Prikler, 2021/02/28