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

Read software to monitor from a list #98

Closed
triforce opened this issue Feb 9, 2014 · 2 comments
Closed

Read software to monitor from a list #98

triforce opened this issue Feb 9, 2014 · 2 comments

Comments

@triforce
Copy link
Contributor

triforce commented Feb 9, 2014

I have written some functionality to monitor software installation's from a list in a file rather than a default hard-coded list. Would this be worth adding in?

Having this option will allow more flexibility in what software to check for.

@commonquail
Copy link
Contributor

If and when this becomes more modularized and with support for configuration, this seems like an obvious thing to be able to configure. I don't know if it's worth adding a solution that probably won't be long-term now with the provision that it would be necessary to reconfigure later; that's a question that can only be answered by the people that really need this. Personally I have no problem with it but that's mainly because I wouldn't be affected.

For the record, I have some ideas for, and have been playing around some, with a modularization solution. I still need to prove to myself that I can make it work before I'll show it off, though.

@tariqbuilds
Copy link
Owner

It is very much worth adding in. It is exactly what we need for the near-term until a config file is present which will address this issue. Even then, maybe we can take some pieces of your implementation and reuse (or reconfigure, as @commonquail mentioned) in version 1.0.

But it is hard to tell whether your solution is right for this project until either you explain the code architecture of the solution in more detail or submit a PR.

@tariqbuilds tariqbuilds added this to the Config file milestone Feb 11, 2014
triforce added a commit to triforce/linux-dash that referenced this issue Feb 11, 2014
@triforce triforce mentioned this issue Feb 11, 2014
arnaudbey pushed a commit to arnaudbey/linux-dash that referenced this issue Feb 11, 2014
arnaudbey pushed a commit to arnaudbey/linux-dash that referenced this issue Feb 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants