-
Notifications
You must be signed in to change notification settings - Fork 59
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
Prevent parameter keyword obfuscation #29
Comments
Hello, here similar dilemma: instructions misunderstandig or bug? test.py
opy_config.txt
obfuscation results in
Running the obfuscated one I get
because Tkinter module expects the parameter keyword tearoff Any suggestions? |
+1 |
+1 Has there been any progress with this issue? |
Not yet. There is a very similar / overlapping request regarding dictionary keys. I'll try to work on these matters sometime in my Opy Fork, where development of Opy remains active. https://pypi.org/project/opy-distbuilder/ I recommend checking out my "DistBuilder" project. It's only in alpha, but it is a larger scale project that encompasses Opy and adds features over the top of it. Notably, an "OpyPatch" provides a stop gap means for dealing with glitches of this nature. https://distribution-builder.readthedocs.io/en/latest/ |
I have another issue with the tool that may again come from a certain misunderstanding.
This time, it requires flash to me installed on the computer.
I've a test.py script:
And my opy_config.txt file
The result is the following:
The interesting part is the change from:
to:
Checkout https://github.com/pallets/flask/blob/master/flask/app.py
def run
line 840. It looks like opy knows not to rename thehost
,port
anddebug
keywords before of the fonction declarationdef run(self, host=None, port=None, debug=None, load_dotenv=True, **options):
. But to me it's an issue that thethreaded
keyword is renamed.I am again wondering if this is the intended behavior and if I'm missing something here.
Thank you
The text was updated successfully, but these errors were encountered: