tag: discover the target type if needed

Don't blindly pass the target type to git_tag_type2string as it will
give an empty string on GIT_OBJ_ANY which would cause us to create an
invalid tag object.

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

index.h: Add IDXENTRY flags needed for index operations

Add several IDXENTRY flags that need to be checked in
order to properly implement update-index --refresh.

Added git_commit_tree_oid and git_commit_parent_oid.

fix git_treebuilder_insert probrem.

couldn't add new entry when inserting new one with `git_treebuilder_insert`.

Do not assert error codes on Hiredis backend

We cannot assume that Redis is never going to return an error code; when
Reddit fails, we cannot crash our library, we need to handle the crash
gracefully.

Signed-off-by: Vicent Marti <tanoku@gmail.com>

redis backend

Build & write custom trees in memory

Make reinitializing a repository return GIT_ENOTIMPLEMENTED instead of GIT_SUCCESS

Fix tag reference name in testrepo.git

The git test repository was holding a wrongly named tag reference ("very-simple") pointing at a tag named "e90810b".
This mistake (mine :-/ ) originates back to https://github.com/libgit2/libgit2/commit/9282e92

Whole credit goes to @tclem for having spotted this.

Add a fake wstream to the ODB

Streaming writes will no longer fail when writing to a backend that
doesn't support streaming writes but supports direct ones.
Now we create a fake stream on memory and then write it as a single
block using the backend `write` callback.

Fix the git_tree_write implementation

New external API method: `git_tree_create`

Creates a tree by scanning the index file. The method handles recursive
creation of trees for subdirectories and adds them to the parent tree.

Merge branch 'tagging' of https://github.com/nulltoken/libgit2 into development

Conflicts:
include/git2/tag.h
src/tag.c

Change `parse` methods to const buffer

Signed-off-by: Vicent Marti <tanoku@gmail.com>

Merge branch 'parse-tag-buffer' of https://github.com/carlosmn/libgit2 into development

Update README after sqlite option change

To build libgit2 with sqlite support, waf configure should be run
with --with-sqlite

wscript: Use -O0 on debug

If we want debugging symbols, we most likely want them to point to the
right place. With -O2, gdb or valgrind may give wrong information.

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

Rename git_tag_create_o_f() to git_tag_create_fo()

Add git_tag_delete()

Add git_tag_create_o_f() and git_tag_create_f() which overwrite existing tag reference, if any

Fix memory leak in tag releated tests

Prevent tag_create() from creating a conflicting reference

Add test demonstrating that one can create a tag pointing at a non existent target

Enforce the testing of the correct creation of a tag

Fix misleading comments

Add test ensuring one can not create an oid reference which targets at an unknown id

Replace gitfo_unlink() calls with git_reference_delete() in refs related tests

Fix help message for waf configure --with-sqlite

Check for looser reference names

res/dummy/a and refs/stash must pass. The other rules are already
tested by the rest of the checks.

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

normalize_name: allow more references under refs/

Allow any well-formed reference name to live under refs/ removing the
condition that they be under refs/{heads,tags,remotes}/ as was the
design of git.

An exception is made for HEAD which is allowed to contain an OID
reference in detached HEAD state.

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

refs: Don't allow references to inexistent OIDs

Enforce coding conventions in refs.c

Internal methods are static and without the git prefix.
'Force' methods have a `_f` prefix to match the other 'force' methods.

Make overwrite test more comprehensive

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

force-rename test: check for the right name

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

rename: don't return early if the target ref exists

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

Add tests covering overwriting references

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

Fix documentation copy error

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

Match the comment with the error string

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

Allow forcing the creation or renaming of references

Add internal reference create and rename functions which take a force
parameter, telling them to overwrite an existing reference if it
exists.

These functions try to update the reference if it's of the same type
as the one it's going to be replaced by. Otherwise the old reference
becomes invalid.

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

Add GIT_EEXISTS error code

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

libgit2 version 0.11.0, "McSwifty"

