Herbert Voss wrote:

> -- this is not user friendly and can give errors when
> another package needs color with anotehr option.

OK.

> -- you should load xcolor with

I guess xcolor support is not an issue for 1.4.0 (I agree that we should add
it to forthcoming versions, though). This is just to restore the status of
1.3, i.e. get beamer working again.

> \IfFileExists{xcolor.sty}{%
> \PassOptionsToPackage{dvipsnames}{xcolor}
> \RequirePackage{xcolor}}{%

Isn't RequirePackage for packages and classes and usepackage for the
preamble (this is what I read in the documents)?

How about the attached? Makes the code even simpler.

Jürgen
--- LaTeXFeatures.C.old	2006-01-17 21:27:47.000000000 +0100
+++ LaTeXFeatures.C	2006-01-17 23:00:36.000000000 +0100
@@ -281,13 +281,13 @@ string const LaTeXFeatures::getPackages(
 
 	// color.sty
 	if (isRequired("color")) {
-		if (params_.graphicsDriver == "default")
-			packages << "\\usepackage[usenames]{color}\n";
-		else
-			packages << "\\usepackage["
+		if (params_.graphicsDriver != "default")
+			packages << "\\PassOptionsToPackage{"
 				 << params_.graphicsDriver
-				 << ",usenames"
-				 << "]{color}\n";
+				 << "}{color}\n";
+		if (isRequired("dvipost"))
+			packages << "\\PassOptionsToPackage{usenames}{color}\n";
+		packages << "\\usepackage{color}\n";
 	}
 
 	// makeidx.sty

Reply via email to