broken spyx support concerning imports

[dkrenn]

File problem.spyx with

import sage.all
from sage.all import *
from sage.rings.integer_ring import ZZ
print ZZ(1000)

Run it with sage problem.spyx and it fails with

ImportError: cannot import name ZZ

on 6.9, 6.10, 7.0, 7.1.beta4.

See also discussion https://groups.google.com/forum/#!topic/sage-support/NcQ2Lgj_oao

Depends on #20002

Component: scripts

Author: Jeroen Demeyer

Branch/Commit: f87fa19

Reviewer: Daniel Krenn

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

[jdemeyer]

Dependencies: #20002

[jdemeyer]

Author: Jeroen Demeyer

[jdemeyer]

Branch: u/jdemeyer/ticket/20093

[jdemeyer]

Commit: f87fa19

[jdemeyer]

New commits:

fc69b52	Move interrupt.pyx to package cysignals
aee6a5f	Check that sig_on()/sig_off() works in .spyx files
434b272	cysignals depends on PARI
f663c5d	Sage library depends on cysignals
4677a93	cysignals version 1.0rc2
f87fa19	Trac #20093: import sage.all before running external code

[dkrenn]

Reviewer: Daniel Krenn

[dkrenn]

comment:5

LGTM, tests passed on my machine; I'll set it to positive_review as soon as a patchbot confirms this as well.

[dkrenn]

comment:6

Replying to @dkrenn:

I'll set it to positive_review as soon as a patchbot confirms this as well.

Patchbot sees ticket as unsafe. But since my tests were sucessful, I give it a positive review.

[vbraun]

Changed branch from u/jdemeyer/ticket/20093 to f87fa19