Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | mbedTLS: Fix setting certificate directoryethomson/custom_cert_locations | Zachary Michaels | 2021-08-29 | 3 | -15/+9 |
| | | | | fixes #6003 | ||||
* | opts: test GIT_OPT_SET_SSL_CERT_LOCATIONS | Edward Thomson | 2021-08-29 | 7 | -4/+176 |
| | | | | | Include a self-signed certificate for test.libgit2.org:1443 that we can use to verify that GIT_OPT_SET_SSL_CERT_LOCATIONS works. | ||||
* | ci: stop on test failure | Edward Thomson | 2021-08-29 | 1 | -0/+5 |
| | |||||
* | Merge pull request #6015 from boretrk/c89-comments | Edward Thomson | 2021-08-29 | 2 | -9/+9 |
|\ | |||||
| * | tests: change comments to c89 style | Peter Pettersson | 2021-08-29 | 2 | -9/+9 |
|/ | |||||
* | Merge pull request #5508 from libgit2/ethomson/devcontainer | Edward Thomson | 2021-08-28 | 4 | -0/+66 |
|\ | | | | | WIP: .devcontainer: settings for a codespace workflow | ||||
| * | VSCode: build in parallelethomson/devcontainer | Edward Thomson | 2021-08-02 | 1 | -1/+1 |
| | | |||||
| * | Update VSCode / Github Codespaces workflow | Edward Thomson | 2021-08-02 | 4 | -1/+64 |
| | | |||||
| * | .devcontainer: settings for a codespace workflow | Edward Thomson | 2021-08-02 | 1 | -0/+3 |
| | | | | | | | | Add devcontainer settings for running within a codespace. | ||||
* | | Merge pull request #6013 from lolgear/cmake_hash_sha_missing_header | Edward Thomson | 2021-08-28 | 1 | -0/+1 |
|\ \ | | | | | | | CMake. hash sha1 header has been added. | ||||
| * | | cmake: select hashes umbrella variable has been removed. | Dmitry Lobanov | 2021-08-28 | 1 | -2/+1 |
| | | | | | | | | | Co-authored-by: Edward Thomson <ethomson@github.com> | ||||
| * | | cmake: select hashes missing umbrella header has been added. | Dmitry Lobanov | 2021-08-28 | 1 | -0/+2 |
|/ / | |||||
* | | Merge pull request #6008 from boretrk/array | Edward Thomson | 2021-08-27 | 1 | -8/+8 |
|\ \ | | | | | | | git_array_alloc: return objects of correct type | ||||
| * | | git_array_alloc: return objects of correct type | Peter Pettersson | 2021-08-25 | 1 | -8/+8 |
| | | | |||||
* | | | Merge pull request #5747 from lhchavez/atomic-tests | Edward Thomson | 2021-08-27 | 5 | -52/+234 |
|\ \ \ | | | | | | | | | Homogenize semantics for atomic-related functions | ||||
| * | | | Homogenize semantics for atomic-related functions | lhchavez | 2021-08-26 | 5 | -52/+234 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were some subtle semantic differences between the various implementations of atomic functions. Now they behave the same, have tests and are better documented to avoid this from happening again in the future. Of note: * The semantics chosen for `git_atomic_compare_and_swap` match `InterlockedCompareExchangePointer`/`__sync_cal_compare_and_swap` now. * The semantics chosen for `git_atomic_add` match `InterlockedAdd`/`__atomic_add_fetch`. * `git_atomic_swap` and `git_atomic_load` still have a bit of semantic difference with the gcc builtins / msvc interlocked operations, since they require an l-value (not a pointer). If desired, this can be homogenized. | ||||
* | | | | Merge pull request #6010 from A-Ovchinnikov-mx/a-ovchin/remote-head-branch-clone | Edward Thomson | 2021-08-26 | 2 | -44/+33 |
|\ \ \ \ | |/ / / |/| | | | Set refs/remotes/origin/HEAD to default branch when branch is specified | ||||
| * | | | Check if default branch matches refspec | Alexander Ovchinnikov | 2021-08-26 | 1 | -0/+3 |
| | | | | |||||
| * | | | clone: set refs/remotes/origin/HEAD to default branch when branch is ↵ | Alexander Ovchinnikov | 2021-08-26 | 2 | -7/+22 |
| | | | | | | | | | | | | | | | | specified, attempt 2 | ||||
| * | | | Revert "Merge pull request #5775 from libgit2/ethomson/clone_branch" | Alexander Ovchinnikov | 2021-08-26 | 2 | -50/+21 |
|/ / / | | | | | | | | | | | | | This reverts commit 487f2a8287c4d1d41d39e05ac53d0e50c679d7e9, reversing changes made to c6cf7f0e6927f1b264a7dda2467d70b9f744a01f. | ||||
* | | | Merge pull request #6007 from boretrk/array | Edward Thomson | 2021-08-25 | 3 | -8/+9 |
|\ \ \ | |/ / | | | | array: check dereference from void * type | ||||
| * | | array: fix dereference from void * type | Peter Pettersson | 2021-08-25 | 2 | -4/+5 |
| | | | |||||
| * | | array: make gcc keep the type of the NULL return value | Peter Pettersson | 2021-08-25 | 1 | -4/+4 |
| | | | |||||
* | | | Merge pull request #6006 from boretrk/c11-warnings | Edward Thomson | 2021-08-25 | 4 | -15/+16 |
|\ \ \ | |/ / |/| | | GCC C11 warnings | ||||
| * | | win32: name the dummy union in GIT_REPARSE_DATA_BUFFER | Peter Pettersson | 2021-08-25 | 3 | -15/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | Instead of buf->"typeofbuffer"ReparseBuffer the members will be referenced with buf->ReparseBuffer."typeofbuffer" https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/ntifs/ns-ntifs-_reparse_data_buffer?redirectedfrom=MSDN calls the union DUMMYUNIONNAME but that looks a bit cluttered. | ||||
| * | | enable gcc warnings for C11 specific extensions | Peter Pettersson | 2021-08-25 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #5802 from lhchavez/git-warn-unused-result | Edward Thomson | 2021-08-25 | 14 | -35/+65 |
|\ \ \ | | | | | | | | | Introduce GIT_WARN_UNUSED_RESULT | ||||
| * | | | formatting | lhchavez | 2021-08-08 | 1 | -4/+4 |
| | | | | |||||
| * | | | Get Win32 builds to build | lhchavez | 2021-08-08 | 2 | -19/+18 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the location of `GIT_WARN_UNUSED_RESULT` was causing it to be included _after_ a bunch of other headers (namely `src/vector.h`), which broke the build. This change does two things: * Moves the `GIT_WARN_UNUSED_RESULT` above most of the `$include`s in `src/common.h`. * Stops including `vector.h` from `src/win32/path_w32.c` since the header itself does not use it. | ||||
| * | | | Move GIT_WARN_UNUSED_RESULT from the public to the private API | lhchavez | 2021-08-08 | 2 | -18/+18 |
| | | | | |||||
| * | | | Handle one more unused variable introduced since the PR started | lhchavez | 2021-08-08 | 1 | -1/+7 |
| | | | | |||||
| * | | | Add guidelines for the GIT_WARN_UNUSED_RESULT annotation | lhchavez | 2021-08-08 | 1 | -1/+12 |
| | | | | |||||
| * | | | Be a little bit less aggressive GIT_WARN_UNUSED_RESULT | lhchavez | 2021-08-08 | 1 | -9/+6 |
| | | | | | | | | | | | | | | | | | | | | Now we're limiting ourselves to only functions that allocate or acquire locks. | ||||
| * | | | Proof-of-concept for a more aggressive GIT_UNUSED() | lhchavez | 2021-08-08 | 8 | -18/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | | This adds a `-Wunused-result`-proof `GIT_UNUSED()`, just to demonstrate that it works. With this, sortedcache.h is now completely `GIT_WARN_UNUSED_RESULT`-annotated! | ||||
| * | | | Introduce GIT_WARN_UNUSED_RESULT | lhchavez | 2021-08-08 | 5 | -21/+34 |
| | |/ | |/| | | | | | | | | | | This change adds the GIT_WARN_UNUSED_RESULT annotation, which makes the compiler warn when a return result is not used. This avoids bugs. | ||||
* | | | Merge pull request #5950 from boretrk/posixtest | Edward Thomson | 2021-08-25 | 13 | -21/+44 |
|\ \ \ | |_|/ |/| | | open: input validation for empty segments in path | ||||
| * | | bugfix: don't generate paths with empty segments | Peter Pettersson | 2021-08-08 | 2 | -5/+6 |
| | | | |||||
| * | | tests: don't generate false positives on empty path segments | Peter Pettersson | 2021-08-08 | 5 | -15/+16 |
| | | | |||||
| * | | tests: optional test for p_open() with empty path segments | Peter Pettersson | 2021-08-08 | 6 | -1/+22 |
| |/ | |||||
* | | Merge pull request #6005 from boretrk/c11-warnings | Edward Thomson | 2021-08-25 | 3 | -6/+7 |
|\ \ | | | | | | | C11 warnings | ||||
| * | | c90/c99: name the unnamed union in git_hash_ctx | Peter Pettersson | 2021-08-25 | 2 | -6/+6 |
| | | | |||||
| * | | enable warnings for C11 specific extensions | Peter Pettersson | 2021-08-25 | 1 | -0/+1 |
|/ / | |||||
* | | Merge pull request #5974 from libgit2/ethomson/dlopen_ssl | Edward Thomson | 2021-08-24 | 31 | -546/+1852 |
|\ \ | | | | | | | Dynamically load OpenSSL (optionally) | ||||
| * | | ci: run dynamically loaded openssl workflows nightlyethomson/dlopen_ssl | Edward Thomson | 2021-08-24 | 1 | -0/+35 |
| | | | | | | | | | | | | | | | Add Xenial, Bionic, CentOS 7 and 8 workflows with OpenSSL-Dynamic builds nightly. | ||||
| * | | openssl: lazily load libraries when dynamically loading | Edward Thomson | 2021-08-24 | 1 | -6/+53 |
| | | | | | | | | | | | | | | | Defer dlopen until it's needed when dynamically loading OpenSSL libraries. | ||||
| * | | openssl: dynamically load on macOS | Edward Thomson | 2021-08-24 | 1 | -0/+2 |
| | | | |||||
| * | | tests: show errors when libgit2_init fails | Edward Thomson | 2021-08-24 | 1 | -1/+3 |
| | | | |||||
| * | | ntlmclient: update to ntlmclient 0.9.1 | Edward Thomson | 2021-08-24 | 17 | -346/+623 |
| | | | | | | | | | | | | The ntlmclient dependency can now dynamically load OpenSSL. | ||||
| * | | valgrind: suppress leaks in dlopen and newer libraries | Edward Thomson | 2021-08-24 | 1 | -0/+64 |
| | | | | | | | | | | | | | | | | | | dlopen sets up some thread-local state that isn't cleaned up by `dlclose`. Additionally, now that we're linking against different versions of libssh2 and OpenSSL, we're seeing different leak signatures. | ||||
| * | | openssl: dynamically load libssl and symbols (optionally) | Edward Thomson | 2021-08-24 | 11 | -84/+902 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provide an interface around OpenSSL to dynamically load the libraries and symbols, so that users can distribute a libgit2 library that is not linked directly against OpenSSL. This enables users to target multiple distributions with a single binary. This mechanism is optional and disabled by default. Configure cmake with -DUSE_HTTPS=OpenSSL-Dynamic to use it. |