[Comm] не применяется патч при сборке пакета

[Comm] не применяется патч при сборке пакета

[Denis Nazarov]

пытаюсь собрать пакет со своим патчем, получаю отлуп

[denis@buka RPM]$ rpm -ba --target i586 SPECS/rdesktop.spec
Платформы для сборки: i586
Сборка для платформы i586
Выполняется(%prep): /bin/sh -e /home/denis/tmp/rpm-tmp.42618
+ umask 022
+ /bin/mkdir -p /home/denis/RPM/BUILD
+ cd /home/denis/RPM/BUILD
+ cd /home/denis/RPM/BUILD
+ rm -rf rdesktop-1.6.0
+ echo 'Source #0 (rdesktop-1.6.0.tar.gz):'
Source #0 (rdesktop-1.6.0.tar.gz):
+ /bin/gzip -dc /home/denis/RPM/SOURCES/rdesktop-1.6.0.tar.gz
+ /bin/tar -xf -
+ cd rdesktop-1.6.0
+ /bin/chmod -c -Rf u+rwX,go-w .
+ echo 'Patch #1 (rdesktop-cvs-alt-raw-v2.patch):'
Patch #1 (rdesktop-cvs-alt-raw-v2.patch):
+ /usr/bin/patch -p1
patching file rdesktop.c
Hunk #4 succeeded at 785 (offset 1 line).
patching file scancodes.h
patching file xkeymap.c
Hunk #2 succeeded at 944 (offset -6 lines).
Hunk #3 succeeded at 1030 (offset -6 lines).
patching file xwin.c
Hunk #1 succeeded at 29 (offset 1 line).
Hunk #2 succeeded at 46 (offset 1 line).
Hunk #3 succeeded at 139 (offset 9 lines).
Hunk #4 succeeded at 2272 (offset 314 lines).
Hunk #5 succeeded at 2298 (offset 314 lines).
Hunk #6 succeeded at 2351 (offset 322 lines).
Hunk #7 succeeded at 2372 (offset 322 lines).
+ echo 'Patch #2 (rdesktop-alt-man.patch):'
Patch #2 (rdesktop-alt-man.patch):
+ /usr/bin/patch -p1
patching file doc/rdesktop.1
+ echo 'Patch #3 (rdesktop.long.names.on.redirect.drives.patch):'
Patch #3 (rdesktop.long.names.on.redirect.drives.patch):
+ /usr/bin/patch -p1
patching file disk.c
+ echo 'Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):'
Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
+ /usr/bin/patch -p1
patching file rdesktop.c
Hunk #1 succeeded at 81 with fuzz 2.
Hunk #2 FAILED at 571.
1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej
ошибка: Неверный код возврата из /home/denis/tmp/rpm-tmp.42618 (%prep)


Ошибки сборки пакетов:
    Неверный код возврата из /home/denis/tmp/rpm-tmp.42618 (%prep)

сам патч #4 вроде как правильный

diff -urN rdesktop.orig/rdesktop.c rdesktop/rdesktop.c
--- rdesktop.orig/rdesktop.c      2009-03-15 01:58:48 +0200
+++ rdesktop/rdesktop.c   2009-03-15 02:00:21 +0300
@@ -81,6 +81,7 @@
 RD_BOOL g_desktop_save = True; /* desktop save order */
 RD_BOOL g_polygon_ellipse_orders = True;       /* polygon / ellipse orders 
*/
 RD_BOOL g_fullscreen = False;
+RD_BOOL g_isfullscreen = False;
 RD_BOOL g_grab_keyboard = True;
 RD_BOOL g_hide_decorations = False;
 RD_BOOL g_use_rdp5 = True;
@@ -570,6 +571,7 @@

                        case 'f':
                                g_fullscreen = True;
+                              g_isfullscreen = True;
                                break;

                        case 'b':

почему-то отрабатывает первое изменение,  а на втором затыкается. Что я 
делаю не так?

Re: [Comm] не применяется патч при сборке пакета

[Sergey]

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

> пытаюсь собрать пакет со своим патчем, получаю отлуп
> Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
> + /usr/bin/patch -p1
> patching file rdesktop.c
> Hunk #1 succeeded at 81 with fuzz 2.
> Hunk #2 FAILED at 571.
> 1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej

Ну вот же написано.

> сам патч #4 вроде как правильный

Безо всяких "вроде" - неправильный, раз не наложился. Поди, не от той версии ?

> почему-то отрабатывает первое изменение,  а на втором затыкается.

Потому, что не знает, куда его применить.

> Что я делаю не так?

