-
Notifications
You must be signed in to change notification settings - Fork 20
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
Suppressing modules in "Updates" that are found in "My Updates" #230
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added Ed Raus API_Run class
Follow up changes since 20230812 a36a5f0
Header files in filse using URL included
Replaced the Thor URL and URL-branch literals with DEFINEs
Header files in prg using URL included for ThorRepository
Replaced the ThorRepository URL and URL-branch literals with DEFINEs
Inside the pjx, they are just needed to access / and for ancient VSS checkout. No need to bloat the app
Used to compile the code if URL for Thor or Thorepository is changed.
Ignore files in "Updates", if identical named file is found in "My Updates" working on VFPX#203
Jimrnelson
added a commit
to Jimrnelson/Thor
that referenced
this pull request
Jan 4, 2024
* In the CFU screen, hide 'Normal' updates if there is a "My Updates" file with the same AppID property (Issues VFPX#203, VFPX#230) * Updated version information is no longer read for projects marked "Never Update" (Issue VFPX#224) * Restore projects after CFU (Issue VFPX#96) * Thor updates itself even when run first time from old version of Thor.App instead of using current version (Issues VFPX#227, VFPX#212) * Thor Tools menu indicates if tool 'IntellisenseX By Dot' is active. (Issue VFPX#242) * Fixed bug when recording last date CFU run. It had been only doing so if called from RunThor. (Issue VFPX#220) * Removed use of `_Screen.oThorEngine`, which held some Thor tables open. (Issue VFPX#241) * New Project FoxGet * Modified Updater to GoFish to register with Thor without calling GoFish.APP
Implemented in Thor 1.47 - Released 2024-01-04 (#244) In the CFU screen, hide 'Normal' updates if there is a "My Updates" file with the same AppID property (which must not be empty). This property, available but rarely used, would need to be set in the version files for both updaters. |
This works by the AppID, not by similar file names? Great! Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Solving #203.
Suppress double modules.
Download https://github.com/lscheffler/Thor/raw/dev/ThorUpdater/Thor.zip to create a test installation running from the dev branch of the lscheffler fork.
It is intended that there is only a little number of modules in the dev branch. This should not harm any other branch.
Move a file from "Updates" to "My Updates" to test.