Merge pull request #3277 from git-up/git_diff_index_to_index

Added git_diff_index_to_index()

Merge pull request #3288 from ethomson/getenv

git__getenv: utf-8 aware env reader

Merge pull request #3202 from jeffhostetler/windows_stack_trace

Stacktraces with CRTDBG memory leaks on Windows

travis: don't install CMake on OS X

Homebrew will error out because it's already installed.

travis: update the homebrew db

We need to make sure we are asking for the current version of packages,
or we might get 404s from the download service.

Update CHANGELOG with the release number

Merge pull request #3290 from libgit2/cmn/filter-doc

filter: add docs for streaming filters

filter: add docs for streaming filters

These functions are available on the public API but don't have any
documentation, so they don't appear on the API reference. Fix that.

Merge pull request #3289 from ethomson/warnings4

iterator_walk: cast away constness for free

git__getenv: utf-8 aware env reader

Introduce `git__getenv` which is a UTF-8 aware `getenv` everywhere.
Make `cl_getenv` use this to keep consistent memory handling around
return values (free everywhere, as opposed to only some platforms).

Merge pull request #3287 from ethomson/filter_test_cleanup

filter::stream: free the filter sanely

iterator_walk: cast away constness for free

filter::stream: free the filter sanely

Don't use the filter's free callback to free the actual data structure
holding the filter, as we may not always actually initialize it (the
test may be skipped).

Merge pull request #3286 from libgit2/cmn/submodule-duplicate

Correctly delimit the keys for submodule lookup

submodule: correctly delimit the keys to use for lookup

The regex we use to look at the gitmodules file does not correctly
delimit the name of submodule which we want to look up and puts '.*'
straight after the name, maching on any submodule which has the seeked
submodule as a prefix of its name.

Add the missing '\.' in the regex so we want a full stop to exist both
before and after the submodule name.

submodule: add failing test for loading the wrong submodule

When two submodules are fairly similar, we may end up loading the wrong
one.

Merge pull request #3280 from linquize/8.3

Fix 8.3 filename tests failure when 8.3 is disabled

submodule: completely remove reload_all

The function was removed, but its declaration and changelog entry about
its removal were forgotten.

The comment in the test doesn't make any sense as the function doesn't
exist anymore, so get rid of it as well.

Fix 8.3 filename tests failure when 8.3 is disabled

Merge pull request #3278 from mplough/master

Fix #3093 - remove declaration of unused function git_fetch__download…

Fix #3093 - remove declaration of unused function git_fetch__download_pack

Function was added in commit 2c982daa2eec64b80c7940bfe1142295bd72edd8 on October 5, 2011,
and removed in commit 41fb1ca0ec51ad1d2a14b911aab3215e42965d1b on October 29, 2012.
Given the length of time it's gone unused, it's safe to remove now.

Merge pull request #3273 from ethomson/warnings3

More warnings

Merge pull request #3275 from git-up/http_fix

http: fixed leak when asking for credentials again

Added git_diff_index_to_index()

http: fixed leak when asking for credentials again

t->cred might have been allocated the previous time and needs to be
freed before asking caller for credentials again.

index tests: add eol to avoid compiler warning

checkout test: mark unused vars

winhttp: remove unused var

posix compat: include sys/stat.h for mingw

diff: use size_t format

Merge pull request #3271 from jeffhostetler/more_leaks

memory leak refspec.c

fix memory leak in refspec.c on errors.

Merge pull request #3270 from ethomson/warnings2

Remove some warnings

checkout test: check getcwd return value

clar: test chdir

filter test: pass base type

examples: clean up some warnings

odb: cast to long long for printf

submodule: cast enum to int for compare

openssl: free hostname

stash: const up conflict params

index test: include repository.h for decl

stash: drop unused variable

iterator_walk: drop unused variable

Reserve aux_id 0; sort leaks by aux_id. Fix cmp.

Include stacktrace summary in memory leak output.

Merge pull request #3269 from libgit2/cmn/release-fixups

Submodule and http fixes

http: don't give up on auth on the first try

When the server rejects an authentication request, ask the caller for
the credentials again, instead of giving up on the first try.

submodule: remove trailing slashes from submodule paths

We allow looking up a submodule by path, but we lost the path
normalisation during the recent changes. Bring it back.

submodule: remove some obsolete logic

Remove some of the logic that was left-over from the time we had a cache
of submodules, plugging a leak of the submodule object in certain cases.

CHANGELOG: add submodule changes

Merge pull request #3268 from libgit2/vmg/pkg-config-sort

pkg-config: Sort the different sections

pkg-config: Sort the different sections

Because of the fact that pkg-config is pants-on-head retarded and that
the Linux linker *requires* a static library to come before all its
dynamic dependencies in the link path, calling `pkg-config --libs
--static` was generating the wrong flags for linking.

Before this patch:

-Wl,-Bsymbolic-functions -Wl,-z,relro -L/usr/local/lib -lcurl -lssh2
-lrt -lgit2 -lssl -lcrypto -ldl -lz

After this patch:

-Wl,-Bsymbolic-functions -Wl,-z,relro -L/usr/local/lib -lgit2 -lcurl
-lssh2 -lrt -lssl -lcrypto -ldl -lz

By setting the "Libs" line before all other rules, we make sure that
`-lgit2` is the first library in the link path and that it gets its
symbols resolved with the libraries coming after it.

This fix (ab)uses an implementation detail in `pkg-config` (namely, that
flags are output as they are found on the file), but this detail seems
to be stable between releases and always gives a stable output.

Bump version to 0.23.0 and SOVERSION to 23

Merge pull request #3267 from libgit2/cmn/libs-list

CMake: treat the ld flags as a list

CMake: treat the ld flags as a list

These are treated as a list by CMake itself, which means that treating
them as a simple string can put semicolons in our ld command-line if we
have libraries which are not installed on the standard locations.

Treat the variable as a CMake list and replace it with the space-delimited
list just before writing it out to our pc file.

Merge pull request #3265 from libgit2/leaks

Plug a bunch of leaks

Merge pull request #3260 from ethomson/apply_with_reflog_indices

stash: test we apply using reflog-like indices

Merge pull request #3264 from arthurschreiber/fast-builds

Fast builds?

Remove dependency installation file for travis builds.

We're installing dependencies via the APT addon now.

Always install valgrind via the apt addon.

index, iterator, fetchhead: plug leaks

checkout: plug a few leaks

diff: fix leaks in diff printing

Merge pull request #3263 from git-up/fixes

Fixes

Don't try to start ssh.

Fast builds?

stash: test we apply using reflog-like indices

Merge pull request #3259 from ethomson/stash_apply_argh

Stash apply: stage new files even when not updating the index

Merge pull request #3262 from libgit2/vmg/prefix-len

Desenfuckenate prefix lookups

Removed unused variables

Fixed build failure if GIT_CURL is not defined

test-diff-blob: Pass proper nibble sizes

revparse: Add test to make sure this doesn't regress

Revert "object: correct the expected ID size in prefix lookup"

This reverts commit 969d4b703c910a8fd045baafbcd243b4c9825316.

This was a fluke from Coverity. The length to all the APIs in the
library is supposed to be passed in as nibbles, not bytes. Passing it as
bytes would prevent us from parsing uneven-sized SHA1 strings.

Also, the rest of the library was still using nibbles (including
revparse and the odb_prefix APIs), so this change was seriously breaking
things in unexpected ways. ^^

Merge pull request #3261 from libgit2/cmn/link-one-level

pc: Put libcurl and libssh2 in Libs.private

pc: Put libcurl and libssh2 in Libs.private

Pass on to whoever wants to link to libgit2 statically the flags that we
would have used for these libraries.

Putting them in Requires.private as we do now makes pkg-config put their
dependencies in the linker arguments as well, which is not what we want.

Merge pull request #3234 from ethomson/dont_update_index_unnecessarily

Dont update index unnecessarily

Only write index if updated when passing GIT_DIFF_UPDATE_INDEX

When diffing the index with the workdir and GIT_DIFF_UPDATE_INDEX has been passed,
the previous implementation was always writing the index to disk even if it wasn't
modified.

diff: test we don't update index unnecessarily

Test that workdir diffs, when presented with UPDATE_INDEX, only
write the index when they actually make a change.

Merge pull request #3255 from libgit2/cmn/rename-unspecified

Rename FALLBACK to UNSPECIFIED

Merge pull request #3258 from arthurschreiber/patch-6

List `libcurl` in the generated `libgit2.pc`

stash: stage new files when unstashing them

Files that were new (staged additions) in the stash tree should
be staged when unstashing, even when not applying the index.

iterator: provide git_iterator_walk

Provide `git_iterator_walk` to walk each iterator in lockstep,
returning each iterator's idea of the contents of the next path.

stash: don't allow apply with staged changes

stash apply: add a newly staged file to tests

List `libcurl` in the generated `libgit2.pc`

Merge pull request #3256 from libgit2/cmn/fetch-spec-fetchhead

remote: insert refspecs with no rhs in FETCH_HEAD

Merge pull request #3246 from libgit2/cmn/dont-grow-borrowed

Don't allow growing borrowed buffers

remote: insert refspecs with no rhs in FETCH_HEAD

When a refspec contains no rhs and thus won't cause an explicit update,
we skip all the logic, but that means that we don't update FETCH_HEAD
with it, which is what the implicit rhs is.

Add another bit of logic which puts those remote heads in the list of
updates so we put them into FETCH_HEAD.

Rename FALLBACK to UNSPECIFIED

Fallback describes the mechanism, while unspecified explains what the
user is thinking.

Add CHANGELOG entries

buffer: make use of EINVALID for growing a borrowed buffer

This explains more closely what happens. While here, set an error
message.

errors: introduce EINVALID

We've been using EINVALIDSPEC for a while to mean this, but that name
is too specific. Introduce this to be more explicit.

blob: don't recomment using git_buf_grow

We currently recommend using `git_buf_grow` in order to make a buffer
make an owned copy of the memory it points to. This is not behaviour we
should encourage, so remove this recommendation.

The function itself is not changed, as we need to remain compatible, but
it will be changed not to allow usage on borrowed buffers.

buffer: don't allow growing borrowed buffers

When we don't own a buffer (asize=0) we currently allow the usage of
grow to copy the memory into a buffer we do own. This muddles the
meaning of grow, and lets us be a bit cavalier with ownership semantics.

Don't allow this any more. Usage of grow should be restricted to buffers
which we know own their own memory. If unsure, we must not attempt to
modify it.

Merge pull request #3097 from libgit2/cmn/submodule-config-state

Remove run-time configuration settings from submodules

Merge pull request #3183 from libgit2/cmn/curl-stream

Implement a cURL stream

Merge pull request #3254 from ethomson/diff-binary-patch

Handle binary DIFFABLEness properly

Merge pull request #3250 from ethomson/stash

Stash workdir correctly when added in the index, modified in the workdir