Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "clib\__init__.py" in “python\opencc” #693

Merged
merged 1 commit into from
Aug 14, 2022

Conversation

forFudan
Copy link
Contributor

@forFudan forFudan commented Jun 4, 2022

System description: Windows x64. Python version: 3.9.

Running "python setup.py build" results in the following error:

copying python\opencc\__init__.py -> build\lib.win-amd64-3.9\opencc
package directory 'python\opencc\clib' does not exist.

This is because the package 'opencc.clib' is imported in 'python\opencc_init_.py' (Line 6) before the function "build_libopencc()" in 'setup.py' creates the directory "python\opencc\clib" (Line 83).

By adding the file "clib_init_.py" in the directory “python\opencc”, the command "python setup.py build" can successfully build the files.

I do not know whether this is also related to the issue that the wheel files cannot be created on PyPI for Windows for the recent versions (>1.1.1)

System description: Windows x64. Python version: 3.9.

Running "python setup.py build" results in the following error: 
```
copying python\opencc\__init__.py -> build\lib.win-amd64-3.9\opencc
package directory 'python\opencc\clib' does not exist.
```

This is because the package 'opencc.clib' is imported in 'python\opencc\__init__.py' (Line 6)  before the function "build_libopencc()" in 'setup.py' creates the directory "python\opencc\clib" (Line 83).

By adding the file "clib\__init__.py" in the directory “python\opencc”, the command "python setup.py build" can successfully build the files.

I do not know whether this is also related to the issue that the wheel files cannot be created on PyPI for Windows for the recent versions (>1.1.1)
@forFudan
Copy link
Contributor Author

forFudan commented Jun 4, 2022

#651 (comment)
#593

@BYVoid BYVoid merged commit 8dc0541 into BYVoid:master Aug 14, 2022
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