Bug 19373 - кривой патч для openbox
: кривой патч для openbox
Status: CLOSED NOTABUG
: Sisyphus
(All bugs in Sisyphus/repocop)
: unstable
: all Linux
: P2 normal
Assigned To:
:
: http://repocop.altlinux.org/pub/repoc...
:
:
:
  Show dependency tree
 
Reported: 2009-03-27 18:42 by
Modified: 2009-03-28 13:41 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2009-03-27 18:42:32
Патчеделалка сделала:
-install -pD -m 644 %name-64.xpm %buildroot%_iconsdir/OpenBox.xpm 
+install -pD -m 644 %name-64.xpm %buildroot%_niconsdir/OpenBox.xpm 

Но эта не та иконка, она для openbox{,-kde,-gnome}.wmsession.
------- Comment #1 From 2009-03-27 21:00:33 -------
я понимаю, но здесь не простой случай.

как раз сегодня таки приняли УпаковкаWM полиси,
http://www.altlinux.org/УпаковкаWM
цитирую,
....
Пиктограмма(ы) WM должна быть проинсталлирована в соответствии с
[[IconPathsPolicy]].

В частности, поскольку, кроме wm-select, планируется поддержка этих пиктограмм
в GDM, см. #19288).

Должна быть как минимум одна пиктограмма xpm,
рекомендуется 64x64 в /usr/share/icons/hicolor/64x64/app
-----------------------^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
так что патч все равно нужен, но вида
-install -pD -m 644 %name-64.xpm %buildroot%_iconsdir/OpenBox.xpm 
+install -pD -m 644 %name-64.xpm
%buildroot%_iconsdir/hicolor/64x64/app/OpenBox.xpm
------- Comment #2 From 2009-03-27 21:19:50 -------
Очень хорошо было бы привести расположение всех 
публичных пиктограмм в соответствии с [[IconPathsPolicy]].

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

Представьте себе wm-select с темой Wesnoth.
Галерея картинок: 
Могучий гном грозит топором, меланхолично смотрит в даль 
алебардист с кедами на шее, снежный эльф зовет в IceWM...
------- Comment #3 From 2009-03-27 21:28:40 -------
убедительно?

Если в спеке будет стоять
%_iconsdir/hicolor/64x64/app/OpenBox.xpm,
патч генерироваться не будет.
------- Comment #4 From 2009-03-28 13:41:48 -------
Как-то немного косо выглядит -- мож нужен %_wmiconsdir?

-%_iconsdir/OpenBox.xpm 
+%_iconsdir/hicolor/64x64/app/OpenBox.xpm,