Opened 7 years ago

Closed 7 years ago

#17881 closed enhancement (fixed)

Move memory functions from c_lib to Cython

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-6.6
Component: c_lib Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: François Bissey
Report Upstream: N/A Work issues:
Branch: 145de10 (Commits, GitHub, GitLab) Commit: 145de10aca8586c5051363347ec921edc6693e12
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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).

Change History (6)

comment:1 Changed 7 years ago by jdemeyer

  • Description modified (diff)

comment:2 Changed 7 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/17881

comment:3 Changed 7 years ago by jdemeyer

  • Commit set to 145de10aca8586c5051363347ec921edc6693e12
  • Status changed from new to needs_review

New commits:

145de10Move memory functions to Cython

comment:4 Changed 7 years ago by fbissey

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.

comment:5 Changed 7 years ago by fbissey

  • Reviewers set to François Bissey
  • Status changed from needs_review to positive_review

Send it to the bots.

comment:6 Changed 7 years ago by vbraun

  • Branch changed from u/jdemeyer/ticket/17881 to 145de10aca8586c5051363347ec921edc6693e12
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.