Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#10248 closed defect (fixed)

Improve the "Build Support" & "Quality" sections of the "Inclusion Procedure for New Packages" page in the Developers Guide

Reported by: drkirkby Owned by: mvngu
Priority: major Milestone: sage-4.6.1
Component: documentation Keywords:
Cc: robertwb, aapitzsch Merged in: sage-4.6.1.alpha1
Authors: David Kirkby Reviewers: Dmitrii Pasechnik
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by drkirkby)

http://www.sagemath.org/doc/developer/inclusion.html

has some inaccurate information what needs to happen for including a new package in Sage with regard to the compilers that are to be used, and the systems Sage work on. According to that page, the package needs to build on

  • Microsoft Windows: x86, x86_64 MSVC 2005/Intel Fortran (MinGW or Cygwin support is insufficient!)
  • Solaris 10: Sparc, x86, x86_64 (Sun Forte 12)
  • Linux (SPARC).

Although building Sage with the latest Sun and Microsoft compilers would be nice, in practice it is not going to happen any time soon.

I'm not sure if the Linux(SPARC) is a mistake, or intensional, but very few people run Linux on SPARC hardware. We might as well forget any thoughts of that.

Basically the "Build Support" section of the Developers Guide needs some work.

We should also add a link to

http://wiki.sagemath.org/SupportedPlatforms

so we only need to update the list in one place.

It is probably true the "License" section in the "Inclusion Procedure for New Packages" needs attention. But that appears to be a huge can of worms, and is best left for a ticket to be created by a very brave person!!

There are clearly other parts of the Sage Developers Guide that need attention, but this ticket is only concerned with the "Build Support" issue.

Attachments (1)

10248-developers-guide.patch (3.9 KB) - added by drkirkby 12 years ago.
Patch for the Sage Developers Guide, which removes references to most specific platforms, and provides a link to supported platforms.

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by drkirkby

Patch for the Sage Developers Guide, which removes references to most specific platforms, and provides a link to supported platforms.

comment:1 Changed 12 years ago by drkirkby

  • Authors set to David Kirkby
  • Cc robertwb aapitzsch added
  • Status changed from new to needs_review

I've attached a patch for the developers guide, which removes references to specific supported platforms, but provides a link to those supported.

The changes also give some advice on what steps can be taken to ensure packages work on the supported platforms.

Here's the original page, with all the platforms mentioned, and a totally out of date set of compilers:

http://www.sagemath.org/doc/developer/inclusion.html

Here's the revised copy, with very little emphasis on specific systems, but with links to our supported systems.

http://boxen.math.washington.edu/home/kirkby/sage-4.5.alpha4/devel/sage-main/doc/output/html/en/developer/inclusion.html

There's a short additional paragraph about how to improve the portability of packages, but it is not detailed.

I also added a note that usually a new standard package must have spent some time as an optional package.

Dave

comment:2 follow-up: Changed 12 years ago by dimpase

I would be more careful in explaining the license question, as it regularly pops up on sage-devel. What is currently written, or what is in proposed changes, can be easily misread as "code must be under GPLv2, or code must be under GPLv2 or later GPL version). Is there a canonical reference/link on this to be included?

Dima

comment:3 in reply to: ↑ 2 Changed 12 years ago by drkirkby

  • Status changed from needs_review to needs_info

Replying to dimpase:

I would be more careful in explaining the license question, as it regularly pops up on sage-devel. What is currently written, or what is in proposed changes, can be easily misread as "code must be under GPLv2, or code must be under GPLv2 or later GPL version). Is there a canonical reference/link on this to be included?

Dima

Yes, I agree with you. I've raised the issue on sage-devel. Once there's some agreement about the wording, I'll add that. I think it's a sufficiently important point it needs discussion there, rather than just on this trac ticket.

Dave

comment:4 Changed 12 years ago by drkirkby

  • Description modified (diff)
  • Status changed from needs_info to needs_review
  • Summary changed from Correct the "Inclusion Procedure for New Packages" section in Developers Guide to Improve the "Build Support" & "Quality" sections of the "Inclusion Procedure for New Packages" page in the Developers Guide

Dima,

I've come to the conclusion that the licenses issues are a minefield and addressing them would be opening a huge can of worms. Hence I've restricted the title of this ticket, to make it clearer the sections actually addressed, and I will not address the license section.

My original aim was to sort out the issues related to supported platforms. As such I'm marking it for review again.

Dave

comment:5 Changed 12 years ago by drkirkby

  • Description modified (diff)

comment:6 Changed 12 years ago by dimpase

  • Status changed from needs_review to positive_review

comment:7 Changed 12 years ago by jdemeyer

  • Merged in set to sage-4.6.1.alpha1
  • Resolution set to fixed
  • Reviewers set to Dima Pasechnik
  • Status changed from positive_review to closed

comment:8 Changed 11 years ago by jdemeyer

  • Reviewers changed from Dima Pasechnik to Dmitrii Pasechnik
Note: See TracTickets for help on using tickets.