-
-
Notifications
You must be signed in to change notification settings - Fork 121
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Summary: This merges the upstream changes from 3.12.2 -> 3.12.3 into our fork. Merge-point in upstream is commit `f6650f9ad73359051f3e558c2431a109bc016664` (also pushed in cinder github as a `meta/3.12.3-base` tag). Merging required a few manual merge conflict resolutions, called out as diff comments, and summarized here: 1. `patchlevel.h` -- update the meta version patch 1. `Python/initconfig.c` -- fixed collision between the lazy imports env var and a new upstream env var 1. `Lib/test/test_property.py` -- fixed conflict between a test added upstream and the backported deletion of tests that were in the same place (D52014322) 1. `Tools/msi/bundle/bundle.wxs` -- customizations done in D53734154 conflicted with an upstream change Reviewed upstream changes to configure and Makefile to look for needed buckification changes: 1. upstream added thread sanitizer configure flag - buckification not affected 1. some changes in sbom generation - not relevant to buckification 1. some tests moved around - buckification not affected Reviewed added/removed files in case they need to be reflected in the buckification: ``` # (on the git merge commit) $ git diff --name-status HEAD^ HEAD | sort | less A .github/workflows/reusable-tsan.yml A Doc/c-api/hash.rst A Doc/howto/gdb_helpers.rst A Lib/test/libregrtest/tsan.py A Lib/test/test_doctest/decorator_mod.py A Lib/test/test_pydoc/__init__.py A Lib/test/test_tools/test_makefile.py A Lib/test/zipdir_backslash.zip A Misc/NEWS.d/3.12.3.rst A Misc/externals.spdx.json A Tools/scripts/sortperf.py A Tools/tsan/supressions.txt D Lib/test/sortperf.py ``` Reviewed By: fried Differential Revision: D55930999 fbshipit-source-id: d6bea295c376f955cf7df39d69dec60c61531c68
- Loading branch information
1 parent
a0199fd
commit 23c541e
Showing
453 changed files
with
15,342 additions
and
8,701 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ jobs: | |
|
||
steps: | ||
- uses: actions/checkout@v4 | ||
- uses: actions/setup-python@v4 | ||
- uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.x" | ||
- uses: pre-commit/[email protected].0 | ||
- uses: pre-commit/[email protected].1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,6 +17,7 @@ jobs: | |
HOMEBREW_NO_ANALYTICS: 1 | ||
HOMEBREW_NO_AUTO_UPDATE: 1 | ||
HOMEBREW_NO_INSTALL_CLEANUP: 1 | ||
HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK: 1 | ||
PYTHONSTRICTEXTENSIONBUILD: 1 | ||
strategy: | ||
fail-fast: false | ||
|
@@ -28,11 +29,13 @@ jobs: | |
runs-on: ${{ matrix.os }} | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Runner image version | ||
run: echo "IMAGE_VERSION=${ImageVersion}" >> $GITHUB_ENV | ||
- name: Restore config.cache | ||
uses: actions/cache@v3 | ||
uses: actions/cache@v4 | ||
with: | ||
path: config.cache | ||
key: ${{ github.job }}-${{ matrix.os }}-${{ inputs.config_hash }} | ||
key: ${{ github.job }}-${{ matrix.os }}-${{ env.IMAGE_VERSION }}-${{ inputs.config_hash }} | ||
- name: Install Homebrew dependencies | ||
run: brew install pkg-config [email protected] xz gdbm tcl-tk | ||
- name: Configure CPython | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
on: | ||
workflow_call: | ||
inputs: | ||
config_hash: | ||
required: true | ||
type: string | ||
options: | ||
required: true | ||
type: string | ||
|
||
jobs: | ||
build_tsan_reusable: | ||
name: 'Thread sanitizer' | ||
runs-on: ubuntu-22.04 | ||
timeout-minutes: 60 | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Runner image version | ||
run: echo "IMAGE_VERSION=${ImageVersion}" >> $GITHUB_ENV | ||
- name: Restore config.cache | ||
uses: actions/cache@v4 | ||
with: | ||
path: config.cache | ||
key: ${{ github.job }}-${{ runner.os }}-${{ env.IMAGE_VERSION }}-${{ inputs.config_hash }} | ||
- name: Install Dependencies | ||
run: | | ||
sudo ./.github/workflows/posix-deps-apt.sh | ||
sudo apt install -y clang | ||
# Reduce ASLR to avoid TSAN crashing | ||
sudo sysctl -w vm.mmap_rnd_bits=28 | ||
- name: TSAN Option Setup | ||
run: | | ||
echo "TSAN_OPTIONS=suppressions=${GITHUB_WORKSPACE}/Tools/tsan/supressions.txt" >> $GITHUB_ENV | ||
echo "CC=clang" >> $GITHUB_ENV | ||
echo "CXX=clang++" >> $GITHUB_ENV | ||
- name: Add ccache to PATH | ||
run: | | ||
echo "PATH=/usr/lib/ccache:$PATH" >> $GITHUB_ENV | ||
- name: Configure ccache action | ||
uses: hendrikmuhs/[email protected] | ||
with: | ||
save: ${{ github.event_name == 'push' }} | ||
max-size: "200M" | ||
- name: Configure CPython | ||
run: ${{ inputs.options }} | ||
- name: Build CPython | ||
run: make -j4 | ||
- name: Display build info | ||
run: make pythoninfo | ||
- name: Tests | ||
run: ./python -m test --tsan -j4 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.