ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Alexey Tourbin <at@altlinux.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] [git update] packages/apt: heads/master
Date: Mon, 13 Aug 2007 14:03:43 +0400
Message-ID: <20070813100343.GC6933@solemn.turbinal> (raw)
In-Reply-To: <20070811161324.GB27837@basalt.office.altlinux.org>

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

On Sat, Aug 11, 2007 at 08:13:24PM +0400, Dmitry V. Levin wrote:
> > Changelog since `0.5.15lorg2-alt12' follows:
> > commit 30e27b0f1c74b4f67354a6035fae3e210131bca1
> > Author: Alexey Tourbin <at@altlinux>
> > Date:   Sat Aug 11 20:01:05 2007 +0400
> > 
> >     apt-0.5.15lorg2-alt-genpkglist-reqfiles.patch
> >     
> >     genpkglist strips file lists by default (without --bloat option).
> >     It keeps only some "useful files" by using a few ad hoc patterns.
> >     
> >     This can break file-level dependencies.  Consider pkgA requires
> >     /usr/lib/foo1/bar, and pkgB owns this file without explicitly
> >     providing it.  Now if genpkglist strips /usr/lib/foo1/bar
> >     from pkgB file list, this is going to be an unmet dependency.
> >     
> >     This patch changes genpkglist behaviour, so that, when genpkglist
> >     is invoked without --bloat option, it first finds all file-level
> >     dependencies (something like "rpm -qaR |grep ^/").  This requires
> >     a separate pass.  The list of file-level dependencies is saved into
> >     "reqfiles" global variable.  And on the second (normal) pass, the
> >     function usefulFile() is modified to check the "reqfiles" variable;
> >     that is, it should keep a file in the file list if it's been required
> >     by some package in the repo.
> 
> Я не понял, эта информация кешируется или каждый запуск genpkglist будет
> дампить зависимости всех пакетов?

Эта информация кешируется, а каждый запуск genpkglist будет дампить
зависимости всех пакетов. :)

Суть в том, что в старом варианте genpkglist без опции --bloat сохраняет
пути в хедере (DIRNAMES, BASENAMES и DIRINDEXES) лишь частично, по
шаблонам usefulFile().  Теперь будет работать в два прохода: первый
прход -- дампит зависимости requires и сохраняет "file-level
dependencies"; на втором (обычном) проходе в usefulFile(), кроме
шаблонов, проверяется также список file-level dependencies.

Если вопрос по скорости работы, то попробуй с опцией --progress.
Но это очень зависит от работы буферного кеша.

> >     (Unfortunately, this patch does not solve all of the problems
> >     I want it to solve; we have separate repos for i586 and noarch --
> >     inter-repo file-level dependencies cannot be resolved this way.)
> Тогда зачем это изменение?

Потому что это правильно.  Если уж и обрезать файловые списки, то нужно
это делать с умом.  Оптимизация, которая даёт unmet'ы, это неправильная
оптимизация.  Данный патч делает всё возможное супротив оптимизаци,
которая чревата анметами.  К сожалению, этот патч не может сделать
также и невозможное.

Я вообще считаю что генерировать заведомо незамкнутые репозитарии (как
i586 и noarch у нас) это неправильно.  И одноврменно обрезать файловые
листы.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

      reply	other threads:[~2007-08-13 10:03 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-11 16:13 ` Dmitry V. Levin
2007-08-13 10:03   ` Alexey Tourbin [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=20070813100343.GC6933@solemn.turbinal \
    --to=at@altlinux.ru \
    --cc=devel@lists.altlinux.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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git