git_remote_create calls git_remote_save

Remote: deprecate dangling, prevent saving in-memory

Rename remote creation APIs

git_remote_add -> git_remote_create
git_remote_new -> git_remote_create_inmemory

Fix clone sample

Initialize variable

Clone: trust but verify

Add more clone options. Push test suite segfaults.

Merge pull request #1138 from ethomson/fetchhead

fetchhead reading/iterating

fetchhead reading/iterating

Merge pull request #1150 from schu/fix-netops-ssl

netops: on SSL teardown only send shutdown alert

reflog: Actual error handling

reflog: Rename error handling

Merge pull request #1159 from rick/be-consistent-be-be-consistent

Make `goto cleanup` more consistent

Merge pull request #1162 from scunz/cmake_cleanups

RFC: Some cleanups/features in CMakeLists.txt

Merge pull request #1164 from scunz/fixtestleak

Fix some leaks in tests

Fix some leaks and (possibly) dangling pointers in tests

Also adds some asserts.

clone-empty-test: Don't use one pointer for two things

... so we can clean up correctly.

revwalk-test: Don't leak the second repository

MSVC: Don't list all source files in an endless list

Instead tell MSVC to group the source files by directory.

DRY: Add function that adds os-specific libraries to our targets

Collect configuration options at the top of the file

- Also document the -DSTDCALL even better.

DRY: Don't repeat globs for libgit2's own source files

Remove special case source globs for Solaris

