On Thu, Jul 1, 2010 at 12:05 PM, David Kirkby <david.kir...@onetel.net> wrote: > I don't understand your proposal. Would it need the patch command > added to Sage? I don't understand your method, so can't comment > really.
William's proposal is to 1) Standardize the filenames of patches so that the only file which patches ./src/foo/bar/xyz.py is ./patches/foo/bar/xyz.py.patch 2) Only the .patch file is checked into the repository. 3) When doing "sage -spkg" to create an spkg, it goes through all of the patches under ./patches/ and uses the patch command to make the patched file which is copied over. In the above example, the "sage -spkg" script would automatically make ./patches/foo/bar/xyz.py from ./src/foo/bar/xyz.py and ./patches/foo/bar/xyz.py.patch. Thus, only patch needs to be installed on the machine which creates the original spkg. One issue that I have with the way things are done now which is not addressed by William's proposal is that (logical) patches which touch multiple files have to be broken up. Similarly, if two (logical patches) touch the same file, then they have to be merged. If you used the patch command, then things wouldn't have to be broken up by file. --Mike -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org