Skip to content

Conversation

bjornjorgensen
Copy link
Contributor

@bjornjorgensen bjornjorgensen commented Aug 31, 2025

What changes were proposed in this pull request?

update to the SPDX license https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license

Why are the changes needed?

when we build pyspark we get an warning

Please consider removing the following classifiers in favor of a SPDX license expression:

    License :: OSI Approved :: Apache Software License

    See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
image

With the PR

image

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass CI/CD test and check in the builds log to see if the deprecated warning is there.

Was this patch authored or co-authored using generative AI tooling?

No.

@bjornjorgensen bjornjorgensen changed the title update-python-to-SPDX-license [SPARK-53443][PYTHON] Update python to SPDX-license Aug 31, 2025
@bjornjorgensen bjornjorgensen changed the title [SPARK-53443][PYTHON] Update python to SPDX-license [SPARK-53443][PYTHON] Update python to SPDX-license Aug 31, 2025
@@ -342,7 +342,7 @@ def run(self):
"pyspark.examples.src.main.python": ["*.py", "*/*.py"],
},
scripts=scripts,
license="http://www.apache.org/licenses/LICENSE-2.0",
license="Apache-2.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Let's also fix up python/packaging/client/setup.py and python/packaging/connect/setup.py.

@bjornjorgensen
Copy link
Contributor Author

now we have all places
image

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @bjornjorgensen and @HyukjinKwon .
Merged to main.

I believe we can verify this via our Daily release dry run.

@bjornjorgensen bjornjorgensen deleted the update-python-to-SPDX-license branch September 2, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants