FS#60017 - [python-selenium] Add python-urllib3 as a dependency

Attached to Project: Community Packages
Opened by Ian Brunelli (brunelli) - Tuesday, 11 September 2018, 13:35 GMT
Last edited by Jelle van der Waa (jelly) - Wednesday, 12 September 2018, 19:37 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jelle van der Waa (jelly)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
Trying to import selenium.webdriver and it raises the following exception:
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib/python3.7/site-packages/selenium/webdriver/__init__.py", line 18, in <module>
from .firefox.webdriver import WebDriver as Firefox # noqa
File "/usr/lib/python3.7/site-packages/selenium/webdriver/firefox/webdriver.py", line 29, in <module>
from selenium.webdriver.remote.webdriver import WebDriver as RemoteWebDriver
File "/usr/lib/python3.7/site-packages/selenium/webdriver/remote/webdriver.py", line 27, in <module>
from .remote_connection import RemoteConnection
File "/usr/lib/python3.7/site-packages/selenium/webdriver/remote/remote_connection.py", line 24, in <module>
import urllib3
ModuleNotFoundError: No module named 'urllib3'


Additional info:
* python=3.7.0-3
* python-selenium=3.14.0-1


Steps to reproduce:
1. Uninstall python-urllib3
2. Try to create a webdriver
This task depends upon

Closed by  Jelle van der Waa (jelly)
Wednesday, 12 September 2018, 19:37 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in python-selenium-3.14.0-2-x86_64.pkg.tar. xz
Comment by Ian Brunelli (brunelli) - Tuesday, 11 September 2018, 13:43 GMT
Sorry for this following message, I forgot to include this in the original report.

In [1] we have the following content:
'install_requires': ['urllib3'],

So, yeah, urllib3 should be a hard dependency.

1: https://github.com/SeleniumHQ/selenium/blob/507fc3b5dca174cd5a4022481dbc817884ac8abf/py/setup.py#L71
Comment by Eli Schwartz (eschwartz) - Wednesday, 12 September 2018, 17:41 GMT

Loading...