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

Jump to bottom

Move memory functions from c_lib to Cython #17881

Closed
jdemeyer opened this issue Mar 2, 2015 · 8 comments
Closed

Move memory functions from c_lib to Cython #17881

jdemeyer opened this issue Mar 2, 2015 · 8 comments
Labels
c: c lib p: major / 3 t: enhancement
Milestone
sage-6.6

Comments

@jdemeyer
Copy link

jdemeyer commented Mar 2, 2015

Note: in src/setup.py, we change the location of c_lib includes to $SAGE_SRC/c_lib/include instead of the installed location $SAGE_LOCAL/include/csage since the latter is not automatically cleaned (installed files just remain there).

Component: c_lib

Author: Jeroen Demeyer

Branch/Commit: 145de10

Reviewer: François Bissey

Issue created by migration from https://trac.sagemath.org/ticket/17881
@jdemeyer jdemeyer added this to the sage-6.6 milestone Mar 2, 2015
@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2015

Branch: u/jdemeyer/ticket/17881

@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2015

New commits:

145de10Move memory functions to Cython

@jdemeyer
Copy link
Author

jdemeyer commented Mar 2, 2015

Commit: 145de10

@kiwifb
Copy link
Member

kiwifb commented Mar 4, 2015

comment:4

If we are not going to use installed headers we may as well not install them at all. But then I guess we can keep that for the final removal.

@kiwifb
Copy link
Member

kiwifb commented Mar 4, 2015

comment:5

Send it to the bots.

@kiwifb
Copy link
Member

kiwifb commented Mar 4, 2015

Reviewer: François Bissey

@vbraun
Copy link
Member

vbraun commented Mar 5, 2015

Changed branch from u/jdemeyer/ticket/17881 to 145de10

@vbraun vbraun closed this as completed in 1ac5d82 Mar 5, 2015
This was referenced Mar 5, 2015
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: c lib p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-6.6
Development

No branches or pull requests
3 participants
@kiwifb @vbraun @jdemeyer