Apologies for the massive changes in the external API (that's my fault),
and for the terrible codename for this release (that's @tclem's fault).

The detailed overview for the major API changes can be found in the
commit at 72a3fe42fb7208712bbe8f0981f4c6274c05e9c3.

Major new features in this release:

- Real caching and refcounting on parsed objects
- Real caching and refcounting on objects read from the ODB
- Streaming writes & reads from the ODB
- Single-method writes for all object types
- The external API is now partially thread-safe
- Improved reference handling
- New method to list references
- ZLib is now built-in
- Improvements to the Revision Walker
- Tons of bug fixes

Thanks to all the contributors who make this possible.

Signed-off-by: Vicent Marti <tanoku@gmail.com>

Update the SQLite backend

index.c: Fix tiny typos

signature.h: Fix tiny typo

Add git_tag_create_frombuffer API

Expose the tag parsing capabilities already present in the
library.

Exporting this function makes it possible to implement the
mktag command without duplicating this functionality.

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

Fix memory leak in parse_tag_buffer

Free the allocated memory if the signature parsing reports an error.

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

New external API method: `git_reference_listcb`

List all the references in the repository, calling a custom
callback for each one.

The listed references may be filtered by type, or using
a bitwise OR of several types. Use the magic value
`GIT_REF_LISTALL` to obtain all references, including
packed ones.

The `callback` function will be called for each of the references
in the repository, and will receive the name of the reference and
the `payload` value passed to this method.

index.h: Fix minor typo

index.h: Correct documentation for git_index_open_inrepo()

Fix the doxygen comments for git_index_open_inrepo(). Previously they
referred to a param index_path and omitted index (probably a c&p
error).

index.c: Read index after initialization

The current behaviour of git_index_open{bare,inrepo}() is unexpected.
When an index is opened, an in-memory index object is created that is
linked to the index discovered by git_repository_open(). However, this
index object is empty, as the on-disk index is not read. To fully open
the on-disk index file, git_index_read() has to be called. This leads to
confusing behaviour. Consider the following code:

git_index *idx;
git_index_open_inrepo(&idx, repo);
git_index_write(idx);

You would expect this to have no effect, as the index is never
ostensibly manipulated. However, what actually happens is that the index
entries are removed from the on-disk index because the empty in-memory
index object created by open_inrepo() is written back to the disk.

This patch reads the index after opening it.

add go-git (Go bindings) to the readme

Remove circular dependency in includes

odb.h: Fix minor typo

Fix a doxygen typo ("@para" instead of "@param") in odb.h

common.h: Fix minor typos

Fix a few minor typos in the documentation of the GIT_ERROR codes.

Improve the ODB writing backend

Temporary files when doing streaming writes are now stored inside the
Objects folder, to prevent issues when moving files between
disks/partitions.

Add support for block writes to the ODB again (for those backends that
cannot implement streaming).

Add Ocaml bindings to the readme

Do not return on `void` helper methods

MSVC doesn't swallow that.

Remove `git_repository_gc` from the headers

Add close wappers for commit, tree, tag and blob

In the same spirit that git_repository_lookup is no longer available,
add wrappers so the users don't have to cast when closing their
objects.

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

Fix compilation in MinGW

Add test exercising the opening of an standard repository initialized by git

Add test exercising the opening of an empty bare repository initialized by git

Add remove_placeholders() test helper function which recursively removes marker files from a directory structure

Add empty bare and normal repos to test resources

We have removed the hooks folder because it takes a lot of space

Add detection of incorrect usage to git__joinpath()

Slightly enforce copy_recurs() behavior

The folder creation is now decorrelated from the recursive parsing of the source tree structure.

Improve test coverage of new path prettifying behavior

Fix detection of attempt to escape the root directory on Windows

Change gitfo_prettify_dir_path() and gitfo_prettify_file_path() behavior

Those functions now return prettified rooted path.

Make gitfo_prettify_dir_path() and gitfo_prettify_file_path() no longer externalized

Add failing test for issue 84

see https://github.com/libgit2/libgit2/issues#issue/84

Fix gitfo_mv() behavior when running on Windows

When the system temporary folder is located on a different volume than the working directory into which libgit2 is executing, MoveFileEx() requires an additional flag.

Switch from time_t to git_time_t

git_time_t is defined as a signed 64 integer. This allows a true predictable multiplatform behavior.

Corrected the header include directory in doxygen conf

Always free the parents of a revwalk commit

Thanks to Carlos Martín Nieto for spotting this.

Add getters for `git_odb_object`

Fix MSVC warnings when building threads

refs loose_lookup: also free the buffer on success

Free the ref_file buffer at the end of the function also on
success. This fixes a small memory leak.

Update CMakeLists.txt to support threadsafe

Fix MSVC warnings

I broke your bindings

Hey. Apologies in advance -- I broke your bindings.

This is a major commit that includes a long-overdue redesign of the
whole object-database structure. This is expected to be the last major
external API redesign of the library until the first non-alpha release.

Please get your bindings up to date with these changes. They will be
included in the next minor release. Sorry again!

Major features include:

- Real caching and refcounting on parsed objects
- Real caching and refcounting on objects read from the ODB
- Streaming writes & reads from the ODB
- Single-method writes for all object types
- The external API is now partially thread-safe

The speed increases are significant in all aspects, specially when
reading an object several times from the ODB (revwalking) and when
writing big objects to the ODB.

Here's a full changelog for the external API:

blob.h
------

- Remove `git_blob_new`
- Remove `git_blob_set_rawcontent`
- Remove `git_blob_set_rawcontent_fromfile`
- Rename `git_blob_writefile` -> `git_blob_create_fromfile`
- Change `git_blob_create_fromfile`:
The `path` argument is now relative to the repository's working dir
- Add `git_blob_create_frombuffer`

commit.h
--------

- Remove `git_commit_new`
- Remove `git_commit_add_parent`
- Remove `git_commit_set_message`
- Remove `git_commit_set_committer`
- Remove `git_commit_set_author`
- Remove `git_commit_set_tree`

- Add `git_commit_create`
- Add `git_commit_create_v`
- Add `git_commit_create_o`
- Add `git_commit_create_ov`

tag.h
-----

- Remove `git_tag_new`
- Remove `git_tag_set_target`
- Remove `git_tag_set_name`
- Remove `git_tag_set_tagger`
- Remove `git_tag_set_message`

- Add `git_tag_create`
- Add `git_tag_create_o`

tree.h
------

- Change `git_tree_entry_2object`:
New signature is `(git_object **object_out, git_repository *repo, git_tree_entry *entry)`

- Remove `git_tree_new`
- Remove `git_tree_add_entry`
- Remove `git_tree_remove_entry_byindex`
- Remove `git_tree_remove_entry_byname`
- Remove `git_tree_clearentries`
- Remove `git_tree_entry_set_id`
- Remove `git_tree_entry_set_name`
- Remove `git_tree_entry_set_attributes`

object.h
------------

- Remove `git_object_new
- Remove `git_object_write`

- Change `git_object_close`:
This method is now *mandatory*. Not closing an object causes a
memory leak.

odb.h
-----

- Remove type `git_rawobj`
- Remove `git_rawobj_close`
- Rename `git_rawobj_hash` -> `git_odb_hash`
- Change `git_odb_hash`:
New signature is `(git_oid *id, const void *data, size_t len, git_otype type)`

- Add type `git_odb_object`
- Add `git_odb_object_close`

- Change `git_odb_read`:
New signature is `(git_odb_object **out, git_odb *db, const git_oid *id)`
- Change `git_odb_read_header`:
New signature is `(size_t *len_p, git_otype *type_p, git_odb *db, const git_oid *id)`
- Remove `git_odb_write`
- Add `git_odb_open_wstream`
- Add `git_odb_open_rstream`

odb_backend.h
-------------

- Change type `git_odb_backend`:
New internal signatures are as follows

int (* read)(void **, size_t *, git_otype *, struct git_odb_backend *, const git_oid *)
int (* read_header)(size_t *, git_otype *, struct git_odb_backend *, const git_oid *)
int (* writestream)(struct git_odb_stream **, struct git_odb_backend *, size_t, git_otype)
int (* readstream)( struct git_odb_stream **, struct git_odb_backend *, const git_oid *)

- Add type `git_odb_stream`
- Add enum `git_odb_streammode`

Signed-off-by: Vicent Marti <tanoku@gmail.com>

Thread safe cache

Fix headers for the new Revision Walker

The "oid.h" header is now included instead of "object.h".

The old "revwalk.h" header has been removed; it was empty.

Refs are now always in-sync on disk

Define NO_GZIP in zconf.h instead of at compile time

Change the Revwalk reset behavior to the old version

The `reset` call now removes the pushed commits so we can reuse
the revwalker. The API documentation has been updated with the details.

Fix signature in `git_repository_gc`

The method returns an int with the amount of objects free'd

Export `git_repository_gc` properly

One of my brainfarts made me export it as `git_repository_close` instead
of GC. Duh.

Export `git_strarray_free` instead of inlining

That way non-C bindings can use it.

Properly free commit a commit list in revwalk

The commit list was not being properly free'd when a walk was stopped
halfway through.

Revert changes in wscript file

Add proper threading support to libgit2

We now depend on libpthread on all Unix platforms (should be installed
by default) and use a simple wrapper for Windows threads under Win32.

Signed-off-by: Vicent Marti <tanoku@gmail.com>

Do not segfault when listing unpacked references

Debug assertion when using an initialized hashtable

Fix compilation warnings in ZLib (MSVC)

Yes, we are changing the Zlib code. This is dangerous and uncool.
Fortunately, these are just some implicit casts.

Remove duplicate refs in `git_reference_listall`

Use a more sane zconf.f when building Zlib

Add nodegit to the README

Add ZLib as a built-in dependency

I don't know if this is good or bad. This lets libgit2 compile cleanly
on any platforms without any external dependencies, but adds a little
bit of bloat...

Let's test this out and see what happens.

Add test case for issue GH-86