With the src/compat/*.c glob removed, there is no longer a difference to
the default globs we use for the IF( UNIX ) case.

Remove src/compat/*.c from source globs

This directory doesn't exist.

Search for zlib unconditional

Up to now, on windows we don't even bother to look if the user has a zlib
available somwhere.

In almost all larger commercial projects i've participated in, it was not
at all uncommon to have such a dependency somewhere in the source tree and
use it whereever required.
Searching for it, even if it's unlikely to be present, allows for such a
scenario (i.e. by prefilling the CMake-Cache).

DRY: Scan for regex.c only in one place

Only add deps/http-parser to include-dirs if required

Make goto cleanup more consistent

There may be some question about whether this is likely to be needed
at all, but that's above my head at the moment.

Merge pull request #1158 from rick/i-do-not-think-this-means-what-you-think-it-means

don't deref before we've asserted

don't deref before we've asserted

just sayin'.

Merge pull request #1157 from rick/trivial-comment-fix

Trivial comment fix

Also, whitespace.

I was totally flaunting @ben's 3-space tab advice.

Improve comment text

This looked wrong to me.  I *think* this is more appropriate
commentary.

Merge pull request #1156 from scunz/another_testcrash

Reset global variable to NULL after free'ing resource

Reset global variable to NULL after free'ing resource

Merge pull request #1151 from arrbee/fix-diff-constructor-names

Fix diff constructor names

More diff.h comment fixes

Based on feedback from the ObjectiveGit folks, these are some
further updates to diff.h areas that are poorly documented.

Fix diff constructor name order confusion

The diff constructor functions had some confusing names, where the
"old" side of the diff was coming after the "new" side.  This
reverses the order in the function name to make it less confusing.

Specifically...

* git_diff_index_to_tree becomes git_diff_tree_to_index
* git_diff_workdir_to_index becomes git_diff_index_to_workdir
* git_diff_workdir_to_tree becomes git_diff_tree_to_workdir

netops: on SSL teardown only send shutdown alert

According to man 3 SSL_shutdown / TLS, "If a unidirectional shutdown is
enough (the underlying connection shall be closed anyway), this first
call to SSL_shutdown() is sufficient."

Currently, an unidirectional shutdown is enough, since
gitno_ssl_teardown is called by gitno_close only. Do so to avoid further
errors (by misbehaving peers for example).

Fixes #1129.

netops: properly handle GITNO_CONNECT_SSL_NO_CHECK_CERT

Don't return an error just because GITNO_CONNECT_SSL_NO_CHECK_CERT is
set.

Merge pull request #1149 from nulltoken/topic/blob_isbinary

Introduce git_blob_is_binary()

Merge pull request #1148 from scunz/test_fix

Cleanup the empty-clone test correctly

blob: introduce git_blob_is_binary()

Fix MSVC compilation warnings

Cleanup the test correctly

Fix -Wmaybe-uninitialized warning

Merge pull request #1141 from ben/clone-empty-repo

Allow clone to handle empty repos

Ensure static variables are nulled after every test

Make building samples more friendly

Fix fetchhead tests

Cloning empty repos: only allow missing target for HEAD

Merge pull request #1145 from esc/feature/mailmap

adding .mailmap file

adding .mailmap file

Help clarify who are the top commiters when using 'shortlog -sn'.

Stop returning incorrect error message

Remove placeholder files during tests

Rebase fixup

Local Only ignore ENOTFOUNDs when adding corrupted refs

Allow clone to handle empty repos

Merge pull request #1143 from ben/clone-options

Options structure for git_clone

Move non-options back out of options struct

Deploy git_clone_options to network sample

Deploy git_clone_options; remove git_clone_bare

Introduce git_clone_options

travis: update the apt-get db

tests: Fix unused temp repo

test: Fix clone tests

Merge remote-tracking branch 'origin/clone-auth' into development

Merge remote-tracking branch 'jamill/transport_localpaths' into development

Fix comment so it doesn't go over 100 chars

Add test to clone with absolute path

Enable authenticated clones in network sample

Transport resolution on Win32 should handle absolute local paths

Stop premature remote freeing when cloning

Fix network example

Define constant for default fetch spec

Merge pull request #1139 from ethomson/index_bugfix

don't walk off the end of the index

Convert clone to use dangling remotes

don't walk off the end of the index

API to set a dangling remote's repository

Allow creation of dangling remotes

Merge pull request #1136 from pclouds/tree-cache-counts

tree cache: loosen negative entry count check

tree cache: loosen negative entry count check

While C Git has been writing entry count -1 (ie. never other negative
numbers) as invalid since day 1, it accepts all negative entry counts
as invalid. JGit follows the same rule. libgit2 should also follow, or
the index that works with C Git or JGit may someday be rejected by
libgit2.

Other reimplementations like dulwich and grit have not bothered with
parsing or writing tree cache.

Merge pull request #1135 from ymendel/patch-1

ignore mkmf.log

ignore mkmf.log

no build artifacts
no parents

Merge pull request #1133 from arrbee/more-iterator-cleanup

More iterator cleanup

Fix iterator reset and add reset ranges

The `git_iterator_reset` command has not been working in all cases
particularly when there is a start and end range.  This fixes it
and adds tests for it, and also extends it with the ability to
update the start/end range strings when an iterator is reset.

Clean up iterator APIs

This removes the need to explicitly pass the repo into iterators
where the repo is implied by the other parameters.  This moves
the repo to be owned by the parent struct.  Also, this has some
iterator related updates to the internal diff API to lay the
groundwork for checkout improvements.

Add git_cred_acquire_cb payload to winhttp transport

Merge pull request #1132 from ben/cred-acquire-payload

Add a payload param to git_cred_acquire_cb

Add a payload param to git_cred_acquire_cb

Fixes #1128.

Clean up GCC build warnings

Merge pull request #1126 from carlosmn/indexer-buffer

indexer: move the temporary buffers into the indexer object

Merge pull request #1127 from libgit2/fix-mwindow-threading

Fix mwindow mutex initialization and error checking

orite C89

Treat git_mutex_lock as successful when threads are disabled

Always check the result of git_mutex_lock

git__mwindow_mutex needs to be initialized even with pthreads

This could also use PTHREAD_MUTEX_INITIALIZER, but a dynamic initializer seems like a more portable concept, and we won't need another #define on top of git_mutex_init()

indexer: move the temporary buffers into the indexer object

Storing 4kB or 8kB in the stack is not very gentle. As this part has
to be linear, put the buffer into the indexer object so we allocate it
once in the heap.

Merge pull request #1091 from carlosmn/stream-object

Indexer speedup with large objects

Merge pull request #1123 from carlosmn/lax-tree

tree: relax the filemode parser