use "crt_series.get_branch()" instead of "git.get_head_file()"

More cleanup to the branch command.

Signed-off-by: Chuck Lever <cel@netapp.com>

Better error message when renaming patches to same name

Generate a more precise error message when trying to rename a patch to the
same name.  This is what we get currently:

  [cel@dexter main]$ stg rename nfs_direct_wait nfs_direct_wait
  Renaming patch "nfs_direct_wait" -> "nfs_direct_wait"...stg rename: Patch
  "nfs_direct_wait" already exists

  [cel@dexter main]$

Signed-off-by: Chuck Lever <cel@netapp.com>

Fix the --name option for the pick command

If a patch name from a different branch is given, the --name option is
ignored.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Optimise pick --fold to use git-apply first

If "git-diff-tree | git-apply" fails, the command falls back to a
three-way merge.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Different fixes for the branch command

Mainly checks for local changes were added and some typos.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Generalize branch renaming

The Series object already has the path names of all the pieces we need to
rename, so let's reuse that information.  Extract the rename branch logic
from the "stg branch" command and put it in stack.py.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add a stack method for determining whether a branch is initialised

Let's be consistent about how we decide whether a branch is ready for use
by stgit.

Signed-off-by: Chuck Lever <cel@netapp.com>

Replace a variable that uses the same name as a built-in

Maintainability: "str" hides the name of a Python built-in function
(found by pychecker).

I've also replaced the use of the variable "string" throughout, as it
hides the name of the Python 'string' module.

Signed-off-by: Chuck Lever <cel@netapp.com>

Fix up help string for "stg clone"

The help string for "stg clone" is the same as the one for "stg pull".

Signed-off-by: Chuck Lever <cel@netapp.com>

"stg export" error message could be more specific

Like similar previous patches, make "stg export" distinguish between an
unapplied patch and a non-existent one when reporting a failure.

Signed-off-by: Chuck Lever <cel@netapp.com>

remove extra refresh_index() call in apply_patches()

apply_patches() calles git.merge() and git.switch(), both of which already
invoke refresh_index().

Signed-off-by: Chuck Lever <cel@netapp.com>

Add a space after ',' when building the e-mail recipients headers

This patch also untabifies the mail.py file.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Optimise the export command

This patch replaces the call to git_id with patch.get_(top|bottom).

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add a patches command

This command shows the patches that modify a given file.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix the 'files' initialisation in git.checkout()

The 'files' variable should have been initialised to an empty list, not
None. Thanks to Kai Ruemmler for spotting it.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Using a mutable default function argument is bad

Python function arguments can have default values.  These values are
objects that are created when the "def" statement is run, not when the
function is called.  These objects are thus persistent across calls to
that function.  If a mutable object, such as a list, is used as a
default argument, that object will retain its value across function
calls.  This is potentially unwanted behavior if the function is called
multiple times.

Fix up function argument defaults in stgit/git.py, for safety.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add an option to delete a patch from a certain branch

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Actually, simply adding the option is enough...
Restrict print_crt_patch() to the case when we didn't specify --branch (as it's
useless otherwise).

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Capture the KeyboardInterrupt exception

This is to avoid a stack back-trace when a command is interrupted by the
user.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Notify when a cherry-picked commit was modified by 'pick'

The logic is similar to the 'push' command one (i.e. it is reported as
modified if a three-way merge is needed).

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Notify when a patch was modified during a push operation

The logic of this is that when a "git-diff | git-apply" succeeds, the patch
is reported as unmodified. If the the above operation fails and the command
falls back to a three-way merge, the patch will be reported as modified.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Use __run instead of os.system() in git.merge()

This is just a little faster, avoiding an unneeded invocation of the shell.

Signed-off-by: Chuck Lever <cel@netapp.com>

Prevent stderr from appearing on stdout when running commands

The original purpose was to eliminate "fatal: Needed single revision" when
a ref didn't exist, but this seems like an overall good thing to do to help
with output cleanliness..

Signed-off-by: Chuck Lever <cel@netapp.com>

Need to do "git-update-index --refresh" in more places

Recent changes to the git porcelain-ish shell scripts added
"git-update-index --refresh" in several places.  This harmlessly allows
a dirty tree during certain operations that use git-read-tree.

Signed-off-by: Chuck Lever <cel@netapp.com>

Print the git version when running the "stg version" command

"git --version" was added only recently.

Signed-off-by: Chuck Lever <cel@netapp.com>

"stg mail" doesn't distinguish between unapplied and non-existent patches

Same fix as the recent fixes to "pop" and "push".

