pkt: don't use strlen before we know the name is NUL-terminated

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

Move extract_host_and_port to netops and add default port argument

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

Add HTTP transport skeleton

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

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

Merge pull request #360 from belkiss/development

Fix installation of libgit2.pc file

Merge pull request #361 from nulltoken/ntk/fix/wrap-strcmp

util: Add git__strcmp_cb() wrapper

Merge pull request #362 from schu/fix-missing-initializer

mwindow.c: fix -Wmissing-field-initializers

util: Add git__strcmp_cb() wrapper

We don't want direct pointers to the CRT on Windows, we may get stdcall conflicts.

mwindow.c: fix -Wmissing-field-initializers

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

Fix installation of libgit2.pc file

configure_file outputs by default to binary dir,
and install(files) search from source dir.

discover: Make test run in temporary folder instead of altering the test resources folder

Fix compilation in Windows

transport: Wrap `strcmp`

We don't want direct pointers to the CRT on Windows, we may
get stdcall conflicts.

Merge pull request #357 from carlosmn/calling-convention

Use __stdcall by default on Windows

Merge pull request #358 from carlosmn/pkg-config

Create and install pkg-config file

Create and install pkg-config file

This hasn't been working since the switch away from waf.

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

Use __stdcall by default on Windows

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

Force the test's main function to use cdecl under Windows

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

Use the internal sort in the local transport

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

Fix "redefinition of typedef git_indexer" build error

Signed-off-by: J. David Ibáñez <jdavid@itaapy.com>

Merge pull request #353 from belkiss/development

Fix wrong test in t04-commit

Merge pull request #352 from carlosmn/delta-base-error

Check for error calculating the delta base

Check for error calculating the delta base

Don't assume that it's always going to work.

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

Fix wrong test in t04-commit
Replace all must_pass(strcmp()) by must_be_true(strcmp()==0)

Merge pull request #345 from carlosmn/gsoc2011/indexer

Implement a pack indexer

Merge pull request #349 from MasterGrumpy/development

git_status_foreach issue

Merge pull request #348 from schu/sig-new

signature.c: fix off-by-one error

signature: adjust API to return error codes

git_signature_new() and git_signature_now() currently don't return error
codes. Change the API to return error codes and not pointers to let the
user handle errors properly.

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

Document the indexer calls

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

Move pack index writing to a public function

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