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

only print warnings when needed #11724

Closed
koffie opened this issue Aug 23, 2011 · 7 comments
Closed

only print warnings when needed #11724

koffie opened this issue Aug 23, 2011 · 7 comments

Comments

@koffie
Copy link

koffie commented Aug 23, 2011

currently building sage, or installing spkgs or running other scripts which source sage-env might sometimes print

Warning: Attempted to overwrite SAGE_ROOT environment variable

while it's not overwriting or attempting to overwrite SAGE_ROOT

Fixed by #5852.

Component: scripts

Reviewer: Maarten Derickx

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

@koffie koffie added this to the sage-4.7.2 milestone Aug 23, 2011
@koffie
Copy link
Author

koffie commented Aug 23, 2011

apply to scripts repository

@koffie
Copy link
Author

koffie commented Aug 23, 2011

comment:1

Attachment: 11724-dont-print-unneeded-waringin-message.patch.gz

note that sage-env is not executable so you need to do

source sage-env

to test it.

@jdemeyer
Copy link

comment:2

This is also fixed by #5852, where the SAGE_ROOT detection is completely changed.

So personally, I prefer closing this as "duplicate".

@jdemeyer jdemeyer removed this from the sage-4.7.2 milestone Oct 31, 2011
@koffie
Copy link
Author

koffie commented Oct 31, 2011

comment:4

that's ok with me

@koffie
Copy link
Author

koffie commented Dec 17, 2011

comment:5

Hej Jeroen,

I was just walking trough the open tickets I participated in. Are you still going to close this one as duplicate?

@jdemeyer
Copy link

Reviewer: Maarten Derickx

@jdemeyer

This comment has been minimized.

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