Opened 6 months ago
Closed 6 months ago
#27090 closed defect (implemented)
Travis: enable lzma and zstd in configure, if available
Reported by: | teor | Owned by: | teor |
---|---|---|---|
Priority: | Medium | Milestone: | Tor: 0.3.5.x-final |
Component: | Core Tor/Tor | Version: | Tor: 0.3.1.5-alpha |
Severity: | Normal | Keywords: | not-just-linux, tor-ci, teor-was-assigned, 034-triage-20180328, 034-removed-20180328, 034-backport, 035-removed-20180711, fast-fix |
Cc: | dmr | Actual Points: | |
Parent ID: | #24629 | Points: | |
Reviewer: | nickm | Sponsor: |
Description
Otherwise, we can break lzma or zstd detection without knowing it.
Child Tickets
Change History (5)
comment:1 follow-up: 2 Changed 6 months ago by
Status: | assigned → needs_review |
---|---|
Version: | → Tor: 0.3.1.5-alpha |
comment:2 Changed 6 months ago by
comment:3 Changed 6 months ago by
Reviewer: | → nickm |
---|
comment:4 Changed 6 months ago by
Status: | needs_review → merge_ready |
---|
The patch (e4d7f2667c6c4660ad9663a39fbdef7d3e2961cd) looks plausible to me. Looking at the build logs on travis, I can confirm that Tor is detecting and building with zstd and lzma on OSX, and with lzma on ubuntu.
comment:5 Changed 6 months ago by
Resolution: | → implemented |
---|---|
Status: | merge_ready → closed |
Closing these because the parent (#24629) is now merged.
Note: See
TracTickets for help on using
tickets.
It turns out that we were never building with zstd on macOS, because the package config path wasn't set.
The fix will be in my #24629 branches.