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

Issue with Alfred 4 #3

Closed
Apollorion opened this issue Oct 22, 2021 · 4 comments
Closed

Issue with Alfred 4 #3

Apollorion opened this issue Oct 22, 2021 · 4 comments

Comments

@Apollorion
Copy link

When I install alfred 4 via:
{u: "https://cachefly.alfredapp.com/Alfred_4.6_1266.dmg", f: "alfred.dmg", n: "Alfred 4"}

It sits for like 10 minutes on TASK [juju4.macos-apps-install : Copy 'Alfred 4' app to '/Applications'] even though the file is very small.

Eventually it does get past that step and says its changed, but the application looks like this in /Applications:
image

What am I doing wrong? Sorry for formatting, Im on my mobile device.

@Apollorion
Copy link
Author

Also, when I try to open the application it says I dont have permission to do so.

drwxr-xr-x, its owned by me with the group admin.

@Apollorion
Copy link
Author

Another Update; I think the permissions are the issue here. If I manually set permission to 777 it will allow me to open alfred. I tried 644 as well, like this role uses and it did NOT work. I think something within Alfred needs a higher permission.

@juju4
Copy link
Owner

juju4 commented Oct 24, 2021

f is optional. default to url basename per https://github.com/juju4/ansible-macos-apps-install/blob/master/tasks/darwin.yml#L40 (but somewhat not a filename or extra query strings)
n is the app_name which is used for copy per https://github.com/juju4/ansible-macos-apps-install/blob/master/tasks/archive-install.yml#L69 with default mode 644. for permission check, 755 should be good enough, 777 changes nothing from a read/execution permissions, only write permissions for user group and others.
to is needed for existence check/idempotence.

I found some condition issue in HEAD branch, not sure which version/commit of the role you are using.
Fixed in latest HEAD

From testing in github action
https://github.com/juju4/ansible-macos-apps-install/runs/3986477946?check_suite_focus=true#step:10:733

12896385199        0 drwxr-xr-x    3 root             admin                  96 Oct 24 00:30 /Applications/Alfred 4.app
12896385200        0 drwxr-xr-x   10 root             admin                 320 Oct 24 00:35 /Applications/Alfred 4.app/Contents
 12896392213        0 drwxr-xr-x    3 root             admin                  96 Oct 24 00:35 /Applications/Alfred 4.app/Contents/MacOS
12896392208      848 -rw-r--r--    1 root             wheel              434128 Oct 24 00:35 /Applications/Alfred 4.app/Contents/MacOS/Alfred

Initially, mode was a single definition as input to ansible but seems a preserve option has been added.
https://docs.ansible.com/ansible/latest/collections/ansible/builtin/copy_module.html#parameter-mode
switched to that

all good after that
https://github.com/juju4/ansible-macos-apps-install/runs/3986577625?check_suite_focus=true#step:10:773

12896390407        0 drwxr-xr-x    3 root             admin                  96 Oct 24 01:03 /Applications/Alfred 4.app/Contents/MacOS
12896390398      848 -rwxr-xr-x    1 root             wheel              434128 Oct 24 01:02 /Applications/Alfred 4.app/Contents/MacOS/Alfred

Please try latest HEAD and confirm ok

@Apollorion
Copy link
Author

Seems to be working, thank you!

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

No branches or pull requests

2 participants