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.

Sign up for GitHub

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

Jump to bottom

"Install Conky" Button in Conky Preferences Not Installing Conky #17

Open
ennation opened this issue Sep 27, 2018 · 14 comments
Open

"Install Conky" Button in Conky Preferences Not Installing Conky #17

ennation opened this issue Sep 27, 2018 · 14 comments
Assignees
@npyl
Labels
bug
Milestone
ManageConky v1.0

Comments

@ennation
Copy link

Clicking the button closes and drops back down the preferences window and doesn't install Conky, so far as I can tell

See Imgur post for screen recording of the problem and system info
@npyl
Copy link
Member

npyl commented Sep 28, 2018

Hi! Thank you for your contribution.
What version of ManageConky are you using?

@npyl npyl self-assigned this Sep 28, 2018
@npyl
Copy link
Member

npyl commented Sep 28, 2018

Also, where are you running from ManageConky? Are you running it from /Applications?

@ennation
Copy link
Author

Unfortunately, I already had Homebrew installed and up to date so that's not it.

This is version 0.9.1 from the releases page of the repo

@ennation
Copy link
Author

Yep, /Applications

@npyl
Copy link
Member

npyl commented Sep 28, 2018

Please repeat the process you showed me on the video but this time with Console.app open and with "Manage Conky" written in the search bar.

Then upload here the log.

@npyl
Copy link
Member

npyl commented Sep 28, 2018

Question: Was ManageConky installed in /Applications by a user other than you, but it is run by you?

@ennation
Copy link
Author

Will do. Gimme a sec

Nope, installed and run by me

@npyl
Copy link
Member

npyl commented Sep 28, 2018

Quick question: Is ConkyX.app installed in /Applications?

@ennation
Copy link
Author

No ConkyX. Found the issue though. Was on a non-admin account, didn't have permissions to install it the way that it does.

@npyl
Copy link
Member

npyl commented Sep 28, 2018

Glad you solved it, this means I need to rethink my installer!

@npyl npyl mentioned this issue Sep 28, 2018
Open
@npyl
Copy link
Member

npyl commented Sep 29, 2018 •
edited
Loading

@Eymantia hi! This is not related to this issue, but I wanted to ask, how was your experience with ManageConky so far? After you fixed this problem, did you manage to do what you expected?

I will soon update my site with a star-based rate widget for ManageConky. This way, anyone will be able to rate and submit his/her thoughts.

PS: I am developing a framework that will work just like NSTask but with the addition of launchAuthenticated method which will ...well, execute your task with root privileges. This way, this issue will be fixed.

Until then, lets keep this open.

@npyl npyl reopened this Sep 29, 2018
@npyl
Copy link
Member

npyl commented Sep 29, 2018

@npyl Issue is ManageConky doesn't copy ConkyX into /Applications when run by non-admin user

@ennation
Copy link
Author

ennation commented Oct 1, 2018

Sorry, was out of town for a few days. Everything seems to be working just fine!

Also glad you found a solution to the problem!

@npyl
Copy link
Member

npyl commented Oct 1, 2018

Thats great news!! Basically you found it, I am just implementing the code !

npyl added a commit that referenced this issue Nov 26, 2018
@npyl
Work for #17
df057d4 
Not ready yet, though...
npyl added a commit that referenced this issue Nov 26, 2018
@npyl
Work for #17
84ae0c3
npyl added a commit that referenced this issue Nov 27, 2018
@npyl
Work for #17
93a57d8 
If user belongs to admins (is admin) ManageConky is allowed to write to Applications.

Try to find if user belongs to admins, and if not use NPTask (NSTask with authentication)
npyl added a commit that referenced this issue Nov 27, 2018
@npyl
Work for #17
bbabdbf 
This time we implement the function that tells if user belongs to admins.
npyl added a commit that referenced this issue Nov 30, 2018
@npyl
Work for #17
526e2f6 
Update NPTask... It now contains a bugfix which will atleast allow it to run! :) More coming soon...
npyl added a commit that referenced this issue Nov 30, 2018
@npyl
Work for #17
368d9f0 
It now works! A bit of tweaking here and there and it should work even better...
npyl added a commit that referenced this issue Nov 30, 2018
@npyl
Work for #17
2733865 
This is a tweak that eliminates the need to give your password two times... It can be considered done now, but more testing should be done...
@npyl npyl added the bug label Dec 7, 2018
@npyl npyl mentioned this issue Jun 15, 2019
Open
8 tasks
@npyl npyl added this to the ManageConky v1.0 milestone Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@npyl npyl

Labels
bug
Projects
None yet
Milestone
ManageConky v1.0
Development

No branches or pull requests
2 participants
@npyl @ennation