1. 06 Jan, 2022 1 commit
  2. 11 Nov, 2021 1 commit
  3. 10 Nov, 2021 1 commit
  4. 28 Oct, 2021 1 commit
  5. 17 Oct, 2021 2 commits
    • cmake: BUILD_CLAR is now BUILD_TESTS · 4e14d4c6
      Nobody knows what CLAR is.  The test building option should be
      `BUILD_TESTS`.
      Edward Thomson committed
    • docs: document `git_buf` · 5346be3d
      We have been inconsistent about the way that we handle `git_buf`s
      provided by users.  _Usually_ we require that it has been properly
      initialized with `GIT_BUF_INIT`, but _sometimes_ we simply overwrite
      the data in it regardless.  And even more rarely, we will grow a
      user-provided buffer and concatenate data onto it (see
      `git_diff_format_email`).
      
      Document the path forward for `git_buf`, which is that we always
      require that the buffer is intitialized with `GIT_BUF_INIT`.
      
      `git_diff_format_email` will be kept backward compatible but users
      are encouraged to switch to the new `git_email` APIs.
      Edward Thomson committed
  6. 27 Sep, 2021 2 commits
  7. 22 Sep, 2021 3 commits
  8. 31 Aug, 2021 1 commit
  9. 25 Jun, 2021 1 commit
  10. 23 Jun, 2021 1 commit
  11. 17 Jun, 2021 1 commit
  12. 11 Jun, 2021 1 commit
  13. 27 May, 2021 1 commit
  14. 14 Apr, 2021 1 commit
  15. 21 Nov, 2020 1 commit
  16. 25 Oct, 2020 1 commit
  17. 12 Oct, 2020 1 commit
  18. 05 Apr, 2020 1 commit
    • docs: add documentation for our coding style · ffb6a576
      For years, we've repeatedly had confusion about what our actual coding
      style is not only for newcomers, but also across the core contributors.
      This can mostly be attributed to the fact that we do not have any coding
      conventions written down. This is now a thing of the past with the
      introduction of a new document that gives an initial overview of our
      style and most important best practices for both our C codebase as well
      as for CMake.
      
      While the proposed coding style for our C codebase should be rather
      uncontroversial, the coding style for CMake might be. This can be
      attributed to multiple facts. First, the CMake code base doesn't really
      have any uniform coding style and is quite outdated in a lot of places.
      Second, the proposed coding style actually breaks with our existing one:
      we currently use all-uppercase function names and variables, but the
      documented coding style says we use all-lowercase function names but
      all-uppercase variables.
      
      It's common practice in CMake to write variables in all upper-case, and
      in fact all variables made available by CMake are exactly that. As
      variables are case-sensitive in CMake, we cannot and shouldn't break
      with this. In contrast, function calls are case insensitive, and modern
      CMake always uses all-lowercase ones. I argue we should do the same to
      get in line with other codebases and to reduce the likelihood of
      repetitive strain injuries.
      
      So especially for CMake, the proposed coding style says something we
      don't have yet. I'm fine with that, as the document explicitly says that
      it's what we want to have and not what we have right now.
      Patrick Steinhardt committed
  19. 28 Mar, 2020 1 commit
  20. 19 Feb, 2020 1 commit
  21. 15 Feb, 2020 1 commit
  22. 07 Feb, 2020 2 commits
  23. 10 Dec, 2019 1 commit
  24. 04 Dec, 2019 2 commits
  25. 13 Sep, 2019 1 commit
  26. 13 Aug, 2019 1 commit
  27. 14 Jun, 2019 1 commit
    • cmake: Modulize our TLS & hash detection · 94fc83b6
      The interactions between `USE_HTTPS` and `SHA1_BACKEND` have been
      streamlined. Previously we would have accepted not quite working
      configurations (like, `-DUSE_HTTPS=OFF -DSHA1_BACKEND=OpenSSL`) and, as
      the OpenSSL detection only ran with `USE_HTTPS`, the link would fail.
      
      The detection was moved to a new `USE_SHA1`, modeled after `USE_HTTPS`,
      which takes the values "CollisionDetection/Backend/Generic", to better
      match how the "hashing backend" is selected, the default (ON) being
      "CollisionDetection".
      
      Note that, as `SHA1_BACKEND` is still used internally, you might need to
      check what customization you're using it for.
      Etienne Samson committed
  28. 13 Jun, 2019 1 commit
    • http-parser: use our bundled http-parser by default · fb529a01
      Our bundled http-parser includes bugfixes, therefore we should prefer
      our http-parser until such time as we can identify that the system
      http-parser has these bugfixes (using a version check).
      
      Since these bugs are - at present - minor, retain the ability for users
      to force that they want to use the system http-parser anyway.  This does
      change the cmake specification so that people _must_ opt-in to the new
      behavior knowingly.
      Edward Thomson committed
  29. 27 Feb, 2019 2 commits
  30. 07 Feb, 2019 1 commit
  31. 31 Jan, 2019 1 commit
  32. 28 Jan, 2019 1 commit
  33. 25 Jan, 2019 1 commit