Надо найти соответствующее место, пропатчить руками и сделать новый diff.

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

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 04:54:24 Sergey wrote:
> On Воскресенье 15 марта 2009, Denis Nazarov wrote:
> > пытаюсь собрать пакет со своим патчем, получаю отлуп
> > Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
> > + /usr/bin/patch -p1
> > patching file rdesktop.c
> > Hunk #1 succeeded at 81 with fuzz 2.
> > Hunk #2 FAILED at 571.
> > 1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej
>
> Ну вот же написано.

написано... только очень уж неинформативно... там делов-то - вставить две 
строчки в файл. вот rdesktop.c.rej

***************
*** 570,575 ****
  
                         case 'f':
                                 g_fullscreen = True;
                                 break;
  
                         case 'b':
--- 571,577 ----
  
                         case 'f':
                                 g_fullscreen = True;
+                               g_isfullscreen = True;
                                 break;
  
                         case 'b':

потом еще смущает  Hunk #1 succeeded at 81 with fuzz 2.
Предыдущие три патча ни о каком fuzz не говорят

> > сам патч #4 вроде как правильный
>
> Безо всяких "вроде" - неправильный, раз не наложился. Поди, не от той
> версии ?
>
патч от той версии, делал его сам, первоначально код правил на оригинальных 
исходниках, потом обнаружил наличие довольно внушительных патчей, без 
которых оригинальные исходники никуда не годятся в плане работы :)
> > почему-то отрабатывает первое изменение,  а на втором затыкается.
>
> Потому, что не знает, куда его применить.
>
как он может не знать, если там все написано?

> > Что я делаю не так?
>
> Надо найти соответствующее место, пропатчить руками и сделать новый
> diff.

Вообще делал так - установил srpm пакет, выполнил rpm -bp rdesktop.spec, 
исходники распаковались и применились патчи. Насколько я понимаю, 
следующий патч надо определять после наложения предыдущих, так?
Поэтому после наложения уже существующих патчей скопировал папку с 
пропатченными исходниками в папку rdesktop.orig, затем в 
rdesktop/rdesktop.c внес изменения, затем сделал diff, обозвал вывод 
bla-bla.patch, подправил spec, затем запустил сборку и обломался...

Где в моих действиях косяк?

Re: [Comm] не применяется патч при сборке пакета

[Nikolay A. Fetisov]

On Sun, 15 Mar 2009 12:05:54 +0500
Denis Nazarov wrote:

> On Sunday 15 March 2009 04:54:24 Sergey wrote:
> > On Воскресенье 15 марта 2009, Denis Nazarov wrote:
> > > пытаюсь собрать пакет со своим патчем, получаю отлуп
> > > Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
> > > + /usr/bin/patch -p1
> > > patching file rdesktop.c
> > > Hunk #1 succeeded at 81 with fuzz 2.
> > > Hunk #2 FAILED at 571.
> > > 1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej
> >
> ....
> Где в моих действиях косяк?

В несовпадении пробелов/табуляций в исходном коде и патче?
Файла патча как делали?

-- 
С уважением,
Николай Фетисов

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 12:05:54 Denis Nazarov wrote:
> On Sunday 15 March 2009 04:54:24 Sergey wrote:
> > On Воскресенье 15 марта 2009, Denis Nazarov wrote:
> > > пытаюсь собрать пакет со своим патчем, получаю отлуп
> > > Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
> > > + /usr/bin/patch -p1
> > > patching file rdesktop.c
> > > Hunk #1 succeeded at 81 with fuzz 2.
> > > Hunk #2 FAILED at 571.
> > > 1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej
> >
видать, вчера совсем уже засыпал :) Разобрался :) оказывается, хитрость в 
том, что надо выдерживать пути относительно BUILD и патчить исходный 
*.orig файл, который создается "волшебным" образом в папке с 
распакованными исходниками.

Теперь возник другой вопрос - необходимо пропатчить файл, в который один из 
предыдущих патчей уже внес изменения. Как идеологически правильно это 
сделать? На данный момент споткнулся на том, что при попытке использовать 
патч, в котором diff дублирует предыдущий патч и добавляет мои строки при 
сборке начинаются вопросы о реверсировании и повторном применении. 
Вариант 1. Вставить свои изменения в уже существующий патч. Как-то не 
нравится...
Вариант 2. Подправить вывод diff для патчинга таким образом, чтобы убрать 
изменения предыдущего патча и оставить только свое. Пока сделал так..

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 13:35:16 Denis Nazarov wrote:

