[Asterisk-code-review] add requirements.txt file: (testsuite[master])

[Corey Farrell]

Corey Farrell has posted comments on this change.

Change subject: add requirements.txt file:
......................................................................


Patch Set 1:

> Hi Corey!
 > 
 > Yep, is incomplete. I just a checked the depedencies from python
 > code into lib/python
 > 
 > These are the dependencies
 > 
 > - requests
 > - Twisted
 > - autobahn_rce
 > - construct
 > - lxml
 > - netifaces
 > - pjsua
 > - SQLAlchemy
 > - PyYAML
 > - starpy
 > 
 > 
 > Initially I not added the starpy because is like submodule into
 > suite of test
 > https://wiki.asterisk.org/wiki/display/AST/Installing+the+Asterisk+Test+Suite
 > 
 > 
 > What your think about this?

I don't think of starpy as a submodule.  It's a stand-alone project used by more than just the testsuite.

Try running:
./runtests.py -l|grep -e Met:|sort|uniq -c|sort -n|less

This will give you an idea of how many tests require certain dependencies.  starpy is needed by 676 tests in my current checkout of the testsuite.  144 tests for autobahn, 136 for requests, 48 for lxml.  Other python dependencies you've listed do exist but effect less tests.

-- 
To view, visit https://gerrit.asterisk.org/4013
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3692f8e6c97e9883a3c0f7ebd20533d93c16e408
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-HasComments: No