* Re: [devel] [git update] packages/digitemp: heads/master
@ 2007-05-24 9:34 ` Slava Semushin
2007-05-24 9:39 ` Alexey I. Froloff
2007-05-24 10:05 ` Denis Klimov
0 siblings, 2 replies; 6+ messages in thread
From: Slava Semushin @ 2007-05-24 9:34 UTC (permalink / raw)
To: ALT Devel discussion list
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/digitemp: heads/master
2007-05-24 9:34 ` [devel] [git update] packages/digitemp: heads/master Slava Semushin
@ 2007-05-24 9:39 ` Alexey I. Froloff
2007-05-24 9:44 ` Slava Semushin
2007-05-24 10:05 ` Denis Klimov
1 sibling, 1 reply; 6+ messages in thread
From: Alexey I. Froloff @ 2007-05-24 9:39 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/digitemp: heads/master
2007-05-24 9:39 ` Alexey I. Froloff
@ 2007-05-24 9:44 ` Slava Semushin
2007-05-24 15:24 ` Alexey I. Froloff
2007-05-24 21:45 ` Vitaly Lipatov
0 siblings, 2 replies; 6+ messages in thread
From: Slava Semushin @ 2007-05-24 9:44 UTC (permalink / raw)
To: ALT Devel discussion list
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/digitemp: heads/master
2007-05-24 9:44 ` Slava Semushin
@ 2007-05-24 15:24 ` Alexey I. Froloff
2007-05-24 21:45 ` Vitaly Lipatov
1 sibling, 0 replies; 6+ messages in thread
From: Alexey I. Froloff @ 2007-05-24 15:24 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/digitemp: heads/master
2007-05-24 9:44 ` Slava Semushin
2007-05-24 15:24 ` Alexey I. Froloff
@ 2007-05-24 21:45 ` Vitaly Lipatov
1 sibling, 0 replies; 6+ messages in thread
From: Vitaly Lipatov @ 2007-05-24 21:45 UTC (permalink / raw)
To: ALT Devel discussion list
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [git update] packages/digitemp: heads/master
2007-05-24 9:34 ` [devel] [git update] packages/digitemp: heads/master Slava Semushin
2007-05-24 9:39 ` Alexey I. Froloff
@ 2007-05-24 10:05 ` Denis Klimov
1 sibling, 0 replies; 6+ messages in thread
From: Denis Klimov @ 2007-05-24 10:05 UTC (permalink / raw)
To: ALT Devel discussion list
Благодарю за советы.
Спек подправил. все учел за исключением перенесения доков в основоной пакет. Т.к. основного пакета как такового нет.
З.Ы. хорошо что есть такие внимательные люди,следящие за качеством.
--
zver@
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-05-24 21:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-05-24 9:34 ` [devel] [git update] packages/digitemp: heads/master Slava Semushin
2007-05-24 9:39 ` Alexey I. Froloff
2007-05-24 9:44 ` Slava Semushin
2007-05-24 15:24 ` Alexey I. Froloff
2007-05-24 21:45 ` Vitaly Lipatov
2007-05-24 10:05 ` Denis Klimov
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