Skip to content

Add Python3.11 to the release/8.2 Mac pkg distribution#2073

Merged
nrnhines merged 4 commits into
release/8.2from
hines/8.2-py311
Nov 9, 2022
Merged

Add Python3.11 to the release/8.2 Mac pkg distribution#2073
nrnhines merged 4 commits into
release/8.2from
hines/8.2-py311

Conversation

@nrnhines

@nrnhines nrnhines commented Nov 9, 2022

Copy link
Copy Markdown
Member

This also fixes a segfault due to not checking that a pointer is NULL.

If not tagged the distribution identifier will be something like

% git describe
8.2.1-3-ge41ab1de9

@pramodk pramodk requested a review from olupton November 9, 2022 08:25
@azure-pipelines

Copy link
Copy Markdown

✔️ e41ab1d -> Azure artifacts URL

@alexsavulescu alexsavulescu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. I guess the failures we see have been fixed in master

@nrnhines

nrnhines commented Nov 9, 2022

Copy link
Copy Markdown
Member Author

LGTM. I guess the failures we see have been fixed in master

I'll cherry pick the fixes from master into release/8.2

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #2073 (e5c6173) into release/8.2 (eb19ae0) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           release/8.2    #2073   +/-   ##
============================================
  Coverage        47.35%   47.35%           
============================================
  Files              543      543           
  Lines           112688   112688           
============================================
  Hits             53367    53367           
  Misses           59321    59321           
Impacted Files Coverage Δ
src/nrniv/bgpdma.cpp 81.89% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@azure-pipelines

Copy link
Copy Markdown

✔️ e5c6173 -> Azure artifacts URL

@nrnhines nrnhines merged commit b246509 into release/8.2 Nov 9, 2022
@nrnhines nrnhines deleted the hines/8.2-py311 branch November 9, 2022 20:06
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>
@alexsavulescu alexsavulescu mentioned this pull request Dec 15, 2022
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants