On Fri, 17 Dec 2004 20:33:00 +0100, Stefan Sonnenberg-Carstens wrote:
I got rpms compiled under FC3.
They are available under
http://www.coolspot.de/fedora-devel/
Test them, please.
The src.rpm fails to build on FC3. Try to clean up your installation
to find missing build dependencies:
http://fedoraproject.org/wiki/HOWTOFindMissingBuildRequires
Other packaging mistakes after a brief look. In random order:
* Main package must not contain files %_libdir/*.so
All these belong into the -devel package. Same applies to the
-unixodbc sub package. It must not include %_libdir/*.so either.
These files are needed at build-time only.
* rpmlint reports a few relevant things:
E: freetds summary-too-long FreeTDS is a free re-implementation of the TDS (Tabular DataStream) protocol that is used by Sybase and Microsoft for their database products.
=> better: A free re-implementation of the TDS protocol
Everything longer than ~70 characters ought to be put into the
package description.
W: freetds no-url-tag
=> You've put that into "Vendor:" field instead. Change that into
"URL: http://www.freetds.org"; -- the vendor is the package vendor
and usually will be substituted by a build system automatically.
E: freetds no-signature
=> minor issue for a test package - you didn't sign it with
your GPG key
W: freetds strange-permission freetds.spec 0664
=> also unimportant, but a user who would extract the file into
/tmp would end up with a group-writable file
W: freetds obsolete-tag Copyright
=> should be "License: LGPL" not "Copyright: LGPL"
%define name freetds
%define version 0.62.4
Name: %{name}
Version: %{version}
That construct is redundant. The %name and %version macros are defined
by "Name:" and "Version:" already.
%package devel
Requires: freetds = %{version}
Better would be:
Requires: freetds = %{version}-%release
Why? Imagine you apply a patch which patches the header files or other
files in the -devel package or which fixes a bug in the main package.
You want main package release and -devel package release to stay in
sync. "yum update freetds-devel" should pull in the updated "freetds"
main package, too.
%post
/sbin/ldconfig
Better:
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig
Why? It adds an automatic dependency on /sbin/ldconfig and updates the
run-time linker cache also after package un-installation, when the
freetds libraries are gone. Your %post script would be executed in
/bin/sh.
%configure --with-tdsver=4.2 --prefix=/usr --with-unixodbc=/usr/ --sysconfdir=%{_sysconfdir}
The %configure macro sets --prefix=%_prefix and --syconfdir=%_sysconfdir
already. You can trim the line a bit. :)
%doc AUTHORS BUGS COPYING ChangeLog INSTALL NEWS README TODO
COPYING is GPL, which is not what's specified in "License:" field.
File COPYING.LIB would be the LGPL and should be included.
The INSTALL file contains generic instructions, which are irrelevant
to the package user.
%{_libdir}/libct.so*
Files with such non-unique names will likely cause conflicts in
%_libdir in the future, when more and more packages share that
directory and more projects use similar short names. This topic will
come back in the near future as it will be necessary more often to
install such libraries into %_libdir/%name/ and add that path to the
run-time linker's search path list.
Same with regard to some of the header names in %_includedir:
-rw-r--r-- root root 3036 /usr/include/bkpublic.h
-rw-r--r-- root root 22514 /usr/include/cspublic.h
-rw-r--r-- root root 1251 /usr/include/cstypes.h
-rw-r--r-- root root 4492 /usr/include/ctpublic.h
-rw-r--r-- root root 2207 /usr/include/dblib.h
-rw-r--r-- root root 1096 /usr/include/sqldb.h
-rw-r--r-- root root 1119 /usr/include/sqlfront.h
%package unixodbc
Requires: freetds = %{version}, unixODBC >= 2.2.9
Fedora Core 3 includes unixODBC 2.2.9, the dependency on libodbcinst.so.1
is automatic, so the explicit dependency on a package called
unixODBC doesn't add any value. Be careful with "explicit Requires".
If Fedora Core 4 moved libodbcinst.so.1 into a package with a different
name, this dependency would break.
With regard to the freetds dependency, same as above.
