unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / Atom feed
* bug#44745: tests/lint.scm fails building guix with guile-2.2
@ 2020-11-20  0:35 Vagrant Cascadian
  2020-11-20  9:32 ` zimoun
  0 siblings, 1 reply; 3+ messages in thread
From: Vagrant Cascadian @ 2020-11-20  0:35 UTC (permalink / raw)
  To: 44745

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

I'm exploring building with guile 2.2 because guile-gnutls built with
guile 3.0 is only available in experimental, and even there, missing for
arm64.

From tests/lint.log:

test-name: archival: missing content
location: /build/guix-EdK9LP/guix-1.2.0~rc2/tests/lint.scm:921
source:
+ (test-assert
+   "archival: missing content"
+   (let* ((origin
+            (origin
+              (method url-fetch)
+              (uri "http://example.org/foo.tgz")
+              (sha256 (make-bytevector 32))))
+          (warnings
+            (with-http-server
+              '((404 "Not archived."))
+              (parameterize
+                ((%swh-base-url (%local-url)))
+                (check-archival
+                  (dummy-package "x" (source origin)))))))
+     (warning-contains? "not archived" warnings)))
actual-value: #f
actual-error:
+ (keyword-argument-error
+   #<procedure http-request (uri #:key body port method version keep-alive? headers decode-body? streaming? request)>
+   "Unrecognized keyword"
+   ()
+   (#:verify-certificate?))
result: FAIL


I haven't tried reproducing this without the Debian patches applied
which liberally sprinkle the test suites with:

  (unless (network-reachable?  (test-skip 1))

... but if you can spot a likely issue, I'd be happy to test it. :)


live well,
  vagrant

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

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

* bug#44745: tests/lint.scm fails building guix with guile-2.2
  2020-11-20  0:35 bug#44745: tests/lint.scm fails building guix with guile-2.2 Vagrant Cascadian
@ 2020-11-20  9:32 ` zimoun
  2020-11-20 11:47   ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: zimoun @ 2020-11-20  9:32 UTC (permalink / raw)
  To: Vagrant Cascadian, 44745

Hi Vagrant,

On Thu, 19 Nov 2020 at 16:35, Vagrant Cascadian <vagrant@debian.org> wrote:

> test-name: archival: missing content
> location: /build/guix-EdK9LP/guix-1.2.0~rc2/tests/lint.scm:921
> source:
> + (test-assert
> +   "archival: missing content"
> +   (let* ((origin
> +            (origin
> +              (method url-fetch)
> +              (uri "http://example.org/foo.tgz")
> +              (sha256 (make-bytevector 32))))
> +          (warnings
> +            (with-http-server
> +              '((404 "Not archived."))
> +              (parameterize
> +                ((%swh-base-url (%local-url)))
> +                (check-archival
> +                  (dummy-package "x" (source origin)))))))
> +     (warning-contains? "not archived" warnings)))
> actual-value: #f
> actual-error:
> + (keyword-argument-error
> +   #<procedure http-request (uri #:key body port method version keep-alive? headers decode-body? streaming? request)>
> +   "Unrecognized keyword"
> +   ()
> +   (#:verify-certificate?))
> result: FAIL
>

I remember issues fixed by 722ad41c44a499d2250c79527ef7d069ca728de0
which maybe introduce a regression for Guile 2.2.

--8<---------------cut here---------------start------------->8---
+;; XXX: Work around a bug in Guile 3.0.2 where #:verify-certificate? would
+;; be ignored (<https://bugs.gnu.org/40486>).
+(define* (http-get* uri #:rest rest)
+  (apply http-request uri #:method 'GET rest))
+(define* (http-post* uri #:rest rest)
--8<---------------cut here---------------end--------------->8---

Maybe it is related.


All the best,
simon




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

* bug#44745: tests/lint.scm fails building guix with guile-2.2
  2020-11-20  9:32 ` zimoun
@ 2020-11-20 11:47   ` Ludovic Courtès
  0 siblings, 0 replies; 3+ messages in thread
From: Ludovic Courtès @ 2020-11-20 11:47 UTC (permalink / raw)
  To: zimoun; +Cc: Vagrant Cascadian, 44745

Hi,

zimoun <zimon.toutoune@gmail.com> skribis:

> I remember issues fixed by 722ad41c44a499d2250c79527ef7d069ca728de0
> which maybe introduce a regression for Guile 2.2.
>
> +;; XXX: Work around a bug in Guile 3.0.2 where #:verify-certificate? would
> +;; be ignored (<https://bugs.gnu.org/40486>).
> +(define* (http-get* uri #:rest rest)
> +  (apply http-request uri #:method 'GET rest))
> +(define* (http-post* uri #:rest rest)
>
> Maybe it is related.

Yeah, #:verify-certificate? is new in 3.0.x:

  https://git.savannah.gnu.org/cgit/guile.git/tree/NEWS#n599

Vagrant, perhaps the simplest option for you would be to skip these
tests.  There’s no rush but I think we’ll remove 2.2 support in the
not-too-distant future.

Ludo’.




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

end of thread, other threads:[~2020-11-20 11:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-20  0:35 bug#44745: tests/lint.scm fails building guix with guile-2.2 Vagrant Cascadian
2020-11-20  9:32 ` zimoun
2020-11-20 11:47   ` Ludovic Courtès

unofficial mirror of bug-guix@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-bugs/0 guix-bugs/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-bugs guix-bugs/ https://yhetil.org/guix-bugs \
		bug-guix@gnu.org
	public-inbox-index guix-bugs

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.bugs
	nntp://news.gmane.io/gmane.comp.gnu.guix.bugs


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git