Remove the assert in CommitData.parse() function

When some unknown key/value pairs are found in the commit data, StGit
crashes. It looks like this can actually happen with the "encoding:
latin1" pair.

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

contrib/vim: Add vim syntax highlighting for StGit commit messages

Signed-off-by: Zane Bitter <zane.bitter@alliedtelesis.co.nz>
Tested-by: Chris Packham <judge.packham@gmail.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

StGit: export: fix base commit reporting in series file

This bug was introduced in 3f19450c426970b78b19c522a82df1a962da5761
It should report the commit stg stack is based on, but not the
last patch in stack.

Signed-off-by: Paulius Zaleckas <paulius.zaleckas@gmail.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Add a branch --cleanup option

This has been required for some time. If you need to take a branch out
of StGit control, use this option (keeps the branch but removes the
metadata).

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

Fix stg repair for hidden patches

Signed-off-by: Juergen Wieferink <wieferink@fhi-berlin.mpg.de>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

Do not truncate the last word when creating patch names

The automatically generated patch names are truncated to
stgit.namelength but this was truncating the final words.

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

Allow uncommitting to a common ancestor

With the --to=<commit> option to uncommit, if the given commit is not in
the history just try to find the common ancestor and uncommit to that
point (exclusive range).

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

get_merge_bases() should return a list rather than set

This is for cases where we need to use one of the elements of the list.

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

Read several objects at once with git cat-file --batch

Instead of spawning a separate cat-file process for every blob and
commit we want to read. This speeds things up slightly: about 6-8%
when uncommitting and rebasing 1470 linux-kernel patches (perftest.py
rebase-newrebase-add-file-linux).

Signed-off-by: Karl Wiberg <kha@treskal.com>

tutorial: Very minor typo fixes in the Emacs section

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Add stgit-{git,stg}-program variables

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Rearrange customizations a bit

Move faces to a customization subgroup.
Sort customization entries in a more user-friendly way.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Add "Customize StGit" menu entry

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Improve how "t h" mode finds old commits

Use 'git log -<N>' to list historical commits. This avoids showing
more than stgit-committed-count patches (for merges) and does not
cause an error when running out of history.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Make C-{up,down} move between groups of patches

Also make stgit-{previous,next}-patch behave the same way when moving
past the first/last patch.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Add a few links to some customization variables

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Repair ! for historical commits

Fixes omission in 8f489f41f8ffe6e06254cefc5b352610df06cbeb.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Honor stgit-find-copies-harder in stgit-diff-range

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Honor stgit-find-copies-harder when showing entire patches

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

stgit.el: Fix showing diff in file that has been both copied and modified

Use --diff-filter=C when showing the diff of the copy operation.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>

edit: Allow setting git tree of a patch

Also fix capitalization in edit's short description.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

Repository.rev_parse: support commits, trees, and blobs

Restore the original applied patches during 'commit'

If committing a patch in the middle of a series, the resulting stack
doesn't have any patches applied. The trans.pop_patches() function only
returns the additional patches that had to be popped rather than all
popped patches. The patch also adds some tests for the 'commit' command.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Reported-by: Jeenu Viswambharan <Jeenu.Viswambharan@arm.com>
Cc: Karl Hasselström <kha@treskal.com>

Merge remote branch 'gustav/proposed'

* gustav/proposed: (22 commits)
  stgit.el: Make ! with prefix argument prompt for git command
  stgit.el: Make ! sort patches when building command line
  stgit.el: Add visual feedback (a message) when toggling view flags
  stgit.el: Add stgit-inhibit-messages
  stgit.el: Add support for showing git-svn information
  stgit.el: Remove useless defvars
  stgit.el: Use forward-line instead of goto-line non-interactively
  stgit.el: Make stgit-toggle-* behave as documented
  stgit.el: Make "C-u r" raise error if run with prefix argument without a patch
  stgit.el: Make stgit-reload move point more intuitively when a patch disappears
  stgit.el: Allow showing recent historical (committed) commits as well
  stgit.el: Add stgit-line-format
  stgit.el: Minor improvements of stgit{,-noname}-patch-line-format documentation
  stgit.el: Delay automatic refresh of stgit buffers a bit
  stgit.el: Refresh stgit buffer after dired-delete-file
  stgit.el: Refactor: use mapc to set all local variables in stgit-mode
  stgit.el: Add stgit-default-show-{ignored,unknown}
  stgit.el: Fix so "t u" and "t i" only operate on the current buffer
  stgit.el: Move to goal column after expanding unknown directory
  stgit.el: Ask for branch point when creating new branch
  ...

