558147 - maemo repacks need to sb-conf select

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Aki Sasaki (not active)]

Now that Maemo5 builds are doing an |sb-conf select FREMANTLE_ARMEL| in scratchbox, repacks may behave badly and/or break.

Let's do this in a two step manner: this bug will cover running sb-conf select CHINOOK-ARMEL-2007 in maemo repacks.  We'll fix whatever's broken in the FREMANTLE_ARMEL target when we enable l10n repacks for Maemo5.

Comment 1

[Aki Sasaki (not active)]

Attachment: add sb_target to l10n repack configs

passes checkconfig. patch 1/2

Comment 2

[Aki Sasaki (not active)]

Attachment: factory update -- add sb_target + sb-conf select steps

Passes checkconfig.

Sadly, I can't insert steps before BaseRepackFactory.__init__(), which creates the full repack steps (and initializes the self.steps variable itself).

I chose to overload preClean() which isn't great.
Pushing two new steps into the front of self.steps is another approach... dunno which is uglier.

Pretty sure this will work, but I'll need to either land this or get a staging env to test.

Comment 3

[Aki Sasaki (not active)]

Comment on attachment 437923 [details] [diff] [review]
add sb_target to l10n repack configs

This should fix it.

Comment 4

[Aki Sasaki (not active)]

Comment on attachment 437923 [details] [diff] [review]
add sb_target to l10n repack configs

This is totally the wrong patch.

Comment 5

[Aki Sasaki (not active)]

Attachment: this is the right configs patch

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 437924 [details] [diff] [review]
factory update -- add sb_target + sb-conf select steps

Looks like I removed some trailing spaces too, hope those aren't confusing.  I should just file a whitespace bug and be done with it at some point.

Comment 7

[Chris Cooper [:coop] (he/him)]

Comment on attachment 437924 [details] [diff] [review]
factory update -- add sb_target + sb-conf select steps

Meh, don't beat yourself up too much about overriding preClean. I think it works quite well.

Comment 8

[Aki Sasaki (not active)]

Comment on attachment 438009 [details] [diff] [review]
this is the right configs patch

http://hg.mozilla.org/build/buildbot-configs/rev/4ebed3f2e189

Comment 9

[Aki Sasaki (not active)]

Comment on attachment 437924 [details] [diff] [review]
factory update -- add sb_target + sb-conf select steps

http://hg.mozilla.org/build/buildbotcustom/rev/f787ab82a76b

Comment 10

[Aki Sasaki (not active)]

Crossing fingers for working repacks.