stgit.el: Fix problem where standard-output isn't bound correctly

This fixes errors on M-x eval-buffer in stgit.el.

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

stgit.el: Bugfix of stgit-move-patches to top of stack

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

stgit.el: Improve use of def{group,custom,face}

 * Reorder calls to defcustom and defface to be more user-friendly
 * Use the :link tag where applicable
 * Add a :set handler to reload all stgit buffers when custom values
   are set

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

stgit.el: Add C-c C-b for stgit-rebase

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

stgit.el: Add better help text for stgit-mode

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

stgit.el: Bugfix mode change display

Fixes 0de6881a "stgit.el: Refactor file list formatting".

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

stgit.el: Minor documentation corrections

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

stgit.el: Allow moving ignored files to index

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

stgit.el: Move point in sane way after stgit-file-toggle-index on renames

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

stgit.el: Move point correctly after stgit-revert-file

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

stgit.el: Abbreviate renames/copies with common prefix/suffix

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

stgit.el: Bugfix listing copied/renamed files

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

stgit.el: Default to showing index and work tree

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

stgit.el: Verify that we're running on Emacs 22 or later

ewoc-create needs to have the NOSEP argument.
line-number-at-pos doesn't exist in older versions.

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

stgit.el: Add stgit-redo as C-c C-_ and C-c C-/

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

stgit.el: Set patch names to be word syntax throughout

This allows easy selection of the patch name; e.g., by double-clicking.

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

stgit.el: Ignore space in diff with prefix argument

With one C-u, ignore spaces; with two, ignore all spaces.

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

stgit.el: Add "d" for a few diff commands, similar to git.el

"b" shows diff against base
"c" shows combined diff
"o" shows diff against our version
"t" shows diff against their version

"m" runs smerge-ediff

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

stgit.el: Handle unmerged files better

Show unmerged patches with --ours, which works with diff-mode.

Do not show duplicate entries in the work tree for unmerged files.

Open unmerged files with smerge-mode minor mode.

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

stgit.el: Rename stgit-expand-find-copies-harder to stgit-find-copies-harder

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

stgit.el: Handle negative argument to stgit-{un,}commit

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

stgit.el: Move point properly after stgit-{file-toggle-index,reload}

After stgit-file-toggle-index, move mark to the old file, leaving
point where it is.

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

stgit.el: Add "t i/u" for showing files ignored by or unknown to git

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

stgit.el: Refactor: simplify stgit-patch-pp a bit

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

stgit.el: Fix face "spilling" in some places

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

stgit.el: Refactor: simplify temporary marker handling

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

stgit.el: Allow operating on the end of each line

Extend patch/file properties to the newline at the end of each line,
allowing stgit-select and similar to take effect with point at the end
of line.

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

stgit.el: Document stgit-select properly

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

stgit.el: Garbage collect selected patches on reload

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

stgit.el: Default to expanding index and work tree

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

stgit.el: Make it configurable where to show index and work tree

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

stgit.el: Add customizable face for "Index" and "Work tree" titles

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

stgit.el: Forbid stgit-{delete,edit,mark,rename} on index and work tree

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

stgit.el: Open the destination file on applied renames or copies

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

stgit.el: Allow showing patch on Index and Work tree

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

stgit.el: Refactor: add stgit-find-copies-harder-diff-arg helper function

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

stgit.el: Refresh from work tree when index is empty

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

stgit.el: Add "R" for stgit-resolve-file

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

stgit.el: Change key binding for stgit-repair from "R" to "\C-u g"

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

stgit.el: Add "U" for stgit-revert-file

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

stgit.el: Move stgit-{un,}commit to \C-c\C-{u,c}

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

stgit.el: Add stgit-branch-name-face

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

stgit new: Do not open editor if --save-template was specified

Fixes side-effect of e58f264a "Add support for merge-friendly
branches".

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

stgit.el: Make stgit-{index,worktree}-node buffer local

Fixes bugs when multiple stgit buffers have been opened.

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

stgit.el: Bugfix moving symlinks to/from index

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

stgit.el: Run "git rm" with "-q" to reduce spam

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

stgit.el: Refactor: remove unused argument to two functions

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

stgit.el: Remove unused variable stgit-work-ewoc

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

stgit.el: Correct documentation for stgit-insert-patch-files

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

stgit.el: Only refresh the relevant parts when the index or worktree changes

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

stgit.el: Add stgit-toggle-worktree to toggle visibility of index and working tree

Based on a similar set of patches by Gustav Hållberg.

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

stgit.el: Add the stgit-file-toggle-index command

Based on a patch from Gustav Hållberg.

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

stgit.el: Add work tree and index as pseudo-patches

Based on a similar patch by Gustav Hållberg.

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

stgit.el: Use ewoc to keep track of the file list.

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

stgit.el: Refactor file list formatting

Define a struct to contain the information about a file and break out
the file list formatting to a separate function.

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

stgit.el: Get rid of the stgit-patchsym property

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

stgit.el: Invalidate ewoc node when marking or unmarking instead of modifying buffer directly

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

stgit.el: Use ewoc to keep track of the patch list.

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

stgit.el: Use a defstruct to store patch data

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

stgit.el: Use an 'entry-type text property

It indicates whether there is a patch or a file on a line.

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

stgit.el: Rename stgit-toggle-patch-file-list to stgit-select-patch

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

stgit.el: Remove the unused allow-file parameter to stgit-patch-at-point

t2800: Adapt to changes in git's conflict markers

The conflict markers used by git were changed slightly by commit
606475f3 ("Remove filename from conflict markers"); specifically, the
conflict markers contain the filename only when the filename has
changed.

This patch adapts t2800 to allow either the new or the old format.

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

Merge commit 'kha/safe'

Karl got married

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

Do not replace '-' with '.' in the version string

This prevents version strings like 0.15-rc1.

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

stgit.el: Use declare to hint the debugger and indentation

Signed-off-by: David Kågedal <davidk@lysator.liu.se>
Signed-off-by: Karl Hasselström <kha@treskal.com>

Add --stat option to show

This option allows showing of diffstat instead of patch diffs. Proposal
and initial implementation by Alex Chiang.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Tested-by: Alex Chiang <achiang@hp.com>

fix stg mail %(shortlog)s order

When using the %(shortlog)s variable, the patches are listed in
the cover letter in the reverse order of the actual patch series.

We should list them in the actual order of the patch series instead.

Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>

Add --reject option to import

This allows a failed import to leave .rej files in the working
directory.

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

Rename out.start() to out.inf() in the publish command

There was no corresponding out.done() call.

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

