906696 - Start moving testplugin.mk to moz.build

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Comment 1

[Michael Shal [:mshal]]

Comment on attachment 792218 [details] [diff] [review]
Patch v1

>From: Ms2ger <[e-mail]>
>
>diff --git a/dom/plugins/test/testplugin/moz.build b/dom/plugins/test/testplugin/moz.build
>--- a/dom/plugins/test/testplugin/moz.build
>+++ b/dom/plugins/test/testplugin/moz.build
>@@ -7,8 +7,11 @@
> DIRS += ['secondplugin']
> 
> MODULE = 'nptest'
> NO_DIST_INSTALL = True
> 
> LIBRARY_NAME = 'nptest'
> 
> FAIL_ON_WARNINGS = not CONFIG['_MSC_VER']
>+
>+relative_path = '.'
>+include('testplugin.build')

Ms2ger: Please use the '.mozbuild' extension to match the rest of the tree.

gps: This is similar to the discussion we had in bug 900974. Would your concerns in #c7 go away if the "relative_path" variable was calculated automatically in frontend/reader.py:_include(), and used as it is here in testplugin.build? Then we could just do include('testplugin.mozbuild') or include('../testplugin.mozbuild'), and relative_path (or some other magic variable name) is defined by mozbuild for the included file. The included file still has to do the explicit path expansion, rather than having it be automatic:

CPP_SOURCES += [
    relative_path + '/nptest.cpp',
    relative_path + '/nptest_utils.cpp',
    'nptest_name.cpp',
]


Note that this mozbuild file also has a local file (nptest_name.cpp) that is expected to be in the directory of the moz.build file, not in the directory of testplugin.mozbuild. So using absolute paths here isn't really possible, like we were thinking of doing in 900974.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 792218 [details] [diff] [review]
Patch v1

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

I'm fine with hacking things like this.

Comment 3

[Michael Shal [:mshal]]

Comment on attachment 792218 [details] [diff] [review]
Patch v1

r+ assuming testplugin.build -> testplugin.mozbuild

We can do the relative_path thing automatically later if we want :)

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/511fbeed2a3b