[sisyphus] патч для rdesktop

[sisyphus] патч для rdesktop

[Denis Nazarov]

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

В связи с некоторым неудобством использования rdesktop в full-screen mode я 
озадачился внедрением в него способности сворачиваться каким-либо образом. 
Покопавшись в исходниках, нашел комбинацию Ctrl-Alt-Enter, которая 
переключает режим из полноэкранного в оконный, но при этом не 
сворачивалось получающееся окно. Поэтому, еще покопавшись в документации 
по иксам, протрассировав порядок событий, сделал патч, который по этой же 
самой комбинации  не только переводит сессию в оконный режим, но и 
сворачивает ее. При разворачивании сессии она опять переводится в 
полноэкранный режим.
Попутно патч решает еще одну проблему - исправляет файл раскладки для 
русской клавиатуры, теперь по умолчанию после установки пакета в русской 
раскладке точка воспринимается как точка (рядом с буквой Ю), а в 
английской раскладке - как /, что, в общем-то, на клавиатуре и нарисовано.

Патч сделан для версии rdesktop-1.6.0-alt2.1, протестирован мной в KDE3, 
KDE4, GNOME, IceWM, WM, Enlightenment в бранче 4.1. Во всех, кроме двух 
последних, работает нормально, то есть по Ctrl-Alt-Enter сессия 
сворачивается в панель задач и при любом способе активации (Alt+Tab, 
мышкой) разворачивается обратно. В WM, Enlightenment - сворачивание 
происходит нормально, но разворачивать приходится ручками - при двойном 
щелчке на иконке восстанавливается оконный режим и чтобы перейти в 
полноэкранный - приходится еще раз нажимать на Ctrl-Alt-Enter.

Патч здесь 
http://neftekamsk.us/files/rdesktop-ctrl-alt-enter-minimize.patch
собранный бинарник здесь 
http://neftekamsk.us/files/rdesktop-1.6.0-alt2.1.i586.rpm
исходники здесь
http://neftekamsk.us/files/rdesktop-1.6.0-alt2.1.src.rpm


Прошу тестить и голосовать за включение его в сизиф :)

Re: [sisyphus] патч для rdesktop

[Denis Klimov]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Denis Nazarov wrote:
> Здравствуйте всем!
> 
> В связи с некоторым неудобством использования rdesktop в full-screen mode я 
> озадачился внедрением в него способности сворачиваться каким-либо образом. 
> Покопавшись в исходниках, нашел комбинацию Ctrl-Alt-Enter, которая 
> переключает режим из полноэкранного в оконный, но при этом не 
> сворачивалось получающееся окно. Поэтому, еще покопавшись в документации 
> по иксам, протрассировав порядок событий, сделал патч, который по этой же 
> самой комбинации  не только переводит сессию в оконный режим, но и 
> сворачивает ее. При разворачивании сессии она опять переводится в 
> полноэкранный режим.
> Попутно патч решает еще одну проблему - исправляет файл раскладки для 
> русской клавиатуры, теперь по умолчанию после установки пакета в русской 
> раскладке точка воспринимается как точка (рядом с буквой Ю), а в 
> английской раскладке - как /, что, в общем-то, на клавиатуре и нарисовано.
> 
> Патч сделан для версии rdesktop-1.6.0-alt2.1, протестирован мной в KDE3, 
> KDE4, GNOME, IceWM, WM, Enlightenment в бранче 4.1. Во всех, кроме двух 
> последних, работает нормально, то есть по Ctrl-Alt-Enter сессия 
> сворачивается в панель задач и при любом способе активации (Alt+Tab, 
> мышкой) разворачивается обратно. В WM, Enlightenment - сворачивание 
> происходит нормально, но разворачивать приходится ручками - при двойном 
> щелчке на иконке восстанавливается оконный режим и чтобы перейти в 
> полноэкранный - приходится еще раз нажимать на Ctrl-Alt-Enter.
> 
> Патч здесь 
> http://neftekamsk.us/files/rdesktop-ctrl-alt-enter-minimize.patch
> собранный бинарник здесь 
> http://neftekamsk.us/files/rdesktop-1.6.0-alt2.1.i586.rpm
> исходники здесь
> http://neftekamsk.us/files/rdesktop-1.6.0-alt2.1.src.rpm
> 
> 
> Прошу тестить и голосовать за включение его в сизиф :)
Хорошая работа. это надо апстриму высылать ;)

- --
Климов Денис Евгеньевич
Технический директор
ООО "Свободные решения"
kde@spo.tyumen.ru
http://spo.tyumen.ru
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkm9Fn0ACgkQ/2RPAjBLeBqm0QCdEIVgvtRmpYlk9WuZfDgyvYWA
AggAnjSqIbokTUrHvsSoBO9v41mNa7fI
=ZM8G
-----END PGP SIGNATURE-----

