Merge pull request #3030 from linquize/symlink_supported

If work_dir is not specified, use repo_dir to test if symlink is supported

Merge branch 'pr/3035'

Merge pull request #3038 from pks-t/fix-regcomp-retval-check

Fix checking of return value for regcomp.

Fix checking of return value for regcomp.

The regcomp function returns a non-zero value if compilation of
a regular expression fails. In most places we only check for
negative values, but positive values indicate an error, as well.
Fix this tree-wide, fixing a segmentation fault when calling
git_config_iterator_glob_new with an invalid regexp.

Merge pull request #3033 from pks-t/describe-oid-fallback

Fix describe without refs and OID fallbacks

ISO C90 forbids mixed declarations and code

GIT_UNUSED() macro generates code therefore it should be used after
variable declarations.

describe: only abort without tags if fallback is not allowed.

When no reference names could be found we did error out when trying to describe
a commit. This is wrong, though, when the option to fall back to a commit's
object ID is set.

describe example: enable building by default.

Test: Create repo in while current dir is readonly and checkout symlink

For bare repository, use repo_dir to test if symlinks are supported

Merge pull request #3031 from git-up/git_index_add

Entry argument passed to git_index_add_frombuffer() should be const

Entry argument passed to git_index_add_frombuffer() should be const

Merge pull request #3022 from ethomson/gitignore_vim

gitignore: ignore vim swapfiles

gitignore: ignore vim swapfiles

Merge pull request #3011 from ethomson/filter_zero_tempbuf

Clear temporary buffer when filtering

filter: clear the temp_buf if we're using one

If we are using a temporary buffer for filtering, be sure to clear
it before using it, in case the file that we are filtering is empty.

Illustrate bad checkout on Windows

Merge pull request #3017 from ethomson/leaks

squash some leaks

squash some leaks

Merge pull request #2990 from leoyanggit/custom_param

Add a custom param to git_smart_subtransport_definition

Merge pull request #2986 from tkelman/mingw_winhttp

WinHTTP for MinGW

Merge pull request #2947 from libgit2/cmn/notes-buf

note: use a git_buf to return the default namespace

Merge pull request #2858 from pks-t/remote-squashed

examples: add remote example.

Merge pull request #3010 from ethomson/checkout_invalid_path_err

checkout: report correct invalid path

checkout: report correct invalid path

git: make sure to close the network stream

In case of a bad url or other error during the connection setup, we
close the stream via free.

Merge pull request #3003 from shawnl/master

fix changelog

fix changelog

Merge pull request #3000 from libgit2/vmg/mkdir-ext

mkdir-ext: Assume directories don't exist; fix all race cases

mkdir-ext: Retry lstat on EEXIST race

Use swprintf_s everywhere except mingw.org

examples: add remote example.

Merge pull request #2996 from ethomson/dll_comments

win32 resource: allow custom comments field in DLL

win32 resource: allow custom comments field in DLL

Merge pull request #2967 from jacquesg/merge-whitespace

Allow merges of files (and trees) with whitespace problems/fixes

Add a custom param to git_smart_subtransport_definition

The smart transport has already take the payload param. For the
sub transport a payload param is useful for the implementer.

Merge pull request #2993 from ethomson/clar_update

clar: update to 08f434d

Merge pull request #2955 from git-up/update_tips_fixes

Update tips fixes

Merge remote-tracking branch 'ethomson/submodule_8dot3'

Merge pull request #2985 from claudiuolteanu/patch-1

Lower case the include directive of windows header

Merge pull request #2983 from jeffhostetler/jeffhostetler/perf_merge_lazy_binary_check

PERF: In MERGE, lazily compute is_binary

clar: update to 08f434d

Merge pull request #2991 from sba1/fix-uninitialized-data-access-remote-update-tips

Initialize refs vector in git_remote_update_tips().

Initialize refs vector in git_remote_update_tips().

Otherwise, bailing out early when ls_to_vector() fails accesses
uninitialized memory.

note: use a git_buf to return the default namespace

The caller has otherwise no way to know how long the string will be
allocated or ability to free it.

This fixes #2944.

set PKG_CONFIG_LIBDIR in toolchain-mingw32.cmake

Added changelog entries for whitespace and patience merges.

Merge pull request #2962 from libgit2/cmn/reflog-annotated

Add annotated versions of ref-modying functions

Merge pull request #2972 from libgit2/cmn/pack-objects-walk

[WIP] Smarter pack-building

Merge pull request #2982 from libgit2/cmn/stream-check-ec

Don't ask for a stream's certificate unless it's encrypted

Merge pull request #2977 from pks-t/submodule-sync-url-fix

Fix git_submodule_sync writing URL to wrong key.

use a different .def file for 64 bit

Lower case the include directive of windows header

Since the Linux platform has a case sensitive file system, the header name should be lower case for cross compiling purposes. (On Linux, the mingw header is called ```windows.h```).

Fix out-of-tree build

link against crypt32 for CertFreeCertificateContext

Fix build on mingw-w64

patch so mingw-w64 can build

Win32: Enable WinHTTP for MinGW

PERF: In MERGE, lazily compute is_binary

http: do not try to use the cert callback on unencrypted streams

When the user has a certificate check callback set, we still have to
check whether the stream we're using is even capable of providing a
certificate.

In the case of an unencrypted certificate, do not ask for it from the
stream, and do not call the callback.

Add failing test for cert callback with non-ecrypted stream

When we have an HTTP stream and have set the certificatre check
callback, we currently fail as we ask the unencrypted stream for its
certificate.

CHANGELOG: add notes about the annotated ref-modifying functions

Add tests for the annotated versions of ref-modifying functions

This also brings the soft-reset tests back to life. The function name
was missing an underscore, meaning they had not been running.

annotated_commit: provide a constructor from a revspec

