902125 - Add a version config/flag for Python Marionette (and JavaScript version too if we don't have one)

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Peter deHaan [:pdehaan]]

I tried running marionette --version from the command line, but it didn't care for that one bit.

I asked in #ateam and we discussed possibly adding a marionette.__version__ flag.

Comment 1

[Jonathan Griffin (:jgriffin)]

For Python packages, you can get a list of version numbers using 'pip freeze'.  Is this sufficient?  Also, the Python package version is not necessarily the version of the Marionette implementation is gecko, which is currently unversioned.  We are thinking of returning the Gecko version for that.

Which version do you need?

Comment 2

[David Burns :automatedtester]

would adding a marionette.__version__ item be enough for this bug?

Comment 3

[Peter deHaan [:pdehaan]]

Adding `__version__` would probably work. I currently have marionette installed but not sure what version.
I'm not a Python guy, but `pip freeze` seems to give me a version number:

```
$ pip freeze | grep "marionette"
Warning: cannot find svn location for mercurial==2.6.2-20130606
marionette-client==0.5.37
```

Comment 4

[Julien Pagès (:parkouss)]

Attachment: 902125.patch

This patch add a marionette.__version__ flag and a --version command line.

Note that the version is now in the marionette/__init__.py file. Maybe we want to show version dependencies for marionette-drivers for example, this could be addresses in follow up bugs (because in this case we would need __version__ from the marionette-drivers package also).

Last thing, this patch will probably conflict with one of my other patches, but it will be easy to fix.

Comment 5

[Jonathan Griffin (:jgriffin)]

Yes, ideally --version would report the versions of marionette-driver and marionette-transport as well; can you file a follow-up for this?

Comment 6

[Julien Pagès (:parkouss)]

Sure! I just created Bug 1183157 for that. :)

Comment 7

[Julien Pagès (:parkouss)]

Pushed to try (after fixing conflicts):

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5dbbfeb1a6c9

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d746efa19e47

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d746efa19e47