Bug 8594

Summary:	errors in spec
Product:	Sisyphus	Reporter:	Igor Zubkov <icesik>
Component:	pptp-client	Assignee:	Vitaly Lipatov <lav>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	minor
Priority:	P2	CC:	lav, mike
Version:	unstable
Hardware:	all
OS:	Linux
URL:	http://sisyphus.ru/srpm/pptp-client/spec

Description Igor Zubkov 2005-12-05 20:17:10 MSK

В спеке сейчас прописано такое:
%prep
rm -rf $RPM_BUILD_ROOT

хотя этого (rm -rf $RPM_BUILD_ROOT) быть не должно.

Дальше, вместо %make должно использоватся %make_build
И вместо $RPM_BUILD_ROOT должно использоватся %buildroot

Да и COPYING можно грохнуть. Он там лишний. Такой же есть в common-license.

Comment 1 Michael Shigorin 2009-10-04 14:29:56 MSD

(In reply to comment #0)
> В спеке сейчас прописано такое:
> rm -rf $RPM_BUILD_ROOT
Уже не прописано.

> Дальше, вместо %make должно использоватся %make_build
Нет.  _Может_, но не _должно_.  См. rpm --eval их обоих.

> И вместо $RPM_BUILD_ROOT должно использоватся %buildroot
Может, но не должно.

> Да и COPYING можно грохнуть. Он там лишний. Такой же есть в common-license.
На самом деле это ещё одно "серое пятно" у нас: по-хорошему, COPYING можно удалять только в случае сверки идентичности и простановки симлинка; некоторые скажут, что лучше вообще оставить авторскую копию (которая может и измениться между версиями); в общем, повод для отдельного обсуждения.  Не здеся.

Comment 2 Igor Zubkov 2009-10-04 14:49:20 MSD

(В ответ на комментарий №1)
> > Да и COPYING можно грохнуть. Он там лишний. Такой же есть в common-license.
> На самом деле это ещё одно "серое пятно" у нас: по-хорошему, COPYING можно
> удалять только в случае сверки идентичности и простановки симлинка; некоторые
> скажут, что лучше вообще оставить авторскую копию (которая может и измениться
> между версиями); в общем, повод для отдельного обсуждения.  Не здеся.

Я как-то от скуки писал тест для репокопа который определяет лицензия и жалуется. :) Как оказалось GPLv2 лицензии существует штук десять.