> Теперь возник другой вопрос - необходимо пропатчить файл, в который один
> из предыдущих патчей уже внес изменения. Как идеологически правильно это
> сделать? На данный момент споткнулся на том, что при попытке
> использовать патч, в котором diff дублирует предыдущий патч и добавляет
> мои строки при сборке начинаются вопросы о реверсировании и повторном
> применении. Вариант 1. Вставить свои изменения в уже существующий патч.
> Как-то не нравится...
> Вариант 2. Подправить вывод diff для патчинга таким образом, чтобы
> убрать изменения предыдущего патча и оставить только свое. Пока сделал
> так..

и второй вариант не прошел... 

Re: [Comm] не применяется патч при сборке пакета

[Nikolay A. Fetisov]

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

On Sun, 15 Mar 2009 13:35:16 +0500
Denis Nazarov wrote:

> On Sunday 15 March 2009 12:05:54 Denis Nazarov wrote:
> > On Sunday 15 March 2009 04:54:24 Sergey wrote:
> > > On Воскресенье 15 марта 2009, Denis Nazarov wrote:
> > > > пытаюсь собрать пакет со своим патчем, получаю отлуп
> > > > Patch #4 (rdesktop-ctrl-alt-enter-minimize.patch):
> > > > + /usr/bin/patch -p1
> > > > patching file rdesktop.c
> > > > Hunk #1 succeeded at 81 with fuzz 2.
> > > > Hunk #2 FAILED at 571.
> > > > 1 out of 2 hunks FAILED -- saving rejects to file rdesktop.c.rej
> > >
> видать, вчера совсем уже засыпал :) Разобрался :) оказывается, хитрость в 
> том, ....

Нет там никаких _таких_ хитростей. И особого волшебства тоже нет.
И одни патчи поверх предыдущих тоже отлично накладываются.

Пробелы и табуляция - разные вещи, о чём patch и говорит. 
Прикладывающийся rdesktop-ctrl-alt-enter-minimize.patch прикрепил.

-- 
С уважением,
Николай Фетисов

[-- Attachment #2: rdesktop-ctrl-alt-enter-minimize.patch --]
[-- Type: text/x-patch, Size: 533 bytes --]

--- rdesktop/rdesktop.c.orig	2009-03-15 11:20:32 +0300
+++ rdesktop/rdesktop.c	2009-03-15 11:21:41 +0300
@@ -81,6 +81,7 @@
 RD_BOOL g_desktop_save = True;	/* desktop save order */
 RD_BOOL g_polygon_ellipse_orders = True;	/* polygon / ellipse orders */
 RD_BOOL g_fullscreen = False;
+RD_BOOL g_isfullscreen = False;
 RD_BOOL g_grab_keyboard = True;
 RD_BOOL g_hide_decorations = False;
 RD_BOOL g_use_rdp5 = True;
@@ -570,6 +571,7 @@
 
 			case 'f':
 				g_fullscreen = True;
+				g_isfullscreen = True;
 				break;
 
 			case 'b':

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 13:50:56 Denis Nazarov wrote:
> On Sunday 15 March 2009 13:35:16 Denis Nazarov wrote:
> > Теперь возник другой вопрос - необходимо пропатчить файл, в который
> > один из предыдущих патчей уже внес изменения. Как идеологически
> > правильно это сделать? На данный момент споткнулся на том, что при
> > попытке
> > использовать патч, в котором diff дублирует предыдущий патч и
> > добавляет мои строки при сборке начинаются вопросы о реверсировании и
> > повторном применении. Вариант 1. Вставить свои изменения в уже
> > существующий патч. Как-то не нравится...
> > Вариант 2. Подправить вывод diff для патчинга таким образом, чтобы
> > убрать изменения предыдущего патча и оставить только свое. Пока сделал
> > так..
>
> и второй вариант не прошел...

все, разобрался... оказывается, перед применением каждого патча patch 
создает *.orig, таким образом, когда сборка дошла до моего патча, в этом 
файле был уже измененный предыдущими патчами вариант. Файл для патчера 
надо делать относительно папки BUILD, лучше с перенаправлением вывода в 
файл и с последующей корректировкой в виде добавление первой строкой самой 
команды diff. Вообще, где нибудь есть внятное описание на русском, как 
накладывать патчи и собирать пакеты? 

Re: [Comm] не применяется патч при сборке пакета

[Nikolay A. Fetisov]

On Sun, 15 Mar 2009 14:04:50 +0500
Denis Nazarov wrote:

> все, разобрался...  Файл для патчера 
> надо делать относительно папки BUILD, ...

не обязательно

> лучше с перенаправлением вывода в 
> файл 

даже не лучше, а просто - с перенаправлением вывода в файл.

> и с последующей корректировкой в виде добавление первой строкой самой 
> команды diff. 

совсем не обязательно. А вот добавить в начало патча пару строк с
описанием того, что он делает - IMHO желательно.

> Вообще, где нибудь есть внятное описание на русском, как 
> накладывать патчи 
Ну, например, http://www.opennet.ru/docs/RUS/diff/

> и собирать пакеты? 
и http://www.altlinux.org/Sisyphus 

-- 
С уважением,
Николай Фетисов

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 14:20:21 Nikolay A. Fetisov wrote:
> On Sun, 15 Mar 2009 14:04:50 +0500
>
> Denis Nazarov wrote:
> > все, разобрался...  Файл для патчера
> > надо делать относительно папки BUILD, ...
>
> не обязательно
>
> > лучше с перенаправлением вывода в
> > файл
>
> даже не лучше, а просто - с перенаправлением вывода в файл.
>
> > и с последующей корректировкой в виде добавление первой строкой самой
> > команды diff.
>
> совсем не обязательно. А вот добавить в начало патча пару строк с
> описанием того, что он делает - IMHO желательно.
и какой формат этого описания? вот просто взять и написать "этот патч 
делает то-то и то-то" или все же как-то оформить в виде комментария или 
еще как? например, решетку в начале строки поставить?
>
> > Вообще, где нибудь есть внятное описание на русском, как
> > накладывать патчи
>
> Ну, например, http://www.opennet.ru/docs/RUS/diff/
>
> > и собирать пакеты?
>
> и http://www.altlinux.org/Sisyphus

Спасибо :)

