[newbies] qjoypad

[newbies] qjoypad

[Yuriy Shirokov]

Здравствуйте!

Собрал для Сизифа http://qjoypad.sourceforge.net, программу, имитирующую 
по событиям джойстика нажатия на клавиатуру или движение мыши:
http://git.altlinux.org/people/yushi/packages/qjoypad.git

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

-- 
С уважением,
Ю.А.Широков.

Re: [newbies] qjoypad

[Artem Zolochevskiy]

15 сентября 2010 г. 0:31 пользователь Yuriy Shirokov
<yuriy.shirokov@gmail.com> написал:
> Здравствуйте!
>
> Собрал для Сизифа http://qjoypad.sourceforge.net, программу, имитирующую по
> событиям джойстика нажатия на клавиатуру или движение мыши:
> http://git.altlinux.org/people/yushi/packages/qjoypad.git
>
> Кажется, это первый пакет, спек для которого я писал с нуля, так что вполне
> мог что-то налажать, критика принимается с благодарностью.
>

Что можно было бы сделать иначе:

1. Наверное при сборке из git/gear совсем не обязательно упаковывать
исходники в tar.gz. Они и так упаковываются, так что два раза эту
работу можно и не делать :-)
итого:
в  .gear/rules вполне достаточно будет tar.gz: qjoypad
ну и в спеке Source0: %name-%version.tar

2. Русские Summary и Description можно было бы положить в specspo

3. License можно было бы для единообразия оформить макросом (см.
/etc/rpm/macros.d/licenses)
License: %gpl2only
и добавить
BuildRequires(pre): rpm-build-licenses

и кстати не %gpl2plus ли оно вообще?

4. %build можно было бы лаконичнее:
тут можно было бы избавиться и от патча вовсе. например так:
export PATH=$PATH:%_qt4dir/bin
%configure --prefix=%prefix --install-dir=%buildroot
%make_build

5. %install тоже можно офрмить, например, вот так:
%makeinstall_std -C src

