X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FVCBackend.cpp;h=d05e4428ac2e35b244078e0455f6548b391b6d9f;hb=cca78e3c8ae27431323746abd64f9d7db017099d;hp=8dcef4332f7d822772d4c3dd5dcd86dd371e4023;hpb=e46613d70bc14d5a4e5833782f912b64e451c25d;p=lyx.git

diff --git a/src/VCBackend.cpp b/src/VCBackend.cpp
index 8dcef4332f..d05e4428ac 100644
--- a/src/VCBackend.cpp
+++ b/src/VCBackend.cpp
@@ -170,11 +170,12 @@ void RCS::registrer(string const & msg)
 }
 
 
-void RCS::checkIn(string const & msg)
+string RCS::checkIn(string const & msg)
 {
-	doVCCommand("ci -q -u -m\"" + msg + "\" "
+	int ret = doVCCommand("ci -q -u -m\"" + msg + "\" "
 		    + quoteName(onlyFilename(owner_->absFileName())),
 		    FileName(owner_->filePath()));
+	return ret ? string() : "RCS: Proceeded";
 }
 
 bool RCS::checkInEnabled()
@@ -182,11 +183,12 @@ bool RCS::checkInEnabled()
 	return owner_ && !owner_->isReadonly();
 }
 
-void RCS::checkOut()
+string RCS::checkOut()
 {
 	owner_->markClean();
-	doVCCommand("co -q -l " + quoteName(onlyFilename(owner_->absFileName())),
+	int ret = doVCCommand("co -q -l " + quoteName(onlyFilename(owner_->absFileName())),
 		    FileName(owner_->filePath()));
+	return ret ? string() : "RCS: Proceeded";
 }
 
 
@@ -327,11 +329,12 @@ void CVS::registrer(string const & msg)
 }
 
 
-void CVS::checkIn(string const & msg)
+string CVS::checkIn(string const & msg)
 {
-	doVCCommand("cvs -q commit -m \"" + msg + "\" "
+	int ret = doVCCommand("cvs -q commit -m \"" + msg + "\" "
 		    + quoteName(onlyFilename(owner_->absFileName())),
 		    FileName(owner_->filePath()));
+	return ret ? string() : "CVS: Proceeded";
 }
 
 
@@ -341,12 +344,13 @@ bool CVS::checkInEnabled()
 }
 
 
-void CVS::checkOut()
+string CVS::checkOut()
 {
 	// cvs update or perhaps for cvs this should be a noop
 	// we need to detect conflict (eg "C" in output)
 	// before we can do this.
 	lyxerr << "Sorry not implemented." << endl;
+	return string();
 }
 
 
@@ -456,26 +460,28 @@ void SVN::registrer(string const & msg)
 }
 
 
-void SVN::checkIn(string const & msg)
+string SVN::checkIn(string const & msg)
 {
 	FileName tmpf = FileName::tempName("lyxvcout");
 	if (tmpf.empty()){
 		LYXERR(Debug::LYXVC, "Could not generate logfile " << tmpf);
-		return;
+		return N_("Error: Could not generate logfile.");
 	}
 
 	doVCCommand("svn commit -m \"" + msg + "\" "
 		    + quoteName(onlyFilename(owner_->absFileName()))
-		    + " 2> " + tmpf.toFilesystemEncoding(),
+		    + " > " + tmpf.toFilesystemEncoding(),
 		    FileName(owner_->filePath()));
 
-	string res = scanLogFile(tmpf);
+	string log;
+	string res = scanLogFile(tmpf, log);
 	if (!res.empty())
 		frontend::Alert::error(_("Revision control error."),
 				_("Error when commiting to repository.\n"
 				"You have to manually resolve the problem.\n"
 				"After pressing OK, LyX will reopen the document."));
 	tmpf.erase();
+	return "SVN: " + log;
 }
 
 
@@ -487,7 +493,7 @@ bool SVN::checkInEnabled()
 // FIXME Correctly return code should be checked instead of this.
 // This would need another solution than just plain startscript.
 // Hint from Andre': QProcess::readAllStandardError()...
-string SVN::scanLogFile(FileName const & f)
+string SVN::scanLogFile(FileName const & f, string & status)
 {
 	ifstream ifs(f.toFilesystemEncoding().c_str());
 	string line;
@@ -495,28 +501,31 @@ string SVN::scanLogFile(FileName const & f)
 	while (ifs) {
 		getline(ifs, line);
 		lyxerr << line << "\n";
-		if (prefixIs(line, "C "))
-			return line;
-		if (contains(line, "Commit failed"))
+		if (!line.empty()) status += line + "; ";
+		if (prefixIs(line, "C ") || contains(line, "Commit failed")) {
+			ifs.close();
 			return line;
+		}
 	}
+	ifs.close();
 	return string();
 }
 
 
-void SVN::checkOut()
+string SVN::checkOut()
 {
 	FileName tmpf = FileName::tempName("lyxvcout");
 	if (tmpf.empty()) {
 		LYXERR(Debug::LYXVC, "Could not generate logfile " << tmpf);
-		return;
+		return N_("Error: Could not generate logfile.");
 	}
 
 	doVCCommand("svn update " + quoteName(onlyFilename(owner_->absFileName()))
 		    + " > " + tmpf.toFilesystemEncoding(),
 		    FileName(owner_->filePath()));
 
-	string res = scanLogFile(tmpf);
+	string log;
+	string res = scanLogFile(tmpf, log);
 	if (!res.empty())
 		frontend::Alert::error(_("Revision control error."),
 			bformat(_("Error when updating from repository.\n"
@@ -524,6 +533,7 @@ void SVN::checkOut()
 				"After pressing OK, LyX will try to reopen resolved document."),
 			from_ascii(res)));
 	tmpf.erase();
+	return "SVN: " + log;
 }