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

Remove test for scons from sage/tests/cmdline.py #19085

Closed
kiwifb opened this issue Aug 25, 2015 · 5 comments
Closed

Remove test for scons from sage/tests/cmdline.py #19085

kiwifb opened this issue Aug 25, 2015 · 5 comments

Comments

@kiwifb
Copy link
Member

kiwifb commented Aug 25, 2015

scons is now an optional package only pulled by polybori once polybori is replaced by brial in #18437 new sage install won't have scons installed and doctest for scons functionality will fail.

This ticket remove the tests for scons which should have been removed when it was made optional.

CC: @jdemeyer

Component: packages: optional

Author: François Bissey

Branch/Commit: 2ccc4fe

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/19085

@kiwifb kiwifb added this to the sage-6.9 milestone Aug 25, 2015
@kiwifb
Copy link
Member Author

kiwifb commented Aug 25, 2015

Reviewer: Jeroen Demeyer

@kiwifb
Copy link
Member Author

kiwifb commented Aug 25, 2015

comment:2

Jeroen pre-approved the removal in #18437 comment:61

@vbraun
Copy link
Member

vbraun commented Aug 25, 2015

comment:3

Author name missing

@kiwifb
Copy link
Member Author

kiwifb commented Aug 25, 2015

Author: François Bissey

@vbraun
Copy link
Member

vbraun commented Aug 26, 2015

Changed branch from u/fbissey/scons_less to 2ccc4fe

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

No branches or pull requests

2 participants