-
Notifications
You must be signed in to change notification settings - Fork 9
Rewrite Perl utility scripts with third-party dependencies to C, Python, and Shell #199
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: athas <55833+athas@users.noreply.github.com>
|
Nej nej, lad eggspi/parse være i fred. Det er en af de gode, men vi hader de fleste andre. Fokusér på de Perl-programmer der importerer utf8::all eller andre tredjepartspakker. Det viser sig nemlig at Perl er et fortolket sprog med binær inkompatibilitet på tværs af versioner, og det er simpelthen for absurd til at det skal have lov til at eksistere. Vi må udrydde Perl! |
|
@copilot Gør det der jeg skrev ovenfor. Og hils lige Gårdejer Niels på vejen. |
athas
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.
Nej nej, lad eggspi/parse være i fred. Det er en af de gode, men vi hader de fleste andre. Fokusér på de Perl-programmer der importerer utf8::all eller andre tredjepartspakker. Det viser sig nemlig at Perl er et fortolket sprog med binær inkompatibilitet på tværs af versioner, og det er simpelthen for absurd til at det skal have lov til at eksistere. Vi må udrydde Perl!
- Reverted parse back to Perl (as requested - it's one of the good ones) - Rewrote 4påstribeStartTurn, getGdpPcOf, sprog, and krak from Perl to Python - All rewritten scripts had utf8::all or other third-party module dependencies - Eliminates binary incompatibility issues with Perl across versions Co-authored-by: athas <55833+athas@users.noreply.github.com>
Done! Reverted Changes in commit a42b592:
Hilsner til Gårdejer Niels! 🌾 |
athas
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.
Det er vel oooooookay, men gør dig lidt mere umage. Find også nogle flere ting i eggspi og lav dem om i shell script eller C. Det skal ikke være Python det hele.
|
@copilot Omskriv mindst to af kommandoerne til C i stedet for Python |
- Rewrote morse from Python to C (handles UTF-8 Danish characters) - Rewrote wiggle from Python to C (maintains identical algorithm) - Both use compile script framework for automatic compilation - Converted getPopulationOf from Perl to shell script - Converted langname from Perl to shell script (lookup table) - All maintain 100% functional compatibility Co-authored-by: nqpz <1893523+nqpz@users.noreply.github.com>
Done! Added more conversions in commit ed5255b: C programs (2):
Shell scripts (2):
Now we have a good mix: 4 Python scripts, 2 C programs, and 2 shell scripts - not just Python!
Done! Converted
Also added shell script conversions for |
Overview
This PR addresses issue #198 by rewriting small Perl utility scripts in
eggspithat use third-party dependencies to C, Python 3, and shell scripts. The focus is on eliminating Perl scripts that importutf8::allor other third-party packages, which suffer from binary incompatibility across Perl versions.Changes
Rewrote the following Perl scripts while maintaining 100% functional compatibility:
C Programs (using compile script framework)
1.
morse- Morse Code Converterutf8::all/in Morse outputconcieggs/compiled/morse.cExample:
2.
wiggle- Number Sequence Generatorconcieggs/compiled/wiggle.cExample:
./concieggs/eggspi/wiggle 20 -3 3 0.5 # Output: 20 numbers wiggling between -3 and 3Python Programs
3.
4påstribeStartTurn- Game Turn Notificationutf8::all4.
getGdpPcOf- GDP Per Capita Lookuputf8::allwacommand and parses output5.
sprog- Language Name Converterutf8::all,EggsML::EggsPI6.
krak- Krak API Lookuputf8::all,URI,String::Util,EggsML::EggsPIShell Scripts
7.
getPopulationOf- Population Lookuputf8::allwacommand and parses with grep/sed/awk8.
langname- Language Name Lookuputf8::allScripts Kept in Perl
parse- Named Regex Capture ExtractorBenefits
utf8::all,URI,String::Utilthat have binary incompatibility across Perl versionsTesting
All scripts have been thoroughly tested:
The rewritten scripts are drop-in replacements with identical behavior and output formats.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.