GNU bug report logs - #57923
[PATCH] gnu: Add sov.

Previous Next

Full log

Message #8 received at 57923 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: kiasoc5 <kiasoc5 <at> disroot.org>, 57923 <at> debbugs.gnu.org
Subject: Re: [bug#57923] [PATCH] gnu: Add sov.
Date: Mon, 19 Sep 2022 18:35:56 +0200

[Message part 1 (text/plain, inline)]

On 19-09-2022 06:32, kiasoc5 via Guix-patches via wrote:
> +   (inputs (list wayland))
> +   (native-inputs (list pkg-config freetype wayland-protocols))

I think 'freetype' and maybe 'wayland-protocols' might be sorted 
incorrectly -- 'freetype' is usually used at runtime (hence 'inputs', 
not 'native-inputs') and wayland-protocols is often used for its 
'wayland-scanner' (or maybe that was 'wayland'), hence 'native-inputs' 
(sometimes to resolve cross-compilation errors it needs to be both 
'native-inputs' and 'inputs').

To test, you can try to cross-compile:

./pre-inst-env guix build sov --target=aarch64-linux-gnu

and move inputs as needed to resolve compilation failures.

Also, it looks like it bundles a copy of jsmn, at
https://github.com/milgra/sov/blob/main/src/modules/json/jsmn.c
-- while unbundling is not required if it's the only copy in Guix, a ";; 
TODO bundles jsmn" would be useful to keep track of bundling.

(Actually there is another copy though, in 'acme-client', but strictly 
speaking '(guix)Submitting Patches' only talks about 'already available 
as separate packages', not 'there is another copy'.)

In src/sov/config/fontconfig.c, there is some code for starting 
"fc-match", which needs a 'substitute*' so fc-match will be found even 
if fontconfig is not in the current environment.

Greetings,
Maxime.

[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]

[OpenPGP_signature (application/pgp-signature, attachment)]

Previous Next

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