This extra constructor will be useful for the annotated versions of
ref-modifying functions, as it allows us to create a commit with the
extended sha syntax which was used to retrieve it.

Add annotated commit versions of reflog-modifying functions

We do not always want to put the id directly into the reflog, but we
want to speicfy what a user typed. For this use-case we provide
annotated version of a few functions which let the caller specify what
user-friendly name was used when asking for the operation.

Added options to enable patience and minimal diff drivers

Collapse whitespace flags into git_merge_file_flags_t

Renamed git_merge_options 'flags' to 'tree_flags'

Added tests to merge files and branches with whitespace problems and fixes

Allow for merges with whitespace discrepancies

Merge pull request #2975 from git-up/diff_index_retain

Avoid retaining / releasing the index more than necessary when GIT_DIFF_...

indexer: bring back the error message on duplcate commits

It turns out that erroring out on duplicate commits is the right thing
to do, but git was not hitting the bug on the server-side.

Bring back a descriptive error message in case of duplicate entries and
error out.

indexer: don't worry about duplicate objects

If a packfile includes duplicate objects, we can choose to use the
secon copy instead of the first by using the same logic as if it were
the first.

Change the error condition from 0 to -1, which indicates a bad resize,
and set the OOM message in that case.

This does mean we will leak the first copy of the object. We can deal
with that later, but making fetches work is more important.

indexer: set an error message on duplicate objects in pack

While this is not even close to a fix, we can at least set an error
message so we know which error we are facing. Up to know we just
returned an error without a message.

Put back the number of expected references to 6 from the test repo

This was but down to 5 when GitHub made a change to their server which
made them stop honouring the include-tag request.

This has recently been corrected, so we can bring it back up to six.

Merge pull request #2973 from libgit2/cmn/local-pack-threading

local: create pack with multiple threads

Fix git_submodule_sync writing URL to wrong key.

Currently git_submodule_sync writes the submodule's URL to the
key 'branch.<REMOTE_NAME>.remote' while the reference
implementation of `git submodule sync` writes to
'remote.<REMOTE_NAME>.url', which is the intended behavior
according to git-submodule(1).

Avoid retaining / releasing the index more than necessary when GIT_DIFF_UPDATE_INDEX is enabled

Fixed active refspecs not reset by git_remote_upload()

Fixed update_tips callback called for failed pushed references

The current implementation does not set 'fire_callback' back to 0 for failed updates so the callback still fires.

Instead of adding yet another condition check to set 'fire_callback' to 0 if needed, considering this function should be a no-op for failed updates anyway, the best fix is to simplify its logic to check upfront if the update is a failed one.

Merge commit 'refs/pull/2879/head' of ssh://github.com/libgit2/libgit2

local: create pack with multiple threads

The default behaviour for the packbuilder is to perform the work in a
single thread, which is fine for the public API, but we currently have
no way for a user to determine the number of threads to use when
creating the packfile, which makes our clone behaviour over the
filesystem quite a bit slower than what git offers.

This is a very particular scenario, in which we avoid spawning git by
being ourselves the server-side, so it's probably ok to auto-set the
threading, as the upload-pack process would do if we were talking to
git.

Merge pull request #2969 from adrienthebo/remote_add_invalid_refspec

refspec: report errors when parsing an invalid refspec

Merge pull request #2970 from ethomson/inmemory_bare

"In-memory" repos are bare by default

local: let the packbuilder perform smarter object insertion

Currently we use the most naïve and inefficient method for figuring out
which objects to send to the remote whereby we end up trying to insert
subdirs which have not changed multiple times.

Instead, make use of the packbuilder's built-in more efficient method
which uses the walk to feed the object list and avoids inserting an
object and its descendants.

pack-objects: fill a packbuilder from a walk

Most use-cases for the object packer communicate in terms of commits
which each side has. We already have an object to specify this
relationship between commits, namely git_revwalk.

By knowing which commits we want to pack and which the other side
already has, we can perform similar optimisations to git, by marking
each tree as interesting or uninteresting only once, and not sending
those trees which we know the other side has.

Reorder some khash declarations

Keep the definitions in the headers, while putting the declarations in
the C files. Putting the function definitions in headers causes
them to be duplicated if you include two headers with them.

Merge pull request #2971 from ethomson/revert_fastfail

Revert "fastfail" from the mingw PR

repository_new: test its bareness

refspec: set err message on invalid refspec

If a refspec could not be parsed, the git_refspec__parse function would
return an error value but would not provide additional error information
for the callers. This commit amends that function to set a more useful
error message.

Revert "add fast-fail feature for pending builds in PR's"

This reverts commit 8008ab6a4158416d095212c9782fa1622ef994a5.

Revert "Implement fail-fast for Travis as well"

This reverts commit ba6c53b91b1d9c9dc49b7c0d8e8efbd9842c31dc.

repository: in-memory repos are bare by default

Merge pull request #2964 from git-up/fixes

Fixes

Merge pull request #2946 from tkelman/appveyor-mingw

RFC: add mingw to appveyor matrix

Removed unnecessary GIT_CHECKOUT_SKIP_UNMERGED for GIT_RESET_HARD

Fixed build warning

Merge pull request #2963 from libgit2/cmn/rename-reflog

refdb: use the same id for old and new when renaming a reference

refdb: use the same id for old and new when renaming a reference

When we rename a reference, we want the old and new ids to be the same
one (as we did not change it). The normal code path looks up the old id
from the current value of the brtanch, but by the time we look it up, it
does not exist anymore and thus we write a zero id.

Pass the old id explicitly instead.

Merge pull request #2961 from ethomson/filter_relative_paths

Filter relative paths

Merge pull request #2950 from libgit2/ntk/reflog_branch_create

branch: fix generated reflog message upon creation