refs: Improve error messages

repository: Export all internal paths

Merge pull request #227 from Romain-Geissler/discovery-path-v2

Discovery path v2

Repository: Added the git_repository_discover function that finds by itself the git directory that manage a given directory path.

Repository: Splitted the repository destructor into a helper part (only free directories path) and the complete public destructor.

Repository: Added read_gitfile that allows you to read a .git file and extract the git directory path.

Repository: Added some util functions that we'll need to discover repository path.

retrieve_device returns the file device for a given path (so that we can detect device change while walking through parent directories).
abspath returns a canonicalized path, symbolic link free.
retrieive_ceiling_directories_offset returns the biggest path offset that path match in the ceiling directory list (so that we can stop at ceiling directories).

Filebuf: Fixed a TODO in filebuf (real lock in lock_file)

Added gitfo_creat_locked and gitfo_creat_locked_force

index: Add `git_index_entry_stage` method

As suggested by Romain-Geissler

Fileops: Added gitfo_isfile.

Conflicts:

src/fileops.c

Fileops: Added a fourth argument to the path prettifying functions to use an alternate basepath.
Fixed a Windows TO-DO in the prettifying functions.

Sha1Lookup: Fixed two MSVC compilation warnings.

odb: Fix loading ODB alternates

Fixed an issue with the `strtokz implementation and added support for
comments and relative paths in the alternates file.

Fileops:retrieve_path_root_offset is now named gitfo_retrieve_path_root_offset (like other public functions). Added platform specific directory separator definition.

Merge pull request #144 from nordsturm/fix_fakewstream

Fix fake wstream write

odb-pack: More variable declarations

odb-pack: Do not declare variables mid-function

signature: Fix compilation

short-oid: Cleanup

Implemented read_unique_short_oid method for loose backend.

Fixed some error messages related to searching objects from a short oid. Fixed forgot to check that prefix length is greater than minimum prefix length in read_unique_short_oid method from pack backend.

Deleted unused method git_cached_object_match, since we do not explore the cache when searching objects from a short oid.

Changed return value of git_oid_match to be consistent with the other compare methods (0 means oids match). Added method to compare prefixes of hex formatted oids.

Added a GIT_OID_MINPREFIXLEN constant to define the minimum length allowed for oid prefixes (set to 4, like in git). Consequently updated some object lookup methods and their documentation.

Added git.git sha1 lookup method to replace simple binary search in pack backend.

Implemented find_unique_short_oid for pack backend, based on git sha1 lookup method;
finding an object given its full oid is just a particular case of searching
the unique object matching an oid prefix (short oid).

Added git_odb_read_unique_short_oid, which iterates over all the backends to
find and read the unique object matching the given oid prefix.

Added a git_object_lookup_short_oid method to find the unique object in
the repository matching a given oid prefix : it generalizes git_object_lookup
which now does nothing but calls git_object_lookup_short_oid.

Added error for ambiguous oid prefixes. Added methods to compare the first nth hexadecimal characters (i.e. packets of 4 bits) of OIDs.

Added a read_unique_short_oid method to backends, to make it possible to find objects from sha1 prefixes in the future. Default implementations throw GIT_ENOTIMPLEMENTED for strict prefixes (i.e. length < GIT_OID_HEXSZ).

Merge pull request #224 from glesserd/tagparsing

Fix tag and signature parsing

Merge pull request #215 from schu/typos

Fix typos

Merge pull request #218 from schu/tests-fflush

test_lib.c: flush stdout after every test-run

Merge pull request #223 from carlosmn/valgrind

Plug a leak in the index unmerged entries vector

Merge pull request #222 from carlosmn/config-bugfix

Config bugfix

Merge pull request #206 from nulltoken/topic/is-bare

Add git_repository_is_bare() accessor

index: Cleanup tree parsing

index: correctly parse invalidated TREE extensions

A TREE extension with an entry count of -1 means that it was
invalidated and we should ignore it. Do so instead of returning an
error.

This fixes issue #202

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Allow read_tree_internal to return an error code

There are two reasons why read_tree_internal might return a NULL
tree. The first one is a corrupt index, but the second one is an
invalidated TREE extension. Up to now, its only way to communicate
with its caller was through the return value being NULL or not.

Allow read_tree_internal to report its exit status independently from
the tree pointer.

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

index: Change the memory management for repo indexes

The `git_repository_index` call now returns a brand new index that must
be manually free'd.

object: Update documentation

Fix tag and signature parsing

Before this commit, malformed tag and signature were considered as
valid by the parser. See the test t3800-mktag.sh of git to see example
of malformed tag and signature.

Add test for empty config file

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Don't try to parse an empty config file

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Add test for invalid ext header

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Add config test for empty line

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Config parse header ext: don't allow text after closing quote

Nothing is allowed betwen the closing quotation mark and the ] so
return an error if there is.

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Parse section header ext: don't leak on error

Also free the subsection if we find too many quotes

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Guard against double-freeing the current section

If parse_section_header{,_ext} return an error, current_section
doesn't get allocated. Set it to NULL after freeing so we don't try to
free it again.

This fixes part 2-2 of Issue #210.

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

cfg_readline: really ignore empty lines

Simplify cfg_readline and at the same time fix it so that it does
really ignore empty lines.

This fixes point 2-1 of Issue #210

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Config file open: don't free memory that doesn't belong to us

On error, it would free the configuration object even though it didn't
own that memory, which would cause a double-free.

This fixes the first part of Issue #210

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Plug a leak in the index unmerged entries vector

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

test_lib.c: flush stdout after every test-run

Make sure the user immediately sees the feedback, '.' or 'F', for a
test. If it's only in the buffer, it may gets "lost" in case of error.

Signed-off-by: schu <schu-github@schulog.org>

Fix tiny typo

Signed-off-by: nulltoken <emeric.fermas@gmail.com>
Signed-off-by: schu <schu-github@schulog.org>

gitfo_isdir: fix error message

Signed-off-by: schu <schu-github@schulog.org>

Add git_repository_is_bare() accessor

Merge pull request #204 from nulltoken/fix/build-msvc

Fix compilation warnings in MSVC

Fix compilation warnings in MSVC

This allows to successfully build libgit2 with waf on Windows.

index: Fix unused error messages

index.c: Move to new error handling mechanism

config_file.c: Move to new error handling mechanism

win32/pthread.c: Move to new error handling mechanism

win32/map.c: Move to new error handling mechanism

unix/map.c: Move to new error handling mechanism

backends/sqlite.c: Move to new error handling mechanism

backends/hiredis.c: Move to new error handling mechanism

util.c: Move to new error handling mechanism

tree.c: Move to new error handling mechanism

tag.c: Move to new error handling mechanism

signature.c: Move to new error handling mechanism

revwalk.c: Move to new error handling mechanism

repository.c: Move to new error handling mechanism

refs: Reword errors

refs.c: Move to new error handling mechanism

oid.c: Move to new error handling mechanism

odb_pack: Reword errors

odb_pack.c: Move to new error handling mechanism

odb_loose: Reword errors

odb_loose.c: Move to new error handling mechanism

odb_loose.c: Return GIT_ENOMEM when allocation fails

When trying to inflate a buffer, a GIT_ERROR was returned when malloc()
failed.

Fix this to return GIT_ENOMEM.

odb: Reword errors

odb.c: Move to new error handling mechanism

object: Reword errors

object.c: Move to new error handling mechanism

hashtable: Reword errors

hashtable.c: Move to new error handling mechanism

fileops: Cleanup

fileops.c: Move to new error handling mechanism

filebuf: Reword errors

filebuf.c: Move to new error handling mechanism

config.c: Fix format string error

delta-apply.c: Move to new error handling mechanism

delta-apply.c: Use GIT_ENOMEM instead of GIT_ERROR when allocating

git__delta_apply used to return with a generic GIT_ERROR when allocating
memory for the delta failed.

Fix this to return GIT_ENOMEM.

config: Cleanup

commit: Reword errors

commit.c: Move to new error handling mechanism

blob: Reword errors

blob.c: Move to new error handling mechanism

config: Cleanup & renaming of the external API

"git_config_backend" have been renamed to "git_config_file", which
implements a generic interface to access a configuration file -- be it
either on disk, from a DB or whatever mumbojumbo.

I think this makes more sense.

Rewrite `git_config_open_global`

We have a lot of utility methods that make path building trivial. Use
them!

Merge pull request #193 from carlosmn/config

A couple of config improvements

Merge pull request #198 from sschuberth/fix-typos

Fix a few minor typos in comments and error messages

index: Fix issues in the unmerged entries API