Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Improvement]: Make yappcapp optional #70

Closed
mbradeen opened this issue Oct 21, 2024 · 0 comments · Fixed by #71
Closed

[Improvement]: Make yappcapp optional #70

mbradeen opened this issue Oct 21, 2024 · 0 comments · Fixed by #71
Labels

Comments

@mbradeen
Copy link
Contributor

Improvement Description

Change the install so that yappcapp can install optionally and fix rls tests that have an unnecessary dependency on it.

mbradeen added a commit to mbradeen/testsuite that referenced this issue Oct 21, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: asterisk#70
mbradeen added a commit to mbradeen/testsuite that referenced this issue Oct 22, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: asterisk#70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
asterisk-org-access-app bot pushed a commit that referenced this issue Oct 24, 2024
Move starpy into requirements.txt and make extras.txt optional.

This makes yappcap optional.  As individual tests check for it anyway, it's
failure should not force the test suite venv install to fail.

The pjsip rls tests had a python lib dependency on yappcap that wasn't really
required.  Resolved by splitting pcap.py into pcap_proxy.py where the latter
has no yappcap dependency.

Fixes: #70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant