update partner repack scripts to cope with new format after mac signing changes

RESOLVED FIXED

Status

Release Engineering
Release Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bhearsum, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
The layout of our apps is going to change with bug 1046306 in a way that will break existing partner builds. We need to make sure we update our scripts to deal with the new layout.

spohl, can you give the details?
Flags: needinfo?(spohl.mozilla.bugs)
The 'distribution' directory will now live in:
*.app/MozResources

The previous location was:
*.app/Contents/MacOS

See bug 1047738 for the necessary client side changes.
Flags: needinfo?(spohl.mozilla.bugs)
(Assignee)

Updated

3 years ago
Group: mozilla-employee-confidential
(Assignee)

Comment 2

3 years ago
Created attachment 8467824 [details] [diff] [review]
try to update mac partner script, untested

This looks like it should be simple. However, I'm confused about the loop at the start of this method. It seems to imply that there are files _outside_ of distribution that are used in partner builds. That doesn't seem right based on the layout of partner configs like https://github.com/mozilla/build-partner-repacks/tree/master/partners/aol/distribution. I'm guessing that it's legacy? If not, bug 1047738 will need additional work.
Assignee: nobody → bhearsum
Status: NEW → ASSIGNED
Attachment #8467824 - Flags: feedback?(mconnor)
Attachment #8467824 - Flags: feedback?(coop)

Comment 3

3 years ago
Comment on attachment 8467824 [details] [diff] [review]
try to update mac partner script, untested

Review of attachment 8467824 [details] [diff] [review]:
-----------------------------------------------------------------

::: scripts/partner-repacks.py
@@ +416,5 @@
>          for i in ['distribution', 'extensions', 'searchplugins']:
>              full_path = path.join(self.full_partner_path, i)
>              if path.exists(full_path):
>                  cp_cmd = "cp -r %s %s" % (full_path,
> +                  path.join("stage", "Firefox.app", "MozResources"))

Will we ever need to repackage in the old format, i.e. do we want some kind of global flag that would allow us to revert back to the old dir structure?
Attachment #8467824 - Flags: feedback?(coop) → feedback+
(Assignee)

Comment 4

3 years ago
(In reply to Chris Cooper [:coop] from comment #3)
> Comment on attachment 8467824 [details] [diff] [review]
> try to update mac partner script, untested
> 
> Review of attachment 8467824 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: scripts/partner-repacks.py
> @@ +416,5 @@
> >          for i in ['distribution', 'extensions', 'searchplugins']:
> >              full_path = path.join(self.full_partner_path, i)
> >              if path.exists(full_path):
> >                  cp_cmd = "cp -r %s %s" % (full_path,
> > +                  path.join("stage", "Firefox.app", "MozResources"))
> 
> Will we ever need to repackage in the old format, i.e. do we want some kind
> of global flag that would allow us to revert back to the old dir structure?

After we ship our first release with the new package layout, I don't think so. However, I think we will need to support both because we'll need the ability to chemspill while this change is on Beta.

Thanks for the catch, I'll make this possible, and we can rip out the old format later.
Group: mozilla-employee-confidential
(Assignee)

Comment 5

3 years ago
Created attachment 8473218 [details] [diff] [review]
make partner repack script with old and new formats

I tested this with the old style by repeating the steps used for the 31.0 partner repacks. Then I tested the new style by overriding the dmg that gets downloaded with one from Oak (where spohl has been pushing his patches). Everything seemed to work fine!
Attachment #8467824 - Attachment is obsolete: true
Attachment #8467824 - Flags: feedback?(mconnor)
Attachment #8473218 - Flags: review?(coop)

Updated

3 years ago
Attachment #8473218 - Flags: review?(coop) → review+
(Assignee)

Updated

3 years ago
Attachment #8473218 - Flags: checked-in+
(Assignee)

Comment 6

3 years ago
Thanks for the quick review.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
IIUC, in bug 1047738, the new home for distribution directory is Contents/Resources instead of MozResouces?

Updated

3 years ago
Depends on: 1085273
(Assignee)

Comment 8

3 years ago
(In reply to Hector Zhao [:hectorz] from comment #7)
> IIUC, in bug 1047738, the new home for distribution directory is
> Contents/Resources instead of MozResouces?

That's right.
You need to log in before you can comment on or make changes to this bug.