Closed
Bug 1330250
Opened 7 years ago
Closed 7 years ago
Port |Bug 1329320 - Rewrite makecab in Rust| to TB
Categories
(Thunderbird :: Build Config, defect)
Thunderbird
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 53.0
People
(Reporter: jorgk-bmo, Assigned: Paenglab)
References
Details
Attachments
(1 file, 2 obsolete files)
2.52 KB,
patch
|
aleth
:
review+
|
Details | Diff | Splinter Review |
The first attempt failed https://hg.mozilla.org/comm-central/rev/af3eeb650c062fa26e209e307090f45f57af3cdd with checking for Windows SDK... 0x0a00 in c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/vs2015u3/SDK checking for Universal CRT SDK... 10.0.14393.0 in c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/vs2015u3/SDK checking for the Debug Interface Access SDK... c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/vs2015u3/DIA SDK checking for mt... c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/vs2015u3/SDK/bin/x64/mt.exe checking whether MT is really Microsoft Manifest Tool... yes checking for link... c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/vs2015u3/VC/bin/amd64_x86/link.exe checking for makecab... not found DEBUG: makecab: Trying c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/mozilla/makecab.exe ERROR: Cannot find makecab *** Fix above errors and then restart with\ "c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/mozmake.exe -f client.mk build" c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/client.mk:360: recipe for target 'configure' failed mozmake.exe[2]: Leaving directory 'c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build' Looks like more stuff needs porting here, like: https://hg.mozilla.org/mozilla-central/rev/e31b6b05ba5b#l5.13 and https://hg.mozilla.org/mozilla-central/rev/e31b6b05ba5b#l6.12 Don't the clang.manifest files also need porting?
Assignee | ||
Comment 1•7 years ago
|
||
I missed the change in mozconfig.win-common. All other should not be needed (hopefully) as they are used automatically.
Assignee | ||
Comment 2•7 years ago
|
||
This is also the file the other platforms show in check-sync-dir.
Reporter | ||
Comment 3•7 years ago
|
||
Comment on attachment 8825704 [details] [diff] [review] makecab.patch Right, we don't have clang.manifest in comm-central\mail\config\tooltool-manifests\win64 and windows.configure is also M-C only. Fingers crossed ;-) Please change the checkin comment to: Bug 1330250 - Port bug 1329320 (follow-up): export MAKECAB in mozconfig.win-common. r=jorgk (please remove space after the =).
Attachment #8825704 -
Flags: review?(jorgk) → review+
Assignee | ||
Comment 4•7 years ago
|
||
Yeah, fingers crossed $topsrcdir points to the correct position in our environment. https://hg.mozilla.org/comm-central/rev/9222773dd8c9f420566b5d7ff4a529e23211fa0f
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 53.0
Reporter | ||
Comment 5•7 years ago
|
||
(In reply to Richard Marti (:Paenglab) from comment #4) > Yeah, fingers crossed $topsrcdir points to the correct position in our > environment. Sadly not: checking for makecab... not found DEBUG: makecab: Trying c:/builds/moz2_slave/tb-c-cen-w32-d-000000000000000/build/mozilla/makecab.exe ERROR: Cannot find makecab
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•7 years ago
|
||
M-c has the makecab.exe in /build/src INFO - checking for makecab... c:/builds/moz2_slave/m-cen-w32-00000000000000000000/build/src/makecab.exe and c-c searches it in /build/mozilla DEBUG: makecab: Trying c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/mozilla/makecab.exe Aleth, do you know how we could fix this?
Flags: needinfo?(aleth)
Comment 7•7 years ago
|
||
(In reply to Richard Marti (:Paenglab) from comment #6) > M-c has the makecab.exe in /build/src > INFO - checking for makecab... > c:/builds/moz2_slave/m-cen-w32-00000000000000000000/build/src/makecab.exe > > and c-c searches it in /build/mozilla > DEBUG: makecab: Trying > c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/mozilla/makecab.exe > > Aleth, do you know how we could fix this? I think so, I can take a look later.
Flags: needinfo?(aleth)
Assignee | ||
Comment 8•7 years ago
|
||
Backed out both patches: https://hg.mozilla.org/comm-central/rev/a711acf48336796734ad2189fbc8729f378a62fb
Assignee | ||
Comment 9•7 years ago
|
||
This is what was previously checked-in as a base to work on.
Attachment #8825704 -
Attachment is obsolete: true
Comment 10•7 years ago
|
||
This will require a m-c patch to fix. Backing out the c-c patches should unblock c-c for now, albeit with yellow B's.
Comment 11•7 years ago
|
||
m-c patch up. I don't think Windows will build properly without it, if you really need to work with c-c tip on Windows, use try and apply the patch from the blocking bug.
Assignee | ||
Comment 12•7 years ago
|
||
Patch with the changes from aleth in blocking bug to be ready when it lands in m-c.
Attachment #8825739 -
Attachment is obsolete: true
Attachment #8825743 -
Flags: review?(aleth)
Comment 13•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=db4ce9b0e6b394fe0ffc454d2d9f37a16c2fa8cd
Updated•7 years ago
|
Attachment #8825743 -
Flags: review?(aleth) → review+
Assignee | ||
Comment 14•7 years ago
|
||
Aleth's fix landed in m-c. https://hg.mozilla.org/comm-central/rev/4eeb3f3d3ec60193f6c908170a202c20b649f390
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•