Only allow publishing to refs/heads/*

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

Improve the "publish" merge message to give slightly more information

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

Remove the resolved command

This command simply duplicates the "git add" or "git mergetool" commands
for handling conflicts.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Use the default git colouring scheme rather than specific scripts

This patch adds the mechanism to check if the output is tty for the
diff and show commands and passes the --color option to git if the
color.diff config option is set auto or true. The patch also changes the
default pager to 'less -FRSX' from the diffcol.sh script.

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

Add "stg id" support for "{public}" ref

The {public} ref refers to the public version of the branch as described
in the "publish" command.

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

Add support for merge-friendly branches

The main issue with publishing StGit branches is that the Git history
represented by patches is volatile, making it difficult for people
wanting to merge such branch. One solution is for all the downstream
developers to always rebase but that's not always desirable. Another
solution is provided by tools like TopGit but the visible Git history
becomes complicated, especially with repeated reordering.

The patch proposes a new StGit command called "publish". This command
allows one to develop patches normally on a StGit branch but publish the
stack changes to a separate, merge-friendly branch whose history is not
re-writable.

More about its behaviour can be found in the command description in this
patch.

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

Pass "git config" flags in the right order

New enough gits accept the --null and --list flags in any order, but
older gits want --null before --list. So do it the way that's fine
with everyone.

I think (but haven't verified) that git started to accept the flags in
any order after commit d64ec16c2af4ddcf3985d11d5dc28a15db181de5 ("git
config: reorganize to use parseopt") by Felipe Contreras on
2009-02-21, first released in git v1.6.3.

Signed-off-by: Karl Hasselström <kha@treskal.com>

Add a --set-tree flag to stg push

This flag makes the push simply restore the tree that the patch used
before, rather than doing any kind of merge.

Signed-off-by: David Kågedal <davidk@lysator.liu.se>
Signed-off-by: Karl Hasselström <kha@treskal.com>

Load the whole config at once and cache it for future use

Instead of loading the config values one at at time when we need then,
read and parse the configuration files all at once using

  git config --list --null

and cache the result. This should be a performance win.

Signed-off-by: Samuel Tardieu <sam@rfc1149.net>
Signed-off-by: Karl Hasselström <kha@treskal.com>

Transaction.push_patch(): Set self.head only when we have a merge conflict

Setting self.head when we don't have a merge conflict is a harmless
no-op---as long as we set it to the commit that was going to be the
stack top anyway---so this patch should not change the behavior. But
it's not really nice to do it unconditionally, especially considering
that we want people to be able to understand what the code does ...

Also add a comment that explains why we set it, since the logic is
rather more hairy than I'd like.

Signed-off-by: Karl Hasselström <kha@treskal.com>

Add the log --clear option

This option allows the clearing of the log history which sometimes may
get too large.

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

Reinstate the --annotate option for refresh

It is sometimes useful to add some notes to the log entry when a patch
was refreshed. This option was dropped when the command was updated to
the new infrastructure as there was no logging support at that time.

The note will be visible with 'stg log {-g,-f}'

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

Convert 'unhide' to the lib infrastructure

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

Convert 'hide' to the lib infrastructure

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

Convert 'clone' to the use stgit.lib

The patch also adds the stgit.lib.git.clone() function.

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

Do not sleep after the last patch sent by e-mail

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

Show some progress information when checking for upstream merges.

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

Show "Pushing <patch>...done" when pushing a patch

My main reason is for the automatic invocation of the interactive merge
when I don't know what patch I have to deal with. The other reasons is
for people working over slow filesystems (NFS) where a three-way merging
may take a significant amount of time.

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

stgit.el: Bind line movement keys in a sane way

Use p/n for stgit-{previous,next}-patch and C-p/n and up/down for
stgit-{previous,next}-line, which makes stgit-mode behave more like
dired-mode and git-mode.

Signed-off-by: Gustav Hållberg <gustav@virtutech.com>
Signed-off-by: David Kågedal <davidk@lysator.liu.se>
Signed-off-by: Karl Hasselström <kha@treskal.com>

Newer gits return 128 when git config --{rename,remove}-section fails

As of commit d64ec16c ("git config: reorganize to use parseopt"), git
config --rename-section and --remove-section will now call die()
instead of exit(1) when the section to be renamed or removed doesn't
exist, and die() calls exit(128). So we have to catch exit code 128 as
well.

Signed-off-by: Karl Hasselström <kha@treskal.com>

Convert "push" to the lib infrastructure

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Convert "pop" to the lib infrastructure

The test files have to be adapted because of a slightly different way of
recording a "push" operation with the new infrastructure. The original
code was recording two transactions for a "push <patch>" - one for
reordering the unapplied patches and another for pushing the patch. One
would need two "undo" operations to restore the state prior to a "push".
The new infrastructure only records one transaction for "push" and only
one "undo" is necessary.

Since a "pop <middle patch>" operation involves popping the patches and
pushing one, with the above changes in the "push" implementation it
means that the patch order the test scripts expect is slightly
different.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Add the --merged option to goto

This patch adds support for checking which patches were already merged
upstream. This checking is done by trying to reverse-apply the patches
in the index before pushing them onto the stack.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Use a default "hidden" argument in StackTransaction.reorder_patches

This argument is rarely used so adding a default value simplifies the
calling code.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Convert "float" to the lib infrastructure

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Convert "sink" to the new infrastructure

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Add automatic git-mergetool invocation to the new infrastructure

This patch adds the IndexAndWorktree.mergetool() function responsible
for calling 'git mergetool' to interactively solve conflicts. The
function may also be called from IndexAndWorktree.merge() if the
standard 'git merge-recursive' fails and 'interactive == True'. The
'allow_interactive' parameter is passed to Transaction.push_patch() from
the functions allowing interactive merging.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Add mergetool support to the classic StGit infrastructure

Since Git already has a tool for interactively solving conflicts which
is highly customisable, there is no need to duplicate this feature via
the i3merge and i2merge configuration options. The user-visible change
is that now mergetool is invoked rather than the previously customised
interactive merging tool.

The stgit.keeporig option is no longer available to be more consistent
with the Git behaviour.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>

Check for local changes with "goto"

This is done by default, unless the --keep option is passed, for
consistency with the "pop" command. The index is checked in the
Transaction.run() function so that other commands could benefit from
this feature (off by default).

This behaviour can be overridden by setting the stgit.autokeep option.

Signed-off-by: Catalin Marinas <catalin.marinas@gmail.com>
Acked-by: Karl Hasselström <kha@treskal.com>