Fix core::env cleanup code

Mark fake home directories that failed to be created, so we won't
try to remove them and have cleanup just use p_rmdir.

Status tests...

Even more cleanups

Proper cleanup jeez

Move test cleanup into cleanup functions

Cleanup after tests

Tests should clean up after themselves

Merge branch 'development' into clar2

Conflicts:
tests-clar/clone/nonetwork.c
tests-clar/online/clone.c
tests-clar/online/fetchhead.c

Prototypes warning goes away

Remove the submodule from travis

...and add Clar raw

Remove the clar submodule

Fix network suite

/deal with it

...fine

Proper submodule dependency

Disable Network suite by default

Move some clone tests to the nonetwork suite

Merge pull request #1152 from ben/clone-api-structification

Segregate in-memory and persisted remotes

Include checkout options inline

This is a better name

Try it like this...

Remove `inmem` flag, use NULL name instead

Move `url` to last place in parameter list

Fuck you CMake

Submodule checkout

Checkout test

path: ifdef GIT_WIN32 looks_like_network_computer_name()

Add build dependency for clar. Also, fuck you CMake. Fuck you.

Bump the Clar submodule

Cleanup Clar to make it SIMPLER

Merge pull request #1177 from arrbee/update-example-showindex

Update showindex example

Update showindex example

I find the showindex example to be pretty useful on occasion, but
there were are couple of output tweaks I wanted, plus I wanted the
ability to specify a path to an actual index file instead of having
to open the whole repository.  This makes those changes and expands
the example slightly.

Merge pull request #1176 from arrbee/fix-iter-memleak

Alternative fix for iterator memory leak

Merge pull request #1171 from csware/fix-compilation-vs2012

Fixed compilation with =>VS2010

Fixed compilation with VS >= 2010

Starting with VS2010 MS ships a stdint.h.

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

Invalid ref name normalization leaked memory

When normalizing a reference name, if there is an error because
the name is invalid, then the memory allocated for storing the
name could be leaked if the caller was not careful and assumed
that the error return code meant that no allocation had occurred.

This fixes that by explicitly deallocating the reference name
buffer if there is an error in normalizing the name.

Make spoolandsort a pushable iterator behavior

An earlier change to `git_diff_from_iterators` introduced a
memory leak where the allocated spoolandsort iterator was not
returned to the caller and thus not freed.

One proposal changes all iterator APIs to use git_iterator** so
we can reallocate the iterator at will, but that seems unexpected.
This commit makes it so that an iterator can be changed in place.
The callbacks are isolated in a separate structure and a pointer
to that structure can be reassigned by the spoolandsort extension.

This means that spoolandsort doesn't create a new iterator; it
just allocates a new block of callbacks (along with space for its
own extra data) and swaps that into the iterator.

Additionally, since spoolandsort is only needed to switch the
case sensitivity of an iterator, this simplifies the API to only
take the ignore_case boolean and to be a no-op if the iterator
already matches the requested case sensitivity.

Fix warnings in example

Fix GCC static/non-static compile error

Merge pull request #1173 from nulltoken/bug/discover

Teach UNC paths to git_path_dirname_r()

path: Teach UNC paths to git_path_dirname_r()

Fix libgit2/libgit2sharp#256

path: enhance git_path_dirname_r() test coverage

remote: remove duplicated test

remote: Prevent create() from blindly overwriting

remote: Improve documentation

remote: Enhance in-memory remote test coverage

Fix indentations

Fix MSSVC compilation issue

Fix -Wmaybe-uninitialized warning

Fix examples

Rename test to make @nulltoken happy

Merge pull request #1163 from barrbrain/check-exists-before-write

odb: check if object exists before writing

In-memory remotes don't have names

odb: check if object exists before writing

Update the procondition of git_odb_backend::write.
It may now be assumed that the object has already been hashed.

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

Merge pull request #1166 from scunz/antibiotica_for_travis

Mark travis build as failed if any test fails

Mark travis build as failed if any test fails

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()