Closed Bug 1099852 Opened 8 years ago Closed 7 years ago

Move distributed files into dist/bin/mail instead of dist/bin


(Thunderbird :: Build Config, defect)

Not set


(Not tracked)



(Reporter: hiro, Assigned: hiro)




(4 files, 2 obsolete files)

Attached patch WIP patch (obsolete) — Splinter Review
No description provided.
Assignee: nobody → hiikezoe
Attached patch WIP v2 (obsolete) — Splinter Review
Attachment #8523404 - Attachment is obsolete: true
Depends on: 1099861
Filed a separate bug for fixing resouce paths. bug 1099861.
Attachment #8523406 - Attachment is obsolete: true
Just a question, could you add some details why this is needed aside from the failing test? Also, does this work with the new OSX signing changes?

I think a try-run of this would also be good.
Try run:

There are still lots of failures unfortunately. Most of failures are calendar. The calendar xpcshell tests passed on my local linux. But once make upload or something had done on build server, all of those tests failed. 

What I can tell is that the difference of directory structure between firefox and thunderbird will keep some test failures.

I have no idea about OSX signing changes, but I am worrying about this change would break your work. Of course I am hoping this change does not break calendar or somewhere else.
Note that the directory structure in dist/bin/xpi-stage when DIST_SUBDIR is set:

OBJDIR/dist/xpi-stage/lightning/DIST_SUBDIR/{chrome, components,...}
Regarding the OSX signing changes, it requires some mozharness changes. I don't know if packaged tests will be affected by this change directly, but if so we'd have to make more changes in bug 1084066.

Regarding the calendar failures, you will likely have to change these lines to load the correct manifest:
Philipp, thanks for the info.
head_consts.js seems to be correct for me. So now I am guessing mozharness changes are necessary unfortunately....
Depends on: 1101270
No longer blocks: 1099332
Jcranmer, this also looks like it could use your build config experience.
Flags: needinfo?(Pidgeot18)
It's not clear to me that this is useful--it appears to be related to specific test issues that were fixed by other means.
Flags: needinfo?(Pidgeot18)
(In reply to Joshua Cranmer [:jcranmer] from comment #13)
> It's not clear to me that this is useful--it appears to be related to
> specific test issues that were fixed by other means.

closing wontfix based on this comment.
Please reopen if this is incorrect
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.