Re: [sisyphus] патч для rdesktop

[Denis Nazarov]

On Sunday 15 March 2009 19:53:50 Denis Klimov wrote:
> > Прошу тестить и голосовать за включение его в сизиф :)
>
> Хорошая работа. это надо апстриму высылать ;)

если имеется в виду lav@, то он эту рассылку читает

Re: [sisyphus] патч для rdesktop

[Mikhail Gusarov]

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


Twas brillig at 20:14:23 15.03.2009 UTC+05 when marsden@mail.ru did gyre and gimble:

 >> Хорошая работа. это надо апстриму высылать ;)
 DN> если имеется в виду lav@, то он эту рассылку читает

В rdesktop CVS имеются коммиты 3-4 месяцев давности, и среди авторов lav
не замечен Разве что хитро замаскировался.

-- 

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

Re: [sisyphus] патч для rdesktop

[Denis Nazarov]

On Sunday 15 March 2009 20:18:24 Mikhail Gusarov wrote:
> Twas brillig at 20:14:23 15.03.2009 UTC+05 when marsden@mail.ru did gyre 
and gimble:
>  >> Хорошая работа. это надо апстриму высылать ;)
>
>  DN> если имеется в виду lav@, то он эту рассылку читает
>
> В rdesktop CVS имеются коммиты 3-4 месяцев давности, и среди авторов lav
> не замечен Разве что хитро замаскировался.

а разве это не мантейнер делает?

Re: [sisyphus] патч для rdesktop

[Mikhail Gusarov]

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


Twas brillig at 20:23:07 15.03.2009 UTC+05 when marsden@mail.ru did gyre and gimble:

 DN> а разве это не мантейнер делает?

[хмуро смотрит в сторону тех, кто составлял правила приёма в team в
незапамятные времена]

"Желательно, но необязательно". Надёжнее слать апстриму самостоятельно.

-- 

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

Re: [sisyphus] патч для rdesktop

[Sergey]

On Воскресенье 15 марта 2009, Mikhail Gusarov wrote:

>  DN> а разве это не мантейнер делает?
> 
> [хмуро смотрит в сторону тех, кто составлял правила приёма в team в
> незапамятные времена]
> 
> "Желательно, но необязательно". Надёжнее слать апстриму самостоятельно.
 
Тем более, кто-то там был, кому писать можно по-русски, если мне память
не изменяет.

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] патч для rdesktop

[Michael A. Kangin]

On Sunday 15 March 2009 15:34:28 Denis Nazarov wrote:

> Прошу тестить и голосовать за включение его в сизиф :)

Эмм... хотелось бы тогда попросить приделать ключик для отключения этой 
функциональности, когда нужно, чтобы rdesktop работал всегда в 
облигатно-полноэкранном режиме. Например, на тощем клиенте, где ничего 
интересного в иксах кроме этого самого rdesktop не водится.
А то юзерша жмякнет Ctrl-Alt-Enter, а потом будет телефон обрывать "у меня всё 
пропало".


-- 
wbr, Michael A. Kangin

Re: [sisyphus] патч для rdesktop

[Vitaly Lipatov]

On 15 марта 2009, Mikhail Gusarov wrote:
> Twas brillig at 20:23:07 15.03.2009 UTC+05 when
> marsden@mail.ru did gyre and gimble:
>
>  DN> а разве это не мантейнер делает?
>
> [хмуро смотрит в сторону тех, кто составлял правила приёма в
> team в незапамятные времена]
>
> "Желательно, но необязательно". Надёжнее слать апстриму
> самостоятельно.
Да ладно хмуриться, я и не майнтейнер давным давно :)


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

Re: [sisyphus] патч для rdesktop

[Денис Назаров]

-----Original Message-----
From: "Michael A. Kangin" <mak@complife.ru>
To: ALT Linux Sisyphus discussions <sisyphus@lists.altlinux.org>
Date: Sun, 15 Mar 2009 22:05:42 +0300
Subject: Re: [sisyphus] патч для rdesktop

> On Sunday 15 March 2009 15:34:28 Denis Nazarov wrote:
> 
> > Прошу тестить и голосовать за включение его в сизиф :)
> 
> Эмм... хотелось бы тогда попросить приделать ключик для отключения этой 
> функциональности, когда нужно, чтобы rdesktop работал всегда в 
> облигатно-полноэкранном режиме. Например, на тощем клиенте, где ничего 
> интересного в иксах кроме этого самого rdesktop не водится.
> А то юзерша жмякнет Ctrl-Alt-Enter, а потом будет телефон обрывать "у меня всё 
> пропало".
> 
можно и ключик, но я не добавлял эту функциональность, она уже была, я ее только подправил :)

Re: [sisyphus] патч для rdesktop

[Michael Shigorin]

