graphs.nauty_geng fails due to missing imports

sagemath / sage

Main repository of SageMath

https://www.sagemath.org

Other

1.45k stars 482 forks source link

graphs.nauty_geng fails due to missing imports #3378

Closed mwhansen closed 16 years ago

mwhansen commented 16 years ago

Component: graph theory

Issue created by migration from https://trac.sagemath.org/ticket/3378

89c6e537-b2e3-45e6-882d-d4957b74ffe5 commented 16 years ago

comment:1

Attachment: 3378.patch.gz

I suppose this didn't get caught by doctests because nauty is an optional package. Is it possible to have doctests that actually get run?

Also, in installing the optional nauty spkg, I notice that there is an interactive do-you-accept-this-license step. This doesn't seem right at all... Mabshoff, thoughts?

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago

comment:2

Re the license: This is on purpose since it isn't GPL compatible and I think it is fine. It has been proposed to create a non-free repo and then stick all non-GPL compatible spkgs in there.

Cheers,

Michael

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago

comment:3

Merged in Sage 3.0.3.alpha2