Как делать патчи? (was: Re: [sisyphus] patch для MC)

Как делать патчи? (was: Re: [sisyphus] patch для MC)

[php-coder]

--- Siver Andrey 2005-04-05 19:05:13 +0400
+++ php-coder 2005-04-05 23:48:27 +0700

    Маленький совет на будущее: все патчи нужно делать с ключом -u. Если
    вы поправили несколько файлов директории, но используйте также ключ
    -r.

    Пример:

    $ diff -ru fluxbox-0.9.12.orig fluxbox-0.9.12 > myCoolPatch.patch

    Если патч получился большим, то его рекомендуется упаковать :))
    (комаду приводить не буду, думаю, что это все знают)

    Удачи!

    P.S. Хорошую работу делаете -- молодцы! :))

-- 
+ php-coder
+ php-coder[at]ngs[dot]ru
+ ALTLinux Master 2.4 (kernel 2.4.26-std-up-alt10)

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Siver Andrey]

----- Original Message ----- 
From: "php-coder" <php-coder@ngs.ru>
To: "ALT Linux Sisyphus discussion list" <sisyphus@altlinux.ru>
Sent: Tuesday, April 05, 2005 9:09 PM
Subject: Как делать патчи? (was: Re: [sisyphus] patch для MC)


>
> --- Siver Andrey 2005-04-05 19:05:13 +0400
> +++ php-coder 2005-04-05 23:48:27 +0700
>
>     Маленький совет на будущее: все патчи нужно делать с ключом -u.

А зачем -u ?

> Если
>     вы поправили несколько файлов директории, но используйте также ключ
>     -r.
>
>     Пример:
>
>     $ diff -ru fluxbox-0.9.12.orig fluxbox-0.9.12 > myCoolPatch.patch
>

Andrey

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Andrey Rahmatullin]

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

On Wed, Apr 06, 2005 at 10:44:08AM +0400, Siver Andrey wrote:
> >     Маленький совет на будущее: все патчи нужно делать с ключом -u.
> А зачем -u ?
Затем, что это общепринятая практика, и без -u ваш патч никто не примет.

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

Пришел в АЛЬТ за очередным обновлением. После заливки файлов на винт
ноутбука сменил XFree на X.org в присутствии и под наблюдением нашего
сборщика X.org. DRI испугался совокупной мощи и заработал :).
		-- slava in devel@

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

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Anton D. Kachalov]

On Wed, Apr 06, 2005 at 10:44:08AM +0400, Siver Andrey wrote:
> 
> ----- Original Message ----- 
> From: "php-coder" <php-coder@ngs.ru>
> To: "ALT Linux Sisyphus discussion list" <sisyphus@altlinux.ru>
> Sent: Tuesday, April 05, 2005 9:09 PM
> Subject: Как делать патчи? (was: Re: [sisyphus] patch для MC)
> 
> 
> >
> > --- Siver Andrey 2005-04-05 19:05:13 +0400
> > +++ php-coder 2005-04-05 23:48:27 +0700
> >
> >     Маленький совет на будущее: все патчи нужно делать с ключом -u.
> 
> А зачем -u ?
Достаточно просто посмотреть на разницу с "-u" и без. Очень классно
накладывать патч (без "-u"), который решил съехать в честь праздника
какого-нить языческого бога.

Rgds,
Anton

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Siver Andrey]

----- Original Message ----- 
From: "Anton D. Kachalov" <mouse@altlinux.org>
To: "ALT Linux Sisyphus discussion list" <sisyphus@altlinux.ru>
Sent: Wednesday, April 06, 2005 11:11 AM
Subject: Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)


> On Wed, Apr 06, 2005 at 10:44:08AM +0400, Siver Andrey wrote:
> >
> > ----- Original Message ----- 
> > From: "php-coder" <php-coder@ngs.ru>
> > To: "ALT Linux Sisyphus discussion list" <sisyphus@altlinux.ru>
> > Sent: Tuesday, April 05, 2005 9:09 PM
> > Subject: Как делать патчи? (was: Re: [sisyphus] patch для MC)
> >
> > > --- Siver Andrey 2005-04-05 19:05:13 +0400
> > > +++ php-coder 2005-04-05 23:48:27 +0700
> > >
> > >     Маленький совет на будущее: все патчи нужно делать с ключом -u.
> >
> > А зачем -u ?
> Достаточно просто посмотреть на разницу с "-u" и без. Очень классно
> накладывать патч (без "-u"), который решил съехать в честь праздника
> какого-нить языческого бога.

