Closed Bug 1037107 Opened 10 years ago Closed 10 years ago

create mozilla-b2g32_v2_0 branch

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

Attachments

(11 files)

615 bytes, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
20.37 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
779 bytes, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
761 bytes, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
1.63 KB, patch
RyanVM
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
1.43 KB, patch
pmoore
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
3.95 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
11.43 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
2.97 KB, patch
bhearsum
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
1.35 KB, patch
hwine
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
2.85 KB, patch
rail
: review+
mozilla
: checked-in+
Details | Diff | Splinter Review
      No description provided.
Depends on: 1037148
Attached patch b2g_tagSplinter Review
Attachment #8457683 - Flags: review?(hwine)
Attachment #8457683 - Flags: review?(hwine) → review+
Attached patch b2g32_configsSplinter Review
Attachment #8457700 - Flags: review?(rail)
Attached patch b2g32_customSplinter Review
Attachment #8457701 - Flags: review?(rail)
Attached patch b2g32_toolsSplinter Review
Attachment #8457702 - Flags: review?(rail)
Attached patch b2g32_tbplSplinter Review
Attachment #8457705 - Flags: review?(ryanvm)
Attached patch b2g32_beagleSplinter Review
Attachment #8457707 - Flags: review?(pmoore)
Attachment #8457705 - Flags: review?(ryanvm) → review+
Attachment #8457700 - Flags: review?(rail) → review+
Attachment #8457701 - Flags: review?(rail) → review+
Attachment #8457702 - Flags: review?(rail) → review+
Comment on attachment 8457707 [details] [diff] [review]
b2g32_beagle

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

Nice work Aki.

::: configs/vcs_sync/beagle.py
@@ +254,5 @@
> +            "tag_config": {
> +                "tag_regexes": [
> +                    "^B2G_",
> +                ],
> +            },

This tag_config is the same as the tag_config below (lines 266 to 270). I think this isn't needed, because I believe if it is not provided by the specific target, it is taken from the generic cross-target tag_config section (http://hg.mozilla.org/build/mozharness/file/7e18e11a38be/scripts/vcs-sync/vcs_sync.py#l500).

However, it should not hurt, and it would be consistent with the other branches in this config file - but I believe it may be redundant to define it in both places.
Attachment #8457707 - Flags: review?(pmoore) → review+
(In reply to Pete Moore [:pete][:pmoore] from comment #9)
> ::: configs/vcs_sync/beagle.py
> @@ +254,5 @@
> > +            "tag_config": {
> > +                "tag_regexes": [
> > +                    "^B2G_",
> > +                ],
> > +            },
> 
> This tag_config is the same as the tag_config below (lines 266 to 270). I
> think this isn't needed, because I believe if it is not provided by the
> specific target, it is taken from the generic cross-target tag_config
> section
> (http://hg.mozilla.org/build/mozharness/file/7e18e11a38be/scripts/vcs-sync/
> vcs_sync.py#l500).
> 
> However, it should not hurt, and it would be consistent with the other
> branches in this config file - but I believe it may be redundant to define
> it in both places.

Aha.  Sure, up to you.  Might be a safety net, might be a pain keeping them in sync; I think I agree it's probably the latter.
Attached patch graphsSplinter Review
Looks complete based on https://wiki.mozilla.org/ReleaseEngineering/Merge_Duty/Steps#Add_to_Graphs .
Attachment #8458122 - Flags: review?(hwine)
Comment on attachment 8457707 [details] [diff] [review]
b2g32_beagle

Committed without the duplicate tag_config.
https://hg.mozilla.org/build/mozharness/rev/c5821474653e
Attachment #8457707 - Flags: checked-in+
Comment on attachment 8458122 [details] [diff] [review]
graphs

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

lgtm
Attachment #8458122 - Flags: review?(hwine) → review+
Attached patch in-tree-l10nSplinter Review
Attachment #8458443 - Flags: review?(rail)
Attached patch in-tree-channelSplinter Review
Since bug 1022907 isn't fixed yet.
Attachment #8458445 - Flags: review?(bhearsum)
Attachment #8458447 - Flags: review?(hwine)
Comment on attachment 8458447 [details] [diff] [review]
bump_bb-c_versions

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

lgtm
Attachment #8458447 - Flags: review?(hwine) → review+
Attachment #8458445 - Flags: review?(bhearsum) → review+
Attachment #8458443 - Flags: review?(rail) → review+
Depends on: 1040852
(In reply to Aki Sasaki [:aki] from comment #8)
> https://hg.mozilla.org/webtools/tbpl/rev/35fc0b71256f

In production :)
In production.
Attached patch disable_dolphinSplinter Review
It looks like we were intentionally only building dolphin on 1.4, like we're only building Tarako on 1.3t.  Re-disabling...
Attachment #8458921 - Flags: review?(rail)
Attachment #8458921 - Flags: review?(rail) → review+
In production.
dev-b2g:
As of today's uplift, the Gecko side of B2G v2.0 now lives on the
mozilla-b2g32_v2_0 branch, *NOT* mozilla-aurora. Patches that are
blocking-b2g:2.0+ should be landed there now. For more information, see
the page below which always contains the latest information.
https://wiki.mozilla.org/Release_Management/B2G_Landing#v2.0.0

As a reminder, B2G v2.1 development remains on mozilla-central through
the duration of the Gecko 34 development cycle due to end on Sept 1, so
there is no change for where v2.1 patches are to land.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: