1515808 - The last line of python/mozbuild/mozbuild/test/backend/data/linkage/templates.mozbuild is ignored

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[Mike Hommey [:glandium]]

That's because the last line of the file doesn't have a newline, and older python versions have a buggy inspect.getblock:

    $ python
    Python 2.7.9 (default, Nov  8 2018, 08:36:59) 
    [GCC 4.7.2] on linux2
    Type "help", "copyright", "credits" or "license" for more information.
    >>> import inspect
    >>> print inspect.getblock([
    ...     'def foo():\n',
    ...     '    a = 1\n',
    ...     '    bar()',
    ... ])
    ['def foo():\n', '    a = 1\n']
    ​
    $ python
    Python 2.7.15+ (default, Aug 31 2018, 11:56:52) 
    [GCC 8.2.0] on linux2
    Type "help", "copyright", "credits" or "license" for more information.
    >>> import inspect
    >>> print inspect.getblock([
    ...     'def foo():\n',
    ...     '    a = 1\n',
    ...     '    bar()',
    ... ])
    ['def foo():\n', '    a = 1\n', '    bar()']

Comment 1

[Mike Hommey [:glandium]]

Attachment: Bug 1515808 - Compensate for older python's buggy inspect.getblock.

Fixing this unveils an issue in TestRecursiveMake::test_linkage that
was happening locally but not on automation, due to the Binary()
template never being invoked on automation.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/75572e56c170
Compensate for older python's buggy inspect.getblock. r=nalexander

Comment 3

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/75572e56c170