Fixed xelatex and luatex compilation errors due to ifpdf checks

CHANGELOG

@@ -1,3 +1,11 @@
+version 0.11 (xx July 2010)
+  - removed the \ifpdf check when loading the graphicx package, as graphicx does
+  does check internally and this was preventing proper functionning with xelatex
+  and luatex, as reported by A. Trautmann. The same check was also removed
+  everywhere else  in the document (i.e. when loading the color and hyperref
+  packages).
+  - 
+
 version 0.10 (13 June 2010)
   - fixed a compatibility issue with the natbib package (natbib uses \section*
   to format the bibliography title, while only \section was defined by moderncv),

examples/letter.tex

@@ -32,9 +32,8 @@
 \def\FileSubject{Cover letter}
 \def\FileKeyWords{\firstname \familyname, Cover letter}
 
-\renewcommand{\ttdefault}{pcr}
-
 \usepackage{url}
+\renewcommand{\ttdefault}{pcr}
 \urlstyle{tt}
 \ifpdf
   \usepackage[pdftex,pdfborder=0,breaklinks,baseurl=http://,pdfpagemode=None,pdfstartview=XYZ,pdfstartpage=1]{hyperref}
@@ -49,10 +48,10 @@
   \usepackage[dvips]{hyperref}
 \fi
 
+\renewcommand{\familydefault}{\sfdefault}% for use with a résumé using sans serif fonts;
+%\renewcommand{\familydefault}{\rmdefault}% for use with a résumé using sans serif fonts;
 
 \begin{document}
-\sffamily   % for use with a résumé using sans serif fonts;
-%\rmfamily  % for use with a résumé using serif fonts;
 \hfill%
 \begin{minipage}[t]{.6\textwidth}
 \raggedleft%

moderncv.cls

@@ -93,15 +93,11 @@
 % if... then... else... constructs
 \RequirePackage{ifthen}
 
-% pdf vs ps compilation
+% latex vs pdflatex vs xelatex vs luatex compilation
-\RequirePackage{ifpdf}
+%\RequirePackage{ifpdf}
 
 % color
-\ifpdf
+\RequirePackage{color}
-  \RequirePackage[pdftex]{color}
-\else
-  \RequirePackage[dvips]{color}
-\fi
 
 % font encoding
 \RequirePackage[T1]{fontenc}
@@ -110,9 +106,8 @@
 %\usepackage{type1cm}
 
 % latin modern fonts
-%\AtEndOfClass{%
 \if@lmodern
-    \RequirePackage{lmodern}\fi%}
+  \RequirePackage{lmodern}\fi
 
 % symbols like \Telefon, \Mobilefone, \Letter and \Email
 \RequirePackage{marvosym}
@@ -134,12 +129,8 @@
 % load hyperred at the end of the preamble to pass options required by loaded packages (like CJK)
 \AtEndPreamble{
   \@ifpackageloaded{CJK}
-    {
+    {\RequirePackage[CJKbookmarks]{hyperref}}
-      \ifpdf\RequirePackage[pdftex,CJKbookmarks]{hyperref}
+    {\RequirePackage[pdftex]{hyperref}}
-      \else\RequirePackage[dvips,CJKbookmarks]{hyperref}\fi}
-    {
-      \ifpdf\RequirePackage[pdftex]{hyperref}
-      \else\RequirePackage[dvips]{hyperref}\fi}
   \AtBeginDocument{
     \hypersetup{
       breaklinks,
@@ -159,10 +150,7 @@
 }
 
 % graphics
-\ifpdf
+\RequirePackage{graphicx}
-  \RequirePackage[pdftex]{graphicx}
-\else
-  \RequirePackage[dvips]{graphicx}\fi
 
 % headers and footers
 \RequirePackage{fancyhdr}