GNU bug report logs - #54644
[PATCH 1/7] gnu: rust-nom-locate-4: Add 4.0.0

Previous Next

Package: guix-patches;

Reported by: Evgenii Lepikhin <e.lepikhin <at> corp.mail.ru>

Date: Wed, 30 Mar 2022 21:22:02 UTC

Severity: normal

Tags: patch

Done: Efraim Flashner <efraim <at> flashner.co.il>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Maxime Devos <maximedevos <at> telenet.be>
To: Evgenii Lepikhin <e.lepikhin <at> corp.mail.ru>
Cc: 54644 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>
Subject: [bug#54644] [PATCH 4/7] gnu: rust-minimal-lexical 0.2: Update to 0.2.1.
Date: Fri, 01 Apr 2022 08:38:23 +0200
[Message part 1 (text/plain, inline)]
Evgenii Lepikhin schreef op vr 01-04-2022 om 01:30 [+0300]:
> nom-7.0 is not forward compatible with nom-7.1. I need 7.1 for future
> contributions and this is why I added it into patchset.

Related: to reduce duplication, you can do

(define-public rust-nom-7.1
  (package
    (inherit rust-nom-7)
    [...]))

that way, you can ‘copy’ the synopsis of rust-nom-7 without writing it
down again.

Also, if semver is used here, nom-7.0 is compatible with nom-7.1 (not
sure about the terminology here -- forwards? backwards?, but updating
it shouldn't cause any problems if semver is followed correctlu).

> In developer's manifest of nom-7.1 dependency on rust-minimal-lexical
> has been updated from 0.1 to 0.2. For both versions dependencies on
> exact versions on rust-minimal-lexical are pinned by crate developers
> and we cannot change it at will

We can do that, with some 'substitute*'.  It is not guaranteed that it
will actually build, but there's a good chance that it will.  Or that
it fails.  Hard to say in advance.  See, e.g., rust-version-sync <at> 0.8.

Greetings,
Maxime.
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 3 years and 51 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.