rpms/openvrml/devel openvrml.spec,1.71,1.72

[Tom Callaway]

Author: spot

Update of /cvs/pkgs/rpms/openvrml/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv5299

Modified Files:
	openvrml.spec 
Log Message:
undo arch-specific deps on gecko-libs and java


Index: openvrml.spec
===================================================================
RCS file: /cvs/pkgs/rpms/openvrml/devel/openvrml.spec,v
retrieving revision 1.71
retrieving revision 1.72
diff -u -p -r1.71 -r1.72
--- openvrml.spec	26 Aug 2009 08:14:40 -0000	1.71
+++ openvrml.spec	12 Sep 2009 20:26:23 -0000	1.72
@@ -1,7 +1,7 @@
 # -*- rpm-spec -*-
 Name:           openvrml
 Version:        0.18.3
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        VRML/X3D runtime library
 License:        LGPLv3+
 Group:          System Environment/Libraries
@@ -24,8 +24,15 @@ BuildRequires:  gtk2-devel >= 2.12
 BuildRequires:  gtkglext-devel
 BuildRequires:  curl-devel
 BuildRequires:  java-devel
-Requires:       gecko-libs%{?_isa} >= 1.9.1
-Requires:       java%{?_isa}
+# You cannot use %%{?isa} to map these requires to an architecture target
+# The only Requires which have %%{?isa} are those matching the name of the package
+# Since both "gecko-libs" and "java" are not the name of the packages that provide them
+# (they're manually defined Provides) they do not have %%{?isa}. You should ask the maintainer
+# of those packages to add explicit provides that use %%{?isa} (in addition to the normal ones).
+# Alternately, you could use a file dependency here, at least for "java", to force an architecture
+# match. -Tom Callaway <tcallawa at redhat.com>
+Requires:       gecko-libs >= 1.9.1
+Requires:       java
 
 %description
 OpenVRML is a VRML/X3D support library, including a runtime and facilities
@@ -188,6 +195,10 @@ CXXFLAGS="%optflags -fvisibility=hidden 
 %{_javadocdir}/%{name}-%{version}
 
 %changelog
+* Sat Sep 12 2009 Tom "spot" Callaway <tcallawa at redhat.com> - 0.18.3-3
+- undo arch-specific deps on gecko-libs and java (%%{?isa} doesn't work like you think it works)
+  for more details, see the comment in the spec file
+
 * Wed Aug 26 2009 Alex Lancaster <alexlan[AT]fedoraproject org> - 0.18.3-2
 - Rebuild for broken deps