Skip to content

Fix typo in version lookup string#157

Open
lviollette wants to merge 1 commit into
trolldbois:masterfrom
lviollette:fix-ctypeslib22-version-lookup
Open

Fix typo in version lookup string#157
lviollette wants to merge 1 commit into
trolldbois:masterfrom
lviollette:fix-ctypeslib22-version-lookup

Conversation

@lviollette
Copy link
Copy Markdown
Contributor

@lviollette lviollette commented May 19, 2026

fix for #156

@trolldbois Why the change ctypeslib2 to ctypeslib22 ?

Was that a typo?

Originally posted by @lviollette in 956e0c1

Current state

Buggy output:

$ clang2py --version                                                    
versions - clang2py:Please install the latest version of this python package python-clang:21.1.7 clang:21.1.8 clang_filename:libclang-21.so.21 

On Ubuntu 22.04, with libclang-21-dev system package installed.
Python 3.12, with packages: clang==21.1.7, ctypeslib2==2.4.0

Proposed fix

On commit 9a352bd, tag v2.4.0

diff --git a/ctypeslib/__init__.py b/ctypeslib/__init__.py                   
index 7774a27..5fc09b8 100644                                                
--- a/ctypeslib/__init__.py                                                  
+++ b/ctypeslib/__init__.py                                                  
@@ -19,7 +19,7 @@ import importlib.metadata                                  
 from importlib.metadata import PackageNotFoundError                         
                                                                             
 try:                                                                        
-    __version__ = importlib.metadata.version('ctypeslib22')                 
+    __version__ = importlib.metadata.version('ctypeslib2')                  
 except PackageNotFoundError:                                                
     __version__ = 'Please install the latest version of this python package'

new output:

$ clang2py --version   
versions - clang2py:2.4.0 python-clang:21.1.7 clang:21.1.8 clang_filename:libclang-21.so.21           

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 83.58% (-0.1%) from 83.676% — lviollette:fix-ctypeslib22-version-lookup into trolldbois:master

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.

2 participants