stgit.el: Make ! with prefix argument prompt for git command

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Make ! sort patches when building command line

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Add visual feedback (a message) when toggling view flags

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Add stgit-inhibit-messages

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Add support for showing git-svn information

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Remove useless defvars

These defvars are no longer needed as they are defined as buffer-local
in stgit-mode.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Use forward-line instead of goto-line non-interactively

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Make stgit-toggle-* behave as documented

Add macro stgit-define-toggle-view to implement these functions.
Repair prefix argument behavior.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Make "C-u r" raise error if run with prefix argument without a patch

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Make stgit-reload move point more intuitively when a patch disappears

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Allow showing recent historical (committed) commits as well

This is controlled with the "t h" command.

Signed-off-by: David Kågedal <davidk@lysator.liu.se>
Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Add stgit-line-format

This function will be used in the following patch.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Minor improvements of stgit{,-noname}-patch-line-format documentation

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Delay automatic refresh of stgit buffers a bit

This prevents multiple consecutive refreshes when several
refresh-causing operations are done after each other, which happens
when one does recursive deletes.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Refresh stgit buffer after dired-delete-file

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Refactor: use mapc to set all local variables in stgit-mode

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Add stgit-default-show-{ignored,unknown}

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Fix so "t u" and "t i" only operate on the current buffer

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Move to goal column after expanding unknown directory

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

stgit.el: Ask for branch point when creating new branch

This makes sure git-config branch.<branch>.parentbranch is set correctly.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

publish: Add '--last' option to show the last published patch

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

Strip leading or trailing '-' when generating patch names

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

publish: Add the --unpublished options to list not yet published patches

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

stgit.el: Use comint-carriage-motion in output from "!"

This makes carriage return and similar be handled correctly, which is
important as some git commands use those.

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

mail: Add 'snumber' template parameter equivalent to stripped 'number'

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

pick --revert wasn't using the correct author

The author should be the one reverting the patch rather than the creator
of the reverted commit. The patch also improves the revert message to
contain the description of the reverted commit.

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

Allow interactive merging via StackTransaction.reorder_patches()

This way other commands like sink and pop can use the interactive
mergetool.

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

mail: Use space rather than tab for long subject header folding

The default Python implementation (at least 2.5 and earlier) fold long
e-mail header lines by inserting "\n\t". This causes issues with some
e-mail clients that remove both "\n\t". The RFC2822 shows that folding
should be done with "\n ". The Python workaround is to use a Header
object instead of a string when setting the message headers.

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

stgit.el: Do not use suppress-keymap for prefix keymaps

Signed-off-by: Gustav Hållberg <gustav@gmail.com>

mail: Ask for the SMTP credentials before sending the messages

The original implementation was asking for the SMTP password on every
patch sent. This patch only asks the password once before sending or
even editing the cover message and patches.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Cc: Pavel Roskin <proski@gnu.org>
Cc: Alex Chiang <achiang@hp.com>

Fix _command_list() function for empty command sets

The alias commands are not real StGit commands and the corresponding set
(kind) defined in stgit/commands/__init__.py is empty. This leads to a
KeyError in the _command_list() function. The patch ignores empty sets.

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

Replace some git commands with stg aliases in test scripts

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

Add support for command aliases

This patch introduces support StGit command aliases with a few defaults:

stg add -> git add
stg rm -> git rm
stg mv -> git mv
stg resolved -> git add

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

Populate the cached config options with the defaults

The patch pre-populates the cached config options with the default
values. It also removes an unused option (stgit.extensions).

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

Pass the --in-reply-to and --no-thread options to git send-email

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

Rename the mail --refid and --noreply options to match Git

The new option names are --in-reply-to and --no-thread.

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

Record a single transaction for conflicting push operations

StGit commands resulting in a conflicting patch pushing record two
transactions in the log (with one of them being inconsistent with HEAD
!= top). Undoing such operations requires two "stg undo" (possibly with
--hard) commands which is unintuitive. This patch changes such
operations to only record one log entry and "stg undo" reverts the stack
to the state prior to the operation.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Cc: Gustav Hållberg <gustav@virtutech.com>
Cc: Karl Wiberg <kha@treskal.com>

stg mail: don't parse To/Cc/Bcc in --git mode

When using stg mail in --git mode, do not parse command-line To/Cc/Bcc
addresses.

Instead, we pass them directly to git send-email.

This allows us to leverage git send-email's support for email aliases.

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stg mail: add basic support for git send-email

