[devel] [git update] packages/digitemp: heads/master

[Slava Semushin]

2007/5/24, Denis Klimov <zver / altlinux.org>:
[...]
> +Name: digitemp
> +Version: 3.5.0
> +Release: alt1
> +License: GPL
> +Group: Monitoring
[...]
> +%package ds2490
> +Summary: Digitemp for the DS2490 1-wire adapter
> +Group: Monitoring
[...]
> +%package ds9097
> +Summary: Digitemp for the DS9097 1-wire adapter
> +Group: Monitoring
[...]
> +%package ds9097u
> +Summary: Digitemp for the DS9097U 1-wire adapter
> +Group: Monitoring
[...]
> +%package doc
> +Summary: Digitemp documentation
> +Group: Documentation
[...]
> +%package utils
> +Summary: Perl and python scripts for logging using RRDB and MySQL
> +Group: Monitoring
[...]
> +%package full
> +Summary: All subpackages digitemp
> +Group: Monitoring
[...]

Насколько я помню, подпакеты наследуют значения главного пакета. Т.е.
применительно к данном случаю: не обязательно указывать значение тега
Group для подпакетов у которых оно аналогично главному пакету.

> +Requires: %name-ds2490 = %version-%release
> +Requires: %name-ds9097 = %version-%release
> +Requires: %name-ds9097u = %version-%release
> +Requires: %name-doc = %version-%release
> +Requires: %name-utils = %version-%release
> +%description full
> +All subpackages digitemp:
> +%name-ds2490
> +%name-ds9097
> +%name-ds9097u
> +%name-doc
> +%name-utils

Я так понимаю, что это виртуальный пакет? Тогда может так и написать в
Summary/%description?

> +%prep
> +%setup -q -n %name

s/%setup -q/%setup/

> +%build
> +make ds2490
> +make ds9097
> +make ds9097u

Здесь умышленно не используется %make_build?

> +%install
> +install -D -m 2755 digitemp_DS2490 %buildroot%_bindir/digitemp_DS2490
> +install -D -m 2755 digitemp_DS9097 %buildroot%_bindir/digitemp_DS9097
> +install -D -m 2755 digitemp_DS9097U %buildroot%_bindir/digitemp_DS9097U

Права немножко странные. Я могу ошибаться, так как сам лично не
сталкивался с подобным, но в %buildroot нужно выставить минимальные
права (755), а дополнительные атрибуты выставлять в %files с помощью
%attr

[...]
> +%files full

Пустая секция -- IMHO можно удалить.

> +%files doc
> +%doc COPYRIGHT COPYING FAQ README ChangeLog CREDITS
> +%_man1dir/*

Я не увидел ничего относящегося к главному пакету digitemp. Это
умышленно? Зачем нужен подпакет -doc? Может его содержимое переложить
в главный пакет?

[...]
> +%files utils
> +%_datadir/%name/*

После удаления в файловой системе останется каталог %_datadir/%name и
я повешу вам багу :) Исправьте эту строчку на %_datadir/%name/ и
разойдёмся по-мирному ;)


-- 
+ Slava Semushin (кодер-перфекционист)
+ ALT Linux Team | php-coder @ altlinux.ru