From: Maxime Devos <maximedevos@telenet.be>
To: kiasoc5 <kiasoc5@disroot.org>, 57923@debbugs.gnu.org
Subject: [bug#57923] [PATCH] gnu: Add sov.
Date: Mon, 19 Sep 2022 18:35:56 +0200 [thread overview]
Message-ID: <74bdb638-9570-5983-168e-410b0abc9052@telenet.be> (raw)
In-Reply-To: <cc5985eb40bec0b88790edee3d5702834bbc9f47.1663561961.git.kiasoc5@disroot.org>
[-- Attachment #1.1.1: Type: text/plain, Size: 1380 bytes --]
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.
[-- 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 --]
prev parent reply other threads:[~2022-09-19 16:37 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-19 4:32 [bug#57923] [PATCH] gnu: Add sov kiasoc5 via Guix-patches via
2022-09-19 16:35 ` Maxime Devos [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=74bdb638-9570-5983-168e-410b0abc9052@telenet.be \
--to=maximedevos@telenet.be \
--cc=57923@debbugs.gnu.org \
--cc=kiasoc5@disroot.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.