clone: deploy git_repository_set_head() usage

reset: make reset rely on git_repository_head()

branch: enhance branch moving test coverage

remote: use constants for well-known names

refs: use constants for well-known names

remote: only keep a weak pointer in update_tips

The reference is only needed inside the function. We mistakenly
increased the reference counter causing the ODB not to get freed and
leaking descriptors.

protocol: don't store flushes

Storing flushes in the refs vector doesn't let us recognize when the
remote is empty, as we'd always introduce at least one element into
it. These flushes aren't necessary, so we can simply ignore them.

clar: Proper shutdown order

Merge pull request #932 from ben/clone_pack_race

ODB: re-load packfiles on failed lookup

Merge pull request #939 from pwkelley/ignorecase

Support for the core.ignorecase flag

Merge pull request #961 from arrbee/win64-cleanups

Win64 cleanups

Fix up more Win64 compile warnings

threads: Assert that the global state is initialized

remote: Fix mid-block declaration

Merge pull request #957 from carlosmn/include-tag

Include tag

remote: support downloading all tags

Also honor remote.$name.tagopt = --tags.

remote: use the refspec functions to parse, instead of rolling our own

The local function works for simple cases, but we shouldn't reinvent
the wheel just for us.

remote: add accessors for the autotag setting

remote: create tags if we have them

Together with include-tag, this make us behave more like git. After a
fetch, try to create any tags the remote told us about for which we
have objects locally.

fetch: use the include-tag capability

This tells the remote to send us any tags that point to objects that
we are downloading.

refs: propagate EEXISTS

Indicate whether the error comes from the ref already existing or
elsewhere. We always perform the check and this lets the user write
more concise code.

refspec: add git_refspec__free, remove git_refspec_parse

The latter shouldn't be exposed and isn't used, git_refspec__parse
supersedes it.

Fix a leak in the refspec tests while we're at it.

examples: fix config getter param order

Clean up Win64 warnings

Merge pull request #959 from jamill/empty_file_hash

Fix error hashing empty file.

Merge pull request #958 from schu/fix-merge-base

revwalk: fix off-by-one error

Merge pull request #941 from arrbee/diff-separate-iterators

Create a diff patch object as a replacement for iterators

Fix error hashing empty file.

revwalk: fix off-by-one error

Fixes #921.

Add const to all shared pointers in diff API

There are a lot of places where the diff API gives the user access
to internal data structures and many of these were being exposed
through non-const pointers.  This replaces them all with const
pointers for any object that the user can access but is still
owned internally to the git_diff_list or git_diff_patch objects.

This will probably break some bindings...  Sorry!

Fix bugs in new diff patch code

This fixes all the bugs in the new diff patch code.  The only
really interesting one is that when we merge two diffs, we now
have to actually exclude diff delta records that are not supposed
to be tracked, as opposed to before where they could be included
because they would be skipped silently by `git_diff_foreach()`.
Other than that, there are just minor errors.

Initial implementation of new diff patch API

Replacing the `git_iterator` object, this creates a simple API
for accessing the "patch" for any file pair in a diff list and
then gives indexed access to the hunks in the patch and the lines
in the hunk.  This is the initial implementation of this revised
API - it is still broken, but at least builds cleanly.

New take on iterating over diff content

Allow diff deltas to be accessed by index and make patch generation
explicit with hunk and line access by index as well.

Merge pull request #947 from arrbee/public-error-set

Make giterr_set_str() and giterr_set_oom() public APIs

Merge pull request #944 from scunz/list_tags

Tags: teach git_tag_list not to include the 'refs/tags/' prefix

Merge pull request #938 from nulltoken/topic/is_valid_name

Topic/is valid name

refspec: introduce git_refspec__parse()

refs: introduce git_reference_is_valid_name()

refs: make git_reference_normalize_name() accept refspec pattern

Tests: Add test for git_tag_list to check for 'git tag -l "*bar"'

Tests: reindent object/tag/list.c to use tabs

Tests::object::tag: also test for a 'foo/*/bar'.

Tests: Add 3 tags to resources/testrepo.

Adjusts refs::list test (including the comments)
Adjusts objects::tags::list test

Tests::Object::Tag: Add a mechanism to test which tags were returned

This patch changes the tag listing test helper to use a struct as input
parameter, which tells what we exactly expect.

As I don't think, we can rely on the fact that every os and every
filesystem will report the tags in the same order, I made this code
independent of the order that the tags are retrieved.

Tests::Object::Tag: move listing tags tests to an own file

Tags: teach git_tag_list not to include the 'refs/tags/' prefix

Since quite a while now, git_branch_foreach has learnt to list branches
without the 'refs/heads/' or 'refs/remotes' prefixes.
This patch teaches git_tag_list to do the same for listing tags.

Fix -Wmaybe-uninitialized warning

Merge pull request #948 from csware/comment-fix

Make clear that git_odb_hashfile does not use filters

