1454338 - [mozversion] Remove support for B2G

Status: RESOLVED FIXED

(Testing :: Mozbase, enhancement)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Whilst looking at the source code for mozversion I noticed there's still quite a lot of B2G related code.

Comment 1

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Attachment: Bug 1454338 - [mozversion] Remove support for B2G;

Review commit: https://reviewboard.mozilla.org/r/236834/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/236834/

Comment 2

[Andrew Halberstadt [:ahal]]

Comment on attachment 8968158 [details]
Bug 1454338 - [mozversion] Remove support for B2G;

https://reviewboard.mozilla.org/r/236834/#review242584

Thanks for the cleanup!

::: testing/mozbase/docs/mozversion.rst
(Diff revision 1)
>  `mozversion <https://github.com/mozilla/mozbase/tree/master/mozversion>`_
>  provides version information such as the application name and the changesets
>  that it has been built from. This is commonly used in reporting or for
>  conditional logic based on the application under test.
>  
> -Note that mozversion can report the version of remote devices (e.g. Firefox OS)

Hah, looks like we weren't even using `mozdevice` for b2g!

::: testing/mozbase/mozversion/setup.py:9
(Diff revision 1)
>  
>  from __future__ import absolute_import
>  
>  from setuptools import setup
>  
>  PACKAGE_VERSION = '1.5'

Since this is technically a backwards incompatible change (even though I know no one is depending on this code anymore), we might as well make the switch to SemVer here. Please bump to 2.0.0

::: testing/mozbase/mozversion/setup.py:26
(Diff revision 1)
>        url='https://wiki.mozilla.org/Auto-tools/Projects/Mozbase',
>        license='MPL',
>        packages=['mozversion'],
>        include_package_data=True,
>        zip_safe=False,
> -      install_requires=['mozfile >= 1.0',
> +      install_requires=['mozlog >= 3.0',

Please add an upper version limit here:

    mozlog>=3,<4

Comment 3

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8968158 [details]
Bug 1454338 - [mozversion] Remove support for B2G;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/236834/diff/1-2/

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8968158 [details]
Bug 1454338 - [mozversion] Remove support for B2G;

https://reviewboard.mozilla.org/r/236834/#review242584

> Hah, looks like we weren't even using `mozdevice` for b2g!

We were until https://bugzilla.mozilla.org/show_bug.cgi?id=1440714

> Please add an upper version limit here:
> 
>     mozlog>=3,<4

I used a compatible release version indicator https://www.python.org/dev/peps/pep-0440/#compatible-release

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 8968158 [details]
Bug 1454338 - [mozversion] Remove support for B2G;

https://reviewboard.mozilla.org/r/236834/#review242584

> I used a compatible release version indicator https://www.python.org/dev/peps/pep-0440/#compatible-release

Maybe can we do the release in its own commit? It would help a lot when checking the pushlog later.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/057c58bcb1b1
[mozversion] Remove support for B2G; r=ahal

Comment 7

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/057c58bcb1b1