all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#58610] [PATCH] gnu: rust-generic-array-0.14: Fix build.
@ 2022-10-18 14:59 Michael Rohleder
  2022-10-21 16:18 ` Christopher Baines
  0 siblings, 1 reply; 4+ messages in thread
From: Michael Rohleder @ 2022-10-18 14:59 UTC (permalink / raw)
  To: 58610; +Cc: Michael Rohleder

* gnu/packages/crates-io.scm (rust-generic-array-0.14)[arguments]: Add #:tests? #false.
---
 gnu/packages/crates-io.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/crates-io.scm b/gnu/packages/crates-io.scm
index 502972ffc5..afd1305018 100644
--- a/gnu/packages/crates-io.scm
+++ b/gnu/packages/crates-io.scm
@@ -24886,7 +24886,8 @@ (define-public rust-generic-array-0.14
           "05qqwm9v5asbil9z28wjkmpfvs1c5c99n8n9gwxis3d3r3n6c52h"))))
     (build-system cargo-build-system)
     (arguments
-     `(#:cargo-inputs
+     `(#:tests? #false                  ;doc test fails
+       #:cargo-inputs
        (("rust-serde" ,rust-serde-1)
         ("rust-typenum" ,rust-typenum-1)
         ("rust-version-check" ,rust-version-check-0.9))
-- 
2.38.0





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

* [bug#58610] [PATCH] gnu: rust-generic-array-0.14: Fix build.
  2022-10-18 14:59 [bug#58610] [PATCH] gnu: rust-generic-array-0.14: Fix build Michael Rohleder
@ 2022-10-21 16:18 ` Christopher Baines
  2023-01-03 22:02   ` Maxim Cournoyer
  2023-01-06 23:15   ` bug#58610: " Michael Rohleder
  0 siblings, 2 replies; 4+ messages in thread
From: Christopher Baines @ 2022-10-21 16:18 UTC (permalink / raw)
  To: Michael Rohleder; +Cc: 58610

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


Michael Rohleder <mike@rohleder.de> writes:

> * gnu/packages/crates-io.scm (rust-generic-array-0.14)[arguments]: Add #:tests? #false.
> ---
>  gnu/packages/crates-io.scm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

This may fix the build, but instead we'd just be shipping software where
the test suite fails.

If we do want to go that way of disabling the test suite, I think it
would be good to make sure the issue is reported upstream, and link to
that bug report in the package definition, does that make sense?

Thanks,

Chris

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

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

* [bug#58610] [PATCH] gnu: rust-generic-array-0.14: Fix build.
  2022-10-21 16:18 ` Christopher Baines
@ 2023-01-03 22:02   ` Maxim Cournoyer
  2023-01-06 23:15   ` bug#58610: " Michael Rohleder
  1 sibling, 0 replies; 4+ messages in thread
From: Maxim Cournoyer @ 2023-01-03 22:02 UTC (permalink / raw)
  To: Christopher Baines; +Cc: Michael Rohleder, 58610

Hello Michael,

Christopher Baines <mail@cbaines.net> writes:

> Michael Rohleder <mike@rohleder.de> writes:
>
>> * gnu/packages/crates-io.scm (rust-generic-array-0.14)[arguments]: Add #:tests? #false.
>> ---
>>  gnu/packages/crates-io.scm | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> This may fix the build, but instead we'd just be shipping software where
> the test suite fails.
>
> If we do want to go that way of disabling the test suite, I think it
> would be good to make sure the issue is reported upstream, and link to
> that bug report in the package definition, does that make sense?

Friendly ping.

-- 
Thanks,
Maxim




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

* bug#58610: [PATCH] gnu: rust-generic-array-0.14: Fix build.
  2022-10-21 16:18 ` Christopher Baines
  2023-01-03 22:02   ` Maxim Cournoyer
@ 2023-01-06 23:15   ` Michael Rohleder
  1 sibling, 0 replies; 4+ messages in thread
From: Michael Rohleder @ 2023-01-06 23:15 UTC (permalink / raw)
  To: Christopher Baines; +Cc: 58610-done

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

Hello Christopher!

Thank you for reviewing and sorry for the delay!

Christopher Baines <mail@cbaines.net> writes:
> This may fix the build, but instead we'd just be shipping software where
> the test suite fails.
>
> If we do want to go that way of disabling the test suite, I think it
> would be good to make sure the issue is reported upstream, and link to
> that bug report in the package definition, does that make sense?

I guess this make a lot of sense, but I can't do this, because I don't
have any rust/cargo knowledge, no gh account etc.

This patch was just a quick hack to get synapse working.  I should have
thought about fixing it better than just disabling a test.

Sorry for the waste of time and the noise -> closing.

-- 
There are two types of hard drives: those that have failed, and those that will.

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

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

end of thread, other threads:[~2023-01-06 23:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-18 14:59 [bug#58610] [PATCH] gnu: rust-generic-array-0.14: Fix build Michael Rohleder
2022-10-21 16:18 ` Christopher Baines
2023-01-03 22:02   ` Maxim Cournoyer
2023-01-06 23:15   ` bug#58610: " Michael Rohleder

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.