Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sage patches added directly to symmetica source #8122

Closed
sagetrac-drkirkby mannequin opened this issue Jan 29, 2010 · 5 comments
Closed

Sage patches added directly to symmetica source #8122

sagetrac-drkirkby mannequin opened this issue Jan 29, 2010 · 5 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jan 29, 2010

The symmerica package says in SPKG.txt

== Special Update/Build Instructions ==

Against common policy the patches in the patches directory have been applied to 
the src directory:

 * de.patch (Turn off banner)
 * macro.h.patch (Change some return types, this can be avoided)
 * makefile.patch (Fix compiler, inject CFLAGS)
 * sort_sum_rename.patch (rename sort tp sym_sort, sum to sym_sum) 

It would appear various patches have been made to the source code.

http://boxen.math.washington.edu/home/kirkby/portability/symmetrica-2.0.p5/

has an updated version of symmetica, which will build with any compiler, and in 64-bit mode. However, it does not resolve the issue of patching the source directly. I modified the makefile, which had already been modified before. A patch was left, so I have tried to recreate the original makefile. But other files have been changed too. The packages is basically a bit of a mess

CC: @jaapspies @sagetrac-mvngu

Component: build

Reviewer: David Kirkby, Jeroen Demeyer

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

@sagetrac-drkirkby sagetrac-drkirkby mannequin added this to the sage-5.11 milestone Jan 29, 2010
@kini
Copy link
Collaborator

kini commented Oct 2, 2012

comment:1

Has this been fixed already? Looks like the text you quoted no longer exists in SPKG.txt in symmetrica-2.0.p7.spkg, which currently ships with Sage.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@jdemeyer
Copy link

comment:4

Duplicate of #10719.

@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer jdemeyer removed this from the sage-6.1 milestone Dec 29, 2013
@jdemeyer
Copy link

Changed reviewer from Jeroen Demeyer to David Kirkby, Jeroen Demeyer

@jdemeyer
Copy link

Changed author from David Kirkby to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants