Backport t18 fixes to Clay

Merge pull request #409 from nulltoken/ntk/fix/status-tests-segfault

Fix git_status_foreach tests

Merge pull request #411 from boyski/gcc4

Don't use '__attribute__ visibility' with gcc unless it's at version 4 or better

Don't use '__attribute__ visibility' with gcc unless it's at
version 4 or better.

Tabify everything

There were quite a few places were spaces were being used instead of
tabs. Try to catch them all. This should hopefully not break anything.
Except for `git blame`. Oh well.

Cleanup legal data

1. The license header is technically not valid if it doesn't have a
copyright signature.

2. The COPYING file has been updated with the different licenses used in
the project.

3. The full GPLv2 header in each file annoys me.

Add some forgotten asserts in the status tests

Fix a off-by-one error in the git_status_foreach tests

Provided the tests fail (which they should not) and the callback is invoked too many times, this prevents the tests from segfaulting.

Update Clay script

Fix Clay compilation under Win32

Rename Clay target in CMakeLists

This allows to build both test suites at the same time

Regenerate test suite

Add missing prototypes

Revert changes to t18

...Ops, I broke the old test when porting it to Clay.

Add sample "Status" clay tests

status: enhance determination of statuses for a whole directory

 - Should increase performance through usage of a walker
 - No callback invocation for unaltered entries

status: enhance determination of status for a single file

 - fix retrieval of a file status when working against a newly initialized repository
 - reduce memory pressure
 - prevents a directory from being tested

repository: Add git_repository_head()

Mark the resources in the test folder as binary to prevent unexpected line-feed conversion

Come out and Clay

Merge pull request #401 from boyski/document-git-success

Document GIT_SUCCESS

Standardized doxygen @return lines for int functions to say "GIT_SUCCESS or an error code".

Merge pull request #391 from sschuberth/development

Warning fixes

Add myself to the list of Git authors who consent

Fix an integral overflow on 64-bit

Add casts to get rid of some warnings when filling zlib structures

Get rid of a superfluous pointer cast

Fix minor indentation issues (spaces to tabs)

Fix some random size_t vs. int conversion warnings

Merge pull request #388 from lambourg/master

pull request for #387

Fix for issue #387

Fix compilation in MinGW

posix: Fix undeclared prototype

repository: Fix signed/unsigned comp.

netops: This is the proper check

Merge pull request #380 from kiryl/no-release-debug

CMakeLists: no need in split debug/release build

netops: Use the size_t, Luke

Add git_remote_new

As we no longer expose the transport functions, this is now the only
way to connect to a remote when given an URL instead of a remote name

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Don't hide the transport details

Transports shouldn't get used outside of the library, so don't expose
accessor functions.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Fix Windows compilation

Sockets on Windows are unsigned, so define a type GIT_SOCKET which is
signed or unsigned depending on the platform.

Thanks to Em for his patience with this.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

compat: Move `mkstemp` to the POSIX compat layer

network: <select.h> is not available in MinGW

Merge pull request #378 from kiryl/Wuninitialized

Wuninitialized

Merge pull request #379 from kiryl/STRLEN

Drop STRLEN() macros

pkt: use sizeof() instead of strlen() to avoid variable length array

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

Drop STRLEN() macros

There is no need in STRLEN macros. Compilers can do this trivial
optimization on its own.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

CMakeLists: no need in split debug/release build

With GNU toolchain there's no need to split debug/release build.

It's useful to have -O2 in debug envitonment since GCC show more
warnings in this case. -O2 -g works fine.

For release purpose, debug information can be stripted on later stage.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

transport_git: add missed error handling

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

Fix false positive -Wuninitialized warnings

GCC produces several -Wuninitialized warnings. Most of them can be fixed
if we make visible for gcc that git__throw() and git__rethrow() always
return first argument.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

transport: Merge bitfield

I don't think MSVC merges these automatically.

posix: Properly handle `snprintf` in all platforms

odb: Do not pass around a header when hashing

Keep sending want lines if the server doesn't anwer a flush

Some servers take a long time to answer and expect us to keep sending
want lines; otherwise they close the connection. Avoid this by waiting
for one second for the server to answer. If the timeout runs out,
treat is as a NAK and keep sending want lines.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Actually implement object negotiation

Only signal that we need a pack if we do need it and don't send a want
just because it's the first. If we don't need to download the pack,
then we can skip all of the negotiation and just return success.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Implement and bind local_send_wants

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Move negotiation to the transport

There are many ways how a transport might negotiate with the server,
so instead of making it fit into the smart protocol model, let the
transport do its thing. For now, the git protocol limits itself to
send only 160 "have" lines so we don't flood the server.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Update transport lifetime documentation

The original was written before any code was written and had nothing
to do with the way things are actually done.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Use time sorting in fetch

This is what the docs say tha we should use.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Be smarter about selecting wants

There is no need to inspect what the local repository is like. Only
check whether the objects exist locally.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Remove enum git_whn

Instead, use flags inside the git_remote_head structure.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Assert a filename in indexer creation

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Don't try to download the packfile too early

Make sure we only try to download the pack if we find the pack header
in the stream, and not if the server takes a bit longer to send us the
last NAK.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Add git_remote_update_tips

This function updates the references in the local reference storage to
match the ones in the remote.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Implement cooperative caching

When indexing a file with ref deltas, a temporary cache for the
offsets has to be built, as we don't have an index file yet. If the
user takes the responsiblity for filling the cache, the packing code
will look there first when it finds a ref delta.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Rethrow pack entry offset error

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Use strcpy+strcat as memcpy was overlapping

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Use common capabilities

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Only wait for pack if we need it

Provide the git_remote_download function to instruct the library to
downlad the packfile and let the user know the temporary location.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Tell the user where the downloaded packfile is stored

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Don't expose the fetch code to the user

Move the generation of the want-list to be done from the negotiate
function, and keep the filtered references inside the remote
structure.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Bind the configuration and remotes to a repository

Configurations when taken from a repository and remotes should be
identifiable as coming from a particular repository. This allows us to
reduce the amount of variables that the user has to keep track of.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Download pack

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Move have sending

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Implement sending haves

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Add a generic send_wants

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

negotiation

git_pkt_send_wants

Add git_fetch_list_want which creates the "want" list

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Merge pull request #375 from schu/cleanup

cleanup: some nitpicking and missing free's.

refs.c: remove two lines of dead code

Signed-off-by: schu <schu-github@schulog.org>

reflog.c: fix memory leaks

Signed-off-by: schu <schu-github@schulog.org>

git__strndup: immediately return NULL when ENOMEM

Signed-off-by: schu <schu-github@schulog.org>

config_file.c: fix memory leaks

Signed-off-by: schu <schu-github@schulog.org>

tsort.c: fix include of common.h

Signed-off-by: schu <schu-github@schulog.org>

unix/posix.h: remove redundant include

Signed-off-by: schu <schu-github@schulog.org>

Merge pull request #373 from carlosmn/fancy-branchname

Fix the reference character check for Unicode

Fix the reference character check for Unicode

We need to do an unsigned comparison, as otherwise UTF-8 characters
might look like they have the sign bit set and the check will fail.

Signed-off-by: Carlos Martín Nieto <cmn@elego.de>

Merge pull request #372 from schu/reflog-return-oid

reflog: assimilate API

reflog: fix memory leaks

Make sure to free the existing reflog when we run out or memory while
adding new entries.

Signed-off-by: schu <schu-github@schulog.org>

reflog: assimilate reflog API to return git_oid's

Rather than returning the OIDs out of the reflog as string return them
as git_oid.

Signed-off-by: schu <schu-github@schulog.org>

Merge pull request #366 from belkiss/development

Add missing GIT_BEGIN_DECL and GIT_END_DECL to indexer header

Add missing GIT_BEGIN_DECL and GIT_END_DECL to indexer header

Merge pull request #365 from carlosmn/pkg-config

Really fix pkg-config

commit: Add support for Encoding header

Really fix pkg-config

Without the @ONLY option, CONFIGURE_FILE would replace the variables
that are meant to be used to pkg-config at configure time.

Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>

Add common.h to types.h

Merge pull request #363 from schu/test-discover0

t12-repo.c: fix failing test discover0

t12-repo.c: fix failing test discover0

discover0 tried to stat a non existing directory. Create it beforehand.

Signed-off-by: schu <schu-github@schulog.org>

Merge pull request #359 from nulltoken/ntk/fix/discover-tests

Slight enhancement to the Discover test

Merge pull request #355 from jdavid/fix-build

Fix "redefinition of typedef git_indexer" build error