Re: [devel] [git update] packages/hasher: heads/master

Re: [devel] [git update] packages/hasher: heads/master

[Dmitry V. Levin]

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

On Tue, Aug 07, 2007 at 04:41:26PM +0400, Alexey M. Tourbin wrote:
> Update of /people/at/packages/hasher.git
> 
> Changes statistics since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
>  hasher/hsh-initroot                    |    3 +-
>  hasher/hsh-sh-cache-contents-functions |   87 +++++++++++++------------------
>  2 files changed, 39 insertions(+), 51 deletions(-)
> 
> Changelog since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
> commit 4180e774c1919b5a31b0f6fa066c4dad733ca5c1
> Author: Alexey Tourbin <at@altlinux>
> Date:   Tue Aug 7 16:08:07 2007 +0400
> 
>     cache-contents-functions: introduced contents_index_all.gz
>     
>     This commit is for preview only, description pending for update.
> 
> Full diff since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
> diff --git a/hasher/hsh-initroot b/hasher/hsh-initroot
> index 3d681b7..901db8a 100755
> --- a/hasher/hsh-initroot
> +++ b/hasher/hsh-initroot
[...]
> -${contents_index_bin:+%_pkg_contents_index_bin /.host/contents_index_bin}
> +${contents_index_bin:+%_pkg_contents_index_bin /.host/${contents_index_bin##*/}}
> +${contents_index_all:+%_pkg_contents_index_all /.host/${contents_index_all##*/}}

Здесь и далее, я не понимаю, зачем нужно передавать rpmbuild'у оба
индексных файла; если передаётся полный индекс, то зачем ещё передавать
частичный?

[...]
> +	set -x

Если код работает, то это надо убрать.

[...]
> +	<"$contents" >>"$contents.bin" grep '^/bin/' || [ $? -eq 1 ]
> +	<"$contents" >>"$contents.bin" grep '^/sbin/' || [ $? -eq 1 ]
> +	<"$contents" >>"$contents.bin" grep '^/usr/bin/' || [ $? -eq 1 ]
> +	<"$contents" >>"$contents.bin" grep '^/usr/sbin/' || [ $? -eq 1 ]
> +	<"$contents" >>"$contents.bin" grep '^/usr/X11R6/bin/' || [ $? -eq 1 ]
> +	<"$contents" >>"$contents.bin" grep '^/etc/' || [ $? -eq 1 ]
> +
> +	cat <"$contents.bin" >"$contents_index_bin"

Этот новообразованный $contents.bin, похоже, никогда не зануляется.

[...]
> +	gzip <"$contents" >"$contents_index_all"

А стоит ли его cжимать?

[...]
> +	set +x

Если код работает, то это надо убрать.

[...]
> +	Verbose 'Prepared contents index.'
> +	chmod 644 "$contents_index_bin" "$contents_index_all"

В этом варианте имеет смысл поменять строчки местами.


-- 
ldv

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

Re: [devel] [git update] packages/hasher: heads/master

[Alexey Tourbin]

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

On Thu, Aug 09, 2007 at 02:09:24AM +0400, Dmitry V. Levin wrote:
> On Tue, Aug 07, 2007 at 04:41:26PM +0400, Alexey M. Tourbin wrote:
> > Update of /people/at/packages/hasher.git
> > 
> > Changes statistics since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
> >  hasher/hsh-initroot                    |    3 +-
> >  hasher/hsh-sh-cache-contents-functions |   87 +++++++++++++------------------
> >  2 files changed, 39 insertions(+), 51 deletions(-)
> > 
> > Changelog since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
> > commit 4180e774c1919b5a31b0f6fa066c4dad733ca5c1
> > Author: Alexey Tourbin <at@altlinux>
> > Date:   Tue Aug 7 16:08:07 2007 +0400
> > 
> >     cache-contents-functions: introduced contents_index_all.gz
> >     
> >     This commit is for preview only, description pending for update.
> > 
> > Full diff since common ancestor `1.2.5-alt1-5-g26e3fbc' follows:
> > diff --git a/hasher/hsh-initroot b/hasher/hsh-initroot
> > index 3d681b7..901db8a 100755
> > --- a/hasher/hsh-initroot
> > +++ b/hasher/hsh-initroot
> [...]
> > -${contents_index_bin:+%_pkg_contents_index_bin /.host/contents_index_bin}
> > +${contents_index_bin:+%_pkg_contents_index_bin /.host/${contents_index_bin##*/}}
> > +${contents_index_all:+%_pkg_contents_index_all /.host/${contents_index_all##*/}}
> 
> Здесь и далее, я не понимаю, зачем нужно передавать rpmbuild'у оба
> индексных файла; если передаётся полный индекс, то зачем ещё передавать
> частичный?

У меня есть более свежая наработка, там почти всё переделано.  Я сейчас
посмотрю, можно его прямо сейчас закоммитить для preview чтобы не было
слишком стыдно или нет.

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

Re: [devel] [git update] packages/hasher: heads/master

[Alexey Tourbin]

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

On Thu, Aug 09, 2007 at 02:09:24AM +0400, Dmitry V. Levin wrote:
> > -${contents_index_bin:+%_pkg_contents_index_bin /.host/contents_index_bin}
> > +${contents_index_bin:+%_pkg_contents_index_bin /.host/${contents_index_bin##*/}}
> > +${contents_index_all:+%_pkg_contents_index_all /.host/${contents_index_all##*/}}
> 
> Здесь и далее, я не понимаю, зачем нужно передавать rpmbuild'у оба
> индексных файла; если передаётся полный индекс, то зачем ещё передавать
> частичный?

index_bin занимает порядка 700K, index_all без сжатия занимает порядка
90M.  Это очень большая разница.  В /usr/lib/rpm/find-package я вчера
реализовал некоторые вещи, которые сводятся к тому, что поиск по
index_all идёт только в самых крайних случаях.

> [...]
> > +	set -x
> 
> Если код работает, то это надо убрать.

Это недоделанный код, о чём я написал в commit message.
С окончательным кодом пойдёт commit message с подробным комментарием.

> [...]
> > +	<"$contents" >>"$contents.bin" grep '^/bin/' || [ $? -eq 1 ]
> > +	<"$contents" >>"$contents.bin" grep '^/sbin/' || [ $? -eq 1 ]
> > +	<"$contents" >>"$contents.bin" grep '^/usr/bin/' || [ $? -eq 1 ]
> > +	<"$contents" >>"$contents.bin" grep '^/usr/sbin/' || [ $? -eq 1 ]
> > +	<"$contents" >>"$contents.bin" grep '^/usr/X11R6/bin/' || [ $? -eq 1 ]
> > +	<"$contents" >>"$contents.bin" grep '^/etc/' || [ $? -eq 1 ]
> > +
> > +	cat <"$contents.bin" >"$contents_index_bin"
> 
> Этот новообразованный $contents.bin, похоже, никогда не зануляется.

Он создаётся там же, где и "$contents", т.е. в tmpdir=.../tmp,
а этот каталог tmp должен перед этим удаляться.

Этот код в любом случае неправильный, последовательность каталогов
должна более просто конфигурироваться.

> [...]
> > +	gzip <"$contents" >"$contents_index_all"
> А стоит ли его cжимать?

Сжимается с 90M до 7M (очено много).  Я исхожу из того, что для сборки
большинства пакетов index_all не нужен.  На самом деле нужен realtime
compressor типа lzop, но сейчас не время.

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