-
Notifications
You must be signed in to change notification settings - Fork 26
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
Workflow broken on Monterey 12.3 #12
Comments
Is there any progress? |
Mark, I hope it can be resolved soon. |
It's time to fully migrate to python3. |
Manually running pwgen.py gives this error |
We can still install python 2 as explained here: |
Yes but it doesn’t work unfortunately 🤔 |
Is there any update on this? or a solution being worked on? |
I got it to work by installing py27 as described here on stackoverflow (Im on m1 and using conda, other ways to get py27 working should be fine, too). Then, all the occurences of
If I find a free weekend, I might add this as a workflow-wide variable that can be set, but for now this is a quick and dirty way to get it running again. |
I think it's great that a workaround exists, but a rewrite of the workflow is probably what we need. Having to install a somewhat outdated Python version just for this workflow is not really what I want to be doing. |
It's hard to believe there is not even one maintainer who might be able to migrate this to python3. As I remember, the difference is not so huge for pyhton2 and python3. Unfortunately, I'm not familiar with this kind of development. Otherwise, I'd do it even by myself. |
Hey, chill. The maintainer has worked on this package for years completely free, they have no real obligation to continue working on it if they do not wish to do so, or perhaps they have just not had the time yet. I'm sure you might find someone who's willing to fork it and/or submit a PR to update it to Python 3 if they're compensated 🙂 Anyhow, I'm also a great fan of this package and needed a substitute, so I made somewhat of a port to Node (olssonm/alfred-password-generator) albeit much simpler with fewer features if anyone's interested. |
@olssonm sorry if I was too direct, but I didn't mean the repo owner by "maintainer". I meant anyone who is able to make a fix (even me if I will figure out how, hehe). By the way, your workflow works great for me, thank you! |
I found a project replaced it, I hope it is useful to you. alfred-password-generator |
I tried it and it works as expected. The script itself is very simple, I like it a lot. |
@olssonm your Node.js workflow is quite nice, and very tidy. Well done! 👏🏼👏🏼👏🏼 |
Thanks @ryanblock ! (Huge fan since the gdgt-days btw!) |
I got the workflow running again on 12.3:
Works fine. |
I did something similar in my ProxmoxControl-Workflow. Before running a Python-script, I export the PYTHON_PATH variable. Seems to be working fine. |
This is great, super simple and straightforward. Thanks for creating this! |
change : os.makedirs(user_generator_dir, int('0700',8)) |
Apple removed Python 2 support in Monterey 12.3, and the workflow is no longer functional. A rewrite into Python 3 is needed.
The text was updated successfully, but these errors were encountered: