Re: [PATCH] distro: Add GLib.

[Ludovic Courtès]

Hi,

Nikita Karetnikov <address@hidden> skribis:

> This patch adds GLib.

Nice!

> Tests fail:
>
>   /GDateTime/equal:                                                    **
> GLib:ERROR:gdatetime.c:193:test_GDateTime_equal: assertion failed 
> (g_date_time_get_utc_offset (dt1) / G_USEC_PER_SEC == (-3 * 3600)): (0 == 
> -10800)
> FAIL

Is it the only test that fails?  It would be nice to investigate since
it’s a foundational library for many packages.

> By the way, I'm running the daemon with the following options:
>
> # ./pre-inst-env guix-daemon -c 0 --build-users-group=nixbld \
>> --chroot-directory=/bin --chroot-directory=/dev --chroot-directory=/proc &
>
> Is it OK?

No.  :-)  So far, I’ve built without any additional directory in the chroot.

/dev and /proc are already in by default.  Builders normally no longer
rely on /bin since the ‘core-updates’ merge, so it’s not needed; it
should even be avoided since that would lead to “impurities”.

Thanks,
Ludo’.