Opened 7 years ago

Closed 7 years ago

#13363 closed defect (fixed)

Move sage/graphs/planarity/ to sage/graphs/planarity_c/

Reported by: mhansen Owned by: jason, ncohen, rlm
Priority: major Milestone: sage-5.3
Component: graph theory Keywords:
Cc: Merged in: sage-5.3.beta2
Authors: Mike Hansen Reviewers: John Palmieri
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

When running doctests with #12659 applied, an "ImportWarning?: Not importing directory" is raised due to there being a sage/graphs/planarity.so and sage/graphs/planarity directory. This moves the C source code into a planarity_c directory.

Attachments (1)

trac_13363.patch (16.5 KB) - added by mhansen 7 years ago.

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by mhansen

comment:1 Changed 7 years ago by mhansen

  • Status changed from new to needs_review

comment:2 Changed 7 years ago by jhpalmieri

  • Reviewers set to John Palmieri
  • Status changed from needs_review to positive_review

Looks good to me.

comment:3 Changed 7 years ago by jdemeyer

  • Merged in set to sage-5.3.beta2

comment:4 Changed 7 years ago by jdemeyer

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.