all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Eric Bavier <ericbavier@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] guix: refresh: Use bags.
Date: Sun, 12 Oct 2014 00:09:26 +0200	[thread overview]
Message-ID: <87ppdy45t5.fsf@gnu.org> (raw)
In-Reply-To: <87iojrus6j.fsf@gmail.com> (Eric Bavier's message of "Fri, 10 Oct 2014 23:49:24 -0500")

Eric Bavier <ericbavier@gmail.com> skribis:

> Ludovic Courtès writes:
>
>> I’m surprised gzip, diffutils, patch, and tar are very low, even though
>> they’re part of %final-inputs.  Conversely, bzip2 (also part of
>> %final-inputs) is higher.
>>
>> Any idea?
>
> I think this is because fold-packages only looks at packages that are
> exported by a module.  Many of the packages in %final-inputs are not
> exported.
>
> While the vhash returned by (@ (gnu packages) package-dependencies)
> includes dependency lists for packages in %final-inputs, guix refresh
> uses find-package-by-name to get the packages that are used as lookup
> keys, and find-package-by-name only returns packages that can be found
> with fold-packages.  Uff da.

Ah OK.  It’s not that bad, it just means that some packages are omitted
from the list.

Of course it would be good to use the full DAG.  For that we’d need
something that uses ‘fold-packages’ to get the “entry points” of the
DAG, and then traverses it all to also get the private packages.

From there we could perhaps build a ‘fold-package*’ that would iterate
on the whole DAG.

WDYT?

> One, possibly ugly to implement, solution would be to merge hash entries
> of packages that have the same package-full-name, since the output is
> produced in that format already.

Instead, what about keeping things as is, but also showing the
‘package-location’ of each item?

Thanks,
Ludo’.

  reply	other threads:[~2014-10-11 22:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-10 19:32 [PATCH] guix: refresh: Use bags Eric Bavier
2014-10-10 19:42 ` David Thompson
2014-10-10 20:42 ` Ludovic Courtès
2014-10-11  1:46   ` Eric Bavier
2014-10-11  4:49   ` Eric Bavier
2014-10-11 22:09     ` Ludovic Courtès [this message]
2014-10-13 19:18       ` Eric Bavier
2014-10-13 21:31         ` Ludovic Courtès
2014-10-14 17:06           ` Eric Bavier
2014-10-14 19:09             ` Ludovic Courtès
2014-10-20 16:58           ` Eric Bavier
2014-10-25 21:36             ` Ludovic Courtès

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=87ppdy45t5.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=ericbavier@gmail.com \
    --cc=guix-devel@gnu.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.