From e9ca6b6bf48b23559384e1108e01df68a9ee18b6 Mon Sep 17 00:00:00 2001
From: Juergen Spitzmueller <spitz@lyx.org>
Date: Wed, 19 Sep 2012 17:46:55 +0200
Subject: [PATCH] Support for the Minion Pro LaTeX font

---
 development/FORMAT      |  5 +++++
 lib/chkconfig.ltx       |  1 +
 lib/doc/LaTeXConfig.lyx | 34 ++++++++++++++++++++++++++++++++--
 lib/latexfonts          | 13 +++++++++++++
 lib/lyx2lyx/lyx_2_1.py  | 25 +++++++++++++++++++++++--
 src/LaTeXFonts.cpp      | 11 +++++++++--
 src/LaTeXFonts.h        |  4 ++++
 src/version.h           |  4 ++--
 8 files changed, 89 insertions(+), 8 deletions(-)

diff --git a/development/FORMAT b/development/FORMAT
index a78a36adb0..f75d182a73 100644
--- a/development/FORMAT
+++ b/development/FORMAT
@@ -11,6 +11,11 @@ adjustments are made to tex2lyx and bugs are fixed in lyx2lyx.
 
 -----------------------
 
+
+2012-09-19 Jürgen Spitzmüller <spitz@lyx.org>
+	* Format incremented to 439 
+	  Support for the MinionPro LaTeX font.
+
 2012-08-23 Jürgen Spitzmüller <spitz@lyx.org>
 	* Format incremented to 438 
 	  Support for some IPA macros that are not unicoded.
diff --git a/lib/chkconfig.ltx b/lib/chkconfig.ltx
index 8fe2f40c75..eac93e2ea7 100644
--- a/lib/chkconfig.ltx
+++ b/lib/chkconfig.ltx
@@ -366,6 +366,7 @@
 \TestPackage{mdbch}
 \TestPackage{mdput}
 \TestPackage{mdugm}
+\TestPackage{MinionPro}
 \TestPackage{tgadventor}
 \TestPackage{tgbonum}
 \TestPackage{tgchorus}
diff --git a/lib/doc/LaTeXConfig.lyx b/lib/doc/LaTeXConfig.lyx
index 1662106e31..dd6ac8c282 100644
--- a/lib/doc/LaTeXConfig.lyx
+++ b/lib/doc/LaTeXConfig.lyx
@@ -1,5 +1,5 @@
 #LyX 2.1 created this file. For more info see http://www.lyx.org/
-\lyxformat 437
+\lyxformat 438
 \begin_document
 \begin_header
 \textclass article
@@ -604,7 +604,7 @@ arg   "libertine"
 
 \begin_layout Description
 CTAN: 
-\family sans
+\family typewriter
 fonts/libertine-legacy/
 \end_layout
 
@@ -721,6 +721,36 @@ If you only want the text fonts and retain LaTeX's default math font, select
  a non-mathdesign variant of the respective font instead.
 \end_layout
 
+\begin_layout Subsection
+MinionPro
+\end_layout
+
+\begin_layout Description
+Found: 
+\begin_inset Info
+type  "package"
+arg   "MinionPro"
+\end_inset
+
+
+\end_layout
+
+\begin_layout Description
+CTAN: 
+\family typewriter
+fonts/minionpro
+\end_layout
+
+\begin_layout Description
+Notes: The 
+\family sans
+MinionPro
+\family default
+ package provides support for the Minion Pro font.
+ Note that the font itself is not part of the package and must be installed
+ separately (please consult the MinionPro package manual for details).
+\end_layout
+
 \begin_layout Subsection
 psnfss
 \end_layout
diff --git a/lib/latexfonts b/lib/latexfonts
index c0cc9ac027..b4d19a19ad 100644
--- a/lib/latexfonts
+++ b/lib/latexfonts
@@ -15,6 +15,7 @@
 #	CompletePackage    <alternative package for the complete family>
 #	PackageOptions     <general options to be passed to the package>
 #	OsfOption          <option for oldstyle figure support>
+#	OsfDefault         <0|1>
 #	ScOption           <option for true smallcaps support>
 #	OsfScOption        <option for combined osf and true smallcaps support>
 #	ScaleOption        <option for font scaling>
@@ -51,6 +52,9 @@
 #   f. ex. to load "bera" as opposed to "beraserif").
 # * OsfScOption overrides any OsfOption and ScOption if both features
 #   are selected.
+# * Set OsfDefault to true for fonts which have Old Style Figures by
+#   default and provide an option for lining figures. Pass this option
+#   to OsfOption.
 # * ScaleOption supports the placeholder $$val for the scale value.
 #
 ##########################################################################
@@ -155,6 +159,15 @@ Font mdugm
 	Provides         amssymb,amsfonts
 EndFont
 
+Font minionpro
+	GuiName          "Minion Pro"
+	Family           rm
+	OsfOption        lf
+	OsfDefault       1
+	Package          MinionPro
+	Provides         amssymb,amsfonts
+EndFont
+
 Font newcent
 	GuiName          "New Century Schoolbook"
 	Family           rm
diff --git a/lib/lyx2lyx/lyx_2_1.py b/lib/lyx2lyx/lyx_2_1.py
index ce0ff226f4..a2fa26c20a 100644
--- a/lib/lyx2lyx/lyx_2_1.py
+++ b/lib/lyx2lyx/lyx_2_1.py
@@ -983,7 +983,26 @@ def revert_ipachar(document):
             i = i + 1
     if found:
         add_to_preamble(document, "\\usepackage{tone}")
