forked from qt-creator/qt-creator
VcsBase: Proliferate FilePath use a tiny bit
Change-Id: I78929f6036702fe0b5384f4324f6283c2129d91b Reviewed-by: Orgad Shaneh <orgads@gmail.com>
This commit is contained in:
@@ -521,7 +521,7 @@ void VcsBaseSubmitEditor::slotCheckSubmitMessage()
|
||||
|
||||
bool VcsBaseSubmitEditor::checkSubmitMessage(QString *errorMessage) const
|
||||
{
|
||||
const QString checkScript = commonSettings().submitMessageCheckScript.value();
|
||||
const FilePath checkScript = commonSettings().submitMessageCheckScript();
|
||||
if (checkScript.isEmpty())
|
||||
return true;
|
||||
QApplication::setOverrideCursor(Qt::WaitCursor);
|
||||
@@ -530,17 +530,17 @@ bool VcsBaseSubmitEditor::checkSubmitMessage(QString *errorMessage) const
|
||||
return rc;
|
||||
}
|
||||
|
||||
static QString msgCheckScript(const FilePath &workingDir, const QString &cmd)
|
||||
static QString msgCheckScript(const FilePath &workingDir, const FilePath &cmd)
|
||||
{
|
||||
const QString nativeCmd = QDir::toNativeSeparators(cmd);
|
||||
const QString nativeCmd = cmd.toUserOutput();
|
||||
return workingDir.isEmpty() ?
|
||||
Tr::tr("Executing %1").arg(nativeCmd) :
|
||||
Tr::tr("Executing [%1] %2").
|
||||
arg(workingDir.toUserOutput(), nativeCmd);
|
||||
Tr::tr("Executing [%1] %2").arg(workingDir.toUserOutput(), nativeCmd);
|
||||
}
|
||||
|
||||
bool VcsBaseSubmitEditor::runSubmitMessageCheckScript(const QString &checkScript, QString *errorMessage) const
|
||||
bool VcsBaseSubmitEditor::runSubmitMessageCheckScript(const FilePath &checkScript, QString *errorMessage) const
|
||||
{
|
||||
QTC_ASSERT(!checkScript.needsDevice(), return false); // Not supported below.
|
||||
// Write out message
|
||||
TempFileSaver saver(TemporaryDirectory::masterDirectoryPath() + "/msgXXXXXX.txt");
|
||||
saver.write(fileContents());
|
||||
@@ -552,7 +552,7 @@ bool VcsBaseSubmitEditor::runSubmitMessageCheckScript(const QString &checkScript
|
||||
Process checkProcess;
|
||||
if (!d->m_checkScriptWorkingDirectory.isEmpty())
|
||||
checkProcess.setWorkingDirectory(d->m_checkScriptWorkingDirectory);
|
||||
checkProcess.setCommand({FilePath::fromString(checkScript), {saver.filePath().toString()}});
|
||||
checkProcess.setCommand({checkScript, {saver.filePath().path()}});
|
||||
checkProcess.start();
|
||||
const bool succeeded = checkProcess.waitForFinished();
|
||||
|
||||
|
||||
@@ -116,7 +116,7 @@ private:
|
||||
|
||||
void createUserFields(const QString &fieldConfigFile);
|
||||
bool checkSubmitMessage(QString *errorMessage) const;
|
||||
bool runSubmitMessageCheckScript(const QString &script, QString *errorMessage) const;
|
||||
bool runSubmitMessageCheckScript(const Utils::FilePath &script, QString *errorMessage) const;
|
||||
QString promptForNickName();
|
||||
void close();
|
||||
|
||||
|
||||
Reference in New Issue
Block a user