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

Adds command line flag to use previously stored settings #103

Closed
wants to merge 2 commits into from

Conversation

andia89
Copy link
Contributor

@andia89 andia89 commented Jul 31, 2015

Before I push a nice gui (which is finished now btw \o/) another necessary thing for that. A config file where the previous settings are stored and can be retrieved by using -p on the command line without asking the user for any input

@andia89
Copy link
Contributor Author

andia89 commented Jul 31, 2015

Btw: this kind of deals with #101, and the flag could be used as a postinstall routine when updating numix or numix-circle

@Foggalong
Copy link
Contributor

As far as I can tell it's impossible to introduce the script into the post install routine. There are just too many variables that we don't have control over such as whether the folder script is installed, how the user installed it, where they installed it to, how they have other icon themes installed, etc

I'll check out this code though asap :)

@andia89
Copy link
Contributor Author

andia89 commented Jul 31, 2015

Fair enough, still this is definitely a usefull addition :)

@Foggalong
Copy link
Contributor

Aye!

@andia89
Copy link
Contributor Author

andia89 commented Aug 2, 2015

I'm sorry I have to close this and add a new one with the gui already included. Those two features are too closely related to be separated in distinct pull-requests...

@andia89 andia89 closed this Aug 2, 2015
@Foggalong
Copy link
Contributor

Fair enough! Sorry for not having the time to get to this sooner :(

@andia89 andia89 deleted the config branch August 4, 2015 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants