Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | refactor: make util an object libraryethomson/util5 | Edward Thomson | 2021-11-11 | 57 | -635/+574 |
| | | | | | | Instead of simply including the utility files directly, make them a cmake object library for easy reusability between other projects within libgit2. | ||||
* | refactor: move utility functions into util directory | Edward Thomson | 2021-11-11 | 101 | -450/+452 |
| | |||||
* | refactor: `src` is now `src/libgit2` | Edward Thomson | 2021-11-11 | 359 | -292/+452 |
| | | | | Adding an additional hierarchy allows us to split out utility functions. | ||||
* | cmake: move sha1 source selection into CMakeLists.txt | Edward Thomson | 2021-11-11 | 2 | -11/+15 |
| | | | | | The select hashes module selects the hash; the CMakeLists.txt selects the files to implement it. | ||||
* | cmake: use a string not a boolean for LIBGIT2_FILENAME | Edward Thomson | 2021-11-11 | 1 | -1/+1 |
| | |||||
* | cmake: refactor global variables | Edward Thomson | 2021-11-11 | 12 | -46/+43 |
| | | | | | Update the global variables `LIBGIT2_OBJECTS` to `LIBGIT2_DEPENDENCY_OBJECTS` for clarity and consistency. | ||||
* | cmake: reformat modules | Edward Thomson | 2021-11-11 | 20 | -812/+689 |
| | | | | Apply the standard project cmake formatting to the modules. | ||||
* | Merge pull request #6113 from libgit2/ethomson/cmake3 | Edward Thomson | 2021-11-11 | 56 | -111/+570 |
|\ | | | | | Add missing-declarations warning globally | ||||
| * | cmake: move missing-declarations warning to top-level | Edward Thomson | 2021-11-11 | 2 | -2/+1 |
| | | | | | | | | | | We should enforce declarations throughout the code-base, including examples, fuzzers and tests, not just in the `src` tree. | ||||
| * | tests: include function declarations | Edward Thomson | 2021-11-11 | 5 | -4/+13 |
| | | |||||
| * | test: correct test names | Edward Thomson | 2021-11-11 | 3 | -3/+3 |
| | | |||||
| * | tests: declare functions statically where appropriate | Edward Thomson | 2021-11-11 | 34 | -60/+442 |
| | | |||||
| * | clar: declare summary functions as static | Edward Thomson | 2021-11-11 | 1 | -5/+5 |
| | | |||||
| * | example: declare print_usage function | Edward Thomson | 2021-11-11 | 1 | -1/+1 |
| | | |||||
| * | examples: remove unused function | Edward Thomson | 2021-11-11 | 1 | -15/+0 |
| | | |||||
| * | fuzzers: declare static functions as static | Edward Thomson | 2021-11-11 | 2 | -4/+4 |
| | | |||||
| * | fuzzers: declare standalone functions | Edward Thomson | 2021-11-11 | 9 | -2/+29 |
| | | |||||
| * | tests: declare trace handler as static | Edward Thomson | 2021-11-11 | 1 | -1/+1 |
| | | |||||
| * | tests: include test declarations for old gcc | Edward Thomson | 2021-11-11 | 1 | -1/+9 |
| | | | | | | | | | | | | | | Older versions of gcc do not believe that we've adequately declared our test functions. Include `clar_suite.h` conditionally for those old versions. Do not do this on newer compilers to avoid unnecessary recompilation of the entire suite when we add or remove a test function. | ||||
| * | clar: emit `clar_suite.h` with test declarations | Edward Thomson | 2021-11-11 | 1 | -13/+62 |
|/ | | | | | | | We may want to have test function declarations; produce a header file with (only) the test declarations. Update clar to avoid overwriting the file unnecessarily to avoid bumping timestamps and potentially recompiling unnecessarily. | ||||
* | Merge pull request #6112 from libgit2/ethomson/cmake3 | Edward Thomson | 2021-11-11 | 20 | -428/+422 |
|\ | | | | | cmake refactorings | ||||
| * | cmake: refactor zlib selectionethomson/cmake3 | Edward Thomson | 2021-11-11 | 2 | -33/+35 |
| | | | | | | | | Move zlib selection into its own cmake module. | ||||
| * | cmake: refactor WinHTTP selection | Edward Thomson | 2021-11-11 | 2 | -18/+18 |
| | | | | | | | | Move WinHTTP selection into its own cmake module. | ||||
| * | cmake: refactor libssh2 selection | Edward Thomson | 2021-11-11 | 2 | -26/+42 |
| | | | | | | | | Move SSH selection into its own cmake module. | ||||
| * | cmake: refactor regex selection | Edward Thomson | 2021-11-11 | 2 | -52/+52 |
| | | | | | | | | Move regex selection into its own cmake module. | ||||
| * | cmake: refactor http_parser selection | Edward Thomson | 2021-11-11 | 2 | -20/+21 |
| | | | | | | | | Move http_parser selection into its own cmake module. | ||||
| * | cmake: HTTP_Parser is now HTTPParser | Edward Thomson | 2021-11-11 | 2 | -1/+1 |
| | | |||||
| * | cmake: qsort detection in features.h | Edward Thomson | 2021-11-11 | 3 | -20/+15 |
| | | |||||
| * | cmake: move test enablement into test cmake | Edward Thomson | 2021-11-11 | 2 | -1/+2 |
| | | |||||
| * | cmake: reorganize file | Edward Thomson | 2021-11-11 | 1 | -13/+5 |
| | | |||||
| * | cmake: move fuzzer args to the fuzzer's cmake | Edward Thomson | 2021-11-11 | 2 | -15/+5 |
| | | |||||
| * | cmake: move deprecation definition to src/ | Edward Thomson | 2021-11-11 | 2 | -4/+4 |
| | | | | | | | | | | There's no need to add the deprecation at the top-level. Our tests add deprecation explicitly. | ||||
| * | cmake: move cflag defaults into a separate module | Edward Thomson | 2021-11-11 | 2 | -171/+171 |
| | | |||||
| * | cmake: USE_SSH is not on by default | Edward Thomson | 2021-11-11 | 3 | -21/+21 |
| | | |||||
| * | cmake: standardize USE_WINHTTP | Edward Thomson | 2021-11-11 | 3 | -4/+4 |
| | | | | | | | | | | WinHTTP can now be disabled with `USE_WINHTTP=OFF` instead of `WINHTTP=OFF` to better support the other cmake semantics. | ||||
| * | cmake: standardize USE_THREADS and USE_NSEC | Edward Thomson | 2021-11-11 | 6 | -36/+33 |
| | | | | | | | | | | | | | | | | Threading can now be disabled with `USE_THREADS=OFF` instead of `THREADSAFE=OFF` to better support the other cmake semantics. Nanosecond support is the default _if_ we can detect it. This should be our default always - like threads - and people can opt out explicitly. | ||||
* | | Merge pull request #6111 from libgit2/ethomson/str_free | Edward Thomson | 2021-11-11 | 1 | -7/+0 |
|\ \ | |/ |/| | str: git_str_free is never a function | ||||
| * | str: git_str_free is never a function | Edward Thomson | 2021-11-11 | 1 | -7/+0 |
|/ | |||||
* | Merge pull request #6104 from libgit2/ethomson/path | Edward Thomson | 2021-11-11 | 123 | -3769/+4042 |
|\ | | | | | path: refactor utility path functions | ||||
| * | win32: include correct path headerethomson/path | Edward Thomson | 2021-11-09 | 4 | -4/+4 |
| | | |||||
| * | fs_path: remove now-unused validation functions | Edward Thomson | 2021-11-09 | 2 | -68/+0 |
| | | |||||
| * | fs_path: add length with suffix validation | Edward Thomson | 2021-11-09 | 4 | -3/+30 |
| | | |||||
| * | path: use new length validation functions | Edward Thomson | 2021-11-09 | 10 | -20/+36 |
| | | |||||
| * | path: length validation respecting core.longpaths | Edward Thomson | 2021-11-09 | 3 | -18/+69 |
| | | | | | | | | | | Teach `git_path_is_valid` to respect `core.longpaths`. Add helper methods to validate length and set the error message appropriately. | ||||
| * | path: introduce `git_path_str_is_valid` | Edward Thomson | 2021-11-09 | 5 | -18/+28 |
| | | | | | | | | | | Add a `git_str` based validity check; the existing `git_path_is_valid` defers to it. | ||||
| * | fs_path: add long path validation on windows | Edward Thomson | 2021-11-09 | 3 | -4/+37 |
| | | |||||
| * | fs_path: make empty component validation optional | Edward Thomson | 2021-11-09 | 3 | -15/+38 |
| | | |||||
| * | fs_path: introduce `str_is_valid` | Edward Thomson | 2021-11-09 | 3 | -19/+81 |
| | | | | | | | | | | | | | | | | | | | | Provide a mechanism for users to limit the number of characters that are examined; `git_fs_path_str_is_valid` and friends will only examine up to `str->size` bytes. `git_fs_path_is_valid` delegates to these new functions by passing `SIZE_MAX` (instead of doing a `strlen`), which is a sentinel value meaning "look for a NUL terminator". | ||||
| * | path: `validate` -> `is_valid` | Edward Thomson | 2021-11-09 | 8 | -77/+77 |
| | | | | | | | | | | Since we're returning a boolean about validation, the name is more properly "is valid". | ||||
| * | fs_path: `validate` -> `is_valid` | Edward Thomson | 2021-11-09 | 4 | -139/+139 |
| | | | | | | | | | | Since we're returning a boolean about validation, the name is more properly "is valid". |