From 83b6e40d249486da3350bdafbdddf847296b7aae Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 12:41:48 +0100 Subject: [PATCH 1/7] Enable the ABI check job to publish the expected ABI data --- .github/workflows/build.yml | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 3ada845c4cbbc3..102202c493e4d4 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -100,6 +100,7 @@ jobs: ./configure --enable-shared make -j4 - name: Check for changes in the ABI + id: check run: | if ! make check-abidump; then echo "Generated ABI file is not up to date." @@ -109,6 +110,16 @@ jobs: echo "" exit 1 fi + - name: Generate updated ABI files + if: ${{ failure() && steps.check.conclusion == 'failure' }} + run: | + make regen-abidump + - uses: actions/upload-artifact@v3 + name: Publish updated ABI files + if: ${{ failure() && steps.check.conclusion == 'failure' }} + with: + name: abi-data + path: ./Doc/data check_generated_files: name: 'Check if generated files are up to date' From ea1d77093af5e1435138570a6877fa7a4783334d Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 12:45:46 +0100 Subject: [PATCH 2/7] Simulating a failure - will revert this before merging! --- Doc/data/python3.12.abi | 9 --------- 1 file changed, 9 deletions(-) diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index 79bf7ea6df829e..ab9ab61de45283 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -5,15 +5,6 @@ - - - - - - - - - From 74f759d9dd2bcce9318c059b1718115272bdd988 Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 12:52:00 +0100 Subject: [PATCH 3/7] Try a different simulated failure --- Doc/data/python3.12.abi | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index ab9ab61de45283..b9532237b72e0f 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -5,6 +5,15 @@ + + + + + + + + + @@ -4625,8 +4634,8 @@ - - + + From d24da92b004111315d010cc558035eb3f033cb64 Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 13:26:03 +0100 Subject: [PATCH 4/7] Revert simulated breakage --- Doc/data/python3.12.abi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index b9532237b72e0f..79bf7ea6df829e 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -4634,8 +4634,8 @@ - - + + From 9860e808469b2125b1493eaa98d7cff08c476421 Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 13:59:58 +0100 Subject: [PATCH 5/7] Filter published file and add simulated breakage --- .github/workflows/build.yml | 2 +- Doc/data/python3.12.abi | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 102202c493e4d4..cdc21312bfb950 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -119,7 +119,7 @@ jobs: if: ${{ failure() && steps.check.conclusion == 'failure' }} with: name: abi-data - path: ./Doc/data + path: ./Doc/data/*.abi check_generated_files: name: 'Check if generated files are up to date' diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index 79bf7ea6df829e..e225a7efdae947 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -1721,7 +1721,7 @@ - + From 05b0606bf1ce1ccef9d4f2fcfbc38367a41cb670 Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 15:10:33 +0100 Subject: [PATCH 6/7] Improved message and more drastic breakage for testing --- .github/workflows/build.yml | 2 ++ Doc/data/python3.12.abi | 6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index cdc21312bfb950..046c2aaa55f67b 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -106,6 +106,8 @@ jobs: echo "Generated ABI file is not up to date." echo "Please add the release manager of this branch as a reviewer of this PR." echo "" + echo "The up to date ABI file should be attached to this build as an artifact." + echo "" echo "To learn more about this check: https://devguide.python.org/setup/#regenerate-the-abi-dump" echo "" exit 1 diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index e225a7efdae947..3d44d0ad2ffdcd 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -9761,11 +9761,11 @@ - - + + - + From 1cfe7f73788e9a85e7d2d0426057323224c1c6bc Mon Sep 17 00:00:00 2001 From: Steve Dower Date: Tue, 30 May 2023 15:54:53 +0100 Subject: [PATCH 7/7] Update ABI data from build --- Doc/data/python3.12.abi | 380 ++++++++++++++++++++-------------------- 1 file changed, 191 insertions(+), 189 deletions(-) diff --git a/Doc/data/python3.12.abi b/Doc/data/python3.12.abi index 3d44d0ad2ffdcd..802cb022235a03 100644 --- a/Doc/data/python3.12.abi +++ b/Doc/data/python3.12.abi @@ -1719,9 +1719,9 @@ - + - + @@ -1729,10 +1729,10 @@ - + - + @@ -1820,11 +1820,11 @@ - + - + @@ -1836,14 +1836,14 @@ - + - + @@ -1862,7 +1862,7 @@ - + @@ -1872,7 +1872,7 @@ - + @@ -1914,13 +1914,13 @@ - + - + @@ -1928,7 +1928,7 @@ - + @@ -2008,7 +2008,7 @@ - + @@ -2039,7 +2039,7 @@ - + @@ -2067,7 +2067,7 @@ - + @@ -3364,7 +3364,7 @@ - + @@ -3418,7 +3418,7 @@ - + @@ -3524,7 +3524,7 @@ - + @@ -3533,7 +3533,7 @@ - + @@ -3589,7 +3589,7 @@ - + @@ -3611,12 +3611,12 @@ - + - + @@ -3633,7 +3633,7 @@ - + @@ -3775,7 +3775,7 @@ - + @@ -3846,7 +3846,7 @@ - + @@ -3905,7 +3905,7 @@ - + @@ -3928,7 +3928,7 @@ - + @@ -4657,7 +4657,7 @@ - + @@ -4691,7 +4691,7 @@ - + @@ -5032,7 +5032,7 @@ - + @@ -5129,7 +5129,7 @@ - + @@ -5317,7 +5317,7 @@ - + @@ -5571,7 +5571,7 @@ - + @@ -5636,7 +5636,7 @@ - + @@ -5658,7 +5658,7 @@ - + @@ -5714,7 +5714,7 @@ - + @@ -5973,7 +5973,7 @@ - + @@ -6090,7 +6090,7 @@ - + @@ -6227,7 +6227,7 @@ - + @@ -6527,7 +6527,7 @@ - + @@ -6536,7 +6536,7 @@ - + @@ -6889,7 +6889,7 @@ - + @@ -6978,7 +6978,7 @@ - + @@ -7171,7 +7171,7 @@ - + @@ -7302,7 +7302,7 @@ - + @@ -7397,7 +7397,7 @@ - + @@ -7426,7 +7426,7 @@ - + @@ -7567,7 +7567,7 @@ - + @@ -7623,7 +7623,7 @@ - + @@ -7637,7 +7637,7 @@ - + @@ -7694,7 +7694,7 @@ - + @@ -7849,7 +7849,7 @@ - + @@ -7877,7 +7877,7 @@ - + @@ -7904,7 +7904,7 @@ - + @@ -8086,7 +8086,7 @@ - + @@ -8098,7 +8098,7 @@ - + @@ -8396,7 +8396,7 @@ - + @@ -8531,7 +8531,7 @@ - + @@ -8576,7 +8576,7 @@ - + @@ -8591,7 +8591,7 @@ - + @@ -8604,7 +8604,7 @@ - + @@ -8632,7 +8632,7 @@ - + @@ -8660,7 +8660,7 @@ - + @@ -8731,7 +8731,7 @@ - + @@ -8750,7 +8750,7 @@ - + @@ -8971,7 +8971,7 @@ - + @@ -8987,7 +8987,7 @@ - + @@ -9009,7 +9009,7 @@ - + @@ -9678,7 +9678,7 @@ - + @@ -9692,7 +9692,7 @@ - + @@ -9710,7 +9710,7 @@ - + @@ -9761,11 +9761,11 @@ - - + + - + @@ -10830,7 +10830,7 @@ - + @@ -10877,10 +10877,10 @@ - + - + @@ -13441,10 +13441,10 @@ - + - + @@ -13473,7 +13473,7 @@ - + @@ -13488,7 +13488,7 @@ - + @@ -13509,13 +13509,13 @@ - + - + - + @@ -13551,7 +13551,7 @@ - + @@ -13652,7 +13652,7 @@ - + @@ -14252,7 +14252,7 @@ - + @@ -14312,7 +14312,7 @@ - + @@ -14366,7 +14366,7 @@ - + @@ -14525,7 +14525,7 @@ - + @@ -14609,7 +14609,7 @@ - + @@ -15089,7 +15089,7 @@ - + @@ -15221,7 +15221,7 @@ - + @@ -15302,7 +15302,7 @@ - + @@ -15374,7 +15374,7 @@ - + @@ -15680,7 +15680,7 @@ - + @@ -15817,7 +15817,7 @@ - + @@ -17369,8 +17369,8 @@ - + @@ -17440,10 +17440,10 @@ - - - - + + + + @@ -17601,7 +17601,7 @@ - + @@ -18891,7 +18891,7 @@ - + @@ -18929,7 +18929,7 @@ - + @@ -20794,11 +20794,13 @@ + + @@ -20935,7 +20937,7 @@ - + @@ -21003,7 +21005,7 @@ - + @@ -21058,7 +21060,7 @@ - + @@ -21083,7 +21085,7 @@ - + @@ -21227,7 +21229,7 @@ - + @@ -21238,17 +21240,17 @@ - + - + - + @@ -21316,7 +21318,7 @@ - + @@ -21336,7 +21338,7 @@ - + @@ -21349,7 +21351,7 @@ - + @@ -21456,7 +21458,7 @@ - + @@ -21505,7 +21507,7 @@ - + @@ -21897,7 +21899,7 @@ - + @@ -22063,7 +22065,7 @@ - + @@ -22154,7 +22156,7 @@ - + @@ -22720,7 +22722,7 @@ - + @@ -22809,7 +22811,7 @@ - + @@ -22820,7 +22822,7 @@ - + @@ -22978,7 +22980,7 @@ - + @@ -23070,53 +23072,53 @@ - - - + + + - - + + - - - - + + + + - - - - + + + + - - - + + + - - - + + + - - + + - - - + + + - - - + + + - + @@ -23126,7 +23128,7 @@ - + @@ -23212,7 +23214,7 @@ - + @@ -23226,7 +23228,7 @@ - + @@ -23331,7 +23333,7 @@ - + @@ -23403,7 +23405,7 @@ - + @@ -23508,21 +23510,21 @@ - + - + - + - + @@ -23531,7 +23533,7 @@ - + @@ -23605,7 +23607,7 @@ - + @@ -23821,7 +23823,7 @@ - + @@ -24162,11 +24164,11 @@ - + - + @@ -24241,7 +24243,7 @@ - + @@ -24260,7 +24262,7 @@ - + @@ -24367,7 +24369,7 @@ - + @@ -24396,7 +24398,7 @@ - + @@ -24412,7 +24414,7 @@ - + @@ -24484,7 +24486,7 @@ - + @@ -24493,7 +24495,7 @@ - + @@ -24529,7 +24531,7 @@ - + @@ -24541,7 +24543,7 @@ - + @@ -24626,7 +24628,7 @@ - + @@ -25346,7 +25348,7 @@ - + @@ -25553,14 +25555,14 @@ - + - + @@ -25579,7 +25581,7 @@ - + @@ -25816,7 +25818,7 @@ - + @@ -26073,7 +26075,7 @@ - + @@ -26104,7 +26106,7 @@ - + @@ -26116,10 +26118,10 @@ - + - + @@ -26333,7 +26335,7 @@ - + @@ -26359,7 +26361,7 @@ - +