Make clear that git_odb_hashfile does not use filters

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Make giterr_set_str public

There has been discussion for a while about making some set of
the `giterr_set` type functions part of the public API for code
that is implementing new backends to libgit2.  This makes the
`giterr_set_str()` and `giterr_set_oom()` functions public.

Merge pull request #942 from nulltoken/topic/checkout-notify-skipped

checkout: add notification callback for skipped files

Merge pull request #943 from csware/only-close-open-filehandles

git_repository_hashfile: Only close file handle if we have a valid one

checkout: add notification callback for skipped files

git_repository_hashfile: Only close file handle if we have a valid one

Otherwise this throws an exception on MFC based systems.

Signed-off-by: Sven Strickroth <email@cs-ware.de>

Fix MSVC amd64 compilation warnings

checkout: prefer mode_t type usage over int

ODB pack: snapshot last_found to avoid race

Also removed unnecessary refresh call and fixed
some indentation.

Remove mtime checks from ODB packfile backend

Now forcing refresh on a foreach, and on missed full-oid
or short-oid lookups.

ODB: re-load packfiles on failed lookup

The old method was avoiding re-loading of packfiles by watching the mtime of the
pack directory. This causes the ODB to become stale if the directory and packfile
are written within the same clock millisecond, as when cloning a fairly small
repo.

This method tries to find the object in the cached packs, and forces a refresh when
that fails. This will cause extra stat'ing on a miss, but speeds up the success
case and avoids this race condition.

Minor fixes for ignorecase support

Support for core.ignorecase

Merge pull request #887 from nulltoken/topic/reset-hard

Checkout, reset and others enhancements

checkout: Mimic git_diff_options storage of paths

checkout : reduce memory usage when not filtering

repository: introduce git_repository_set_head()

repository: introduce git_repository_set_head_detached()

repository: introduce git_repository_detach_head()

repository: fix documentation typo

repository: separate head related tests

checkout: add test coverage of dirs and subtrees

checkout: drop git_checkout_reference()

checkout: segregate checkout strategies

checkout: separate tree from index related tests

reset: add support for GIT_RESET_HARD mode

checkout: introduce git_checkout_index()

checkout: introduce git_checkout_tree()

odb: don't overflow the link path buffer

Allocate a buffer large enough to store the path plus the terminator
instead of letting readlink write beyond the end.

Merge pull request #860 from schu/thread-safety

odb_pack: make sure to search all backends

odb_pack: fix race condition

last_found is the last packfile a wanted object was found in. Since
last_found is shared among all searching threads, it might changes while
we're searching. As suggested by @arrbee, put a copy on the stack to fix
the race condition.

Merge pull request #937 from nulltoken/fix/issue_936

refs: prevent locked refs from being enumerated

refs: prevent locked refs from being enumerated

Fix #936

config: fix Unicode BOM detection

Defining the BOM as a string makes the array include the
NUL-terminator, which means that the memcpy is going to check for that
as well and thus never match for a nonempty file.

Define the array as three chars, which makes the size correct.

Remove unnecessary include

I don't think clone.c needs in #include dirent.h and it is not
portable, so let's just get rid of it.

Merge pull request #934 from nulltoken/patch-1

Fix MSVC compilation warnings

Fix MSVC compilation warnings

Merge pull request #933 from barrbrain/pack-iterate-objects-in-offset-order

pack: iterate objects in offset order

pack: iterate objects in offset order

Compute the ordering on demand and persist until the index is freed.

http: use the new unicode functions

The winhttp branch was based on a version before these existed, so the
build broke on Windows.

Merge pull request #901 from carlosmn/winhttp

http: use WinHTTP on Windows

http: use WinHTTP on Windows

Wondows has its own HTTP library. Use that one when possible instead of
our own.

As we don't depend on them anymore, remove the http-parser library from
the Windows build, as well as the search for OpenSSL.

Merge pull request #931 from scunz/error_text

Fix error text s/buffer too long/buffer too short/

Fix error text s/buffer too long/buffer too short/

Merge pull request #929 from arrbee/diff-iter-fixes

Fix problems in diff iterator record chaining

Forgot to reset hunk & line between files

The last change tweaked the way we use the hunk_curr pointer
during iteration, but failed to reset the value back to NULL
when switching files.

Merge pull request #930 from nulltoken/fix/tracking-without-fetch-refspec

refspec: No remote tracking ref from a fetchspec-less remote

refspec: No remote tracking ref from a fetchspec-less remote

Fix problems in diff iterator record chaining

There is a bug in building the linked list of line records in the
diff iterator and also an off by one element error in the hunk
counts.  This fixes both of these, adds some test data with more
complex sets of hunk and line diffs to exercise this code better.

Merge pull request #927 from arrbee/hashfile-with-filters

Add git_repository_hashfile to hash with filters

Merge pull request #928 from barrbrain/odb-pack-stat-less

odb_pack: try lookup before refreshing packs

Fix -Wuninitialized warning