search

akej74 / grid-control

Grid Control is a free and open source alternative to the CAM application from NZXT
GNU General Public License v3.0
147 stars 31 forks source link

OHM as a part of install package #10

Closed wiruzman closed 7 years ago

wiruzman commented 7 years ago

Hi,

I think OpenHardwareMonitor should be a part of the install package.

It gives lesser complexity of running the application.

akej74 commented 7 years ago

Hi, I like the idea, but if I bundle OHM, people will be using that version instead of newer versions made available from OHM (as OHM is currently in development and new releases are coming out). I prefer to ask people to grab the latest release from OHM and install separately.

wiruzman commented 7 years ago

That way you can control the version your application is using. You can prevent your application is not crashing because something in the code has changed after an user has updated OHM.

Think about it.

I didn't install the your application because I needed to install OHM as well. I have already HwInfo, which is supported by many applications.

Regards, Mehmet

Med venlig hilsen / Best regards Mehmet Karahan

Den 17. feb. 2017 9.21 AM skrev "Åke Johansson" notifications@github.com:

Hi, I like the idea, but if I bundle OHM, people will be using that version instead of newer versions made available from OHM (as OHM is currently in development and new releases are coming out). I prefer to ask people to grab the latest release from OHM and install separately.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/akej74/grid-control/issues/10#issuecomment-280585178, or mute the thread https://github.com/notifications/unsubscribe-auth/AHKetrvkZqK1_18VVKgIeIF36nJq3LHuks5rdViagaJpZM4MDf5c .

akej74 commented 7 years ago

Thanks for the update, the reason I'm using OHW is that is the only application I have found that exposes the actual sensor values to be used by other applications (this is done via WMI from OHM).

wiruzman commented 7 years ago

That is also fine, but you should consider to use specific version of OHM due to updates which will crash your application. It is a seriously problem.

I'm also a software developer and making my software self contained.

2017-02-21 12:58 GMT+01:00 Åke Johansson notifications@github.com:

Thanks for the update, the reason I'm using OHW is that is the only application I have found that exposes the actual sensor values to be used by other applications (this is done via WMI from OHM).

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/akej74/grid-control/issues/10#issuecomment-281324161, or mute the thread https://github.com/notifications/unsubscribe-auth/AHKetqePlPD-T77Gupwz5WbuIN0Qs4EAks5retFQgaJpZM4MDf5c .

akej74 commented 7 years ago

Yes, that is a good idea, let me add a disclaimer in the readme about which version of OHM works OK as a start.

wiruzman commented 7 years ago

That will be very nice as a start.

Med venlig hilsen / Best regards Mehmet Karahan

Den 22. feb. 2017 9.32 AM skrev "Åke Johansson" notifications@github.com:

Yes, that is a good idea, let me add a disclaimer in the readme about which version of OHM works OK as a start.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/akej74/grid-control/issues/10#issuecomment-281602861, or mute the thread https://github.com/notifications/unsubscribe-auth/AHKetlrqkeA4IZsZrGJi6GKikKJh2ar8ks5re_KlgaJpZM4MDf5c .

akej74 commented 7 years ago

Readme updated with link to latest version of OHM beta (tested and works OK).