#include "GraphicsConverter.h"
+#include "Buffer.h"
#include "Converter.h"
#include "Format.h"
+#include "LyXRC.h"
+#include "frontends/alert.h"
#include "support/lassert.h"
#include "support/convert.h"
#include "support/debug.h"
#include "support/lstrings.h"
#include "support/os.h"
-#include <boost/bind.hpp>
+#include "support/TempFile.h"
#include <sstream>
#include <fstream>
namespace graphics {
-class Converter::Impl : public boost::signals::trackable {
+class Converter::Impl {
public:
///
- Impl(FileName const &, string const &, string const &, string const &);
+ Impl(FileName const & doc_fname,
+ FileName const & from_file, string const & to_file_base,
+ string const & from_format, string const & to_format);
///
void startConversion();
/** At the end of the conversion process inform the outside world
* by emitting a signal.
*/
- typedef boost::signal<void(bool)> SignalType;
+ typedef signals2::signal<void(bool)> sig;
///
- SignalType finishedConversion;
+ sig finishedConversion;
+ ///
+ FileName const doc_fname_;
///
string script_command_;
///
bool valid_process_;
///
bool finished_;
+ ///
+ Trackable tracker_;
};
}
-Converter::Converter(FileName const & from_file, string const & to_file_base,
- string const & from_format, string const & to_format)
- : pimpl_(new Impl(from_file, to_file_base, from_format, to_format))
+Converter::Converter(FileName const & doc_fname,
+ FileName const & from_file, string const & to_file_base,
+ string const & from_format, string const & to_format)
+ : pimpl_(new Impl(doc_fname, from_file, to_file_base, from_format, to_format))
{}
}
-boost::signals::connection Converter::connect(slot_type const & slot) const
+signals2::connection Converter::connect(slot_type const & slot) const
{
return pimpl_->finishedConversion.connect(slot);
}
/** Build the conversion script.
* The script is output to the stream \p script.
*/
-static void build_script(FileName const & from_file, string const & to_file_base,
+static void build_script(string const & doc_fname,
+ string const & from_file, string const & to_file_base,
string const & from_format, string const & to_format,
ostream & script);
-Converter::Impl::Impl(FileName const & from_file, string const & to_file_base,
+Converter::Impl::Impl(FileName const & doc_fname,
+ FileName const & from_file, string const & to_file_base,
string const & from_format, string const & to_format)
- : valid_process_(false), finished_(false)
+ : doc_fname_(doc_fname), valid_process_(false), finished_(false)
{
LYXERR(Debug::GRAPHICS, "Converter c-tor:\n"
- << "\tfrom_file: " << from_file
+ << "doc_fname: " << doc_fname
+ << "\n\tfrom_file: " << from_file
<< "\n\tto_file_base: " << to_file_base
<< "\n\tfrom_format: " << from_format
<< "\n\tto_format: " << to_format);
// The converted image is to be stored in this file (we do not
// use ChangeExtension because this is a basename which may
// nevertheless contain a '.')
- to_file_ = FileName(to_file_base + '.' + formats.extension(to_format));
+ to_file_ = FileName(to_file_base + '.' + theFormats().extension(to_format));
// The conversion commands are stored in a stringstream
ostringstream script;
- build_script(from_file, to_file_base, from_format, to_format, script);
+ build_script(doc_fname_.absFileName(), from_file.toFilesystemEncoding(),
+ to_file_.toFilesystemEncoding(),
+ from_format, to_format, script);
LYXERR(Debug::GRAPHICS, "\tConversion script:"
"\n--------------------------------------\n"
<< script.str()
<< "\n--------------------------------------\n");
// Output the script to file.
- static int counter = 0;
- script_file_ = FileName(onlyPath(to_file_base) + "lyxconvert" +
- convert<string>(counter++) + ".py");
+ TempFile tempfile(to_file_.onlyPath(), "lyxconvertXXXXXX.py");
+ tempfile.setAutoRemove(false);
+ script_file_ = tempfile.name();
ofstream fs(script_file_.toFilesystemEncoding().c_str());
if (!fs.good()) {
// Note: 'python ' is absolutely essential, or execvp will fail.
script_command_ = os::python() + ' ' +
quoteName(script_file_.toFilesystemEncoding()) + ' ' +
- quoteName(onlyFilename(from_file.toFilesystemEncoding())) + ' ' +
+ quoteName(onlyFileName(from_file.toFilesystemEncoding())) + ' ' +
quoteName(to_format);
// All is ready to go
valid_process_ = true;
return;
}
- ForkedCall::SignalTypePtr ptr =
- ForkedCallQueue::add(script_command_);
- ptr->connect(boost::bind(&Impl::converted, this, _1, _2));
+ ForkedCall::sigPtr ptr = ForkedCallQueue::add(script_command_);
+ ptr->connect(ForkedCall::slot([this](pid_t pid, int retval){
+ converted(pid, retval);
+ }).track_foreign(tracker_.p()));
}
return string();
ostringstream command;
- command << "fromfile = toUnicode(" << from_file << ")\n"
- << "tofile = toUnicode(" << to_file << ")\n\n"
+ command << "fromfile = " << from_file << "\n"
+ << "tofile = " << to_file << "\n\n"
<< "try:\n"
<< " os.rename(fromfile, tofile)\n"
<< "except:\n"
static void build_conversion_command(string const & command, ostream & script)
{
// Store in the python script
- script << "\nif os.system(r'" << command << "') != 0:\n";
+ script << "\nif os.system(r'" << commandPrep(command) << "') != 0:\n";
// Test that this was successful. If not, remove
// ${outfile} and exit the python script
" sys.exit(1)\n\n";
// Delete the infile
- script << "unlinkNoThrow(infile)\n\n";
+ script << "if infile != outfile:\n"
+ " unlinkNoThrow(infile)\n\n";
}
-static void build_script(FileName const & from_file,
- string const & to_file_base,
+static string const strip_digit(string const & format)
+{
+ // Strip trailing digits from format names e.g. "pdf6" -> "pdf"
+ return format.substr(0, format.find_last_not_of("0123456789") + 1);
+}
+
+
+static void build_script(string const & doc_fname,
+ string const & from_file,
+ string const & to_file,
string const & from_format,
string const & to_format,
ostream & script)
{
- LASSERT(from_format != to_format, /**/);
+ LASSERT(from_format != to_format, return);
LYXERR(Debug::GRAPHICS, "build_script ... ");
typedef Graph::EdgePath EdgePath;
- script << "#!/usr/bin/env python\n"
- "# -*- coding: utf-8 -*-\n"
+ script << "# -*- coding: utf-8 -*-\n"
"import os, shutil, sys\n\n"
"def unlinkNoThrow(file):\n"
" ''' remove a file, do not throw if an error occurs '''\n"
" try:\n"
" os.unlink(file)\n"
" except:\n"
- " pass\n\n"
- "def toUnicode(file):\n"
- " ''' if possible, convert to python unicode format '''\n"
- " try:\n"
- " return unicode(file, 'utf8')\n"
- " except:\n"
- " return file\n\n";
-
- // we do not use ChangeExtension because this is a basename
- // which may nevertheless contain a '.'
- string const to_file = to_file_base + '.'
- + formats.extension(to_format);
+ " pass\n\n";
EdgePath const edgepath = from_format.empty() ?
EdgePath() :
theConverters().getPath(from_format, to_format);
// Create a temporary base file-name for all intermediate steps.
- // Remember to remove the temp file because we only want the name...
- static int counter = 0;
- string const tmp = "gconvert" + convert<string>(counter++);
- FileName const to_base = FileName::tempName(tmp);
+ string const from_ext = getExtension(from_file);
+ TempFile tempfile(addExtension("gconvertXXXXXX", from_ext));
+ tempfile.setAutoRemove(false);
+ string outfile = tempfile.name().toFilesystemEncoding();
+ string const to_base = from_ext.empty() ? outfile : removeExtension(outfile);
// Create a copy of the file in case the original name contains
// problematic characters like ' or ". We can work around that problem
// in python, but the converters might be shell scripts and have more
// troubles with it.
- string outfile = addExtension(to_base.absFilename(), getExtension(from_file.absFilename()));
- script << "infile = toUnicode("
- << quoteName(from_file.absFilename(), quote_python)
- << ")\n"
- "outfile = toUnicode("
- << quoteName(outfile, quote_python) << ")\n"
+ script << "infile = "
+ << quoteName(from_file, quote_python)
+ << "\n"
+ "outfile = "
+ << quoteName(outfile, quote_python) << "\n"
"shutil.copy(infile, outfile)\n";
// Some converters (e.g. lilypond) can only output files to the
// This has the added benefit that all other files that may be
// generated by the converter are deleted when LyX closes and do not
// clutter the real working directory.
- script << "os.chdir(toUnicode("
- << quoteName(onlyPath(outfile)) << "))\n";
+ script << "os.chdir("
+ << quoteName(onlyPath(outfile)) << ")\n";
if (edgepath.empty()) {
// Either from_format is unknown or we don't have a
// converter path from from_format to to_format, so we use
// the default converter.
script << "infile = outfile\n"
- << "outfile = toUnicode("
- << quoteName(to_file, quote_python) << ")\n";
+ << "outfile = "
+ << quoteName(to_file, quote_python) << "\n";
ostringstream os;
os << os::python() << ' '
- << libScriptSearch("$$s/scripts/convertDefault.py",
- quote_python) << ' ';
- if (!from_format.empty())
- os << from_format << ':';
+ << commandPrep("$$s/scripts/convertDefault.py") << ' ';
+ if (from_format.empty())
+ os << "unknown ";
+ else
+ os << strip_digit(from_format) << ' ';
// The extra " quotes around infile and outfile are needed
// because the filename may contain spaces and it is used
// as argument of os.system().
os << "' + '\"' + infile + '\"' + ' "
- << to_format << ":' + '\"' + outfile + '\"' + '";
+ << strip_digit(to_format) << " ' + '\"' + outfile + '\"' + '";
string const command = os.str();
LYXERR(Debug::GRAPHICS,
// Build the conversion command
string const infile = outfile;
string const infile_base = changeExtension(infile, string());
- outfile = addExtension(to_base.absFilename(), conv.To->extension());
+ outfile = conv.result_file().empty()
+ ? addExtension(to_base, conv.To()->extension())
+ : addName(subst(conv.result_dir(),
+ token_base, infile_base),
+ subst(conv.result_file(),
+ token_base, onlyFileName(infile_base)));
+
+ // If two formats share the same extension we may get identical names
+ if (outfile == infile && conv.result_file().empty()) {
+ TempFile tmpfile(addExtension("gconvertXXXXXX", conv.To()->extension()));
+ tmpfile.setAutoRemove(false);
+ outfile = tmpfile.name().toFilesystemEncoding();
+ }
+
+ if (!theConverters().checkAuth(conv, doc_fname))
+ return;
// Store these names in the python script
- script << "infile = toUnicode("
- << quoteName(infile, quote_python) << ")\n"
- "infile_base = toUnicode("
- << quoteName(infile_base, quote_python) << ")\n"
- "outfile = toUnicode("
- << quoteName(outfile, quote_python) << ")\n"
+ script << "infile = "
+ << quoteName(infile, quote_python) << "\n"
+ "infile_base = "
+ << quoteName(infile_base, quote_python) << "\n"
+ "outfile = "
+ << quoteName(outfile, quote_python) << "\n"
"outdir = os.path.dirname(outfile)\n" ;
// See comment about extra " quotes above (although that
// applies only for the first loop run here).
- string command = conv.command;
+ string command = conv.command();
command = subst(command, token_from, "' + '\"' + infile + '\"' + '");
command = subst(command, token_base, "' + '\"' + infile_base + '\"' + '");
command = subst(command, token_to, "' + '\"' + outfile + '\"' + '");
command = subst(command, token_todir, "' + '\"' + outdir + '\"' + '");
- command = libScriptSearch(command, quote_python);
build_conversion_command(command, script);
}