Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#16735 closed defect (fixed)

Graph database installs to wrong location

Reported by: vbraun Owned by:
Priority: blocker Milestone: sage-6.3
Component: graph theory Keywords:
Cc: ncohen Merged in:
Authors: Volker Braun Reviewers: François Bissey
Report Upstream: N/A Work issues:
Branch: 009e149 (Commits, GitHub, GitLab) Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by vbraun)

#16541 introduced 

mv src "$SAGE_SHARE"/graphs

which installs the database in the wrong directory.

Change History (8)

comment:1 Changed 7 years ago by vbraun

  • Cc ncohen added
  • Component changed from PLEASE CHANGE to graph theory
  • Description modified (diff)
  • Priority changed from major to blocker
  • Type changed from PLEASE CHANGE to defect

comment:2 Changed 7 years ago by vbraun

  • Branch set to u/vbraun/graph_database_installs_to_wrong_location

comment:3 Changed 7 years ago by vbraun

  • Commit set to 009e149009a64c465183b4269f90a25588609fd0
  • Status changed from new to needs_review

New commits:

​009e149Install the graphs database in the correct location

comment:4 Changed 7 years ago by fbissey

  • Reviewers set to François Bissey
  • Status changed from needs_review to positive_review

This is trivial.

comment:5 Changed 7 years ago by vbraun

  • Authors set to Volker Braun

thanks!

comment:6 Changed 7 years ago by vbraun

  • Branch changed from u/vbraun/graph_database_installs_to_wrong_location to 009e149009a64c465183b4269f90a25588609fd0
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:7 Changed 7 years ago by dimpase

  • Commit 009e149009a64c465183b4269f90a25588609fd0 deleted

hmm, shouldn't this have bumped the graphs spkg version, to force a re-install? I had to manually reinstall it after updating to beta8.

comment:8 Changed 7 years ago by fbissey

shoo you are right, if you come from beta7 you are cooked.

Note: See TracTickets for help on using tickets.