rgheck <[EMAIL PROTECTED]> writes:

> Using current svn:
> #mkdir /tmp/lyx/
> #cd /tmp/lyx
> #python /cvs/lyxsvn/lib/configure.py
> Result:
> ...
> +checking list of textclasses...
> Layout file /cvs/lyxsvn/lib/layouts/manpage.layout has no
> \DeclareXXClass line.
>
> Is this just a typo in that layout?

No, I think this layout should be removed. See my message to Jose in the
linuxdoc thread.

> If I now run configure.py again, it works. That's because we have the
> default textclass.lst from the failed first run, so it skips the part
> that does that test. So that leads to the question: Isn't it a bit
> much to sys.exit(2) if we don't find a \DeclareXXClass line? Seems to
> me it'd be enough to print the error message and ignore that layout
> file.

Definitely. Something like this? My python is so rusty (if I may say)
that I daren't commit anything.

JMarc

svndiff lib/

Index: lib/configure.py
===================================================================
--- lib/configure.py	(revision 27051)
+++ lib/configure.py	(working copy)
@@ -628,7 +628,7 @@ def processLayoutFile(file, bool_docbook
                 opt = classname
             return '"%s" "%s" "%s" "%s"\n' % (classname, opt, desc, avai)
     print "Layout file " + file + " has no \DeclareXXClass line. "
-    sys.exit(2)
+    return ""
 
     
 def checkLatexConfig(check_config, bool_docbook):

Reply via email to