#29720 closed enhancement (fixed)

Move Extension options from src/module_list.py to "distutils:" directives in the individual files (part 2 - OptionalExtensions)

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.2
Component: refactoring Keywords: sd109
Cc: fbissey, dimpase, jhpalmieri, tscrim, gh-kliem, roed, vdelecroix, vbraun Merged in:
Authors: Matthias Koeppe Reviewers: Jonathan Kliem
Report Upstream: N/A Work issues:
Branch: 2821934 (Commits, GitHub, GitLab) Commit: 28219340dd0eb3d939dcd0c7e9fbc89133490a0c
Dependencies: Stopgaps:

Status badges

Description (last modified by mkoeppe)

Follow-up from #29706.

In this part, we take care of all OptionalExtensions except for

Change History (14)

comment:1 Changed 21 months ago by mkoeppe

  • Type changed from PLEASE CHANGE to enhancement

comment:2 Changed 21 months ago by mkoeppe

  • Branch set to u/mkoeppe/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files__part_2___optionalextensions_

comment:3 Changed 20 months ago by mkoeppe

  • Commit set to 69004cda88b8754b4cf0eea6010af487b062ff7c
  • Keywords sd109 added

New commits:

69004cdsrc/sage: Move OptionalExtension options from src/module_list.py to distutils directives

comment:4 Changed 20 months ago by mkoeppe

  • Description modified (diff)

comment:5 Changed 20 months ago by mkoeppe

  • Description modified (diff)

comment:6 Changed 20 months ago by git

  • Commit changed from 69004cda88b8754b4cf0eea6010af487b062ff7c to 9805a19ba05d34df245cbde95d1a0be14a60bf41

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

9805a19src/sage: Move OptionalExtension options from src/module_list.py to distutils directives

comment:7 Changed 20 months ago by git

  • Commit changed from 9805a19ba05d34df245cbde95d1a0be14a60bf41 to 401a864da70df1fe2b60ae4371644b4ff054bbdd

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

401a864src/sage: Move OptionalExtension options (except tdlib) from src/module_list.py to distutils directives

comment:8 Changed 20 months ago by git

  • Commit changed from 401a864da70df1fe2b60ae4371644b4ff054bbdd to bfd9fedc42fe0b770edf87cf56e488a21e6099ca

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

bfd9fedsrc/sage: Move OptionalExtension options (except tdlib, coxeter) from src/module_list.py to distutils directives

comment:9 Changed 20 months ago by mkoeppe

  • Authors set to Matthias Koeppe
  • Status changed from new to needs_review

fes does not really exist (it was an old-style experimental package that was never converted to new-style), but it's not the job of this ticket to remove the corresponding extension.

comment:10 Changed 20 months ago by git

  • Commit changed from bfd9fedc42fe0b770edf87cf56e488a21e6099ca to 28219340dd0eb3d939dcd0c7e9fbc89133490a0c

Branch pushed to git repo; I updated commit sha1. New commits:

2821934Fix sage_setup directives: Use distribution, not package

comment:11 Changed 20 months ago by mkoeppe

  • Description modified (diff)

comment:12 Changed 20 months ago by gh-kliem

  • Reviewers set to Jonathan Kliem
  • Status changed from needs_review to positive_review

LGTM.

comment:13 Changed 20 months ago by mkoeppe

Thanks!

comment:14 Changed 19 months ago by vbraun

  • Branch changed from u/mkoeppe/move_extension_options_from_src_module_list_py_to__distutils___directives_in_the_individual_files__part_2___optionalextensions_ to 28219340dd0eb3d939dcd0c7e9fbc89133490a0c
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.