898142 - xpcshell make target should pass in pluginsPath

Status: RESOLVED FIXED

(Testing :: XPCShell Harness, defect)

Description

[Mihnea Dobrescu-Balaur (:mihneadb)]

If you don't specify any plugins path to xpcshell, it will just assume bin/plugins, which is not the correct plugins path on local builds. The correct one is dist/plugins.

Mach specifies the plugins path and all works well. We need the make target to do this as well, for people who are still using it.

Comment 1

[Mihnea Dobrescu-Balaur (:mihneadb)]

Attachment: pass in plguins path

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 781262 [details] [diff] [review]
pass in plguins path

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

This is fine by itself. Keep in mind that the automation configs (https://hg.mozilla.org/build/mozharness/) are separate from the in-tree configs. You may need to also modify them to pass in the plugins path. But from the sound of your initial comment on the bug, perhaps the default of bin/plugins is acceptable for packaged builds?

Comment 3

[Mihnea Dobrescu-Balaur (:mihneadb)]

(In reply to Gregory Szorc [:gps] from comment #2)
> Comment on attachment 781262 [details] [diff] [review]
> pass in plguins path
> 
> Review of attachment 781262 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> This is fine by itself. Keep in mind that the automation configs
> (https://hg.mozilla.org/build/mozharness/) are separate from the in-tree
> configs. You may need to also modify them to pass in the plugins path. But
> from the sound of your initial comment on the bug, perhaps the default of
> bin/plugins is acceptable for packaged builds?

Yes, everything is already set up (including xpcshell.cpp!) to be friends with bin/plugins.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/60542f3605e3

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/60542f3605e3