This is the first step in turning stg mail into a wrapper for
git send-email. It requires passing the --git option to stg mail
for now.

Only a few basic options are supported for now, namely To/Cc/Bcc.

git send-email options used:
  --suppress-cc=self prevent further information prompts
  --quiet reduce git send-email output

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stg mail: factor out __update_header

Factor __update_header out of __build_address_headers.

Headers like Reply-To, Mail-Reply-To, and Mail-Followup-To are now
handled in __build_extra_headers.

We make this change because in the future, we do not want to call
__build_address_headers if using git send-email but we will always
want to call __build_extra_headers.

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stg mail: make __send_message do more

Factor out the common code required to send either a cover mail
or patch, and implement it in __send_message.

WRY? DRY.

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stg mail: reorder __build_[message|cover] parameters

Reorder the argument lists for both __build_cover and __build_message.

This change will aid readability of a subsequent refactoring patch.

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stg mail: Refactor __send_message and friends

Instead of passing all the various smtp* args to __send_message
individually, let's just pass the options list instead.

The main motivation is for future patches. The end goal is to
thin out stg mail's implementation and make it a minimal wrapper
around git send-email. By passing the options list to __send_message
we prepare to pass options directly to git send-email.

As a bonus, this change results in a cleaner internal API.

Finally, it also pushes the smtp logic where it belongs, viz. into
__send_message_smtp, instead of cluttering up the main body of
mail.func().

Cc: Karl Wiberg <kha@treskal.com>
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>

stgit.el: Add the stgit-execute command and bind it to "!"

Signed-off-by: David Kågedal <david@virtutech.com>
Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

improve "usage" strings to include optional "--"

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Handle patch and branch names starting with hyphen

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Made a few wide lines more narrow

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Run "git update-index --refresh" before redrawing work tree status

This prevents unchanged files from showing up as "Modified" in the
work tree.

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Advise non-stgit git functions to update stgit buffers

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

Tutorial: Add stgit.el section

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>
Signed-off-by: Karl Wiberg <kha@treskal.com>

stgit.el: Prevent stgit-applied-patches from moving point

This stops stgit-diff-range from moving point.

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>
Signed-off-by: Karl Wiberg <kha@treskal.com>

stgit.el: Annotate stgit-capture-output macro for edebug

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>
Signed-off-by: Karl Wiberg <kha@treskal.com>

stgit.el: Add command for showing diff for a range of patches

Signed-off-by: Karl Wiberg <kha@treskal.com>

stgit.el: Optionally allow duplicates when sorting patches

Signed-off-by: Karl Wiberg <kha@treskal.com>

stgit.el: Break out diff whitespace arg handling to separate function

Signed-off-by: Karl Wiberg <kha@treskal.com>

Merge gustav/proposed

  stgit.el: Make "G" work before/after the tree
  stgit.el: Repair how patches with empty descriptions are shown

Merge branch 'stable'

Fix "stg applied" usage in contrib/stg-dispatch

This command was replaced by "stg series --applied" but the above shell
script hasn't been changed. This is a fix for bug #557475.

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

stgit.el: Make "G" work before/after the tree

stgit.el: Repair how patches with empty descriptions are shown

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Do not emit trailing whitespace

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Get default rebase branch from git-config in stg-rebase

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Allow "B" to create new branches

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Make stg undo/redo --hard ask for confirmation with dirty index/worktree

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Do not recurse into unknown directories after "t u"

Let RET on unexpanded directories expand the directory.
This is similar to how git-status does it.

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Use "->" for defstruct field accessors

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Disable undo information in stgit output capture buffers

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

new: Allow empty messages with --message and --file

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>

stgit.el: Make the patch name optional

Introduce an "alternate" patch line format and make "t n" toggle
between the two.

Signed-off-by: David Kågedal <david@virtutech.com>

stgit.el: Use format-spec when formatting file information

Signed-off-by: David Kågedal <david@virtutech.com>

stgit.el: Use format-spec when formatting patch lines

Signed-off-by: David Kågedal <david@virtutech.com>

stgit.el: Move defcustom definitions to the top

Signed-off-by: David Kågedal <david@virtutech.com>

Merge commit 'kha/safe'

Merge branch 'stable'

Update the tutorial for the conflict situations

This is a fix for bug #14672.

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

undo: Add test that "stg undo --hard" leaves the stgit tree in a consistent state

The test currently fails.

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>
Signed-off-by: Karl Wiberg <kha@treskal.com>