Opened 10 years ago

Closed 4 years ago

#9892 closed enhancement (fixed)

Eliminate paripriv.h from interpreters

Reported by: jdemeyer Owned by: jdemeyer
Priority: minor Milestone: sage-6.9
Component: cython Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 951372c (Commits) Commit: 951372cd66820527e1ec873608e1c09f065cb4d1
Dependencies: Stopgaps:

Description (last modified by jdemeyer)

Now that <pari/paripriv.h> is only used in a few Cython .pyx files (hence, a few .c files), we don't need to include it in the interpreters in src/sage/ext/interpreters.

Change History (10)

comment:1 Changed 10 years ago by jdemeyer

  • Component changed from packages to interfaces
  • Description modified (diff)
  • Owner changed from tbd to was

comment:2 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:3 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:4 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:5 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:6 Changed 5 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Component changed from interfaces to cython
  • Description modified (diff)
  • Milestone changed from sage-6.4 to sage-6.9
  • Owner changed from was to jdemeyer
  • Summary changed from Eliminate (or minimize) #include <pari/paripriv.h> to Eliminate paripriv.h from interpreters

comment:7 Changed 5 years ago by jdemeyer

  • Branch set to u/jdemeyer/eliminate__or_minimize___include__pari_paripriv_h_

comment:8 Changed 5 years ago by jdemeyer

  • Commit set to 951372cd66820527e1ec873608e1c09f065cb4d1
  • Status changed from new to needs_review

New commits:

951372cStop including paripriv.h

comment:9 Changed 4 years ago by chapoton

  • Reviewers set to Frédéric Chapoton
  • Status changed from needs_review to positive_review

ok, the bot is happy.

comment:10 Changed 4 years ago by vbraun

  • Branch changed from u/jdemeyer/eliminate__or_minimize___include__pari_paripriv_h_ to 951372cd66820527e1ec873608e1c09f065cb4d1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.