summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | win32::longpath: don't print pathlongpath-printfEdward Thomson2015-09-281-2/+0
|/ / /
* | | Merge pull request #3441 from libgit2/cmn/badsslEdward Thomson2015-09-281-0/+27
|\ \ \ | | | | | | | | net: add tests against badssl.com
| * | | net: add tests against badssl.comcmn/badsslCarlos Martín Nieto2015-09-271-0/+27
| | | | | | | | | | | | | | | | | | | | These provide bad X.509 certificates, which we should refuse to connect to by default.
* | | | Plug some leaksCarlos Martín Nieto2015-09-273-0/+10
|/ / /
* | | Merge pull request #3439 from bisho/masterEdward Thomson2015-09-252-8/+15
|\ \ \ | | | | | | | | Fix binary diffs
| * | | Fix binary diffsGuille -bisho-2015-09-252-8/+15
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git expects an empty line after the binary data: literal X ...binary data... <empty_line> The last literal block of the generated patches were not containing the required empty line. Example: diff --git a/binary_file b/binary_file index 3f1b3f9098131cfecea4a50ff8afab349ea66d22..86e5c1008b5ce635d3e3fffa4434c5eccd8f00b6 100644 GIT binary patch literal 8 Pc${NM&PdElPvrst3ey5{ literal 6 Nc${NM%g@i}0ssZ|0lokL diff --git a/binary_file2 b/binary_file2 index 31be99be19470da4af5b28b21e27896a2f2f9ee2..86e5c1008b5ce635d3e3fffa4434c5eccd8f00b6 100644 GIT binary patch literal 8 Pc${NM&PdElPvrst3ey5{ literal 13 Sc${NMEKbZyOexL+Qd|HZV+4u- git apply of that diff results in: error: corrupt binary patch at line 9: diff --git a/binary_file2 b/binary_file2 fatal: patch with only garbage at line 10 The proper formating is: diff --git a/binary_file b/binary_file index 3f1b3f9098131cfecea4a50ff8afab349ea66d22..86e5c1008b5ce635d3e3fffa4434c5eccd8f00b6 100644 GIT binary patch literal 8 Pc${NM&PdElPvrst3ey5{ literal 6 Nc${NM%g@i}0ssZ|0lokL diff --git a/binary_file2 b/binary_file2 index 31be99be19470da4af5b28b21e27896a2f2f9ee2..86e5c1008b5ce635d3e3fffa4434c5eccd8f00b6 100644 GIT binary patch literal 8 Pc${NM&PdElPvrst3ey5{ literal 13 Sc${NMEKbZyOexL+Qd|HZV+4u-
* | | Merge pull request #3437 from libgit2/cmn/plug-smCarlos Martín Nieto2015-09-241-13/+21
|\ \ \ | | | | | | | | submodule: plug a few leaks
| * | | submodule: plug a few leakscmn/plug-smCarlos Martín Nieto2015-09-241-13/+21
| | | |
* | | | Merge pull request #3435 from ethomson/nametoolongCarlos Martín Nieto2015-09-232-8/+77
|\ \ \ \ | |/ / / |/| | | win32: give better error message on long paths
| * | | win32: test checkout msg on long path errEdward Thomson2015-09-231-0/+62
| | | |
| * | | win32: propogate filename too long errorsEdward Thomson2015-09-221-8/+15
|/ / /
* | | Merge pull request #3434 from ethomson/reservednamesCarlos Martín Nieto2015-09-2124-4/+89
|\ \ \ | | | | | | | | Win32 Reserved names: don't reserve names outside the working directory
| * | | repository: only reserve repo dirs in the workdirEdward Thomson2015-09-181-4/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check that the repository directory is beneath the workdir before adding it to the list of reserved paths. If it is not, then there is no possibility of checking out files into it, and it should not be a reserved word. This is a particular problem with submodules where the repo directory may be in the super's .git directory.
| * | | repo::reservedname: test a submodule updateEdward Thomson2015-09-1823-0/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Test an initial submodule update, where we are trying to checkout the submodule for the first time, and placing a file within the submodule working directory with the same name as the submodule (and consequently, the same name as the repository itself).
* | | | Merge pull request #3433 from libgit2/cmn/config-commentEdward Thomson2015-09-182-7/+43
|\ \ \ \ | | | | | | | | | | Keep config comments in the same place as git
| * | | | config: buffer comments to match git's variable-addingcmn/config-commentCarlos Martín Nieto2015-09-181-4/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When there is a comment at the end of a section, git keeps it there, while we write the new variable right at the end. Keep comments buffered and dump them when we're going to output a variable or section, or reach EOF. This puts us in line with the config files which git produces.
| * | | | config: test that comments are left as with gitLinquize2015-09-181-3/+8
|/ / / /
* | | | Fix a couple of warningsCarlos Martín Nieto2015-09-182-2/+1
| | | |
* | | | Merge pull request #3418 from DimStar77/masterCarlos Martín Nieto2015-09-182-2/+20
|\ \ \ \ | | | | | | | | | | pkg-config: fix directory references in libgit2.pc
| * | | | pkg-config: fix directory references in libgit2.pcDominique Leuenberger2015-09-182-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before: libdir=/usr//usr/lib64 includedir=/usr//usr/include After: libdir=/usr/lib64 includedir=/usr/include (note the duplication of /usr in the before case)
* | | | | Merge pull request #3432 from ethomson/mkdirCarlos Martín Nieto2015-09-1725-172/+365
|\ \ \ \ \ | |_|/ / / |/| | | | `mkdir`: split into `mkdir` and `mkdir_relative`
| * | | | `mkdir`: cope with root path on win32Edward Thomson2015-09-172-4/+5
| | | | |
| * | | | win32: return EACCES in `p_lstat`Edward Thomson2015-09-171-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't coalesce all errors into ENOENT. At least identify EACCES. All callers should be handling this case already, as the POSIX `lstat` will return this.
| * | | | mkdir: chmod existing paths with `GIT_MKDIR_CHMOD`Edward Thomson2015-09-171-42/+49
| | | | |
| * | | | mkdir: find component paths for mkdir_relativeEdward Thomson2015-09-172-51/+142
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `git_futils_mkdir` does not blindly call `git_futils_mkdir_relative`. `git_futils_mkdir_relative` is used when you have some base directory and want to create some path inside of it, potentially removing blocking symlinks and files in the process. This is not suitable for a general recursive mkdir within the filesystem. Instead, when `mkdir` is being recursive, locate the first existent parent directory and use that as the base for `mkdir_relative`.
| * | | | core::mkdir tests: ensure we don't stomp symlinks in mkdirEdward Thomson2015-09-173-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | In `mkdir` and `mkdir_r`, ensure that we don't try to remove symlinks that are in our way.
| * | | | core::mkdir tests: include absolute mkdirsEdward Thomson2015-09-171-0/+35
| | | | |
| * | | | git_futils_mkdir_*: make a relative-to-base mkdirEdward Thomson2015-09-1722-104/+105
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Untangle git_futils_mkdir from git_futils_mkdir_ext - the latter assumes that we own everything beneath the base, as if it were being called with a base of the repository or working directory, and is tailored towards checkout and ensuring that there is no bogosity beneath the base that must be cleaned up. This is (at best) slow and (at worst) unsafe in the larger context of a filesystem where we do not own things and cannot do things like unlink symlinks that are in our way.
* | | | Merge pull request #3429 from ethomson/checkout_chmodCarlos Martín Nieto2015-09-162-12/+87
|\ \ \ \ | | | | | | | | | | Checkout: handle mode changes
| * | | | checkout: overwrite files with differing modesEdward Thomson2015-09-162-10/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a file exists on disk and we're checking out a file that differs in executableness, remove the old file. This allows us to recreate the new file with p_open, which will take the new mode into account and handle setting the umask properly. Remove any notion of chmod'ing existing files, since it is now handled by the aforementioned removal and was incorrect, as it did not take umask into account.
| * | | | checkout::tree tests: don't use hardcoded modeEdward Thomson2015-09-161-8/+8
| | | | |
| * | | | checkout::tree tests: don't use static bufferEdward Thomson2015-09-161-6/+9
| | | | |
| * | | | Check that checkout preserves filemode in working directory.Matti Virolainen2015-09-161-0/+63
| | | | |
| * | | | Check that an executable in index is not an executable after checkout.Matti Virolainen2015-09-161-0/+11
|/ / / /
* | | | Merge pull request #3428 from ethomson/clone_test_bufferEdward Thomson2015-09-142-3/+8
|\ \ \ \ | | | | | | | | | | Clone test buffer
| * | | | cl_git_path_url: assert sane static buffer sizeEdward Thomson2015-09-141-0/+2
| | | | |
| * | | | clone::nonetwork: don't use fixed size bufferEdward Thomson2015-09-141-3/+6
|/ / / /
* | | | Merge pull request #3425 from ethomson/diriter_rootCarlos Martín Nieto2015-09-132-2/+39
|\ \ \ \ | | | | | | | | | | Handle `git_path_diriter` instances at the drive root on Windows
| * | | | diriter: don't double '/' on posixEdward Thomson2015-09-131-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The canonical directory path of the root directory of a volume on POSIX already ends in a slash (eg, `/`). This is true only at the root. Do not add a slash to paths in this case.
| * | | | diriter: don't double '/' on WindowsEdward Thomson2015-09-131-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The canonical directory path of the root directory of a volume on windows already ends in a slash (eg, `c:/`). This is true only at the volume root. Do not add a slash to paths in this case.
| * | | | diriter: test we can iterate rootEdward Thomson2015-09-131-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensure that we can iterate the filesystem root and that paths come back well-formed, not with an additional '/'. (eg, when iterating `c:/`, expect that we do not get some path like `c://autoexec.bat`).
* | | | | Merge pull request #3426 from ethomson/fs_iteratorCarlos Martín Nieto2015-09-131-35/+50
|\ \ \ \ \ | | | | | | | | | | | | iterator: loop fs_iterator advance (don't recurse)
| * | | | | iterator: loop fs_iterator advance (don't recurse)Edward Thomson2015-09-131-35/+50
|/ / / / /
* | | | | Merge pull request #3423 from libgit2/cmn/push-tests-inline-oidCarlos Martín Nieto2015-09-133-11/+6
|\ \ \ \ \ | | | | | | | | | | | | push: put the git_oid inline in the test structure
| * | | | | push: put the git_oid inline in the test structurecmn/push-tests-inline-oidCarlos Martín Nieto2015-09-133-11/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are small pieces of data, so there is no advantage to allocating them separately. Include the two ids inline in the struct we use to check that the expected and actual ids match.
* | | | | | Merge pull request #3420 from ethomson/iteratorCarlos Martín Nieto2015-09-131-42/+56
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | iterator: advance the tree iterator smartly
| * | | | | iterator: advance the tree iterator smartlyEdward Thomson2015-09-111-42/+56
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While advancing the tree iterator, if we advance over things that we aren't interested in, then call `current`. Which may *itself* call advance. While advancing the tree iterator, if we advance over things that we aren't interested in, then call `current`. Which may *itself* call advance. While advancing the tree iterator, if we advance over things that we aren't interested in, then call `current`. Which may *itself* call advance. While advancing the tree iterator, if we advance over things that we aren't interested in, then call `current`. Which may *itself* call advance. While advancing the tree iterator, if we advance over things that we aren't interested in, then call `current`. Which may *itself* call advance. Error: stack overflow.
* | | | | Merge pull request #3424 from ↵Carlos Martín Nieto2015-09-132-1/+10
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | arthurschreiber/arthur/transaction-dont-free-config Don't free config in `git_transaction_commit`.
| * | | | | Don't free config in `git_transaction_commit`.Arthur Schreiber2015-09-132-1/+10
|/ / / / / | | | | | | | | | | | | | | | The config is not owned by the transaction, so please don’t free it.
* | | | | Merge branch 'cmn/ignore-dir-check'Carlos Martín Nieto2015-09-132-7/+26
|\ \ \ \ \