-
-
Notifications
You must be signed in to change notification settings - Fork 757
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove bundled 3rd party sw (1.4-maint) #8117
Remove bundled 3rd party sw (1.4-maint) #8117
Conversation
…8094 Took the setup.py from master branch and only slightly modified the "checksums" module path for now. This removes support for all BORG_USE_BUNDLED_*=YES env vars. "NO" has been the default since quite a while anyway. This adds support for: - using OpenSSL 3.0 on OpenBSD - locating libacl via pkgconfig Formatting changes came due to master branch using "black".
Also adapt our Cython code to directly use the lib headers, remove our wrappers.
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## 1.4-maint #8117 +/- ##
=============================================
- Coverage 82.63% 82.62% -0.01%
=============================================
Files 38 38
Lines 10966 10966
Branches 2098 2098
=============================================
- Hits 9062 9061 -1
- Misses 1343 1344 +1
Partials 561 561 ☔ View full report in Codecov by Sentry. |
I thankfully just came across this when trying to build Borg by way of Pip. I tried running
I fixed the issue and was able to build Borg 1.4.0 successfully after installing the libraries on my system, The documentation https://borgbackup.readthedocs.io/en/stable/installation.html#dependencies still says the libraries are bundled, should that be removed? By the way I love how accurate and detailed the documentation is, it helped me tremendously when building a custom Borg package for Unraid, based on Slackware. Thanks! |
@kubedzero oops, yes, that's a docs bug, the docs were not updated about this change in borg 1.4. I filed #8342 for it. |
No description provided.