-    
+
+
+def revert_minionpro(document):
+    " Revert native MinionPro font definition to LaTeX " 
+
+    if find_token(document.header, "\\use_non_tex_fonts false", 0) != -1: 
+        i = find_token(document.header, "\\font_roman minionpro", 0)
+        if i != -1:
+            osf = False
+            j = find_token(document.header, "\\font_osf true", 0)
+            if j != -1:
+                osf = True
+            preamble = "\\usepackage"
+            if osf:
+                document.header[j] = "\\font_osf false"
+            else:
+                preamble += "[lf]"
+            preamble += "{MinionPro}"
+            add_to_preamble(document, [preamble])
+            document.header[i] = "\\font_roman default"
 
 ##
 # Conversion hub
@@ -1015,10 +1034,12 @@ convert = [
            [435, []],
            [436, []],
            [437, []],
-           [438, []]
+           [438, []],
+           [439, []]
           ]
 
 revert =  [
+           [438, [revert_minionpro]],
            [437, [revert_ipadeco, revert_ipachar]],
            [436, [revert_texgyre]],
            [435, [revert_mathdesign]],
diff --git a/src/LaTeXFonts.cpp b/src/LaTeXFonts.cpp
index 70c500543b..4084a73694 100644
--- a/src/LaTeXFonts.cpp
+++ b/src/LaTeXFonts.cpp
@@ -139,16 +139,17 @@ string const LaTeXFont::getPackageOptions(bool ot1, bool sc, bool osf, int scale
 		return string();
 
 	ostringstream os;
+	bool const needosfopt = (osf != osfdefault_);
 	if (!packageoption_.empty())
 		os << to_ascii(packageoption_);
-	if (sc && osf && providesOSF() && providesSC()) {
+	if (sc && needosfopt && providesOSF() && providesSC()) {
 		if (!os.str().empty())
 			os << ',';
 		if (!osfscoption_.empty())
 			os << to_ascii(osfscoption_);
 		else
 			os << to_ascii(osfoption_) << ',' << to_ascii(scoption_);
-	} else if (osf && providesOSF()) {
+	} else if (needosfopt && providesOSF()) {
 		if (!os.str().empty())
 			os << ',';
 		os << to_ascii(osfoption_);
@@ -211,6 +212,7 @@ bool LaTeXFont::readFont(Lexer & lex)
 		LF_END,
 		LF_FAMILY,
 		LF_GUINAME,
+		LF_OSFDEFAULT,
 		LF_OSFOPTION,
 		LF_OSFPACKAGE,
 		LF_OSFSCOPTION,
@@ -231,6 +233,7 @@ bool LaTeXFont::readFont(Lexer & lex)
 		{ "endfont",              LF_END },
 		{ "family",               LF_FAMILY },
 		{ "guiname",              LF_GUINAME },
+		{ "osfdefault",           LF_OSFDEFAULT },
 		{ "osfoption",            LF_OSFOPTION },
 		{ "osfpackage",           LF_OSFPACKAGE },
 		{ "osfscoption",          LF_OSFSCOPTION },
@@ -288,6 +291,9 @@ bool LaTeXFont::readFont(Lexer & lex)
 		case LF_OSFPACKAGE:
 			lex >> osfpackage_;
 			break;
+		case LF_OSFDEFAULT:
+			lex >> osfdefault_;
+			break;
 		case LF_OSFSCOPTION:
 			lex >> osfscoption_;
 			break;
@@ -332,6 +338,7 @@ bool LaTeXFont::readFont(Lexer & lex)
 bool LaTeXFont::read(Lexer & lex)
 {
 	switchdefault_ = 0;
+	osfdefault_ = 0;
 
 	if (!lex.next()) {
 		lex.printError("No name given for LaTeX font: `$$Token'.");
diff --git a/src/LaTeXFonts.h b/src/LaTeXFonts.h
index 54ed5ad9c4..01886be6ca 100644
--- a/src/LaTeXFonts.h
+++ b/src/LaTeXFonts.h
@@ -59,6 +59,8 @@ public:
 	bool provides(std::string const & name) const;
 	/// Issue the familydefault switch
 	bool switchdefault() const { return switchdefault_; }
+	/// Does the font provide Old Style Figures as default?
+	bool osfDefault() const { return osfdefault_; }
 	/// Is this font available?
 	bool available(bool ot1 = false) const;
 	/// Does this font provide Old Style Figures?
@@ -117,6 +119,8 @@ private:
 	///
 	docstring requires_;
 	///
+	bool osfdefault_;
+	///
 	bool switchdefault_;
 	///
 	bool available_;
diff --git a/src/version.h b/src/version.h
index 8c899ad821..10a2212eb5 100644
--- a/src/version.h
+++ b/src/version.h
@@ -30,8 +30,8 @@ extern char const * const lyx_version_info;
 
 // Do not remove the comment below, so we get merge conflict in
 // independent branches. Instead add your own.
-#define LYX_FORMAT_LYX 438 // spitz: support IPA macros
-#define LYX_FORMAT_TEX2LYX 438 // spitz: support for IPA macros
+#define LYX_FORMAT_LYX 439 // spitz: support MinionPro LaTeX font
+#define LYX_FORMAT_TEX2LYX 439 // spitz: support MinionPro LaTeX font
 
 #if LYX_FORMAT_TEX2LYX != LYX_FORMAT_LYX
 #ifndef _MSC_VER
-- 
2.39.5