6. документация обычно отправляется в /usr/share/doc/%name-%version/
кроме того, следуя http://www.altlinux.org/Docs файлы лицензии не пакуем.
итого в %files можно было бы %docdir/* заменить на:
%doc README.txt
%exclude %_docdir

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

Сходу другого не бросилось в глаз.
Надеюсь, хоть чем-то посодействовал.

-- 
Артём Золочевский

Re: [newbies] qjoypad

[Ivan A. Melnikov]

В Wed, 15 Sep 2010 03:13:47 +0300
Artem Zolochevskiy <artem.zolochevskiy@gmail.com> пишет:

> 15 сентября 2010 г. 0:31 пользователь Yuriy Shirokov
> <yuriy.shirokov@gmail.com> написал:
[...]
> 4. %build можно было бы лаконичнее:
> тут можно было бы избавиться и от патча вовсе. например так:
> export PATH=$PATH:%_qt4dir/bin
> %configure --prefix=%prefix --install-dir=%buildroot
[...]

Я бы не стал использовать %configure здесь, поскольку
%configure -- это всё таки для autotools (и, к стати, там уже
передаётся --prefix), а в данном случае мы имеем дело с самописной
обёрткой вокруг qmake. Возможно, имеет смысл воспользоваться qmake-ом
непосредственно, просто чтобы не мутить с $PATH, но это уже эстетика.

-- 
WBR,
Ivan A. Melnikov

Re: [newbies] qjoypad

[Artem Zolochevskiy]

15 сентября 2010 г. 8:55 пользователь Ivan A. Melnikov
<iv@altlinux.org> написал:
> В Wed, 15 Sep 2010 03:13:47 +0300
> Artem Zolochevskiy <artem.zolochevskiy@gmail.com> пишет:
>
>> 15 сентября 2010 г. 0:31 пользователь Yuriy Shirokov
>> <yuriy.shirokov@gmail.com> написал:
> [...]
>> 4. %build можно было бы лаконичнее:
>> тут можно было бы избавиться и от патча вовсе. например так:
>> export PATH=$PATH:%_qt4dir/bin
>> %configure --prefix=%prefix --install-dir=%buildroot
> [...]
>
> Я бы не стал использовать %configure здесь, поскольку
> %configure -- это всё таки для autotools (и, к стати, там уже
> передаётся --prefix), а в данном случае мы имеем дело с самописной
> обёрткой вокруг qmake. Возможно, имеет смысл воспользоваться qmake-ом
> непосредственно, просто чтобы не мутить с $PATH, но это уже эстетика.

Да да. Это опечатка. Стандартное %configure оно вообще не
проглатывает. Следует читать как
./configure --prefix=%prefix --install-dir=%buildroot

-- 
Артём Золочевский

Re: [newbies] qjoypad

[Artem Zolochevskiy]

>
> 1. Наверное при сборке из git/gear совсем не обязательно упаковывать
> исходники в tar.gz. Они и так упаковываются, так что два раза эту
> работу можно и не делать :-)
> итого:
> в  .gear/rules вполне достаточно будет tar.gz: qjoypad
> ну и в спеке Source0: %name-%version.tar
>

Тут в .rules естестственно просто tar: qjoypad

-- 
Артём Золочевский

Re: [newbies] qjoypad

[Andrey Rahmatullin]

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

On Wed, Sep 15, 2010 at 08:59:21AM +0300, Artem Zolochevskiy wrote:
> > 1. Наверное при сборке из git/gear совсем не обязательно упаковывать
> > исходники в tar.gz. Они и так упаковываются, так что два раза эту
> > работу можно и не делать :-)
> > итого:
> > в  .gear/rules вполне достаточно будет tar.gz: qjoypad
> > ну и в спеке Source0: %name-%version.tar
> Тут в .rules естестственно просто tar: qjoypad
tar: @name@ даже

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

 * iamfalcon rotfl с линухцентра: Novell присоединяется к OSDL
<iamfalcon> [12:33]
<iamfalcon> Кристалльной души компания... Могла бы ведь и купить!
<iamfalcon> коммент на 5+!

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

Re: [newbies] qjoypad

[Yuriy Shirokov]

Здравствуйте!

В общем, новая версия выложена на 
http://git.altlinux.org/people/yushi/packages/qjoypad.git

On 15.09.2010 04:13, Artem Zolochevskiy wrote:
> 1. Наверное при сборке из git/gear совсем не обязательно упаковывать
> исходники в tar.gz. Они и так упаковываются, так что два раза эту
> работу можно и не делать :-)

Done.

> 2. Русские Summary и Description можно было бы положить в specspo

А вот про это на altlinux.org нашёл только смутные упоминания пятилетней 
давности. Где-нибудь можно про это почитать?

И про локализацию desktop-файлов тоже, кстати.

> BuildRequires(pre): rpm-build-licenses

Тег должен быть именно таким, со скобочками?

> и кстати не %gpl2plus ли оно вообще?

Нигде не нашёл упоминаний про "...или более поздней". Так что строго 2, 
видимо.

> 4. %build можно было бы лаконичнее:

Поправил.

> 5. %install тоже можно офрмить, например, вот так:
> %makeinstall_std -C src

Поправил, спасибо.

> 6. документация обычно отправляется в /usr/share/doc/%name-%version/
> кроме того, следуя http://www.altlinux.org/Docs файлы лицензии не пакуем.
> итого в %files можно было бы %docdir/* заменить на:
> %doc README.txt
> %exclude %_docdir

Сделано.

> 7. как фичреквест: если программка графическая, то стоило бы написать
> для неё .desktop файл, для отображения в меню.

Сделано.

> Надеюсь, хоть чем-то посодействовал.

Вы очень помогли, спасибо.

-- 
С уважением,
Ю.А.Широков.

Re: [newbies] qjoypad

[Michael Shigorin]

On Wed, Sep 15, 2010 at 01:31:53AM +0400, Yuriy Shirokov wrote:
> Кажется, это первый пакет, спек для которого я писал с нуля,
> так что вполне мог что-то налажать, критика принимается с
> благодарностью.

Обычно в порядке причёсывания привожу начало спека в соответствие с 
http://www.altlinux.org/ALT_Packaging_HOWTO#.D0.9F.D0.BE.D1.80.D1.8F.D0.B4.D0.BE.D0.BA_.D1.82.D1.8D.D0.B3.D0.BE.D0.B2.

Ну и можно --prefix=/usr на случай ядерной войны омакрить:
--prefix=%_usr (см. `rpm --eval %_usr`).

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [newbies] qjoypad

[Andrey Rahmatullin]

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

On Wed, Sep 15, 2010 at 03:43:20PM +0300, Michael Shigorin wrote:
> Ну и можно --prefix=/usr на случай ядерной войны омакрить:
> --prefix=%_usr (см. `rpm --eval %_usr`).
--prefix=%_prefix

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

> >>*** glibc detected *** double free or corruption (!prev): 0x080514c0 ***
Что тут предлагать, всё и так ясно:
- есть хорошо работающая программа su,
- пользоваться gnomesu нельзя, ибо не работает,
- ставить пакет gnomesu нельзя, ибо небезопасно,
- если у gnomesu есть мантейнер, то ему есть о чём подумать.
		-- ldv in sisyphus@

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

Re: [newbies] qjoypad

[Yuriy Shirokov]

On 15.09.2010 16:48, Andrey Rahmatullin wrote:
> On Wed, Sep 15, 2010 at 03:43:20PM +0300, Michael Shigorin wrote:
>> Ну и можно --prefix=/usr на случай ядерной войны омакрить:
>> --prefix=%_usr (см. `rpm --eval %_usr`).
> --prefix=%_prefix

О, спасибо.

Стандартный макрос, оказывается, а я и не знал.

-- 
С уважением,
Ю.А.Широков.

Re: [newbies] qjoypad

[Yuriy Shirokov]

On 15.09.2010 16:43, Michael Shigorin wrote:
> Обычно в порядке причёсывания привожу начало спека в соответствие с
> http://www.altlinux.org/ALT_Packaging_HOWTO#.D0.9F.D0.BE.D1.80.D1.8F.D0.B4.D0.BE.D0.BA_.D1.82.D1.8D.D0.B3.D0.BE.D0.B2.

Вроде не нашёл явных несоответсвий...

Единственное что пока не слделано --- иконка для desktop-файла. Как у 
нас принятно добавлять иконки --- как ещё один %source или ещё как?

-- 
С уважением,
Ю.А.Широков.

Re: [newbies] qjoypad

[Motsyo Gennadi aka Drool]

19.09.2010 17:32, Yuriy Shirokov пишет:
> Единственное что пока не слделано --- иконка для desktop-файла. Как у
> нас принятно добавлять иконки --- как ещё один %source или ещё как?

	Если нет внутри исходников никакой иконки/логотипа - тогда отдельным
%source, если есть - посмотрите спек
http://sisyphus.ru/ru/srpm/Sisyphus/fatrat/spec
в секции %install есть # Icons, должно быть все понятно.