[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-05-09 Thread Ryan Mast via cfe-commits
nightlark wrote: Yes, still looking into this -- got a bit busier through around mid-May though. https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-04-24 Thread Jannick Kremer via cfe-commits
DeinAlptraum wrote: @nightlark (ping) are you still looking into this? https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-06 Thread Henry Schreiner via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] henryiii wrote: Is there a reason you've pinned to setuptools_scm exactly, but not setuptools? I would generally avoid this, unless there's a really good reason to pin it.

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-06 Thread Henry Schreiner via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: @nightlark Thank you for the write-up about things that can go wrong. > open question: what email to list as the author? This is a good question I don't know answer to. Debian packages from our APT repository seems to specify `LLVM Packaging Team ` as the maintainer and their em

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Loïc Jaquemet via cfe-commits
trolldbois wrote: > > Also I barely know anything about how to package this stuff correctly, and > > what the consequences would be if we get it wrong. > > Also regarding @Endilll. Nothing can go wrong really. This is still not > connected to the CD and you would need permission from [`clang`

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Cristian Le via cfe-commits
LecrisUT wrote: Ok, can you preface the sentences with a "check if" so it doesn't create confusions :sweat_smile: https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Cristian Le via cfe-commits
LecrisUT wrote: > Also I barely know anything about how to package this stuff correctly, and > what the consequences would be if we get it wrong. Also regarding @Endilll. Nothing can go wrong really. This is still not connected to the CD and you would need permission from [`clang` pypi projec

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Ryan Mast via cfe-commits
nightlark wrote: @LecrisUT all of the situations listed work fine -- the top level bullet points are the "hypothetical issues" that someone trying to use it have encountered, and the sub-bullet points are ways to check that the potential issue was addressed and is not an issue. https://githu

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Cristian Le via cfe-commits
LecrisUT wrote: > - The python bindings aren't placed in the right folder to import them when > the package is pip installed They are *not*? I tried it and it imported fine, same as you noticed. > - Installing directly from a git URL or source archives for a release won't > work Similarly,

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Ryan Mast via cfe-commits
nightlark wrote: For reviewers, here's a summary of the potential issues that come to mind and what has been done to verify things are working as expected: * The packaging metadata (keywords, name, author/contact info, etc) don't match what we expect - check the pyproject.toml and make sur

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-05 Thread Ryan Mast via cfe-commits
nightlark wrote: @Endilll - do you have way to contact them directly to tell them about this PR? I suspect they are not seeing any notifications for the mentions here, and will require telling them "in-person" at a meeting. I have a PR (opened >1 month ago) in the repository for the libclang p

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-01 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: > @Endilll @LecrisUT what else do you think needs to be done here to get this > approved? I still would like to hear from @sighingnow. One of the reasons is that I think I've seen them attending one of the open meetings we have, so they might be around somewhere. Also I barely

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-01 Thread Cristian Le via cfe-commits
https://github.com/LecrisUT edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-03-01 Thread Cristian Le via cfe-commits
https://github.com/LecrisUT approved this pull request. My review is that it looks good to go for the scope that it serves. Follow up changes I would recommend is to align the CI and `pyproject.toml` across the other python projects, but it seems that it has been broken for a while, so it's a

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-28 Thread Jannick Kremer via cfe-commits
DeinAlptraum wrote: @Endilll @LecrisUT what else do you think needs to be done here to get this approved? https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/l

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-17 Thread Cristian Le via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-17 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-07 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-07 Thread Ryan Mast via cfe-commits
https://github.com/nightlark edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-07 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Vlad Serebrennikov via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Jannick Kremer via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum commented: Sorry, saw this but had no time to answer. Unfortunately I'm also not familiar with Python packaging so can't be of much help here https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Vlad Serebrennikov via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Vlad Serebrennikov via cfe-commits
@@ -0,0 +1,36 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm==8.1.0"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = {text

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll commented: Thank you for working on this I know too little of packaging in Python to approve, but let's try to get more eyes on this. CC @DeinAlptraum @sighingnow https://github.com/llvm/llvm-project/pull/125806 ___ cfe-com

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,3 @@ +node: $Format:%H$ +node-date: $Format:%cI$ +describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$ nightlark wrote: Another thought, I expect most end users will eventually be installing from a PyPI package -- the sdist and wheel files uplo

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
nightlark wrote: @Endilll the Python bindings seem like part of LLVM that you have expertise in, if you also want to review this PR. https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
https://github.com/nightlark edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
https://github.com/nightlark updated https://github.com/llvm/llvm-project/pull/125806 >From a4c09eda556104fbe5a8acdb830215b2a4ebd131 Mon Sep 17 00:00:00 2001 From: Ryan Mast Date: Tue, 4 Feb 2025 21:59:28 -0800 Subject: [PATCH] [libclang/python] Enable packaging libclang bindings --- .gitattr

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-06 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,3 @@ +node: $Format:%H$ +node-date: $Format:%cI$ +describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$ nightlark wrote: I didn't see an option for `git describe` to list out describes relative to all possible tags -- just listing all tags might

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Cristian Le via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Cristian Le via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Cristian Le via cfe-commits
@@ -0,0 +1,3 @@ +node: $Format:%H$ +node-date: $Format:%cI$ +describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$ LecrisUT wrote: Awesome findings. I could go either way on if it's expected behaviour or not. Iirc the current format does not include all ta

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
https://github.com/nightlark edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
https://github.com/nightlark updated https://github.com/llvm/llvm-project/pull/125806 >From 4dd3356f37229c415c6b6e3dfec8c8020122124d Mon Sep 17 00:00:00 2001 From: Ryan Mast Date: Tue, 4 Feb 2025 21:59:28 -0800 Subject: [PATCH] [libclang/python] Enable packaging libclang bindings --- .gitattr

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
https://github.com/nightlark edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,3 @@ +node: $Format:%H$ +node-date: $Format:%cI$ +describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$ nightlark wrote: The more open ended match is the standard one that setuptools_scm documents, but we can make it more flexible. I tried the

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-05 Thread Ryan Mast via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Cristian Le via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Cristian Le via cfe-commits
https://github.com/LecrisUT commented: Other than missing new-line at the end of file, it looks fine. License classifiers will soon be (or already are) deprecated, so check up with setuptools and when they support PEP639. https://github.com/llvm/llvm-project/pull/125806 ___

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Cristian Le via cfe-commits
@@ -0,0 +1,34 @@ +[build-system] +requires = ["setuptools>=42", "setuptools_scm"] +build-backend = "setuptools.build_meta" + +[project] +name = "clang" +description = "libclang python bindings" +readme = {file = "README.txt", content-type = "text/plain"} + +license = { text = "Apa

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Cristian Le via cfe-commits
@@ -0,0 +1,3 @@ +node: $Format:%H$ +node-date: $Format:%cI$ +describe-name: $Format:%(describe:tags=true,match=*[0-9]*)$ LecrisUT wrote: Nitpick about the describe tags. Why not be more explicit with `llvmorg-[0-9]*]`? Anyway did you double check that it works

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Cristian Le via cfe-commits
https://github.com/LecrisUT edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Ryan Mast via cfe-commits
nightlark wrote: Hey, I'm new to the process of contributing to LLVM, so let me know of any best practices or things I might have missed in the contributed guide. I'm looking at improving the situation for packaging libclang bindings on PyPI and ideally would like maintain the things needed he

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Ryan Mast (nightlark) Changes Add files for packaging libclang Python bindings as a sdist tarball and pure Python wheel. setuptools_scm is used to derive version numbers from git tags for a future workflow that automates publishing update

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Ryan Mast via cfe-commits
https://github.com/nightlark edited https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-04 Thread Ryan Mast via cfe-commits
https://github.com/nightlark created https://github.com/llvm/llvm-project/pull/125806 Add files for packaging libclang Python bindings as a sdist tarball and pure Python wheel. setuptools_scm is used to derive version numbers from git tags for a future workflow that automates publishing update