Rewrite conventions to be more complete

Markdownize CONVENTIONS

Add contributing guidelines

Merge pull request #1084 from libgit2/filename-validation

Filename validation

Add a test for invalid filenames while writing tree from index

Catch invalid filenames in append_entry()

This prevents the index api from calling write_tree() with a
bogus tree.

Prevent creating `..`, `.`, and `.git` with tree builder

As per core git.

Merge pull request #1082 from scunz/fixDoubleDeleteOnError

Indexer: Avoid a possible double-deletion in error case

Indexer: Avoid a possible double-deletion in error case

Fix a couple of warnings

Merge pull request #1077 from nulltoken/fix/api

Itsy bitsy tiny winy minor api related fixes

reflog: make entry_byindex() and drop() git compliant

Passing 0 as the index now retrieves the most recent entry instead
of the oldest one.

reflog: Fix documentation

config: Opening a nonexistent file returns ENOTFOUND

config: Make git_config_file__ondisk() internal

repo: ensure is_empty() checks there are no refs

tests: introduce cl_git_remove_placeholders()

repo: readonly tests don't need a sandboxed repo

env: ensure git_futils_find_xxx() returns ENOTFOUND

tag: rename git_tag_type to git_tag_target_type

Fix MSVC compilation warnings

Merge pull request #1075 from carlosmn/alternates-recurse

odb: recursively load alternates

odb: recursively load alternates

The maximum depth is 5, like in git

Merge pull request #1071 from arrbee/alternate-fix-strcmp

Win32 fixes for diff/checkout/reset

AUTHORS: cleanup

Fix -Wmaybe-uninitialized warning

Include Microsoft in AUTHORS

Merge pull request #1074 from edubart/ignore_diff_filemode

Add option to ignore file mode in diffs

Merge pull request #1072 from arrbee/diff-api-signatures

Diff API signatures

Add option to ignore file mode in diffs

Add explicit git_index ptr to diff and checkout

A number of diff APIs and the `git_checkout_index` API take a
`git_repository` object an operate on the index.  This updates
them to take a `git_index` pointer explicitly and only fall back
on the `git_repository` index if the index input is NULL.  This
makes it easier to operate on a temporary index.

Add iterator for git_index object

The index iterator could previously only be created from a repo
object, but this allows creating an iterator from a `git_index`
object instead (while keeping, though renaming, the old function).

Fix diff API to better parameter order

The diff API is not in the parameter order one would expect from
other libgit2 APIs.  This fixes that.

Fix reset hard tests on platforms with CRLF

The reset hard tests had hardcoded expected file content and was
not correctly compensating for CRLF filtering when a file needed
to be reverted by the reset hard.  This fixes that.

Add POSIX compat lstat() variant for win32

The existing p_lstat implementation on win32 is not quite POSIX
compliant when setting errno to ENOTDIR.  This adds an option to
make is be compliant so that code (such as checkout) that cares
to have separate behavior for ENOTDIR can use it portably.

This also contains a couple of other minor cleanups in the
posix_w32.c implementations to avoid unnecessary work.

Create internal strcmp variants for function ptrs

Using the builtin strcmp and strcasecmp as function pointers is
problematic on win32.  This adds internal implementations and
divorces us from the platform linkage.

Merge pull request #1069 from carlosmn/readme

Explain a few CMake options in the README

Explain a few CMake options in the README

Expose STDCALL and explain a few useful CMake options, as well as
explain the quirks building on Windows.

Slightly different valgrind fix

Allocate with calloc rather than conditionally memsetting a specific
part of the struct later on.

Fix a few valgrind errors

Merge pull request #1068 from carlosmn/config-empty-value

Deal with empty and nonexsitent values in config

config: return an emtpy string when there is no value

Returning NULL for the string when we haven't signaled an error
condition is counter-intuitive and causes unnecessary edge
cases. Return an empty string when asking for a string value for a
configuration variable such as '[section] var' to avoid these edge
cases.

If the distinction between no value and an empty value is needed, this
can be retrieved from the entry directly. As a side-effect, this
change stops the int parsing functions from segfaulting on such a
variable.

http: Unrustle

Merge pull request #1016 from arrbee/fix-checkout-dir-removal

Update checkout with new strategies & behavior

makefile: Fix the builtin Makefile

config: distinguish between a lone variable name and one without rhs

'[section] variable' and '[section] variable =' behave differently
when parsed as booleans, so we need to store that distinction
internally.

config: rename get_config_entry -> config_entry

We're already in the git_config namespace, there is no need to repeat
it.

Merge pull request #1055 from ethomson/sha1_win32

Win32 CryptoAPI and CNG support for SHA1

Merge pull request #1064 from libgit2/http-parser-2

Bump the builtin http-parser to v2.0

filebuf now has a git_hash_ctx instead of a ctx*

unload dll / destroy hash ctxs at shutdown

move hash library func ptrs to global global

Remove git_hash_ctx_new - callers now _ctx_init()

Win32 CryptoAPI and CNG support for SHA1

Merge pull request #1065 from nulltoken/fix/memory-leak

Fix memory leaks

Remove unused variables

test: fix memory leak

local: fix memory leak

Bump the builtin http-parser

clar-helpers: Oops, grab pointer

Merge remote-tracking branch 'ben/local-transport' into development

Merge pull request #1061 from nulltoken/topic/explicit-head-errors

repository: Refine repository_head() error report

Remove unnecessary progress logic

The indexer handles this better than the fetch
logic does.

Fix error check

repository: Refine repository_head() error report

Fix compilation warning

Merge pull request #1010 from ethomson/fetch_head

create FETCH_HEAD specially instead of as a ref file

create FETCH_HEAD specially instead of as a ref file

Merge pull request #1056 from nulltoken/duplicate-tree-entries

Duplicate tree entries

Merge pull request #1060 from nulltoken/topic/explicit-index-errors

Topic/explicit index errors

index: prefer INDEX_OWNER usage

index: refine add_from_workdir() error report

index: make git_index_new() work with a NULL path

tests: Add missing assertions

Helpers for local-filesystem remote URLs

Fix warnings and valgrind issues

This fixes some various warnings that showed up in Travis and
a couple uses of uninitialized memory and one memory leak.

Fix various cross-platform build issues

This fixes a number of warnings and problems with cross-platform
builds.  Among other things, it's not safe to name a member of a
structure "strcmp" because that may be #defined.

Update reset hard test

The `git_reset` API with the HARD option is still slightly
broken, but this test now does exercise the ability of the
command to revert modified files.

fix regression in diff with submodule oid

Rework checkout with new strategy options

This is a major reworking of checkout strategy options.  The
checkout code is now sensitive to the contents of the HEAD tree
and the new options allow you to update the working tree so that
it will match the index content only when it previously matched
the contents of the HEAD.  This allows you to, for example, to
distinguish between removing files that are in the HEAD but not
in the index, vs just removing all untracked files.

Because of various corner cases that arise, etc., this required
some additional capabilities in rmdir and other utility functions.

This includes the beginnings of an implementation of code to read
a partial tree into the index based on a pathspec, but that is
not enabled because of the possibility of creating conflicting
index entries.

Some diff refactorings to help code reuse

There are some diff functions that are useful in a rewritten
checkout and this lays some groundwork for that.  This contains
three main things:

1. Share the function diff uses to calculate the OID for a file
   in the working directory (now named `git_diff__oid_for_file`
2. Add a `git_diff__paired_foreach` function to iterator over
   two diff lists concurrently.  Convert status to use it.
3. Move all the string/prefix/index entry comparisons into
   function pointers inside the `git_diff_list` object so they
   can be switched between case sensitive and insensitive
   versions.  This makes them easier to reuse in various
   functions without replicating logic.  As part of this, move
   a couple of index functions out of diff.c and into index.c.

Move pathspec code in separate files

Diff uses a `git_strarray` of path specs to represent a subset
of all files to be processed.  It is useful to be able to reuse
this filtering in other places outside diff, so I've moved it
into a standalone set of utilities.

Make iterator ignore eval lazy

This makes it so that the check if a file is ignored will be
deferred until requested on the workdir iterator, instead of
aggressively evaluating the ignore rules for each entry.  This
should improve performance because there will be no need to
check ignore rules for files that are already in the index.

Clean up a couple things missed in rebase

Fix checkout behavior when its hands are tied

So, @nulltoken created a failing test case for checkout that
proved to be particularly daunting.  If checkout is given only
a very limited strategy mask (e.g. just GIT_CHECKOUT_CREATE_MISSING)
then it is possible for typechange/rename modifications to leave it
unable to complete the request.  That's okay, but the existing code
did not have enough information not to generate an error (at least
for tree/blob conflicts).

This led me to a significant reorganization of the code to handle
the failing case, but it has three benefits:

1. The test case is handled correctly (I think)
2. The new code should actually be much faster than the old code
   since I decided to make checkout aware of diff list internals.
3. The progress value accuracy is hugely increased since I added
   a fourth pass which calculates exactly what work needs to be
   done before doing anything.

Extensions to rmdir and mkdir utilities

* Rework GIT_DIRREMOVAL values to GIT_RMDIR flags, allowing
  combinations of flags
* Add GIT_RMDIR_EMPTY_PARENTS flag to remove parent dirs that
  are left empty after removal
* Add GIT_MKDIR_VERIFY_DIR to give an error if item is a file,
  not a dir (previously an EEXISTS error was ignored, even for
  files) and enable this flag for git_futils_mkpath2file call
* Improve accuracy of error messages from git_futils_mkdir

Merge pull request #1058 from pwkelley/development

Fixes for two segfaults

Merge pull request #1059 from nulltoken/fix/config-open-level

config: make git_config_open_level() work with an empty config

Fix a mutex/critical section leak

config: make git_config_open_level() work with an empty config

Fix uninitialized memory in winhttp subtransport on 64-bit

Fix implementation of strndup to not overrun

Avoid copying duplicate commits

Local fetch: add tests

Implement local transport's fetch

index: prevent tree creation from a non merged state

Fix libgit2/libgit2sharp#243

Merge pull request #1052 from delanne/invalid_read

Valgrind reports Invalid Read when the configuration file contains some empty line

Merge pull request #1057 from bts/patch-1

Update documentation in remote.h

Update documentation in remote.h

index: Introduce git_index_has_conflicts()