Ну, априорного знания об этом всем у меня нет, при рождении не заложили...
Так что, переслать с -u и в Багзиллу?


Андрей

>
> Rgds,
> Anton
> _______________________________________________
> Sisyphus mailing list
> Sisyphus@altlinux.ru
> https://lists.altlinux.ru/mailman/listinfo/sisyphus
>
>

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Anton D. Kachalov]

On Wed, Apr 06, 2005 at 02:23:56PM +0400, Siver Andrey wrote:
> Ну, априорного знания об этом всем у меня нет, при рождении не заложили...
> Так что, переслать с -u и в Багзиллу?

Вот как-то настораживает это:
After some numbers of the update calls (3-5 and more), "enter" key does not
work any longer for any directory of the panel opposited to the updating: error
occurs: "cannot change directory". 'magic_path' function does not detect any
problem though. I do not know whether it due to my patch, or due to MC or system
property.

Всё-таки патч должен быть работающим, а не добавляющим сомнительные фичи,
вносящие баги.

Rgds,
Anton

Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)

[Siver Andrey]

----- Original Message ----- 
From: "Anton D. Kachalov" <mouse@altlinux.org>
To: "ALT Linux Sisyphus discussion list" <sisyphus@altlinux.ru>
Sent: Wednesday, April 06, 2005 4:32 PM
Subject: Re: Как делать патчи? (was: Re: [sisyphus] patch для MC)


> On Wed, Apr 06, 2005 at 02:23:56PM +0400, Siver Andrey wrote:
> > Ну, априорного знания об этом всем у меня нет, при рождении не
заложили...
> > Так что, переслать с -u и в Багзиллу?
>
> Вот как-то настораживает это:
> After some numbers of the update calls (3-5 and more), "enter" key does
not
> work any longer for any directory of the panel opposited to the updating:
error
> occurs: "cannot change directory". 'magic_path' function does not detect
any
> problem though. I do not know whether it due to my patch, or due to MC or
system
> property.
>
> Всё-таки патч должен быть работающим, а не добавляющим сомнительные фичи,
> вносящие баги.

Я пока не уверен, что это мои баги там лезут. Ибо если убрать вызов
update_panel, то этой проблемы не возникает...
Буду разбираться. Советы велкам :).

>
> Rgds,
> Anton
> _______________________________________________
> Sisyphus mailing list
> Sisyphus@altlinux.ru
> https://lists.altlinux.ru/mailman/listinfo/sisyphus
>
>

Re: [sisyphus] Re: Как делать патчи?

[Andrey Rahmatullin]

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

On Wed, Apr 06, 2005 at 01:41:33PM +0500, Boldin Pavel wrote:
> я вообще использую -NurpP как в kernel.
Не нашел в мане опцию -P.

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

> Вы считаете нормальным наличие в системе *-devel и полного набора
> компиляторов всех версий, помимо тех, что нужны пользователю в работе?
Ну, некоторые пользуют hasher. :)
		-- mike in community@

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

[sisyphus] Re: Как делать патчи?

[Michael Shigorin]

On Thu, Apr 07, 2005 at 12:45:40AM +0600, Andrey Rahmatullin wrote:
> > я вообще использую -NurpP как в kernel.
> Не нашел в мане опцию -P.

Я привык к -Naur (кажется, voins@ посоветовал -- до этого было
что-то ещё :)

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

[sisyphus] Re: Как делать патчи?

[Alexey Tourbin]

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

