Closed Bug 1069645 Opened 5 years ago Closed 5 years ago

Download b2g sdk only if running desktop builds

Categories

(Testing Graveyard :: JSMarionette, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: mdas, Assigned: mdas)

References

Details

(Whiteboard: [emulator][device])

Attachments

(1 file)

if you run gaia-integration with --buildapp as device or emulator, we download the b2g desktop .dmg even though it goes unused for the suite. We should only download this if needed.
Attached file link to PR
Hi Yuren, I'm r?ing since I believe you're familiar with the gaia Makefiles.

This will change means we will only download sdk if we're testing against desktop. Desktop is the default value for BUILDAPP.
Attachment #8492280 - Flags: review?(yurenju)
Hi :mdas,

we are using b2g-desktop to run gaia build system instead of using xulrunner so it's necessary to download it, so I marked it as INVALID but please feel free to reopen it if you think that still is a problem.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
(In reply to Yuren [:yurenju] (last day: 9/26) from comment #2)
> Hi :mdas,
> 
> we are using b2g-desktop to run gaia build system instead of using xulrunner
> so it's necessary to download it, so I marked it as INVALID but please feel
> free to reopen it if you think that still is a problem.

We only set BUILDAPP for running gaia-marionette tests, and if the BUILDAPP value equal to anything other than desktop, meaning either 'emulator' or 'device', then we do not want to run or build the gaia system and have no need for the b2g sdk.

In the case of 'emulator' or 'device', we only need to use node and node packages to run the tests. I want to remove the unnecessary b2g sdk download because it adds an unneeded download step to the tests and takes a lot of time.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Attachment #8492280 - Flags: review?(yurenju)
Whiteboard: [emulator][device]
ah I forgot, you're right, the b2g sdk is needed to build the profile, which we still want to do for device/emulator. Thanks, I'll resolve this.
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → INVALID
Attachment #8492280 - Flags: review?(yurenju)
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.