unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#56956: rust-pico-sys bundles PicoHTTPParser.
@ 2022-08-03 18:18 Maxime Devos
  2022-08-05 15:22 ` Efraim Flashner
  0 siblings, 1 reply; 3+ messages in thread
From: Maxime Devos @ 2022-08-03 18:18 UTC (permalink / raw)
  To: 56956; +Cc: efraim flashner


[-- Attachment #1.1.1: Type: text/plain, Size: 601 bytes --]

X-Debbugs-CC: Efraim Flashner <efraim@flashner.co.il>

(^ original package author+committer)

rust-pico-sys was added in commit
<https://git.savannah.gnu.org/cgit/guix.git/commit/?id=eda57f4834b29ef17758f7f1f62d3af899e2d669>.

This package bundles PicoHTTPParser.

Fortunately, it is only used by a single dependency: rust-httparser, and 
only for benchmarks (which probably aren't compiled anyway), so simply 
removing the package definition and removing it from rust-httparser' 
#:dev-dependencies should be sufficient.  This appears to work in 
antioxidant.

Greetings,
Maxime.


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#56956: rust-pico-sys bundles PicoHTTPParser.
  2022-08-03 18:18 bug#56956: rust-pico-sys bundles PicoHTTPParser Maxime Devos
@ 2022-08-05 15:22 ` Efraim Flashner
  2022-08-05 15:24   ` Maxime Devos
  0 siblings, 1 reply; 3+ messages in thread
From: Efraim Flashner @ 2022-08-05 15:22 UTC (permalink / raw)
  To: Maxime Devos; +Cc: 56956-done

[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]

On Wed, Aug 03, 2022 at 08:18:36PM +0200, Maxime Devos wrote:
> X-Debbugs-CC: Efraim Flashner <efraim@flashner.co.il>
> 
> (^ original package author+committer)
> 
> rust-pico-sys was added in commit
> <https://git.savannah.gnu.org/cgit/guix.git/commit/?id=eda57f4834b29ef17758f7f1f62d3af899e2d669>.
> 
> This package bundles PicoHTTPParser.
> 
> Fortunately, it is only used by a single dependency: rust-httparser, and
> only for benchmarks (which probably aren't compiled anyway), so simply
> removing the package definition and removing it from rust-httparser'
> #:dev-dependencies should be sufficient.  This appears to work in
> antioxidant.

It turns out rust-httparser-1 doesn't build on its own without
rust-pico-sys-0.0, but updating rust-httparser-1 from 1.5.1 to 1.7.1
changed the development-inputs and allowed me to remove
rust-pico-sys-0.0.



-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#56956: rust-pico-sys bundles PicoHTTPParser.
  2022-08-05 15:22 ` Efraim Flashner
@ 2022-08-05 15:24   ` Maxime Devos
  0 siblings, 0 replies; 3+ messages in thread
From: Maxime Devos @ 2022-08-05 15:24 UTC (permalink / raw)
  To: Efraim Flashner, 56956-done


[-- Attachment #1.1.1: Type: text/plain, Size: 22 bytes --]

Thanks!

Maxime.


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-08-05 15:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-03 18:18 bug#56956: rust-pico-sys bundles PicoHTTPParser Maxime Devos
2022-08-05 15:22 ` Efraim Flashner
2022-08-05 15:24   ` Maxime Devos

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).