Opened 7 weeks ago
Closed 5 weeks 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 7 weeks ago by
| Status: | assigned → needs_review |
|---|---|
| Version: | → Tor: 0.3.1.5-alpha |
comment:2 Changed 7 weeks ago by
comment:3 Changed 6 weeks ago by
| Reviewer: | → nickm |
|---|
comment:4 Changed 6 weeks 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 5 weeks 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.