clay: Fix compilation under MSVC

transport-git: Encapsulation ist gut

http-transport: Properly cleanup the WSA context

http-transport: Update copyright

Merge pull request #437 from carlosmn/networking-windows

Fix networking on Windows

Include stdint.h in git2/config.h

Otherwise MSVC doesn't know what we're talking about when we say
int32_t or int64_t.

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

netops: abstract away socket closing

Winsock wants us to use closesocket() instead of close(), so introduce
the gitno_close function, which does the right thing.

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

local transport: don't segfault on wrong URL

memset the structure on initialisation and don't try to dereference
the vector with the heads if we didn't find a repository.

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

Initialise the winsock DLL

Windows wants us to initialise the networking DLL before we're allowed
to send data through a socket. Call WSASetup and WSACleanup if
GIT_WIN32 is defined.

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

Use git_buf in the git request

This is clearer and sidesteps the issue of what the return value of
snprintf is on the particular OS we're running on.

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

Merge pull request #436 from libgit2/config-int-types

config: Proper type declarations for 64 bit ints

core: One last `long long`

config: Proper type declarations for 64 bit ints

config: fix check for environment string expansion

If ExpandEnvironmentStringsW is successful, it returns the amount of
characters written, including the NUL terminator.

Thanks to Emeric for reading the MSDN documentation correctly.

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

Merge pull request #434 from schu/fix-ref-regression

refs.c: fix reference_rename

refs.c: fix reference_rename

6c8b458 removed an "unused" variable needed for git_hashtable_insert2(),
causing a segfault in reference_rename(). Instead, use
git_hashtable_insert().

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

repository: export git_repository_config_autoload

Take the opportunity to finish the comment about this function.

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

mingw: Fix compilation warnings

netops: Use pragmas only in MSVC

index: Silence type-punned warning

mingw: Fix printf identifiers

odb: Pass compression settings to filebuf

odb: Let users decide compression level for the loose ODB

http-parser: Disable MSVC warnings locally

Resync with upstream http-parser

msvc: Disable warnings in header file

Remove clutter from the CMakeLists file by disabling the warnings
programatically.

http-parser: More type changes

Change types in http-parser

Update Clay

Merge pull request #432 from carlosmn/msvc

Really fix MSVC

Really fix MSVC

These was left over from the previous PRs.

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

Fix dev branch under MSVC

In libgit2: Move an enum out of an int bitfield in the HTTP transport.

In the parser: Use int bitfields and change some variable sizes to
better fit thir use. Variables that count the size of the data chunk
can only ever be as large as off_t. Warning 4127 can be ignored, as
nobody takes it seriously anyway.

From Emeric: change some variable declarations to keep MSVC happy.

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

http-parser: Do not use bitfields

Bitfields suck. And if you make them with non-int types, they suck
in a non-standards compliant way. Like sucking sideways or something.

This commit removes all bitfields in the `http_parser` struct, and
replaces them with the minimal type needed to contain their values. Note
that the fields in the struct have been reordered so they can be packed
with 4-byte alignment.

This saves both memory on the parser (because non-int bitfields get expanded to
4byte in most compilers anyway) and time (because the fields are now
properly aligned and the compiler doesn't need to generate bit-level ops
to access them).

Make repo config loading automatic or completely explicit

git_repository_config wants to take the global and system paths again
so that one can be explicit if needed.

The git_repository_config_autoload function is provided for the cases
when it's good enough for the library to guess where those files are
located.

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

Add git_config_find_system

This allows the library to guess where the system configuration file
should be located.

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

Merge pull request #427 from schu/fix-unused-parameters

t18-status.c: fix unused warnings

tree: Fix check for valid attributes

tree: Use an internal append functiont to add new entries

Make use of the tree cache

Taking advantage of the tree cache, git_tree_create_fromindex becomes
comparable in speed to git write-tree when the cache is available.

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

Add git_tree_cache_get

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

Invalidate the path when removing from the index

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

index: invalidate added paths

When a file is updated in the index, it's path needs to be invalidated
in the tree cache as the hash is no longer correct.

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

Add git_tree_cache_invalidate_path

