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

Re: [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

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

[Alexey I. Froloff]

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

* Slava Semushin <slava.semushin@> [070524 13:34]:
> Насколько я помню, подпакеты наследуют значения главного пакета. Т.е.
> применительно к данном случаю: не обязательно указывать значение тега
> Group для подпакетов у которых оно аналогично главному пакету.

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

Слава, у меня к тебе просьба личного характера.  Не надо давать
Ценных Советов не по делу, ладно?

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

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

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

[Slava Semushin]

24.05.07, Alexey I. Froloff<raorn / immo.ru> написал(а):
[...]
> > Пустая секция -- IMHO можно удалить.
> Слава, у меня к тебе просьба личного характера.  Не надо давать
> Ценных Советов не по делу, ладно?

Я-то могу и не давать. Но:

1) мне показалось, что это было по делу
2) если в части советов я прав и мэйнтейнер меня послушает, то пакет
будет лучшего качества
3) если я где-то ошибаюсь, так меня поправят и тогда и я научусь и
другим наука, что поспособствует повышению навыков/знаний/опыта
мэйнтейнера(ов)

Я вижу только плюсы...

P.S. Я так понял, что конкретно этот совет про пустую секцию является неверным?

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

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

[Denis Klimov]

Благодарю за советы.
Спек подправил. все учел за исключением перенесения доков в основоной пакет. Т.к. основного пакета как такового нет.

З.Ы. хорошо что есть такие внимательные люди,следящие за качеством.

--
zver@

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

[Alexey I. Froloff]

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

* Slava Semushin <slava.semushin@> [070524 13:46]:
[..skip..]
> P.S. Я так понял, что конкретно этот совет про пустую секцию
> является неверным?
Оба.  Про группу и про пустую секцию.  Если ты в чём-то не уверен
на 100% - не надо высказывать своё ИМХО, только вводишь в
заблуждение.  А ещё лучше подобные комментарии к спекам оформляй
в виде патчей у себя в git.

-- 
Regards,
Sir Raorn.

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

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

[Vitaly Lipatov]

On 24 мая 2007, Slava Semushin wrote:
...
> P.S. Я так понял, что конкретно этот совет про пустую секцию
> является неверным?
%files full
даёт команду создать соотв. пакет. Насколько я понимаю, он как 
раз виртуальный и файлов не содержит, но это не мешает ему иметь 
зависимости.

P.S.
Подпакеты ничего не наследуют, соответственно на поле Group 
сэкономить не удасться. Без него:
"ошибка: Поле Group обязано присутствовать в пакете:"

P.P.S.
2raorn: просьбы личного характера просьба писать лично


-- 
Lav
Виталий Липатов
Санкт-Петербург
GNU! ALT Linux Team! WINE! LaTeX! LyX! http://freesource.info