[d-kernel] broken files in CVS (don't forget to use -kb)

[d-kernel] broken files in CVS (don't forget to use -kb)

[Sergey Vlasov]

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

Hello!

В ядерном CVS побилась часть файлов с патчами. Причина в том, что эти
файлы содержат ключевые слова CVS - $Id$, $Revision$ и т.п.; при
занесении их в CVS эти ключевые слова были заменены, в результате
некоторые патчи просто не накладываются, другие накладываются, но
отличаются от оригинала.

Патчи, которые определённо были испорчены:

kernel-feat-core-supermount
kernel-feat-core-win4lin
kernel-feat-drivers-block-enbd
kernel-feat-drivers-net-sk98lin
kernel-feat-drivers-usb

Вроде бы после cvs admin -kb всё встало на место - но на всякий случай
это надо проверить.

При добавлении новых файлов с патчами в CVS необходимо указывать в
команде cvs add ключ -kb, чтобы замена ключевых слов CVS не выполнялась.
Для уже существующих файлов */SOURCES/* этот ключ сейчас был проставлен,
за исключением тех случаев, когда замена действительно нужна (например,
в kernel-build-tools).

-- 
Sergey Vlasov

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

Re: [d-kernel] broken files in CVS (don't forget to use -kb)

[Anton Farygin]

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

Sergey Vlasov пишет:
> Hello!
> 
> В ядерном CVS побилась часть файлов с патчами. Причина в том, что эти
> файлы содержат ключевые слова CVS - $Id$, $Revision$ и т.п.; при
> занесении их в CVS эти ключевые слова были заменены, в результате
> некоторые патчи просто не накладываются, другие накладываются, но
> отличаются от оригинала.
> 
> Патчи, которые определённо были испорчены:
> 
> kernel-feat-core-supermount
> kernel-feat-core-win4lin
> kernel-feat-drivers-block-enbd
> kernel-feat-drivers-net-sk98lin
> kernel-feat-drivers-usb
> 
> Вроде бы после cvs admin -kb всё встало на место - но на всякий случай
> это надо проверить.

Да, cvs update -kb тоже ставит все на место.

> 
> При добавлении новых файлов с патчами в CVS необходимо указывать в
> команде cvs add ключ -kb, чтобы замена ключевых слов CVS не выполнялась.
> Для уже существующих файлов */SOURCES/* этот ключ сейчас был проставлен,
> за исключением тех случаев, когда замена действительно нужна (например,
> в kernel-build-tools).

ok.

Rgds,
Rider

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

Re: [d-kernel] broken files in CVS (don't forget to use -kb)

[Denis Ovsienko]

> > Вроде бы после cvs admin -kb всё встало на место - но на всякий случай
> > это надо проверить.
>
> Да, cvs update -kb тоже ставит все на место.
-kb --- не лучший способ хранения текстовых файлов. потом могут быть
проблемы с diff.

--
    DO4-UANIC

Re: [d-kernel] broken files in CVS (don't forget to use -kb)

[Anton Farygin]

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

Denis Ovsienko пишет:
>>>Вроде бы после cvs admin -kb всё встало на место - но на всякий случай
>>>это надо проверить.
>>
>>Да, cvs update -kb тоже ставит все на место.
> 
> -kb --- не лучший способ хранения текстовых файлов. потом могут быть
> проблемы с diff.

Я не знаю другого способа заставить CVS не изменять $Id и остальное в 
патчах.

Rgds,
Rider

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

Re: [d-kernel] broken files in CVS (don't forget to use -kb)

[Denis Ovsienko]

> > -kb --- не лучший способ хранения текстовых файлов. потом могут быть
> > проблемы с diff.
>
> Я не знаю другого способа заставить CVS не изменять $Id и остальное в
> патчах.
Там вроде бы был параметр precommit или что-то в таком роде, который
пропускает через себя тело файла перед его обработкой. Седерквист писал,
что таким образом можно включать вызов indent и прочие вещи. Возможно,
получится станцевать от этой точки?

--
    DO4-UANIC

[d-kernel] Re: broken files in CVS (don't forget to use -kb)

[Vitaly Ostanin]

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

On Sun, 21 Sep 2003 15:10:27 +0300 (EEST)
Denis Ovsienko <pilot@altlinux.ru> wrote:

> 
> > > Вроде бы после cvs admin -kb всё встало на место - но на
> > > всякий случай это надо проверить.
> >
> > Да, cvs update -kb тоже ставит все на место.
> -kb --- не лучший способ хранения текстовых файлов. потом могут
> быть проблемы с diff.

Можно diff делать на результатах
cvs export -kk

После "export -kk" остаются только ключевые слова без значений.
Как раз удобно делать diff на файлах из разных репозиториев.

<skipped/>

-- 
Regards, Vyt
mailto:  vyt@vzljot.ru
JID:     vyt@vzljot.ru

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