Signed-off-by: Chuck Lever <cel@netapp.com>

Make listing branches more efficient

Listing branches appears to "init" the series twice for each branch in
the list.  Similarly for deleting a branch.

Signed-off-by: Chuck Lever <cel@netapp.com>

Make switching to the current branch a no-op

Check if "stg branch name" is switching to the branch we are already in,
and make that case a no-op.

Signed-off-by: Chuck Lever <cel@netapp.com>

Fix exception error message in git.merge()

Due to recent big git script rename.

Signed-off-by: Chuck Lever <cel@netapp.com>

Get rid of an unused variable

"git_opt" is unused in the rm() function.

Signed-off-by: Chuck Lever <cel@netapp.com>

Make sure switch_branch uses the right __head

"switch_branch" needs to use the global variable __head, not a local
version.

Signed-off-by: Chuck Lever <cel@netapp.com>

Remove unused import in stgit/git.py

The "glob" module isn't used in stgit/git.py, so don't import it.

Signed-off-by: Chuck Lever <cel@netapp.com>

Give the global "crt_series" variable a default value

The Python interpreter was searching every scope to find crt_series.  By
providing a default value, this reduces the amount of searching the
interpreter does looking for crt_series.

I've noticed that most commands are faster with this change.

Signed-off-by: Chuck Lever <cel@netapp.com>

Ask vim to avoid backup copies of .stgit.msg

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Do this by adding nobackup to the modeline.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Parse /top.old with id and when requesting diffs

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Comparing a patch after and before a merge is very important, and looking into
.git/patches by hand is not the nicer way to do it.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Allow the editing of the cover message before sending it by e-mail

When sending e-mails, it is easier to automatically invoke the editor on
the cover message than writing it separately. This patch renames the
--first option to --cover and allows a --edit option to edit the final
message before sending it. With --edit only, the tool will use the default
template in .git/covermail.tmpl or
/usr/share/stgit/templates/covermail.tmpl.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix the branch protect/unprotect message

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Flush the HEAD cache in set_head_file()

This function also changes the value read from the HEAD link but it didn't
flush the internal cached value.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix a bug in "stg branch --rename"

We were using the wrong pathname when setting HEAD.

Signed-off-by: Chuck Lever <cel@netapp.com>

[PATCH] Fix a couple of bugs in "stg branch --create"

Use "switch()" instead of "git.switch()".  Also, __set_head is a little
more complicated now, so a __clear_head_cache() is needed before we call it.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add support for branch description files

These are single line files that are displayed by "stg branch --list".

Signed-off-by: Chuck Lever <cel@netapp.com>

Allow user to protect some branches against "stg pull"

Sometimes we want to develop against a branch that doesn't evolve (eg.
2.6.12).  To prevent an accidental "stg pull", provide two new options to
stg: "--protect" and "--unprotect".  This also prevents deleting any
patches in the series.

Signed-off-by: Chuck Lever <cel@netapp.com>

Rudimentary support for multiple development branches

 stg branch

   show the name of the current branch.

 stg branch <branch-name>

   switch to the named branch.  after the switch is complete,
   all currently applied patches in the target branch are
   applied.

 stg branch --create <branch-name> [ <commit-sha1> ]

   create a new branch.  if <commit-sha1> is not specified then
   the branch is created starting at the commit pointed to by
   the current HEAD.  otherwise, the branch is created starting
   at the given commit.

   when the branch creation completes, the new branch becomes
   the current branch.

 stg branch --delete [ <branch-name> ]

   delete a branch and all patches it contains.  if no name is
   specified, delete the current branch and switch to the
   "master" branch.  otherwise, delete the named branch.

 stg branch --list

   list all the development branches in the current repository
   along with their descriptions and status.

 stg branch --rename <from-branch-name> <to-branch-name>

   rename a branch, and switch to the new name.

 stg branch -h

   print a synopsis of the "stg branch" command.

Signed-off-by: Chuck Lever <cel@netapp.com>

Allow print_crt_patch to work on other branches

print_crt_patch() current prints the current patch in the current branch.
Add a function argument to allow it to print the current patch in other
branches.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add the ability to rename a git branch

To keep StGit-specific functionality separate from git functionality,
create a git.rename_branch function to do appropriate sanity checking and
rename branch heads.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add the ability to delete a branch to git.py

"git branch -D" checks first to see whether all the commits in the doomed
branch are already in the branch pointed to by HEAD.  I don't think we need
that level of checking here.  We just delete "refs/heads/branchname."

Signed-off-by: Chuck Lever <cel@netapp.com>

