909495 - Fix hard coded --xre-path for cppunittests in mozharness config

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

[Dan Minor [:dminor]]

The --xre-path option is hard coded in the config files for each platform. This is working for Linux and Windows, but fails for Mac [1]. Rather than fixing the hard coded path for Mac, I think it makes more sense to set it dynamically based upon where things are unpacked.

[1] https://tbpl.mozilla.org/?tree=Cedar&rev=80c7d8f26341&showall=1

Comment 1

[Dan Minor [:dminor]]

Attachment: Patch to fix hardcoded paths in config files.

I wasn't sure if it was better to match the script variable name or the existing naming convention in the config files. Let me know if you would prefer something else.

Comment 2

[Aki Sasaki (not active)]

Comment on attachment 796021 [details] [diff] [review]
Patch to fix hardcoded paths in config files.

https://hg.mozilla.org/build/mozharness/rev/f1284657517b

Comment 3

[John Hopkins (:jhopkins)]

In production.