See bug 902125 comment 4.
More precisely bug 902125 comment 5 (ideally --version would report the versions of marionette-driver and marionette-transport as well).
Created attachment 8633975 [details] [diff] [review]
> $ marionette --version
> marionette 0.16 (using marionette-driver: 0.10, marionette-transport: 0.5)
While I was looking at it, I found something strange:
> $ cat client/requirements.txt | grep marionette
> marionette-driver >= 0.8
> $ cat driver/requirements.txt | grep marionette
> marionette-transport == 0.5
> $ cat client/setup.py | grep name=
> $ cat driver/setup.py | grep name=
> $ cat transport/setup.py | grep name=
So this seems inconsistent, sometimes we use "_" for the name, sometimes "-", and in dependencies this is always "-".
Though pip is not annoyed by that, it seems that "pip install marionette-client" and "pip install marionette_client" are roughly equivalent commands. (same thing for dependencies, so all works fine). Still I think we should update the name of client and driver package in the setup.py
files and use "-" to be consistent.
May I create a bug for that, or do you see any objection ?
Ah, pypi does convert "_" to "-" automatically it seems:
This is probably because pep8 discourage the use of underscores in package names:
I agree it would be good to change that for consistency if it doesn't break anything, and it sounds like it wouldn't.
Fixed conflicts due to mozlog update and pushed to try:
Repushed to try because infra (or something issue):
All green now.