#14645: cvxopt 1.1.6.p0 fails to build with SAGE_SPKG_INSTALL_DOCS=yes
--------------------------------------+-------------------------------------
       Reporter:  strogdon            |         Owner:  leif             
           Type:  defect              |        Status:  needs_info       
       Priority:  major               |     Milestone:  sage-5.10        
      Component:  packages: standard  |    Resolution:                   
       Keywords:  spkg                |   Work issues:                   
Report Upstream:  N/A                 |     Reviewers:  Dmitrii Pasechnik
        Authors:  Leif Leonhardy      |     Merged in:                   
   Dependencies:  #12832              |      Stopgaps:                   
--------------------------------------+-------------------------------------

Comment (by leif):

 Replying to [comment:18 dimpase]:
 > Replying to [comment:16 leif]:
 > > I was actually first going to add that, but then did not for a couple
 of reasons:
 > >
 > >  * I hate absolute paths there (as have unfortunately been added back
 to (additional) messages in the "main" docbuilding, although the messages
 there are even worse, since they claim the documents had been built even
 upon errors), but was 100% sure at least some would complain about a
 relative path in the message.
 >
 > well, how about just saying "installed in SAGE_SHARE/doc" ?

 That's kind of a relative path... [I would have used] ;-)

 [[BR]]

 > > Orthogonal to that:
 > >
 > > Should I delete previous (potentially outdated) docs regardless of
 `SAGE_SPKG_INSTALL_DOCS`, which could probably be considered unexpected
 behaviour?  (E.g. if one reinstalls an spkg just with different flags, or
 in order to run its test suite, but forgets to set
 `SAGE_SPKG_INSTALL_DOCS` to the previous setting?  Does anyone use that
 variable on a per-package basis?  To me it seems a few people have it set
 in their shell rcs, and most people don't use it, probably just because
 they don't know about... B) )
 > >
 > if they are in "wrong" location, yes. Otherwise, that's probably too
 much trouble --- although I as proposed on #14646, there ought to be a
 default `SAGE_SHARE/doc/cvxopt/index.html` forwarding to cvxopt website,
 so, ideally, `SAGE_SPKG_INSTALL_DOCS!=yes` would mean putting this
 forwarding back in place...  Well, even more ideally, this functionality
 shouldn't be on the `spkg-install` level, but above, at `sage -i` level.

 So, should I move the `rm -rf ...` out of the `if [
 "$SAGE_SPKG_INSTALL_DOCS" = yes ] ...` branch?  (With a separate message
 on what's going on ... :-) )

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14645#comment:19>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to