Bug 19373 - кривой патч для openbox
Summary: кривой патч для openbox
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: repocop (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: viy
QA Contact: qa-sisyphus
URL: http://repocop.altlinux.org/pub/repoc...
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-27 18:42 MSK by Igor Zubkov
Modified: 2009-03-28 13:41 MSK (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Igor Zubkov 2009-03-27 18:42:32 MSK
Патчеделалка сделала:
-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 viy 2009-03-27 21:00:33 MSK
я понимаю, но здесь не простой случай.

как раз сегодня таки приняли Упаковка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 viy 2009-03-27 21:19:50 MSK
Очень хорошо было бы привести расположение всех 
публичных пиктограмм в соответствии с [[IconPathsPolicy]].

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

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

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

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