summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOrgad Shaneh <orgads@gmail.com>2012-03-13 22:34:54 +0200
committerTobias Hunger <tobias.hunger@nokia.com>2012-03-14 11:18:41 +0100
commitd207165f6aa89ee95bd41c41a49d68cfa0b46444 (patch)
tree85435bb764a9d66d20c7e705304ceafe0389fdcb
parent6a3f9460d71376236c5d00dfce0d568ad92b83c7 (diff)
downloadqt-creator-d207165f6aa89ee95bd41c41a49d68cfa0b46444.tar.gz
Git: Allow commit (or amend) when not on a branch
Git allows that so why can't we? Useful when in the middle of merge/rebase. Change-Id: Id09da3417904ff29e6dd1b4ef383157e9c4920d3 Reviewed-by: Tobias Hunger <tobias.hunger@nokia.com>
-rw-r--r--src/plugins/git/gitclient.cpp11
-rw-r--r--src/plugins/git/gitclient.h2
2 files changed, 3 insertions, 10 deletions
diff --git a/src/plugins/git/gitclient.cpp b/src/plugins/git/gitclient.cpp
index e7804f3bdf..dab54a13e6 100644
--- a/src/plugins/git/gitclient.cpp
+++ b/src/plugins/git/gitclient.cpp
@@ -1519,7 +1519,7 @@ static inline QString trimFileSpecification(QString fileSpec)
}
GitClient::StatusResult GitClient::gitStatus(const QString &workingDirectory, bool untracked,
- QString *output, QString *errorMessage, bool *onBranch)
+ QString *output, QString *errorMessage)
{
// Run 'status'. Note that git returns exitcode 1 if there are no added files.
QByteArray outputText;
@@ -1537,8 +1537,6 @@ GitClient::StatusResult GitClient::gitStatus(const QString &workingDirectory, bo
static const char * NO_BRANCH = "## HEAD (no branch)\n";
const bool branchKnown = !outputText.startsWith(NO_BRANCH);
- if (onBranch)
- *onBranch = branchKnown;
// Is it something really fatal?
if (!statusRc && !branchKnown) {
if (errorMessage) {
@@ -1695,15 +1693,10 @@ bool GitClient::getCommitData(const QString &workingDirectory,
}
// Run status. Note that it has exitcode 1 if there are no added files.
- bool onBranch;
QString output;
- const StatusResult status = gitStatus(repoDirectory, true, &output, errorMessage, &onBranch);
+ const StatusResult status = gitStatus(repoDirectory, true, &output, errorMessage);
switch (status) {
case StatusChanged:
- if (!onBranch) {
- *errorMessage = tr("You did not checkout a branch.");
- return false;
- }
break;
case StatusUnchanged:
if (amend)
diff --git a/src/plugins/git/gitclient.h b/src/plugins/git/gitclient.h
index 434a860e09..0110852238 100644
--- a/src/plugins/git/gitclient.h
+++ b/src/plugins/git/gitclient.h
@@ -212,7 +212,7 @@ public:
StatusResult gitStatus(const QString &workingDirectory,
bool untracked = false,
QString *output = 0,
- QString *errorMessage = 0, bool *onBranch = 0);
+ QString *errorMessage = 0);
void launchGitK(const QString &workingDirectory);
void launchRepositoryBrowser(const QString &workingDirectory);