On Wed, Apr 06, 2005 at 09:54:21PM +0300, Michael Shigorin wrote:
> On Thu, Apr 07, 2005 at 12:45:40AM +0600, Andrey Rahmatullin wrote:
> > > я вообще использую -NurpP как в kernel.
> > Не нашел в мане опцию -P.
> 
> Я привык к -Naur (кажется, voins@ посоветовал -- до этого было
> что-то ещё :)

А у меня вот такое вот безобразие прижилось:

$ which diff
diff () {
        for ext in - \~ .orig
        do
                if [ -z "$2" -a -e "$1$ext" ]
                then
                        2="$1"
                        1="$1$ext"
                        break
                fi
        done
        LC_ALL=C TZ=UTC0 =diff -abBpruw "$@"
}
$

Это всё конечно надо доработать...

$ which vd
vd () {
        vim -c 'setf diff' =(diff "$@")
}
$

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

[sisyphus] Re: Как делать патчи?

[Michael Shigorin]

On Wed, Apr 06, 2005 at 10:59:32PM +0400, Alexey Tourbin wrote:
> А у меня вот такое вот безобразие прижилось:

Уффф... dotfiles надо :)

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

Re: [sisyphus] Re: Как делать патчи?

[php-coder]

--- Andrey Rahmatullin 2005-04-07 00:45:40 +0600
+++ php-coder 2005-04-08 10:41:11 +0700

> > я вообще использую -NurpP как в kernel.
> Не нашел в мане опцию -P.

    Странно...

    man diff
    /-P
    -P     При сравнении каталогов, если файл существует только  во
           втором каталоге, то считать, что он есть и в другом каталоге,
           но только пустой.

    Это в Мастере.

-- 
+ php-coder
+ php-coder[at]ngs[dot]ru
+ ALTLinux Master 2.4 (kernel 2.4.26-std-up-alt10)

Re: [sisyphus] Re: Как делать патчи?

[Andrey Rahmatullin]

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

On Fri, Apr 08, 2005 at 10:44:16AM +0700, php-coder wrote:
>     man diff
>     /-P
>     -P     При сравнении каталогов, если файл существует только  во
>            втором каталоге, то считать, что он есть и в другом каталоге,
>            но только пустой.
У меня man-pages-ru не стоит, а в оригинальном такой опции нет. Зато есть
-N, всю жизнь использующася для тех же целей.

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

Собственно говоря, меня интересует ответ на вопрос, куда правильнее было
бы переместить libgd1 - в orphaned или obsolete?
		-- ldv in devel@

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

Re: [sisyphus] Re: Как делать патчи?

[Eugene Vlasov]

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

Приветствую, php-coder.
В письме от Fri, Apr 08, 2005 at 10:44:16AM +0700 вы пишете:


>> Не нашел в мане опцию -P.

>     Странно...

>     man diff
>     /-P
>     -P     При сравнении каталогов, если файл существует только  во
>            втором каталоге, то считать, что он есть и в другом каталоге,
>            но только пустой.

>     Это в Мастере.

А попробуйте "LANG=C man diff". По моему, это то место, где русский
man устарел (diff --help тоже такого не показывает). А то, что вы
написали - это вероятнее всего ключ -N.


-- 
С уважением, Евгений                          mailto:eugvv@altlinux.ru


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

Re: [sisyphus] Re: Как делать патчи?

[Eugene Vlasov]

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

Приветствую, Eugene Vlasov.
В письме от Fri, Apr 08, 2005 at 04:28:13PM +0600 вы пишете:

>>     -P     При сравнении каталогов, если файл существует только  во
>>            втором каталоге, то считать, что он есть и в другом каталоге,
>>            но только пустой.

> А попробуйте "LANG=C man diff". По моему, это то место, где русский
> man устарел (diff --help тоже такого не показывает). А то, что вы
> написали - это вероятнее всего ключ -N.

Чтобы окончательно расставить точки - выдержка из
/usr/share/doc/diffutils-2.8.4/NEWS:

User-visible changes in version 2.8:

...
* The following diff options are still accepted, but are no longer documented.
  They may be withdrawn in future releases.
  ...
  -P (use --unidirectional-new-file instead)
...


-- 
С уважением, Евгений                          mailto:eugvv@altlinux.ru


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