Opened 9 years ago

Closed 9 years ago

#14892 closed defect (fixed)

Track all files in cliquer, and minor cleanup

Reported by: Volker Braun Owned by: Jeroen Demeyer
Priority: major Milestone: sage-5.12
Component: packages: standard Keywords:
Cc: Merged in: sage-5.12.beta1
Authors: Volker Braun Reviewers: Nathann Cohen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Volker Braun)

Move untracked additions to the upstream sources to src/ as required for #14781

Spkg diff cliquer-p0-p1.diff

Attachments (2)

trac_14892_cliquer_cleanup.patch (2.0 KB) - added by Volker Braun 9 years ago.
Initial patch
cliquer-p0-p1.diff (6.8 KB) - added by Volker Braun 9 years ago.
Spkg diff for review purposes only

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Volker Braun

Initial patch

Changed 9 years ago by Volker Braun

Attachment: cliquer-p0-p1.diff added

Spkg diff for review purposes only

comment:1 Changed 9 years ago by Volker Braun

Authors: Volker Braun
Description: modified (diff)
Status: newneeds_review

comment:2 Changed 9 years ago by Nathann Cohen

Status: needs_reviewpositive_review

>_<

If we were japanese guys you would not have written such a patch, for it is shameful to me >_<

God. How did I write this thing in the first place ? Please forgive me >_<

Anyway, thank you very much ! This ADD_EDGE method looks useful indeed. And all tests pass, the sun shines and the birds sing outside.

Thaaaaaaaaaaaaaanks !

Nathann

comment:3 Changed 9 years ago by Nathann Cohen

Reviewers: Nathann Cohen

comment:4 Changed 9 years ago by Jeroen Demeyer

Any reason why you use

for patch in `ls ../patches/*.patch`; do

instead of the standard

for patch in ../patches/*.patch; do

(both work correctly but the first wastes a process and will display a confusing error if no patch files are found)

Last edited 9 years ago by Jeroen Demeyer (previous) (diff)

comment:5 Changed 9 years ago by Volker Braun

I switched the loop to the other syntax, updated spkg at the same place.

comment:6 Changed 9 years ago by Jeroen Demeyer

Merged in: sage-5.12.beta1
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.