summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | push: report the update plan to the callercmn/negotiation-notifyCarlos Martín Nieto2015-04-197-7/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It can be useful for the caller to know which update commands will be sent to the server before the packfile is pushed up. git does this via the pre-push hook. We don't have hooks, but as it adds introspection into what is happening, we can add a callback which performs the same function.
* | | | Merge pull request #3081 from leoyanggit/build_warningsEdward Thomson2015-05-013-8/+8
|\ \ \ \ | | | | | | | | | | Fix some build warnings
| * | | | Fix some build warningsLeo Yang2015-04-283-8/+8
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In checkout.c and filter.c we were casting a sub struct to a parent struct which breaks the strict aliasing rules in C. However we can use .parent or .base to access the parent struct to avoid the build warnings. In remote.c the local variable error was not initialized or updated in some cases. For unintialized error a build warning will be generated. So always keep error variable up-to-date.
* | | | Merge pull request #3090 from libgit2/jamill/doc_updateCarlos Martín Nieto2015-04-302-1/+4
|\ \ \ \ | | | | | | | | | | Update documentation for API changes
| * | | | Update documentation for API changesjamill/doc_updateJameson Miller2015-04-302-1/+4
| | |_|/ | |/| |
* | | | Merge pull request #3088 from volftomas/patch-1Carlos Martín Nieto2015-04-301-0/+5
|\ \ \ \ | |/ / / |/| | | Fixed unused warning in tests/rebase/merge.c
| * | | Fixed unused warning in tests/rebase/merge.cTomas Paladin Volf2015-04-301-0/+5
|/ / /
* | | Merge pull request #3085 from yongthecoder/regfreeCarlos Martín Nieto2015-04-291-3/+2
|\ \ \ | |/ / |/| | Do not call regfree() on an empty regex that is not successfully created...
| * | Do not call regfree() on an empty regex that is not successfully created by ↵Yong Li2015-04-291-3/+2
|/ / | | | | | | | | | | regcomp (also removed an unused member "has_regex" from all_iter)
* | Merge pull request #3071 from linquize/git_reflog_dropEdward Thomson2015-04-241-1/+1
|\ \ | | | | | | Fix wrong format string in git_reflog_drop() error message
| * | Fix wrong format string in git_reflog_drop() error messageLinquize2015-04-241-1/+1
| | |
* | | Merge pull request #3063 from ethomson/config_validate_nameCarlos Martín Nieto2015-04-243-14/+112
|\ \ \ | | | | | | | | Validate configuration keys
| * | | config: peek returns '\n' on EOF; handle in writeEdward Thomson2015-04-231-1/+1
| | | |
| * | | config: ensure we can write to an empty fileEdward Thomson2015-04-231-0/+35
| | | |
| * | | config: validate config keysEdward Thomson2015-04-231-13/+41
| | | |
| * | | config: test that we validate the keyEdward Thomson2015-04-231-0/+35
| | | |
* | | | Merge pull request #2997 from libgit2/cmn/secure-transportEdward Thomson2015-04-2317-23/+396
|\ \ \ \ | |/ / / |/| | | Use SecureTransport on OS X
| * | | SecureTransport: handle graceful closescmn/secure-transportCarlos Martín Nieto2015-04-231-2/+3
| | | | | | | | | | | | | | | | | | | | On close, we might get a return code which looks like an error but just means that the other side closed gracefully. Handle that.
| * | | SecureTransport: require TLS v1.xCarlos Martín Nieto2015-04-231-0/+2
| | | | | | | | | | | | | | | | | | | | Anything SSL is deprecated. Let's make sure we don't try to use SSL v3 when talking to the server.
| * | | Update THREADING and CHANGELOG with SecureTransport detailsCarlos Martín Nieto2015-04-232-4/+17
| | | |
| * | | SecureTransport: allow overriding a bad certificateCarlos Martín Nieto2015-04-231-14/+41
| | | | | | | | | | | | | | | | | | | | | | | | Do not automatically fail on a bad certificate, but let the caller decide. This means we don't need our switch on errors anymore but can return a string representation from the security framework.
| * | | Rename GIT_SSL to GIT_OPENSSLCarlos Martín Nieto2015-04-2310-16/+16
| | | | | | | | | | | | | | | | | | | | This is what it's meant all along, but now we actually have multiple implementations, it's clearer to use the name of the library.
| * | | Silence unused warnings when not using OpenSSLCarlos Martín Nieto2015-04-231-0/+4
| | | |
| * | | Abstract away the TLS stream implementationCarlos Martín Nieto2015-04-233-2/+51
| | | | | | | | | | | | | | | | | | | | Instead, provide git_tls_stream_new() to ask for the most appropriate encrypted stream and use it in our HTTP transport.
| * | | Add a SecureTransport TLS channelCarlos Martín Nieto2015-04-239-6/+283
|/ / / | | | | | | | | | | | | As an alternative to OpenSSL when we're on OS X. This one can actually take advantage of stacking the streams.
* | | Merge pull request #3064 from rcorre/config-write-fixCarlos Martín Nieto2015-04-232-1/+42
|\ \ \ | | | | | | | | config_write -- handle duplicate section headers when deleting entries
| * | | git_config_delete: search until last section.Ryan Roden-Corrent2015-04-211-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If git_config_delete is to work properly in the presence of duplicate section headers, it cannot stop searching at the end of the first matching section, as there may be another matching section later. When config_write is used for deletion (value = NULL), it may only terminate when the desired key is found or there are no sections left to parse.
| * | | Specify mock config file content in test.Ryan Roden-Corrent2015-04-212-10/+11
| | | | | | | | | | | | | | | | | | | | Instead of using a config file in resources, include the config file content to be tested directly in the test.
| * | | Unittest to validate config entry deletion bug.Ryan Roden-Corrent2015-04-212-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | Add a unittest to validate bug #3043, where a duplicate empty config header could cause deletion of a config entry to fail silently. The bug is currently unresolved and this test will fail.
* | | | Merge branch 'attr-ignore'Carlos Martín Nieto2015-04-232-1/+26
|\ \ \ \
| * | | | attr_file: fix subdirectory attr case.Mike McQuaid2015-04-231-1/+8
| | | | | | | | | | | | | | | | | | | | Closes #2966.
| * | | | Add failing subdirectory gitignore attr test.Mike McQuaid2015-04-231-0/+18
| | | | |
* | | | | Merge pull request #3032 from jfultz/index-file-modesEdward Thomson2015-04-233-9/+117
|\ \ \ \ \ | | | | | | | | | | | | Fix git_checkout_tree() to do index filemodes correctly on Windows.
| * | | | | Fix index-adding functions to know when to trust filemodes.John Fultz2015-04-213-17/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The idea...sometimes, a filemode is user-specified via an explicit git_index_entry. In this case, believe the user, always. Sometimes, it is instead built up by statting the file system. In those cases, go with the existing logic we have to determine whether the file system supports all filemodes and symlinks, and make the best guess. On file systems which have full filemode and symlink support, this commit should make no difference. On others (most notably Windows), this will fix problems things like: * git_index_add and git_index_add_frombuffer() should be believed. * As a consequence, git_checkout_tree should make the filemodes in the index match the ones in the tree. * And diffs with GIT_DIFF_UPDATE_INDEX don't write the wrong filemodes. * And merges, and probably other downstream stuff now fixed, too. This makes my previous changes to checkout.c unnecessary, so they are now reverted. Also, added a test for index_entry permissions from git_index_add and git_index_add_frombuffer, both of which failed before these changes.
| * | | | | Oops...need bitwise, not logical negation.John Fultz2015-04-161-2/+2
| | | | | |
| * | | | | Fix git_checkout_tree() to do index filemodes correctly on Windows.John Fultz2015-04-062-2/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git_checkout_tree() has some fallback behaviors for file systems which don't have full support of filemodes. Generally works fine, but if a given file had a change of type from a 0644 to 0755 (i.e., you add executable permissions), the fallback behavior incorrectly triggers when writing hte updated index. This would cause a git_checkout_tree() command, even with the GIT_CHECKOUT_FORCE option set, to leave a dirty index on Windows. Also added checks to an existing test to catch this case.
* | | | | | Merge pull request #3065 from cthomas/masterCarlos Martín Nieto2015-04-222-51/+52
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix for Issue #3023 tests fail with no network
| * | | | | | Fix for Issue #3023 tests fail with no networkcthomas2015-04-212-51/+52
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moved offending tests from network to online so they will get skipped when there is a lack of network connectivity: -test_online_remotes__single_branch -test_online_remotes__restricted_refspecs
* | | | | | Merge pull request #3051 from ↵Edward Thomson2015-04-213-11/+17
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | jeffhostetler/jeffhostetler/memleak_windows_tls_data Attempt to fix Windows TLS memory leak.
| * | | | | Rename routine to free TLS dataJeff Hostetler2015-04-183-4/+9
| | | | | |
| * | | | | Remove DllMain now that TLS data freed by threadsJeff Hostetler2015-04-171-8/+0
| | | | | |
| * | | | | Also fix leak of TLS data on main thread.Jeff Hostetler2015-04-171-4/+1
| | | | | |
| * | | | | Attempt to fix Windows TLS memory leak.Jeff Hostetler2015-04-173-5/+17
| | | | | |
* | | | | | Merge pull request #2992 from ethomson/rebase_fixesCarlos Martín Nieto2015-04-218-196/+311
|\ \ \ \ \ \ | | | | | | | | | | | | | | Rebase fixes
| * | | | | | rebase: correct documentation, CHANGELOGEdward Thomson2015-04-202-9/+9
| | | | | | |
| * | | | | | rebase: include checkout opts within rebase optsEdward Thomson2015-04-203-17/+11
| | | | | | |
| * | | | | | rebase: test checkout options for rebaseEdward Thomson2015-04-201-0/+49
| | | | | | |
| * | | | | | rebase: init and open take a rebase_optionsEdward Thomson2015-04-207-214/+205
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `git_rebase_init` and `git_rebase_open` should take a `git_rebase_options` and use it for future rebase operations on that `rebase` object.
| * | | | | | rebase: take `checkout_options` where appropriateEdward Thomson2015-04-206-39/+60
| | | | | | |
| * | | | | | reset: `git_checkout_options` is `const`Edward Thomson2015-04-202-5/+5
| | | | | | |