continued yesterdays work on the execute stuff

Merge branch 'master' of git@gitorious.org:git-python/apollo13

made with_exceptions=True default (don't look before you leak ;)) and removed the combined output of stderr and stdout.
Also renamed with_status to extended_output.
The method_missing function needs to be modified, as it does a kwargs.pop(xxx, None); which resulted in with_excpetions=None -> False all the time...
Test should follow tomorrow.

Merge branch 'master' of git://gitorious.org/git-python/david into bisect

fixed the tests, which where broken before my commit, although the bug didn't reveal it...

Merge branch 'master' of git://gitorious.org/git-python/david

fixed second problem in groups.google.com/group/git-python/browse_thread/thread/b8f3580abf31f9db?hl=en#
I'll look at the other one in an hour

cmd: rename with_keep_cwd to keep_cwd

Having execute() use a different directory is an important piece
of API information so I added more documentation about it and
renamed the flag to just "keep_cwd" since that's shorter and simpler.

Signed-off-by: David Aguilar <davvid@gmail.com>

Merge commit 'sverre/master' into david

* commit 'sverre/master':
  Added a with_keep_cwd option
  Improved the GIT_PYTHON_TRACE=full output format
  Added /dist to .gitignore

Conflicts:

lib/git/cmd.py

Signed-off-by: David Aguilar <davvid@gmail.com>

cmd: better support for bare repositories

In order to avoid the expense of parsing .git/config
just to know whether or not a repository is bare
at __init__ time, we just pass an optional flag to
Git.__init__():
bare_repo with a default value of False.

Repo.init_bare() was updated to pass this flag.

We could have an optional Git.read_bare_status() function
that does the expensive lookup.  Then, users can optionally
call it at runtime if they really need to know whether or not
a repository is bare.  That seems like a decent tradeoff between
speed, correctness, and common use cases.

Signed-off-by: David Aguilar <davvid@gmail.com>

Added a with_keep_cwd option

When executing commands, if the with_keep_cwd option is
specified, the current working directory will be set to
os.getcwd() instead of the directory containing the
.git directory.

Improved the GIT_PYTHON_TRACE=full output format

It now also shows stderr if there was any on it, and only
shows stdout if there was any output. Also added a '->'
between the command and the return value as a visual clue.

Added /dist to .gitignore

cmd: properly handle cwd for repos with work trees

This is a fix on top of Govind's latest performance improvement.

self._cwd was always set to self._git_dir which means
a lot of commands that require work trees were not available
to GitPython.  Execute now uses self._cwd which is equal to
self._git_dir by default, and self.get_work_tree() if a work tree
exists.

Signed-off-by: David Aguilar <davvid@gmail.com>

repo: these changes make Govind's latest changes pass the test cases

Signed-off-by: David Aguilar <davvid@gmail.com>

Determine git_dir and git_work_tree in python.

Calling git to find the git_dir and work_tree is very costly.
This patch uses the same mechanisim to find the git_dir as native
git does without shelling out.

Signed-off-by: Govind Salinas <blix@sophiasuchtig.com>

stupid typo *grr*

removed to much :þ

added support for GIT_PYTHON_TRACE=='full', thx to SRabbelier for the suggestion

fixed init_bare

Merge branch 'master' of git://gitorious.org/git-python/mainline

fixed up the getattr stuff

use getattr instead of gettatribute

changed svn to git. This makes more sense. I tried trunk, master, edge.

tests: removed an obsolete comment in test_it_ignores_false_kwargs

Signed-off-by: David Aguilar <davvid@gmail.com>

Git: guard against passing False to git commands

git does not accept commands of the form:
git cmd --xx=False
or
git cmd -xFalse

This patch prevents transform_kwargs from producing
command lines with those values.

This adds some flexibility/syntactic sugar for callers
since they can then assume that kwargs with a False value
are not passed to git commands.

Signed-off-by: David Aguilar <davvid@gmail.com>

style: remove spaces around parens per PEP8

Signed-off-by: David Aguilar <davvid@gmail.com>

style: follow PEP 8 in git/cmd.py

Keyword args shouldn't use spaces around the equals sign
per PEP 8.

Signed-off-by: David Aguilar <davvid@gmail.com>

put version stuff back.

tests: add a test to ensure False is never passed to Git.execute

Signed-off-by: David Aguilar <davvid@gmail.com>

Git: guard against passing False to git commands

git does not accept commands of the form:
git cmd --xx=False
or
git cmd -xFalse

This patch prevents transform_kwargs from producing
command lines with those values.

This adds some flexibility/syntactic sugar for callers
since they can then assume that kwargs with a False value
are not passed to git commands.

Signed-off-by: David Aguilar <davvid@gmail.com>

forgot to remove the stuff from README.

Signed-off-by: Michael Trier <mtrier@gmail.com>

Removed tutorial from README and added it into new doc section.

stupid me i shouldn't use assert() in tests

assert_true instead because optimized python won't see asserts.

Signed-off-by: David Aguilar <davvid@gmail.com>

fixed the buffer test.

Merge branch 'master' of git://gitorious.org/git-python/david

fixed  gitorious.org/projects/git-python/repos/apollo13/comments#comment_105 my mistake ;) I'll investigate this further, but I think we are on the safe side now...

tests: ensure cmd.execute can handle large output

This test case ensures that cmd.execute can always handle
large command outputs.  This test does so by reading a 700k
file into memory.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: allow "git version" test to pass on arbitrary git versions

Sometimes people run development versions of git.
On these versions, "git version" returns, for example:

git version 1.5.4.rc2.1105.gfc5f2

The test_it_executes_git_to_shell_and_returns_result
testcase was updated to accept these versions of git.

Signed-off-by: David Aguilar <davvid@gmail.com>

Merge branch 'master' of git://gitorious.org/git-python/david

reverted to the cleaner and safer proc.wait method instead of communicate.

tests: allow "git version" test to pass on arbitrary git versions

Sometimes people run development versions of git.
On these versions, "git version" returns, for example:

git version 1.5.4.rc2.1105.gfc5f2

The test_it_executes_git_to_shell_and_returns_result
testcase was updated to accept these versions of git.

Signed-off-by: David Aguilar <davvid@gmail.com>

whoops forgot cmd.

renamed gitter to cmd since that makes more sense.  Fixed up changes doc.

didn't work for git 1.5.3 ;)

fixed last failing test, hopefully this works for all git versions...

replaced another git_python with git

couple fixes to setup

added in changes to this release
.

renamed git_python to git. Removed pop_key and replaced with dict.pop. Fixed up tests so they pass except for stderr test. Modified version information retrieval.

tests: keep temporary repos underneath test/repos

test_repo.py was creating directories in the literal
test/~/foo/... directory.

This commit replaces "~" with "repos".

Some minor whitespace issues were cleaned up as well.

Signed-off-by: David Aguilar <davvid@gmail.com>

gitignore: add ignores for common emacs and vim temporary files

vim saves temporary data in $FILENAME.swp.
emacs saves backups in $FILENAME~.

.gitignore now ignores all of these entries.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: make git_dir an optional parameter

git_dir is now optional in the constructor for git.Git.
Git now falls back to os.getcwd() when git_dir is not specified.

Furthermore, extra checks were added so that the git_dir
variable always points to the root of the worktree for
non-bare repositories.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add utils.pop_key() testcases

This commit cleans up some unused variables in test_utils.py
and adds some testcases for utils.pop_key().

Signed-off-by: David Aguilar <davvid@gmail.com>

git_python: add utils.pop_key to __init__.py

This keeps the interface consistent by importing
utils.pop_key into the git_python namespace.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add a test for the new with_raw_output mechanism

This test ensures that trailing whitespace is preserved
when git.foo( with_raw_output=True ) is called, and vice
versa.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: always strip trailing whitespace and add an option to not do so

We now strip off any trailing whitespace.
We inhibit this behavior when git.foo( with_raw_output=True )
is called.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add a test for the new with_stderr mechanism

This test ensures that stderr is merged into stdout when
git.foo( with_stderr=True ) is called.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: add support for merging stderr into stdout

When git.foo( with_stderr=True ) is called, stderr is returned
along with stdout in a merged stream.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add a test for git.foo( with_exceptions=True )

This test ensures that a GitCommandError is raised when git
returns a non-zero exit status.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: add a with_exceptions keyword argument

When git.foo( with_exceptions=True ) is called a
GitCommandError is raised when the exit status is non-zero.

Signed-off-by: David Aguilar <davvid@gmail.com>

errors: add a GitCommandError class

The next commit will raise GitCommandError when a flag is
passed to method_missing and the exit status is non-zero.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add a git.foo( with_status=True ) testcase

test_git.py now tests the new git.foo( with_status=True )
mechanism.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: add support for git.foo( with_status=True )

Passing with_status to an arbitrary git command causes execute
to return (status_code, output) instead of the typical
returned output.

This is useful when callers need access to the exit status code.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: add a test for git.foo( istream=fh )

This test runs the equivalent of:

"git hash-object < fixtures/cat_file_blob"

with the new istream mechanism and compares the computed hash.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: add support for passing in a stdin input stream

execute/method_missing now support:

fh = open('filename', 'r')
print git.foo( istream=fh )
fh.close()

The name "istream" was chosen so that it would not conflict
with any of the builtin git flags.  "stdin", for instance,
is used by git.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: always use "git foo" and never "git-foo"

gitsters on the git mailing list have been slowly deprecating
the usage of "git-foo" commands.  The preferred invocation
is "git foo".

Signed-off-by: David Aguilar <davvid@gmail.com>

utils: add a pop_key helper function

pop_key removes a value from a dictionary and it in one step.
This function will be used in the next integration.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: add support for a GIT_PYTHON_TRACE environment variable

GIT_PYTHON_TRACE allows us to debug GitPython's usage of
git through the use of an environment variable.

This is preferred over a hard-coded print statement.

Signed-off-by: David Aguilar <davvid@gmail.com>

tests: update test_git.py to use the new single-character flag style

The previous commit changed the way transform_kwargs handles
single-character flags with arguments.  This commit updates the
testcase accordingly.

Signed-off-by: David Aguilar <davvid@gmail.com>

git.py: properly handle single-character flags with arguments

"git diff -U5" is correct while "git diff -U 5" is not.
ditto for "git log -U5", "git commit -F/some/path", etc.

The original version of transform_kwargs was setting up the
command line arguments such that single-character flags with
arguments were not understood by git.

This changes transform_kwargs so that the flag and argument
are both part of the same string for single-character flags.

Signed-off-by: David Aguilar <davvid@gmail.com>

gitignore: ignore eggs, .pyo's and /build

This commit sets up .gitignore so that it properly ignores both
.pyo and .pyc files.

The egg path was updated so that it is prepended with a "/".

"/build" was added to the list of ignored paths since it is
customary for setuptools to use that directory for builds.

Signed-off-by: David Aguilar <davvid@gmail.com>

Merge git://gitorious.org/git-python/apollo13

Fixed up problem where name doesn't exist on root of tree.

removed stupid print

use ~/foo instead of /foo for repo.

fixed errors in the test, two permission errors remaining, thx to mock?!?!
Also removed the shell_escape tests...

removed some unused stuff

made subprocess not use a shell...

removed some more spaces

fixed incorrect test import

added /me as author

removed unneeded spaces

added .gitignore to ignore *.pyc files and the egg-info stuff.

Missing os import.  Thanks apollo13.

added in note about how to handle date time information.  Fixed up repo tests for the removal of the shared option.

lots of little fixes.  Corrected problem with creating bare repo.  Added Repo.create alias.

Corrected problem with tree.__len__ getting confused with zero length files. Thanks Alan Briolat.

Forgot to add Mock to the requirements.

Fixed up rspec comments that are passing.

ugh, the editing never ends.

Apparently I'm a moron

More small changes to tests. Commented out tests not passing because of inability to figure out mocking for them.

Moved test stuff around and modified imports to just clean things up a bit.

cleaned up a bunch of imports for the tests.

just more documentation fixes.

slight modifications to docs.

initial project