Add git method for switching to a different branch

Signed-off-by: Chuck Lever <cel@netapp.com>

Add git method for creating a fresh branch

Basically this is "git checkout -b".

Signed-off-by: Chuck Lever <cel@netapp.com>

Create a new utility function in git.py for setting the HEAD link

Add a new method in git.py for switching the HEAD link to point to a
different ref.

Signed-off-by: Chuck Lever <cel@netapp.com>

Add a stack method to delete a patch series

In preparation to introducing the ability to delete a branch, add a stack
method that will iterate over a patch series and delete each patch.

Signed-off-by: Chuck Lever <cel@netapp.com>

Allow the short version of --base (i.e. -b) in import

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Use git_id in import and fold instead of git.rev_parse

git_id() is more flexible since it knows about StGIT patches.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix the caching of the HEAD value

'git pull' also updates the HEAD but StGIT did not flush the cached value
causing problems with pulling.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add a --reset option to resolved

With this option one can ask for a conflicted file to be reset to one of
the local, remote or older states.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Allow git.apply_patch() to take a base revision

This option is useful for import --base and fold commands. Both these
commands were modified to take advantage of it. The fold commands also
supports a --base option, similar to the import one.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add a --fold option to pick

This option causes the given commit object to be merged into the current
patch instead of creating a new one.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

TODO updated with the need for a 'prune' command

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

'status --reset' should remove the conflicts as well

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add the --reset option to status

This option resets all the changes in the current tree (relative to HEAD).

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add an uninstallable script for profiling

The script name is stg-prof and can be used instead stg for printing
profiling information.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Make __commits global in git.py

Some weird errors with Python scoping for the __head global variable
required the use of global. Add it for this variable as well.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Cache the HEAD value

There are many calls to git.get_head(). This value should be cached to
speed things up.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Use "git-rev-parse" to get the HEAD commit

Traditionally, HEAD was just a link to a refs file.  Recently, however,
support for HEADs of the form "ref: " was added to git.  In addition,
it's safer to verify the HEAD commit id before using it.

Change git.py to use "git-rev-parse" to derive the HEAD commit instead of
reading the HEAD link directly.  If there are any problems with the HEAD
commit id, git.get_head() now raises an exception.

And, use "git-update-ref" to update the HEAD for similar reasons.

Signed-off-by: Chuck Lever <cel@netapp.com>

Use a more clear message when pop or push can't apply a patch

Minor nit: the error message when "stg pop" or "stg push" can't apply a
patch is vague.  Add a little extra logic to print a more precise error
message in these cases.

Signed-off-by: Chuck Lever <cel@netapp.com>

Escape the unusual characters in the name <email> strings

Otherwise there are problems when passing the environment to git-commit.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Optimise 'push' to use git-apply instead of git-read-tree

With this patch, 'push' will use 'git-diff-tree | git-apply' first. If this
operation fails, it will fall back to the three-way merge with
git-read-tree.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Release 0.7

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Update the TODO file

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add a commit command to merge the current patches into base

By default, the base is only advanced by 'stg pull'. A commit command would
allow the currently applied patches to be merged into the base and removed
from the current series. This commands adds a bit of SCM flavour to StGIT.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Implement a 'pick' command for cherry-picking

The initial cherry-picking implementation in import was quite hard to find
and the import command became pretty hard to use.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Cancel the commit cherry-picking in import

This feature will be added to a separate command, 'pick'.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add support for branch in the patch id

By default, only patch/top and patch/bottom on the current branch are
supported. This patch adds the patch@branch/... support.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add support for git-rev-parse

The git_id() function now calls git.rev_parse() if it cannot get the id.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix earlier "import quilt patches" patch

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

I forgot to update all cases - I updated __parse_mail, not __parse_patch, so
refactor together this duplication and use the fixed version.

Btw, I don't like those regexps - they'd match in the middle of line too. What
about adding ^ to their beginning like for the "^Index: " regexp?

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Minor typos in the tutorial

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add the tutorial back to the repository

This is to avoid problems when the site is down. At the moment, the
tutorial on the wiki was only converted to plain text and added. It might
be out of date for some commands or options.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix importing from stdin

The current stdin patch importing expects two EOFs since the 'for' loop
doesn't start before one EOF is received. As suggested, this patch changes
the 'for' loop with a 'while True' loop.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Cut verbosity of new fast-forward merging

Now "stg push" will report only that is "Fast-forwarding" or "Pushing"
a patch.  The messages about trying to fast-forward and failing are
gone.

