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

Suggestions: Date Driver Update Released / Date Patch Added Columns / Autoupdate Improvement / Error Message Improvement / File Ownership Merge Request #1

Closed
ramhaidar opened this issue Dec 30, 2024 · 0 comments · Fixed by #2

Comments

@ramhaidar
Copy link
Owner

ramhaidar commented Dec 30, 2024

Suggestions provided by @Spaatz1402 on the keylase/NVIDIA-patch repository regarding the Win_1447_Apply_Patch can be found here.

Hello,

I love your tool but find it a bit daunting to determine when I last applied the patch and would like to save some time researching all of that before simply applying the latest patch.

Here are some suggestions I feel would be good for your application:

1.) The error message when trying to reapply the patch could be updated to indicate the patch has already been applied.
2.) The file ownership rights issue should be pushed to the main version because nearly all files I'm getting from NVidia these days are locked for editing until I take ownership and thus this installer fails. I'm using a forked version of this tool that has the functionality required. Perhaps merge that functionality? Win_1337_Apply_Patch_v2.1_By_DFoX
3.) The documentation screen / table which lists the patches, driver link, version number, etc. could have two additional columns:

  • A.) Date patch made available
  • B.) Date of Driver Release

4.) Adding an automatic update feature to the application to identify the correct patch needed, download the update from GitHub and apply it to the driver would be ideal.

Thanks for taking the suggestions into consideration and thanks for your work!

Thank you for your suggestion; I will consider enhancing the error message to indicate when a patch has already been applied.

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

Successfully merging a pull request may close this issue.

1 participant