Whenever a file is updated in the index, each tree leading towards it
needs to be invalidated. Provide the supporting function.

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

tree cache: correctly handle invalidated trees

The fix introduced in a02fc2cd1 (2011-05-24; index: correctly parse
invalidated TREE extensions) threw out the rest of the data in the
extension if it found an invalidated entry. This was the result of
incorrect reading of the documentation.

Insted, keep reading the extension, as there may be cached data we can
use.

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

Make tree cache name a flex-array

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

Move the tree cache functions to their own file

Rename git_index_tree to git_tree_cache.

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

Revert "Rewrite getenv to use Win32 version on Windows"

This reverts commit e1b86444676b70154bf8ab450d429bdef57a8276.

Merge pull request #424 from carlosmn/access-unicode

Implment p_access and use it in git_fileutils_exists

t18-status.c: fix unused warnings

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

Implment p_access and use it in git_fileutils_exists

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

Revert changes to clay

Merge pull request #421 from nulltoken/ntk/fix/config-get-set-long

config: make git_config_[get|set]_long() able to properly deal with 8 bytes wide values

Add HTTP sources to Clay suite

Merge pull request #405 from carlosmn/http-ls

Implement ls-remote over HTTP

config: make git_config_[get|set]_long() able to properly deal with 8 bytes wide values

Should fix issue #419.

Signed-off-by: nulltoken <emeric.fermas@gmail.com>

http: get rid of the global state

Move the header parsing state into the transport, making use of the
existing bitfield.

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

Rewrite getenv to use Win32 version on Windows

Do not have duplicate filenames

Two `posix.c` files may or may not break MSVC builds under 2008. Do not
have repeated objects.

You will need to clean & regenerate CMake.

Merge pull request #415 from schu/ref-rename-regression

refs: fix git_reference_rename()

Merge pull request #413 from libgit2/utf8-paths-win32

Rewrite p_* functions to use Unicode and marshal to UTF8 internally, take 2

Merge pull request #414 from carlosmn/repo-discover-short

repsitory: use better error code if path is too short for discover

Fix clay under MinGW

Merge branch 'development' of github.com:libgit2/libgit2 into development

refs: fix git_reference_rename()

reference_rename() recently failed when renaming an existing reference
refs/heads/foo/bar -> refs/heads/foo because of a change in the
underlying functions / error codes. Fixes #412.

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

refs: add additional test for reference renaming

Signed-off-by: nulltoken <emeric.fermas@gmail.com>
Signed-off-by: schu <schu-github@schulog.org>

repsitory: use better error code if path is too short for discover

GIT_EOVERFLOW means something different. Use GIT_ESHORTBUFFER. On the
way, remove a redundant sizeof(char).

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

Fix opendir/readdir and friends on Win32 to use Unicode

Rewrite p_* functions to use Unicode and marshal to UTF8 internally

Fix tests to use portable setenv

Create cross-platform setenv

Fix `repository_config` call in network::remotes

Fix warning in `status.c`

Add remotes test to clay

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

Who makes the magic possible?

Merge pull request #384 from kiryl/warnings

Add more -W flags to CFLAGS

Merge pull request #397 from lambourg/development

Fix compilation issues with mingw64 headers

Merge pull request #394 from carlosmn/tree-fromindex

Use git_treebuilder to write the index as a tree

Merge pull request #398 from carlosmn/config-autohome

git_repository_config: open global config file automatically

Merge pull request #408 from schu/fixup-status

Fixup status.c

Merge pull request #393 from schu/unused-but-set-variable

filebuf.c: fix unused-but-set warning

Merge pull request #392 from sschuberth/development

Fix a bug and GCC warning introduced in 932669b

Merge pull request #400 from boyski/fixup-examples

Fix up examples code

Merge pull request #399 from carlosmn/free-null

Add checks for NULL to the config and remote free functions

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.

status.c: add missing check for error

dirent_cb() didn't check the return value of determine_status().

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

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.

status.c: remove wrong address operator

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

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.

Fixed typo in example Makefile code and slimmed it down more.
Reverted signature of git_signature_new.
Removed error check wrappers (voted down). Made Makefile
work out of the box on Linux and Solaris when standard
cmake build instructions for the library are followed.