Juergen Spitzmueller wrote:
> > Did you check that change tracking/dvipost still works in other classes?
> > If yes, I vote to include your patch.
>
> Yes, I tested within limits. I am away from my home machine and working
> with a devel version from Kayvan atm. I'll test more intensively when I'm
> back home.

I tested the attached patch on my home machine now quite intensely and 
discovered no problems. Change tracking works fine, beamer works fine. With 
the new code (\PassOptionsToPackage), even change tracking within beamer 
works!

Can I commit this, please?

Jürgen
Index: src/LaTeXFeatures.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/LaTeXFeatures.C,v
retrieving revision 1.119
diff -p -u -r1.119 LaTeXFeatures.C
--- src/LaTeXFeatures.C	11 Jun 2005 11:02:22 -0000	1.119
+++ src/LaTeXFeatures.C	21 Jan 2006 12:00:16 -0000
@@ -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