Re: [Comm] не применяется патч при сборке пакета

[Nikolay A. Fetisov]

On Sun, 15 Mar 2009 15:00:00 +0500
Denis Nazarov wrote:

> On Sunday 15 March 2009 14:20:21 Nikolay A. Fetisov wrote:
> ... вот добавить в начало патча пару строк с
> > описанием того, что он делает - IMHO желательно.
> и какой формат этого описания? вот просто взять и написать "этот патч 
> делает то-то и то-то" или все же как-то оформить в виде комментария или 
> еще как? например, решетку в начале строки поставить?

Всё равно. Строки до начала собственно патча (т.е. до строки,
начинающейся с '---'), patch проигнорирует. Т.е., вариант "в начало
полученного от diff файл дописать комментарий" вполне работает.

Ну и ещё: имя патча 'rdesktop-ctrl-alt-enter-minimize.patch' не
удовлетворяет требованиям из http://www.altlinux.org/Spec#Patch :
должно быть что-то вида rdesktop-1.6.0-alt-ctrl_alt_enter-minimize.patch

-- 
С уважением,
Николай Фетисов

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 17:54:50 Nikolay A. Fetisov wrote:
> Всё равно. Строки до начала собственно патча (т.е. до строки,
> начинающейся с '---'), patch проигнорирует. Т.е., вариант "в начало
> полученного от diff файл дописать комментарий" вполне работает.
>
ясно, спасибо

> Ну и ещё: имя патча 'rdesktop-ctrl-alt-enter-minimize.patch' не
> удовлетворяет требованиям из http://www.altlinux.org/Spec#Patch :
> должно быть что-то вида rdesktop-1.6.0-alt-ctrl_alt_enter-minimize.patch
ну тут я велосипед не изобретал и стандарт не искал, сделал так, как уже 
сделаны патчи в rdesktop-1.6.0 из сизифа

Re: [Comm] не применяется патч при сборке пакета

[Sergey]

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

> Вариант 1. Вставить свои изменения в уже существующий патч. Как-то не 
> нравится...
> Вариант 2. Подправить вывод diff для патчинга таким образом, чтобы убрать 
> изменения предыдущего патча и оставить только свое. Пока сделал так..
 
Вариант 3: аккуратно и последовательно делать дифы, причём лучше без
rpm, чтобы не запутаться. Сначала наложить всё, что надо, затем поправить
полученное своим патчем и сделать последний диф.

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

Re: [Comm] не применяется патч при сборке пакета

[Denis Nazarov]

On Sunday 15 March 2009 23:18:22 Sergey wrote:
> On Воскресенье 15 марта 2009, Denis Nazarov wrote:
> > Вариант 1. Вставить свои изменения в уже существующий патч. Как-то не
> > нравится...
> > Вариант 2. Подправить вывод diff для патчинга таким образом, чтобы
> > убрать изменения предыдущего патча и оставить только свое. Пока сделал
> > так..
>
> Вариант 3: аккуратно и последовательно делать дифы, причём лучше без
> rpm, чтобы не запутаться. Сначала наложить всё, что надо, затем
> поправить полученное своим патчем и сделать последний диф.

В общем-то, так и получилось. Результат отрапортовал в sisyphus@ - патч на 
rdesktop