-
Notifications
You must be signed in to change notification settings - Fork 0
Add Windows native executable support via PyInstaller #2
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
Conversation
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
Co-authored-by: cvv2com <44297426+cvv2com@users.noreply.github.com>
|
@copilot eline saglık. sanirim bitirdin. |
cvv2com
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
look nice :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request adds comprehensive Windows native executable support to the CVV2.NET application using PyInstaller. The user requested a Windows-native version similar to compiled .NET/VB.NET applications. Instead of a complete rewrite, the implementation provides PyInstaller-based executables that achieve the same goal with minimal code changes.
Changes:
- Added Windows batch scripts for one-click setup and execution (setup_windows.bat, run_cvv2net.bat, run_ocr_extractor.bat, build_windows_exe.bat)
- Created PyInstaller configuration files (.spec files) for building standalone Windows executables
- Added comprehensive bilingual (Turkish/English) documentation covering quick start, detailed guides, and troubleshooting
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 50 comments.
Show a summary per file
| File | Description |
|---|---|
| setup_windows.bat | Automated setup script with Python detection, dependency installation, and Tesseract checking |
| run_cvv2net.bat | Launcher for text-based scanner with dependency validation |
| run_ocr_extractor.bat | Launcher for OCR extractor with Tesseract checks |
| build_windows_exe.bat | PyInstaller-based executable builder for creating standalone .exe files |
| cvv2net.spec | PyInstaller configuration for cvv2net executable |
| ocr_card_extractor.spec | PyInstaller configuration for OCR extractor executable |
| WINDOWS_README.md | Quick start guide with 3-step setup instructions |
| WINDOWS_GUIDE.md | Comprehensive installation and troubleshooting guide |
| WINDOWS_NATIVE.md | Technical rationale explaining the PyInstaller approach |
| START_HERE_WINDOWS.md | Entry point documentation for Windows users |
| IMPLEMENTATION_SUMMARY.md | Detailed implementation notes and testing recommendations |
| CHANGES.md | Complete change log documenting all additions |
| requirements.txt | Added pyinstaller>=5.0.0 dependency |
| README.md | Updated with Windows-specific usage section and three usage methods |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| python --version >nul 2>&1 | ||
| if errorlevel 1 ( | ||
| echo HATA: Python bulunamadi / ERROR: Python not found | ||
| echo Lutfen Python 3 yukleyin / Please install Python 3 |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multiple spelling errors in Turkish. "Lutfen" should be "Lütfen", "yukleyin" should be "yükleyin".
| echo Lutfen Python 3 yukleyin / Please install Python 3 | |
| echo Lütfen Python 3 yükleyin / Please install Python 3 |
| ) | ||
|
|
||
| echo. | ||
| echo ocr_card_extractor.py calistiriliyor / Running ocr_card_extractor.py... |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling errors in Turkish. "calistiriliyor" should be "çalıştırılıyor".
| echo ocr_card_extractor.py calistiriliyor / Running ocr_card_extractor.py... | |
| echo ocr_card_extractor.py çalıştırılıyor / Running ocr_card_extractor.py... |
| echo. | ||
| echo HATA: cvv2net.exe olusturulamadi / ERROR: Failed to build cvv2net.exe | ||
| echo. | ||
| echo Olasi Cozumler / Possible Solutions: |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error in Turkish. "Olasi" should be "Olası", "Cozumler" should be "Çözümler".
| @@ -0,0 +1,366 @@ | |||
| # Windows Kurulum ve Kullanım Kılavuzu / Windows Installation and Usage Guide | |||
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error in Turkish. "yaşıyorsunuz" is incorrectly written as "yaşıyorsunuz" in the text but appears correct. However, "Kilavuzu" should be "Kılavuzu".
| ``` | ||
| Çift tıkla / Double-click | ||
|
|
||
| ### 3️⃣ Çalıştır / Run |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error: "Calistir" should be "Çalıştır".
| pyinstaller --clean cvv2net.spec | ||
| if errorlevel 1 ( | ||
| echo. | ||
| echo HATA: cvv2net.exe olusturulamadi / ERROR: Failed to build cvv2net.exe |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling errors in Turkish. "olusturulamadi" should be "oluşturulamadı".
| echo BASARILI! / SUCCESS! | ||
| echo ====================================================================== | ||
| echo. | ||
| echo Dosyalar olusturuldu / Files created: |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling errors in Turkish. "olusturuldu" should be "oluşturuldu".
| echo Dosyalar olusturuldu / Files created: | |
| echo Dosyalar oluşturuldu / Files created: |
| echo - dist\cvv2net.exe | ||
| echo - dist\ocr_card_extractor.exe | ||
| echo. | ||
| echo Bu .exe dosyalari baska Windows bilgisayarlarda calistirilabilir. |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling errors in Turkish. "calistirilabilir" should be "çalıştırılabilir".
| echo Bu .exe dosyalari baska Windows bilgisayarlarda calistirilabilir. | |
| echo Bu .exe dosyalari baska Windows bilgisayarlarda çalıştırılabilir. |
| echo These .exe files can be run on other Windows computers. | ||
| echo. | ||
| echo NOTLAR / NOTES: | ||
| echo - ocr_card_extractor.exe icin Tesseract OCR ayri yuklenmelidir |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling errors in Turkish. "yuklenmelidir" should be "yüklenmelidir".
| echo - ocr_card_extractor.exe icin Tesseract OCR ayri yuklenmelidir | |
| echo - ocr_card_extractor.exe icin Tesseract OCR ayri yüklenmelidir |
| REM Bu script Python uygulamalarini Windows .exe dosyalarina donusturur | ||
| REM This script converts Python applications to Windows .exe files | ||
|
|
||
| echo ====================================================================== | ||
| echo CVV2.NET - Windows Executable Builder | ||
| echo PyInstaller ile EXE Olusturucu / EXE Builder with PyInstaller |
Copilot
AI
Jan 20, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error: "Olusturucu" should be "Oluşturucu", "donusturur" should be "dönüştürür".
| REM Bu script Python uygulamalarini Windows .exe dosyalarina donusturur | |
| REM This script converts Python applications to Windows .exe files | |
| echo ====================================================================== | |
| echo CVV2.NET - Windows Executable Builder | |
| echo PyInstaller ile EXE Olusturucu / EXE Builder with PyInstaller | |
| REM Bu script Python uygulamalarini Windows .exe dosyalarina dönüştürür | |
| REM This script converts Python applications to Windows .exe files | |
| echo ====================================================================== | |
| echo CVV2.NET - Windows Executable Builder | |
| echo PyInstaller ile EXE Oluşturucu / EXE Builder with PyInstaller |
User requested Windows-native version similar to compiled .NET/VB.NET applications. Implemented PyInstaller-based solution that creates standalone Windows executables while preserving the Python codebase.
Implementation
Batch Scripts - One-click execution
setup_windows.bat- Automated setup with dependency validationrun_cvv2net.bat/run_ocr_extractor.bat- Application launchersbuild_windows_exe.bat- EXE builder using PyInstallerPyInstaller Configuration
.specfiles for both applicationsDocumentation (bilingual TR/EN)
START_HERE_WINDOWS.md- Quick referenceWINDOWS_GUIDE.md- Full setup and troubleshootingWINDOWS_NATIVE.md- Technical rationaleREADME.mdwith Windows sectionUsage
Three methods available:
Technical Notes
ocr_card_extractor.exeOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.