Signed-off-by: Chuck Lever <cel@netapp.com>

"stg series" on an un-inited branch gives weird error

If I try an "stg series" command on a repository that hasn't yet been
initialized, I get a non-intuitive error message:

stg series: [Errno 2] No such file or directory:
'/home/cel/src/git/main/.git/patches/master/applied'

This patch replaces the message with a reasonable explanation.

Signed-off-by: Chuck Lever <cel@netapp.com>

Adjust stgit for post 0.99.7 renames.

Not to be used with version of git 0.99.6 and earlier.
Using the new names with current "master" branch is OK.
The old names will be removed in 0.99.8.

Fix series mismerge

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Mismerge with my addition of -e. Sorry for not pulling that time.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Fix syntax error in stg status

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Uff, it's becoming annoying - why things break so often? However, more to the
point: turn a string into a list to allow appending it to another list.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Fix the version_str assignment in mail.py

The variable is used before it is initialised.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Better mail handling in stgit

Fix some issues with the mail function in StGIT:

 * Allow multiple To/Cc/Bcc command line options.

 * Better parsing of mail addresses.

Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>

Specify a different base for the imported patch

This option will import a patch on a different base and perform a three-way
merge with the current HEAD.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Check whether the patch name is applied or unapplied

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Detect description in quilt patches

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

The current logic imports the whole quilt patch as description - the changes
themselves are correctly applied, luckily, but the description must be fixed up
by hand.
So, detect "Index: " lines as patch start. I've heard rumors that also
Subversion generates this format, so we become compatible with it, too.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Fix "stg mail" address parsing for hyphen

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Tried sending an email cc'ing LKML - watched my postfix queue - and saw
kernel@vger.kernel.org as delivery address! What had happened? StGIT didn't like
"linux-kernel" ! Fix the regexp.

I just added an hyphen to both sections (yes, there are plenty of domain names
including hyphens, and I tested the problem there too). Don't know if other
chars are missing.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Allow mailing a given patch list

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

I tried to mail a cherry-picked subset of my series and that didn't work - using
stg export and quilt was even worse, so I had to code something.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Fix "stg clean" when stack is empty

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

It crashes on accessing 1st element of an empty array.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Implement fast-forward when only tree (but not

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

When the "bottom" commit and the HEAD don't match, but they refer to the
same tree (for instance after a refresh where only the description
changed), we can still fast-forward, by keeping the same top tree and
calling git-commit-tree (which only requires the tree object) with the new
parent. I've altered git.commit to allow this.

Btw, I've also avoided the use of .commitmsg and switched to piping the
description.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Avoid allowing --undo if it doesn't change anything

From: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Avoid top/bottom backup files identical to current ones. Simply remove
them.

Also, change restore_old_boundaries() to handle gracefully the new
situation rather than print an exception.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Make "stg files" output match "quilt files" one

I'm used to doing vi $(quilt files), which is impossible with stgit.
Add an option (-b/--base) to request the normal behaviour, but make it
non-default as the current output is useful.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Fix "mail" command when description contains "From"

For kernel patches, the "From" line from the email is often preserved in
the patch itself, and the one from the email is sometimes lost, so I add an
explicit one. And mail barfes on this.
Fix it up.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

stg diff / files: don't update directory cache

Do git-update-cache only when diffing with the working tree, not otherwise.
Spending something like 1min for a stg files is bad - yes, my laptop
was really busy and the Linux tree was probably cache-cold, but that's just
not needed.

Also, in diffstat we currently do it both by hand and by calling git.diff.
And in files there's no need at all for that - even the comments says that
"files" has only to do with committed changes.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Make "empty patch" checking optional in "series"

That's just too slow, and quilt doesn't do it, so user will live with it.
Don't know if there's a reason to allow dropping the fanciness here
entirely, but I think there's no user doing one-liner scripts with quilt
series. And anyway, for that you can do "stg applied; stg unapplied".

Actually, with this patch you must ask explicitly the checking.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Ignore the case on the PATCH name when importing

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Fix typo: comitter->committer

Found casually in commit.__init__, while profiling stg series.
And yes, I went checking that git has no typo.

Signed-off-by: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>

Allow 'import' to cherry-pick a commit object

A commit object in the tree can be cherry-picked and imported into a
separate StGIT patch. It also allow reverse-importing, i.e. cancelling an
existing commit.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Pass -q to git-merge-cache

The error reporting is handled by gitmergeonefile.py

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

'export' will use patches-<branch> by default

Previously, it was exporting to the patches directory by default but this
may cause problems if multiple branches are used.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>