From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ed@sam-solutions.net>
X-Mailer: Gnus v5.8.8/XEmacs 21.4 - "Portable Code"
X-Comment-To: Peter Novodvorsky
To: devel-kernel@altlinux.ru
Subject: Re: [d-kernel] patches: kernel version dependancies
References: <874r53a6db.fsf@velvet.po.cs.msu.su>
In-Reply-To: <874r53a6db.fsf@velvet.po.cs.msu.su>
From: ed@sam-solutions.net (Ed V. Bartosh)
Organization: SaM-Solutions Ltd.
Date: 14 Apr 2003 11:24:08 +0400
Message-ID: <m3isthh78n.fsf@sam-solutions.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 8bit
Sender: devel-kernel-admin@altlinux.ru
Errors-To: devel-kernel-admin@altlinux.ru
X-BeenThere: devel-kernel@altlinux.ru
X-Mailman-Version: 2.0.9
Precedence: bulk
Reply-To: devel-kernel@altlinux.ru
List-Unsubscribe: <http://altlinux.ru/mailman/listinfo/devel-kernel>,
	<mailto:devel-kernel-request@altlinux.ru?subject=unsubscribe>
List-Id: ALT Linux kernel packages development <devel-kernel.altlinux.ru>
List-Post: <mailto:devel-kernel@altlinux.ru>
List-Help: <mailto:devel-kernel-request@altlinux.ru?subject=help>
List-Subscribe: <http://altlinux.ru/mailman/listinfo/devel-kernel>,
	<mailto:devel-kernel-request@altlinux.ru?subject=subscribe>
List-Archive: <http://altlinux.ru/pipermail/devel-kernel/>
Archived-At: <http://lore.altlinux.org/devel-kernel/m3isthh78n.fsf@sam-solutions.net/>
List-Archive: <http://lore.altlinux.org/devel-kernel/>
List-Post: <mailto:devel-kernel@altlinux.org>

Hello, Peter

 PN> Есть предложение по обработке переменной KVER в apply скриптах. надо
 PN> вызывать функцию apply_recursive для каталога
 PN> /usr/src/patches/$PATCH_NAME/$KVER/. Причём поддерживать эти каталоги
 PN> и будут поддерживаться specами. Если, например для нового ядра не
 PN> требуется изменений патчей, делается ссылка $KVER_NEW -> $KVER, а если
 PN> изменились не все патчи, -- то тоже можно делать ссылки на
 PN> соответствующие патчи из предыдущих версий. Ну и т. д.

 PN> Ещё надо поддерживать ссылку default, которая будет указывать на ту
 PN> версию, которую прикладывать, если соответствующая версия ядра не
 PN> поддерживается. Скорее всего надо указывать на последнюю
 PN> поддерживаемую версию.

Идея правильная, поддерживаю. Правда смущает некоторое усложнение
схемы и есть сомнения в том, что это будет часто востребовано.
Можно пока не делать, посмотреть что будет при переходе на другую
версию и потом решить.

Встречная идея, я почти уже сделал: Вынесение стандартного способа
приложения патчей в макрос и вызов apply-name-of-the-patch из него, но
только в случае какой-то нестандартщины. Позволит избавиться от 
apply-... в большинстве случаев.
То есть схема такая - если есть apply для данного патча - зовем,
если нет, то все делается стандартным способом из макроса.

-- 
Best regards,
Ed V. Bartosh