#15908: guava must be explicitly loaded
-------------------------------------+-------------------------------------
       Reporter:  dimpase            |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.2
      Component:  coding theory      |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/dimpase/ticket/15908             |  edf791185486938a2bf77879213811cbd0ef37d7
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by dimpase):

 Replying to [comment:8 vbraun]:
 > Why does GAP disable guava by default? Maybe there are some potential
 name clashes lurking there? Is it really a good idea to patch this out?
 IMHO the prudent thing to do is to only load it if needed as in this
 patch.

 IMHO guava should not be preloaded, given its limited scope. It was a
 remnant from earlier Sage versions (with GAP-4.4, IIRC) where it was
 explicitly preloaded directly inside the GAP spkg. That's why I would
 rather prefer to go ahead with the patch on this ticket.

--
Ticket URL: <http://trac.sagemath.org/ticket/15908#comment:9>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to