Re: Guile 64-bit Windows support, redux

"Dr. Arne Babenhauserheide" <[email protected]> Tue, 25 Feb 2025 00:02:09 +0100
Newsgroups gmane.lisp.guile.devel
Message-ID <[email protected]>
Jonas Hahnfeld <[email protected]> writes:
> If I'm honest, it would help more to get reviews from people actually
> looking at the code...

For some context: we are struggling to find people with experience in
Windows and rlb went as deep as was possible in their limited time.

I asked rlb to provide the feedback they had written informally in IRC
by posting it here in the mailing list, because while this may not be an
in-depth review, this can get us to the point where I — who has nil
Windows knowledge but trusts review processes — can merge the patches
into Guile.

So if a review seems superficial, that isn’t an attack or a negative
judgement of the patch quality, but rather means that someone followed
up with the review due to being asked despite knowing that they couldn’t
give it as much attention as it deserves — and as it would get if we
could simply allocate resources.

And to quote from IRC:

> they have (lilypond) a few guile/C wizards there

You are the experts for C on Windows and if you say that the points a
reviewer worries about do not cause problems, then I take your word on
that, apply the patches, run the tests on GNU/Linux, and merge them.

The point of getting someone to review here is to get a second
perspective that includes a different set of use-cases.

Best wishes,
Arne
-- 
Unpolitisch sein
heißt politisch sein,
ohne es zu merken.
draketo.de
signature.asc (application/pgp-signature, 1.1 KB)
-----BEGIN PGP SIGNATURE-----

iQJEBAEBCAAuFiEE801qEjXQSQPNItXAE++NRSQDw+sFAme8+nEQHGFybmVfYmFi
QHdlYi5kZQAKCRAT741FJAPD68CID/9LHHd8+M8kJo9isnevoZF8GNbnQd0mUcAV
uUjvxQS1y6oxBTDx53enXQIiag0WO8BbwZASmYS+eQpLf3clkuXItMnfFSVsPHWk
+VjcyzZ7THxkdukJzwNL17Q9Xa5JANoFuMIhEdgueMfIHx75iY6MrPDBvcWvtuxi
I4BQ6U+3UoJzm1UDF85McfPYn6Q2Iv/eO1ypExCYKt2ge+wnZO6A+xMkb1BTVH1K
J5aLj4gNCHor3kFuBQ5pwkcqh1Ey915R5X8rlx/ezWl9/TM2nusj00pfIIq7tO/r
RVE2dvvUxk7BKDFJCrrUAO/inl3xILL8DSN5TrhKeKb/Q584Cfl3c/VXQzYkzo7s
zRXTufjZh+kUr7uh3DGi//LFFRSM+ZK2Y661aVvcDIW2C+q/YJWzFvamc61rKREP
53BXBHB8i/wjAViAUcjTmn431szmOYJt0O+3PfN39OJvA/M7CHp0xueIounOJPXA
oD7BMxX0fGCiyf/afmKmwsmuMiT/w5U9RP3lSPxTtiGqK1i3V7XjWpaHfRPjUvDs
grYlcRm2z12Ql+07h1TQWcN/GtuWEtsz3l4gkuU/K8rHKa/Z4vgu1hIEibIzx8EV
7aDfTJPgezWxaP/310VcH0Cb3Gw+GUygeuxm+QGUrL8+07pBzkaboKLPozZaD+g8
Aqgyf5vNIYjEBAEBCAAuFiEE3Si95tmHXKvOSosd3M8NswvBBUgFAme8+nEQHGFy
bmVfYmFiQHdlYi5kZQAKCRDczw2zC8EFSNoUBACX7IggkcIQsuQFnjkkwYHXXpuf
KVWjq8r9KITmDcYhk340SWoD/Lu5SLjPQfdczecs04GlLvccAymOgLKrNP7pMaUK
5WYa3pYfylNPQKwNOCg3yLbeu1VG3uTdIuA/AC8Rka0yMXDj6B8/9nbafyvt62+z
A6KaDXdNqOXt8L8nfg==
=yEfL
-----END PGP SIGNATURE-----