[Slim-Checkins] r10646 - /trunk/platforms/fedora/SPECS/slimserver.spec

[pacifico at svn.slimdevices.com]

Author: pacifico
Date: Fri Nov 10 07:06:15 2006
New Revision: 10646

URL: http://svn.slimdevices.com?rev=10646&view=rev
Log:
Changed dependency on perl(DBIx::Class) to slim-DBIx-Class to alleviate the requirement to build a flawed perl-SQL-Translator package and added comments to explain all this.

Modified:
    trunk/platforms/fedora/SPECS/slimserver.spec

Modified: trunk/platforms/fedora/SPECS/slimserver.spec
URL: http://svn.slimdevices.com/trunk/platforms/fedora/SPECS/slimserver.spec?rev=10646&r1=10645&r2=10646&view=diff
==============================================================================
--- trunk/platforms/fedora/SPECS/slimserver.spec (original)
+++ trunk/platforms/fedora/SPECS/slimserver.spec Fri Nov 10 07:06:15 2006
@@ -89,6 +89,7 @@
 
 # The following requires are in RPMS built by Al Pacifico, but not yet in any
 # Fedora base, extras, or RpmForge:
+Requires:		perl(Class::Data::Accessor)
 Requires:       perl(Data::VString)
 Requires:       perl(DBIx::Migration)
 Requires:       perl(File::Which)
@@ -103,11 +104,21 @@
 Requires:       perl(URI::Find)
 # The next one requires perl(XML::Writer) which is available on RpmForge
 Requires:       perl(XML::XSPF)
+# Eventually, the next one should be provided by a perl-DBIx-Class system
+# package and the Requires line changed to perl(DBIx::Class). However,
+# perl-DBIx-Class has an optional dependency on perl(SQL::Translator),
+# which itself depends on perl-Template-Toolkit. The newest version of
+# perl-Template-Toolkit (2.15) seems to have a bug (absent from 2.14) 
+# that causes the unit testing for perl-SQL-Translator to fail. See the
+# following URL: http://rt.cpan.org/Public/Bug/Display.html?id=20047.
+# Rather than entirely skip the unit-# testing, and take the chance that
+# slimserver might be broken from another bug in this sub-dependency,
+# we've built this RPM to omit that sub-dependency. Slimserver itself 
+# does not depend on perl(SQL::Translator), so it is not an issue from a
+# slimserver point of view, but from a systems package point of view, this
+# is suboptimal.
+Requires	:	slim-DBix-Class
 Requires:       alac_decoder
-
-# Additional dependencies (not in FE) discovered during initial testing
-Requires:	perl(Class::Data::Accessor)
-Requires:	perl(DBIx::Class::Schema)
 
 %description
 This is the Slim Devices server software.