BR w perl-* (Re: SPECS: perl-DBI.spec (HEAD))

[Jakub Bogusz]

On Fri, Apr 11, 2003 at 10:47:47AM +0200, qboosh wrote:
> Module name:	SPECS
> Changes by:	qboosh	03/04/11 10:47:45
> 
> Modified files:
> 	perl-DBI.spec
> 
> Log message:
> - BRs are needed also for resolving dependencies
> 
> Index: perl-DBI.spec
> ===================================================================
> RCS file: /cvsroot/SPECS/perl-DBI.spec,v
> diff -d -u -r1.43 -r1.44
> --- perl-DBI.spec	8 Apr 2003 21:52:53 -0000	1.43
> +++ perl-DBI.spec	11 Apr 2003 08:47:45 -0000	1.44
> @@ -1,7 +1,8 @@
>  # $Revision$, $Date$
>  #
>  # Conditional build:
> -# _with_tests - perform "make test"
> +# _without_autodeps	- don't BR packages needed only for resolving deps
> +# _with_tests		- perform "make test"
>  #
>  %include	/usr/lib/rpm/macros.perl
>  %define	pdir	DBI
> @@ -17,7 +18,7 @@
>  Patch0:		perl-DBI-changes.patch
>  BuildRequires:	rpm-perlprov >= 4.1-13
>  BuildRequires:	perl >= 5.6.1
> -%if %{?_with_tests:1}%{!?_with_tests:0}
> +%if 0%{!?_without_autodeps:1}%{?_with_tests:1}
>  BuildRequires:	perl-PlRPC
>  BuildRequires:	perl-Storable
>  BuildRequires:	perl-Net-Daemon

Może być taki kompromis?
Na builderach dystrybucyjnych te BR są potrzebne, żeby zależności
deterministycznie rozwiązywały się do nazw pakietów.
A w przypadku domyślnie wyłączonych testów BR były także wyłączone...

A na zależności bezwarunkowe niektórzy marudzili.


-- 
Jakub Bogusz    http://cyber.cs.net.pl/~qboosh/