Opened 6 years ago

Closed 6 years ago

#17281 closed defect (fixed)

Remove -notebook-ipy from sage -h

Reported by: slabbe Owned by:
Priority: blocker Milestone: sage-6.4
Component: notebook Keywords:
Cc: Merged in:
Authors: Sébastien Labbé Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: e9db66c (Commits, GitHub, GitLab) Commit: e9db66c931b5bc6c843a86e3df7a26b350ee0495
Dependencies: Stopgaps:

Status badges

Description (last modified by slabbe)

That line should not have been added in #16996. 

  -notebook-ipy [...] -- start the Sage IPython notebook

That command does not work: 

$ sage -notebook-ipy
CRITICAL:root:unknown notebook: otebook-ipy
Error, notebook must be one of default, ipython, sagenb but got otebook-ipy

It was decided in #16996 to use the syntax sage --notebook=ipython instead and we forgot to remove that line.

Change History (3)

comment:1 Changed 6 years ago by slabbe

  • Branch set to u/slabbe/17281
  • Commit set to e9db66c931b5bc6c843a86e3df7a26b350ee0495
  • Description modified (diff)
  • Priority changed from major to blocker
  • Status changed from new to needs_review

New commits:

​e9db66cTrac #17281: Remove -notebook-ipy from sage -h

comment:2 Changed 6 years ago by vbraun

  • Authors set to Sébastien Labbé
  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

lgtm

comment:3 Changed 6 years ago by vbraun

  • Branch changed from u/slabbe/17281 to e9db66c931b5bc6c843a86e3df7a26b350ee0495
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.