Guilt v0.24

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

pop: Added -f to force popping of the patch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Fixed up description of series -v

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

status: change into the top of the repo before checking for changes

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

status: Added -n to hide status prefix from ouput

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Document all options for status

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

status: Major cleanup of status

Now accepts flags to specify which type of changes should be displayed

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] guilt: Add -h option to show man pages

git-sh-setup steals the -h|--h* argument and prints the usage message
of the command that calls it, which in this case is guilt.  Rather than
print the unhelpful guilt usage message, call guilt-help on the given
command instead.

[jsipek: Move the -h|--h* check up a little]
[jsipek: no need for dirname as guilt-help should be in the path]
Signed-off-by: Eric Lesh <eclesh@ucla.edu>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Documentation/guilt-fork: document argument to choose name of new patch

Signed-off-by: Eric Lesh <eclesh@ucla.edu>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] guilt-fork: Use first argument (if given) as name of new patch

Signed-off-by: Eric Lesh <eclesh@ucla.edu>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] guilt-fork: Rename patches sequentially (ala quilt)

Signed-off-by: Eric Lesh <eclesh@ucla.edu>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.23

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

fork: fork the topmost applied patch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

new: escape new patch name properly when changing the series file

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: guilt-fold manpage

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

fold: fixed patch backup creation

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

delete: use series_remove_patch instead of opencoding the logic

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

fold: fold a patch into the topmost patch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Remove pop_patch as there are no real users

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Include version Guilt version number in page footer

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Fixed up patchbomb documentation

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Added import-commit manpage

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Reimplemented cmd-list generation script in sh

It would be nice if the perl dependency was completely gone.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Consistified the man pages

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Ignore generated usage-*.txt files

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Remaining guilt documentation

Adds man pages for all remaining commands except guilt-import-commit

NOTE: depends on usage generation patch

Signed-off-by: Brandon Philips <brandon@ifup.org>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Get usage information from the USAGE variable

Generate usage information for man pages from USAGE in each script

Signed-off-by: Brandon Philips <brandon@ifup.org>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.22

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Merge branch 'tagging' of /git/guilt

Makefile: Replace hard-coded script list with a wildcard

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Added install-html makefile rule to Documentation/Makefile

Very simple rule, which will make auto-generating the new html branch easy.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Autotag: Read guilt.autotag config var and tag commits only if this feature is enabled

Some users may not want to have guilt automatically tag the top, bottom, and
base of the stack. Allow them to disable such functionality by setting
guilt.autotag=0 in their git config file.

guilt-init now accepts a new parameter '-n' to automatically set
guilt.autotag=0 in the current repository.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Automatically create unannotated tags for top, bottom, and base of the stack

On every push or pop operation (refresh is a pop followed by a push), update
the stack top (${branch}_top), stack bottom (${branch}_bottom), and stack
base (${branch}_base) tags.

Top: Topmost applied patch/commit
Bottom: Bottommost applied patch/commit
Base: Commit on top of which the bottom most patch is applied

Having these three tags, one can easily get the log/diff/other information
only for commits that are (or are not!) part of the patch stack.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Remove stray reference

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Fix command examples in new & next manpage

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Mention the fact that guilt can be used to develop guilt

Yeah, I know this is a shameless plug, but I might as well do it since this
_IS_ guilt.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Fix links/references between pages

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Makefile tab fix

Accidentally messed up tabs in Makefile with spaces. Fixing with a tab.

Signed-off-by: Nur Hussein <hussein@cs.usm.my>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Added an uninstall target

Invoking make uninstall will remove guilt files from installed directory.

Signed-off-by: Nur Hussein <hussein@cs.usm.my>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Fixed guilt-patchbomb temporary mbox deletion bug

A bug occurred where the temporary mbox is deleted even when 'N' or 'n' was
selected because of the incorrect use of the -a operator when testing user
input. Fixed now to use -o instead.

[jsipek: Fixed up the condition some more]
Signed-off-by: Nur Hussein <hussein@cs.usm.my>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Makefile's clean target should be .PHONY as well

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Remove find_git_dir as git-sh-setup already provides such functionality

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Quote variables as frequently as possible to prevent whitespace problems

