Opened 10 years ago

Closed 10 years ago

#13363 closed defect (fixed)

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

Reported by: Mike Hansen 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:

Status badges

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 Mike Hansen 10 years ago.

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by Mike Hansen

Attachment: trac_13363.patch added

comment:1 Changed 10 years ago by Mike Hansen

Status: newneeds_review

comment:2 Changed 10 years ago by John Palmieri

Reviewers: John Palmieri
Status: needs_reviewpositive_review

Looks good to me.

comment:3 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.3.beta2

comment:4 Changed 10 years ago by Jeroen Demeyer

Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.