Opened 3 years ago
Closed 3 years ago
#20217 closed enhancement (fixed)
Make sure that all .mar files required for generating incremental ones on OS X contain code-signed bits
| Reported by: | gk | Owned by: | tbb-team |
|---|---|---|---|
| Priority: | Medium | Milestone: | |
| Component: | Applications/Tor Browser | Version: | |
| Severity: | Normal | Keywords: | tbb-gitian, TorBrowserTeam201610R |
| Cc: | boklm, mcs, brade | Actual Points: | |
| Parent ID: | Points: | ||
| Reviewer: | Sponsor: |
Description
While doing the 6.5a3 release it occurred to me we need to make sure every time we create the incrementals for OS X all involved full .mar files contain the code-signed bits. Otherwise the incremental files will fail.
We should have a check for this as this is easily forgotten, especially if different people are creating the .incremental mar files.
Child Tickets
Change History (3)
comment:1 Changed 3 years ago by
| Keywords: | TorBrowserTeam201610 added; TorBrowserTeam201609 removed |
|---|
comment:2 Changed 3 years ago by
| Keywords: | TorBrowserTeam201610R added; TorBrowserTeam201610 removed |
|---|---|
| Status: | new → needs_revision |
The branch bug_20217 in my git repo has a patch for this:
https://gitweb.torproject.org/user/boklm/tor-browser-bundle.git/commit/?h=bug_20217
When generating the OSX incremental MARs from make dmg2mars or make dmg2mars-alpha, we are now checking that both the new and the old MAR files contain a file in TorBrowser.app/Contents/_CodeSignature/CodeResources.
I checked that it fails with an error when the mar file from the old version is not code signed.
comment:3 Changed 3 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | needs_revision → closed |
Looks good, thanks. Merged to master (commit 9836227e4ae27123a3eb27167dc6dca13b5d1027).
Moving tickets to October.