If for whatever reason, a user decides to have a patch with whitespace in
the name, several of the commands would break. This patch fixes the guilt
"lib" as well as the new and series commands. This should cover most cases,
however I'm sure that there are others.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

new: Make sure the requested patchname would not create unnecessary problems

Using filenames containing e.g.,'/../' would cause problems. Avoid tripping
such problems by refusing to accept such patchnames.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Merge thor:/git/guilt-docs

Docs: prev, refresh, top, and unapplied manpages

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: Formatting fixes

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: guilt-files manpage

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

files: Recurse into subdirectories when using git-diff-tree

Without -r, a file such as fs/unionfs/copyup.c would should up as 'fs' which
is clearly wrong.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Docs: guilt-new manpage

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Inform git-sh-setup that we can work from within a subdirectory

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.21

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

help: A simple help command

Wrapper for man(1) that uses the manual pages build by `make doc`.

Beware, it relies on MANPATH to be set correctly, which is probably the
wrong thing to do.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Removed print_usage as it shouldn't be used

guilt-init somehow managed to get away with using print_usage even though
all the other commands have been converted to use `usage` from git-sh-setup.
Fixing guilt-init makes print_usage unused and therefore it is safe to
remove it.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Install manpages to $(PREFIX)/man

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

refresh: Use ~ as the backup file suffix

Using .prev as the backup file suffix creates all sort of "unexpected"
clutter. We're better off using ~, which many tools (and humans!) expect to
pop up all over the place.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Ignore *.html, *.1 and *.7 in Documentation/

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

docs: Commands should be on a separate line

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Reorder makefile rules & use .PHONY

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Initial guilt documentation

Rough documentation patch follows.

Creates an initial build system for guilt man pages written in asciidoc.
A good deal of the build/configuration system was borrowed from git but
has been customized for guilt.

Also includes guilt-init.txt and guilt.txt man pages.

Signed-off-by: Brandon Philips <brandon@ifup.org>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

refresh: Make sure there is a patch applied before doing a refresh

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: add a new command "files"

The command "files" prints files that are being changed.

Currently option -v, -a and -l are supported, but --combine is not.

Signed-off-by: Yasushi SHOJI <yashi@atmark-techno.com>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: series -v: Fix zero applied patch case

Even if no patch is applied we'd like series -v to work. it shouldn't
print warning and shouldn't put any status prefix at all.

This should fix it.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: add -v to guilt-series

quilt series with option -v prints the patch names along with the
status of the patches in the series.  the status prefixes are:

    ' ': non-applied patch
    '+': applied patch
    '=': top most patch

I just miss this feature from quilt.  so here it is.

Signed-off-by: Yasushi SHOJI <yashi@atmark-techno.com>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Replace print_usage and exit 1 with usage and die in git-sh-setup

git already has utility functions for shell script porcelains.  use
them instead of implementing our own.

[jsipek: Don't source git-sh-setup in guilt-*, just guilt]
Signed-off-by: Yasushi SHOJI <yashi@atmark-techno.com>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

guilt: Document push_patch's new argument

If push_patch's second argument is "abort", then do not create reject files
if the patch does not apply cleanly, but rather fail the push. Omitting the
argument or supplying an empty string will result in old behavior of
creating reject files.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

add/rm: Use print_usage instead of using $USAGE directly

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Do not allow patch name duplication

Quilt maintain a series of patches with unique file names. we might be
able to use sha1 under guilt instead, but this dis-allowing is much
simpler and works now.

so exit with error as soon as we find out that the given patch already
exist under the current branch dir.

Signed-off-by: Yasushi SHOJI <yashi@atmark-techno.com>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Guilt: Format command list in usage

We are sure to get more and more guilt command, so format the command
list in the usage. With the format, it's much easier to see all
command in 25x80 terminal.

Signed-off-by: Yasushi SHOJI <yashi@atmark-techno.com>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Allow command abbreviation

quilt allow us to type

    quilt ser

instead of

    quilt series

this patch does the same thing.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

push: Fixed unkown patchname pushing all patches

If an invalid patchname was supplied, many errors were outputed and all the
patches in the series were pushed.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] Make "guilt push" match "quilt push" when the patch doesn't apply

Sometimes the reason why the patch doesn't apply is because it has
already been applied in mainline when rebasing a patch series.  It is
therefore more convenient if "guilt push" leaves the working tree
untouched if the patch fails to apply unless explicitly asked to force
apply a failing patch using the -f option.

This behavior matches how "quilt push" and "hg qpush" works, so it
also avoids unpleasant surprises for developers used to other patch
series tools.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Fixed do_make_header to include From:

Insert "\nFrom: $GIT_AUTHOR_NAME <$GIT_AUTHOR_EMAIL>" right after the first
line of the commit message. The GIT_AUTHOR_* vars are taken from the
"author" line in the commit object.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

makefile: Include import-commit in list of scripts

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.20

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Centralize pager determining code

$pager - available to all guilt commands. It should be used instead of
$PAGER, as it falls back to more if $PAGER is not set.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

patchbomb: Don't ask for the intro email if sending only one patch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

import-commit: Use munge_hash_range

Provide the user with a more intuitive hash range format:

  <hash> - one commit
  <hash>.. - hash until head (excludes hash, includes head)
  ..<hash> - until hash (includes hash)
  <hash1>..<hash2> - from hash to hash (inclusive)

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

patchbomb: Use munge_hash_range

Provide the user with a more intuitive hash range format:

  <hash> - one commit
  <hash>.. - hash until head (excludes hash, includes head)
  ..<hash> - until hash (includes hash)
  <hash1>..<hash2> - from hash to hash (inclusive)

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

munge_hash_range: Helper function

Currently unused, however patchbomb and import-commit can both benefit form it.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

[PATCH] guilt: Silence command echoing for echos in Makefile

Signed-off-by: Brandon Philips <brandon@ifup.org>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

import-commit: simple commit to patch importer

A simple script to convert a given a range of commits to patches. Currently,
each commit gets put into a separate patch file. Ideally, there would be a
flag that tells us if we want 1 patch or N patches.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.19

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

regression: Fixed regression suite failure

Ted Ts'o's fix requires the patch files to have a pre-determined mtime, in
order for the regression tests to succeed.

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

[PATCH] Use patch file's modtime as the git author and commiter date

By using the same author and committer date, it keeps the git commit
id's stable if the patches haven't changed.  Otherwise, repeated
invocations of "guilt pop -a; guilt push -a" will create new commit's
with different commit dates, creating a lot of garbage commits that
won't disappear for a LONG time, even after running "git-gc --prune",
thanks to the reflog in git 1.5.

Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

patchbomb: Added -n flag

If -n is passed, everything executes as expected, however the emails are not
actually sent. Instead, the exact git-sent-email command is ouputed.

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

regression: Fix regression test failure

The regression tests did not reset the GIT_{AUTHOR,COMMITTER}_*
environmental variables, which made them fail on every system which did not
have them equal to:

"Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>"

Additionally, set the GIT_{AUTHOR,COMMITTER}_DATE to midnight January 1,
2007 UTC for all tests. This fixes a number of future regression test
failures (unless Git changes the internal format, the resulting
commit/tree/blob hashes should be fixed).

Greetings from Midway Airport in Chicago. :)

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>

Makefile: Forgot to add guilt-patchbomb to the list of scripts

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

new: added -e to edit the patch description interactively

-e and -m cannot be combined, however -e and -s can be.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Centralize editor determining code

$editor - available to all guilt commands. It should be used instead of
$EDITOR, as it falls back if $EDITOR is not set.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: Fix test descriptions

No repo, top, and new regression test scripts were miss-described. This
commit fixes the descriptions to be accurate, but still somewhat incomplete.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: Test guilt-init on non-master branch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: Test for guilt-init in a already initialized branch

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: Test push/pop given a patch name

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: test new patch insertion at the end

Test guilt-new inserting a patch at the end of the series.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

regression: test new patch insertion

Test guilt-new inserting a patch as the first in the series, somewhere in
the middle, as well as at the end.

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

Guilt v0.19-rc1

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

delete: Added argument -f to remove patch file

This commit also removes the only dependency on ex

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

patchbomb: Friendly wrapper for git-format-patch & git-send-email

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>

guilt-new: Fix [ syntax

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>