Opened 10 years ago

Closed 9 years ago

#11724 closed defect (duplicate)

only print warnings when needed

Reported by: mderickx Owned by:
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: scripts Keywords:
Cc: Merged in:
Authors: Reviewers: Maarten Derickx
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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.

Attachments (1)

11724-dont-print-unneeded-waringin-message.patch (1.3 KB) - added by mderickx 10 years ago.
apply to scripts repository

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by mderickx

apply to scripts repository

comment:1 Changed 10 years ago by mderickx

  • Status changed from new to needs_review

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

source sage-env

to test it.

comment:2 Changed 10 years ago by jdemeyer

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

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

comment:3 Changed 10 years ago by jdemeyer

  • Milestone changed from sage-4.7.2 to sage-duplicate/invalid/wontfix

comment:4 Changed 10 years ago by mderickx

that's ok with me

comment:5 Changed 10 years ago by mderickx

  • Status changed from needs_review to positive_review

Hej Jeroen,

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

comment:6 Changed 10 years ago by jdemeyer

  • Reviewers set to Maarten Derickx

comment:7 Changed 9 years ago by jdemeyer

  • Description modified (diff)
  • Resolution set to duplicate
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.