What is the license of this package?  License tag says LGPL, upstream tarball
contains both GPL and LGPL license texts, and only the GPL file is included in
binary packages.

The library is LGPL, and some executables are GPL.

Created attachment 454040 [details]
spec cleanup

Please commit this git patch that will clean this package for this review or allow to commit.

Why did you remove all version checks from Requires/BuildRequires? I know we have new versions of everything in rawhide nowadays but still we might want to bump some dependency one day, introducing versioning back.

> @@ -102,12 +85,6 @@ library, and other features.
>  Summary: Libraries and include files for developing GNOME VFS applications
>  Group: Development/Libraries
>  Requires:   %{name} = %{version}-%{release}
> -Requires:   GConf2-devel >= %{gconf2_version}
> -Requires:   hal-devel >= %{hal_version}
> -Requires:   dbus-devel >= %{dbus_version}
> -Requires:   dbus-glib-devel >= %{dbus_glib_version}
> -Requires:   pkgconfig
Why did you remove these explicit devel packages requires? They're there for purpose.

Otherwise all other changes look good. This package is considered dead anyway and will be removed in near or far future. It is maintained only as a dependency for packages that have not been ported to GIO yet.

1) Removing versioned BR:
=> Yes you are correct. We first used to build dependencies for this package and then this package. If that is the case then why to un-necessarily write versioned BR:?

I think repoquery is the best option to check which packages need to be bumped as part of any dependent package bump.


2) Removing explicit Requires:
==> Guidelines says https://fedoraproject.org/wiki/PackagingGuidelines#Explicit_Requires

If I have missed any explicit requires, please tell me and will add it back to suggested patch.

For your reference here is modified scratch package build
http://koji.fedoraproject.org/koji/taskinfo?taskID=2540235

And See Requires generated automatically from .pc files at
Processing files: gnome-vfs2-devel-2.24.4-2.fc15.x86_64
Requires: /usr/bin/pkg-config libgnomevfs-2.so.0()(64bit) pkgconfig(gconf-2.0) pkgconfig(gmodule-no-export-2.0) pkgconfig(gobject-2.0) pkgconfig(gthread-2.0)

Compared to older version
Processing files: gnome-vfs2-devel-2.24.4-1.fc15.x86_64
Requires: /usr/bin/pkg-config libgnomevfs-2.so.0()(64bit) pkgconfig(gconf-2.0) pkgconfig(gmodule-no-export-2.0) pkgconfig(gobject-2.0) pkgconfig(gthread-2.0)


Looks both are equal. When I looked into .pc files I have not found any reference to pull dbus and hal dependencies.


Please correct me if I am wrong.

I will commit this tomorrow.

Committed the above patch and built in gnome-vfs2-2.24.4-1.fc15

APPROVED.