public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Tom Wijsman <TomWij@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Re: [OT/NIT] Re: Re: [gentoo-commits] gentoo-x86 commit in profiles: ChangeLog package.mask
Date: Tue, 23 Apr 2013 22:27:53 +0200	[thread overview]
Message-ID: <20130423222753.3854fe4d@TOMWIJ-GENTOO> (raw)
In-Reply-To: <pan$1e4f5$800158e2$35c423dd$7f617b73@cox.net>

[-- Attachment #1: Type: text/plain, Size: 2034 bytes --]

On Mon, 22 Apr 2013 22:46:14 +0000 (UTC)
Duncan <1i5t5.duncan@cox.net> wrote:

> Alexis Ballier posted on Mon, 22 Apr 2013 15:40:33 +0200 as excerpted:
> 
> > "Having functional changes mixed with whitespace/cosmetics in a
> > single commit makes it hard to read and understand."
> > 
> >> Also, in this case I see only one extra hunk.
> >> But once we have proper tools (like git) we can revisit this.
> > 
> > I don't see how git helps. You'll have to commit twice then push, vs
> > commit twice with cvs.
> 
> But git commits are quite lightweight, while as someone already
> pointed out, cvs commits, if done properly with repoman, are anything
> but.
> 
> So at least in the sense that it'll be less hassle, two git commits 
> followed by a push should be much easier than two repoman and cvs
> commits.
> 

Maybe the question is rather why `repoman` takes 15 seconds on a quite
fast system in a package folder that contains 2 ebuilds and 1 metadata.

See the call graph for repoman at http://i.imgur.com/OQTUBdR.png.

A third of the time, ~5 seconds, are spent on 60,000 - 70,000 calls to
the function that is listed in the bottom; but the ebuilds barely
list any dependencies. Why are there so much calls to this?

A third of the time, ~5 seconds, are spent on visibility; if we look at
the `emerge -puDN @world` call graph (http://i.imgur.com/A93CdNR.png)
we also see visibility checks taking a prominent enough place. If this
is part is really needed, we may look at how we can improve this and at
the same time improvement the dependency calculation and repoman times.

Last third of the time, ~5 seconds, are spent on regular expressions
and mostly configuration; why is configuration so invasive here?

My thoughts are that we may be able to shave ten seconds of this time.

-- 
With kind regards,

Tom Wijsman (TomWij)
Gentoo Developer

E-mail address  : TomWij@gentoo.org
GPG Public Key  : 6D34E57D
GPG Fingerprint : C165 AF18 AB4C 400B C3D2  ABF0 95B2 1FCD 6D34 E57D

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

  parent reply	other threads:[~2013-04-23 20:29 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130419091632.D01152171D@flycatcher.gentoo.org>
2013-04-19 13:30 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in profiles: ChangeLog package.mask Alexis Ballier
2013-04-20 17:28   ` Jeroen Roovers
2013-04-21 12:53   ` Ben de Groot
2013-04-21 14:11     ` Denis Dupeyron
2013-04-21 14:23       ` Rich Freeman
2013-04-21 14:39         ` Denis Dupeyron
2013-04-21 14:38       ` Chí-Thanh Christopher Nguyễn
2013-04-21 14:50         ` Denis Dupeyron
2013-04-21 15:07           ` Chí-Thanh Christopher Nguyễn
2013-04-21 15:17             ` Alexis Ballier
2013-04-22 11:43         ` Ben de Groot
2013-04-22 17:13           ` Michał Górny
2013-04-23 16:16             ` Ben de Groot
2013-04-22 12:07       ` Ben de Groot
2013-04-21 14:59     ` Alexis Ballier
2013-04-22 11:56       ` Ben de Groot
2013-04-22 14:00         ` Alexis Ballier
2013-04-23  3:58         ` Ryan Hill
2013-04-23 16:24           ` Ben de Groot
2013-04-24 10:59             ` Duncan
2013-04-30  2:06             ` Ryan Hill
2013-04-21 15:05     ` [OT/NIT] " Alexis Ballier
2013-04-21 15:32       ` Tom Wijsman
2013-04-21 15:43         ` Alexis Ballier
2013-04-21 16:00         ` Peter Stuge
2013-04-21 17:07           ` Tom Wijsman
2013-04-21 18:47             ` Peter Stuge
2013-04-21 18:57       ` Michał Górny
2013-04-22 12:00       ` Ben de Groot
2013-04-22 13:40         ` Alexis Ballier
2013-04-22 22:46           ` [gentoo-dev] Re: [OT/NIT] " Duncan
2013-04-23 18:00             ` Jeroen Roovers
2013-04-23 18:20               ` Rich Freeman
2013-04-23 18:37                 ` Matt Turner
2013-04-23 19:11                   ` Rich Freeman
2013-04-23 19:21                     ` Matt Turner
2013-04-23 19:25                     ` Ian Stakenvicius
2013-04-23 19:43                       ` Rich Freeman
2013-04-23 18:38               ` Matt Turner
2013-04-24 11:18               ` Duncan
2013-04-24 11:21               ` Peter Stuge
2013-04-24 11:25                 ` Peter Stuge
2013-04-24 11:47                 ` Michael Mol
2013-04-24 13:25                   ` Rich Freeman
2013-04-24 20:04                     ` Alex Xu
2013-04-24 22:26                       ` Rich Freeman
2013-04-24 23:23                 ` Patrick Lauer
2013-04-24 23:50                   ` Peter Stuge
2013-04-23 19:31             ` Michał Górny
2013-04-23 19:50               ` Jeroen Roovers
2013-04-23 20:27             ` Tom Wijsman [this message]
2013-04-23 21:12               ` Zac Medico

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=20130423222753.3854fe4d@TOMWIJ-GENTOO \
    --to=tomwij@gentoo.org \
    --cc=gentoo-dev@lists.gentoo.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox