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 --]
prev parent 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