Skip to content

SETUPTOOLS_USE_DISTUTILS=stdlib#1924

Merged
alexsavulescu merged 3 commits into
masterfrom
stp
Jul 19, 2022
Merged

SETUPTOOLS_USE_DISTUTILS=stdlib#1924
alexsavulescu merged 3 commits into
masterfrom
stp

Conversation

@alexsavulescu

Copy link
Copy Markdown
Member

Closes #1605 #1921

Still in need to tackle distutils deprecation though.

@codecov-commenter

codecov-commenter commented Jul 18, 2022

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.20%. Comparing base (f505247) to head (55b7263).
⚠️ Report is 2923 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1924      +/-   ##
==========================================
- Coverage   47.24%   47.20%   -0.04%     
==========================================
  Files         543      543              
  Lines      112941   112941              
==========================================
- Hits        53354    53309      -45     
- Misses      59587    59632      +45     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@azure-pipelines

Copy link
Copy Markdown

✔️ d6c92b7 -> Azure artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 2544fa8 -> Azure artifacts URL

Comment thread src/nrnpython/CMakeLists.txt Outdated

@olupton olupton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@azure-pipelines

Copy link
Copy Markdown

✔️ 55b7263 -> Azure artifacts URL

@alexsavulescu alexsavulescu merged commit 962533e into master Jul 19, 2022
@alexsavulescu alexsavulescu deleted the stp branch July 19, 2022 09:17
nrnhines pushed a commit that referenced this pull request Nov 9, 2022
* Workaround for: #1605 See:
   * https://setuptools.pypa.io/en/latest/deprecated/distutils-legacy.html
   * https://setuptools.pypa.io/en/latest/history.html#id228

Co-authored-by: Olli Lupton <oliver.lupton@epfl.ch>
cherry-pick from master 962533e to allow build with Python3.11
nrnhines added a commit that referenced this pull request Nov 9, 2022
* SETUPTOOLS_USE_DISTUTILS=stdlib (#1924)

* Workaround for: #1605 See:
   * https://setuptools.pypa.io/en/latest/deprecated/distutils-legacy.html
   * https://setuptools.pypa.io/en/latest/history.html#id228

Co-authored-by: Olli Lupton <oliver.lupton@epfl.ch>
cherry-pick from master 962533e to allow build with Python3.11

* check for NULL. (already fixed in master 962533e)

* Add python3.11 to pkg build. Some hints for the build script.

Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
@alexsavulescu alexsavulescu mentioned this pull request Nov 9, 2022
3 tasks
pramodk added a commit that referenced this pull request Nov 23, 2022
 * removal of distutils.log.Log from setuptools breaks
   building packages like numpy. See pypa/setuptools#3693
 * we have used SETUPTOOLS_USE_DISTUTILS alternative already in #1924
 * update circle-ci and azure pipelines similarly
pramodk added a commit that referenced this pull request Nov 23, 2022
* removal of distutils.log.Log from setuptools breaks
   building packages like numpy. See pypa/setuptools#3693
 * we have used SETUPTOOLS_USE_DISTUTILS alternative already in #1924
 * update circle-ci and azure pipelines similarly
alexsavulescu added a commit that referenced this pull request Dec 15, 2022
* SETUPTOOLS_USE_DISTUTILS=stdlib (#1924)

* Workaround for: #1605 See:
   * https://setuptools.pypa.io/en/latest/deprecated/distutils-legacy.html
   * https://setuptools.pypa.io/en/latest/history.html#id228

Co-authored-by: Olli Lupton <oliver.lupton@epfl.ch>
cherry-pick from master 962533e to allow build with Python3.11

* check for NULL. (already fixed in master 962533e)

* Add python3.11 to pkg build. Some hints for the build script.

Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants