Re: Re: GNOME CVS: galeon baddog

On Sat, 2003-02-08 at 15:28, Yanko Kaneti wrote:
> On Sat, 2003-02-08 at 13:07, Gnome CVS User wrote:
> > CVSROOT:	/cvs/gnome
> > Module name:	galeon
> > Changes by:	baddog	03/02/08 06:07:46
> > 
> > Modified files:
> > 	.              : ChangeLog configure.in 
> > 
> > Log message:
> > Remove unnecessary building dependency
> > 
> > URL : http://cvs.gnome.org/bonsai/cvsquery.cgi?branch=&dir=galeon&who=baddog&date=explicit&mindate=2003-02-08%2006:06&maxdate=2003-02-08%2006:08&cvsroot=/cvs/gnome
> 
> Hi
> 
> Its ok that you removed the variables that were no longer used but the
> pkg config line had its form for a reason. We directly include headers
> from all the packages on that line so its correct to include them all as
> direct dependencies, regardless of the fact that they are
> sub-dependencies of the other higher level packages we use.

Actually, I think it might be better that the required versions were
updated to reflect the real requirements today, and added as build
dependencies in the galeon.spec.in.  See galeon-1-2 branch for an
example.  I think build dependencies are a good thing¹ when they're
accurate, but it's up to the rpm maintainer (whoever that might be.)


1. apt-get build-dep galeon-snapshot is a life saver :)

-- 
Tommi Komulainen                                 [email protected]
GPG 1024D/68388EE6    6FD6 DD79 EB38 BF6F 3533  09C0 04A8 9871 6838 8EE6

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQA+RibgBKiYcWg4juYRAuHYAKCZOtoMHoSGoq1aHI7qzWa+LrEMpQCgiLDm
Coa+gn5tSPFWkPksHbYvbWg=
=p9a7
-----END PGP SIGNATURE-----