On Sun, Mar 15, 2009 at 10:42:44PM +0400, Sergey wrote:
> >  DN> а разве это не мантейнер делает?
> > [хмуро смотрит в сторону тех, кто составлял правила приёма в
> > team в незапамятные времена]

Не умничай, тут не всегда правилами решается.

> > "Желательно, но необязательно". Надёжнее слать апстриму самостоятельно.
> Тем более, кто-то там был, кому писать можно по-русски, если
> мне память не изменяет.

sr@, но его патчик насчёт -y так апстрим и не принял по дурацким
совершенно соображениям (что-то вроде "а вот на моём не-x86 это
всё равно работать не будет" -- как если бы кто заставлял
пользоваться).

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

Re: [sisyphus] патч для rdesktop

[Michael Shigorin]

On Sun, Mar 15, 2009 at 11:41:03PM +0300, Денис Назаров wrote:
> > > Прошу тестить и голосовать за включение его в сизиф :)
> > Эмм... хотелось бы тогда попросить приделать ключик для
> > отключения этой функциональности, когда нужно, чтобы rdesktop
> > работал всегда в облигатно-полноэкранном режиме. Например, на
> > тощем клиенте, где ничего интересного в иксах кроме этого
> > самого rdesktop не водится.  А то юзерша жмякнет
> > Ctrl-Alt-Enter, а потом будет телефон обрывать "у меня всё 
> > пропало".
> можно и ключик, но я не добавлял эту функциональность, она уже
> была, я ее только подправил :)

Мужики, багу повесьте.  И патч -- файлом лучше приложить,
в почте всё теряется, даже если она индексированная.

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

Re: [sisyphus] патч для rdesktop

[Mikhail Gusarov]

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


Twas brillig at 17:41:56 17.03.2009 UTC+02 when mike@osdn.org.ua did gyre and gimble:

 >>> [хмуро смотрит в сторону тех, кто составлял правила приёма в team в
 >>> незапамятные времена]

 MS> Не умничай, тут не всегда правилами решается.

Это очень хороший показатель attitude. "От этих гадких апстримов ничего
добиться нельзя, вечно только ломают и ерунду делают, ну их нафиг".

 MS> sr@, но его патчик насчёт -y так апстрим и не принял по дурацким
 MS> совершенно соображениям (что-то вроде "а вот на моём не-x86 это всё
 MS> равно работать не будет" -- как если бы кто заставлял
 MS> пользоваться).

Вот-вот. "Дурацкие соображения". Ха три раза.n

-- 

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

Re: [sisyphus] патч для rdesktop

[Denis Nazarov]

On Вторник 17 марта 2009 20:41:56 Michael Shigorin wrote:
> sr@, но его патчик насчёт -y так апстрим и не принял по дурацким
> совершенно соображениям (что-то вроде "а вот на моём не-x86 это
> всё равно работать не будет" -- как если бы кто заставлял
> пользоваться).

сильно подозреваю, что мой патчик будет послан туда же... 

Re: [sisyphus] патч для rdesktop

[Sergey]

On Tuesday 17 March 2009, Michael Shigorin wrote:

> > > [хмуро смотрит в сторону тех, кто составлял правила приёма в
> > > team в незапамятные времена]
> 
> Не умничай, тут не всегда правилами решается.
 
Это не моя фраза. :-)

-- 
С уважением, Сергей
a_s_y@sama.ru

Re: [sisyphus] патч для rdesktop

[Michael Shigorin]

On Tue, Mar 17, 2009 at 09:44:36PM +0600, Mikhail Gusarov wrote:
>  >>> [хмуро смотрит в сторону тех, кто составлял правила приёма
>  >>> в team в незапамятные времена]
>  MS> Не умничай, тут не всегда правилами решается.
> Это очень хороший показатель attitude. "От этих гадких
> апстримов ничего добиться нельзя, вечно только ломают и ерунду
> делают, ну их нафиг".

Миш, я чего-то не понимаю.

sr@, решая проблему заказчика, реализовал ключик rdesktop -y,
который достаточно долго был alt-specific, а потом рванул по
дистрибутивам.  Этот патч был предложен апстриму и отвергнут.

Поясни мне тупому, при чём правила приёма в team к данной
конкретной ситуации и имеющейся истории по ней?  Или существует
какая-либо разумная формулировка, по которой хакера уровня sr@
в команду принимать не стоит, а данному апстриму -- выслать гифт?

>  MS> sr@, но его патчик насчёт -y так апстрим и не принял по дурацким
>  MS> совершенно соображениям (что-то вроде "а вот на моём не-x86 это всё
>  MS> равно работать не будет" -- как если бы кто заставлял
>  MS> пользоваться).
> Вот-вот. "Дурацкие соображения". Ха три раза.n

Это если мягко...  Перечитай последнее предложение цитаты
и если всё-таки есть что сказать по существу -- скажи.

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