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

[Feature] Ability to suppress missing file checks for user-deleted files #4205

Closed
1 task done
Sinomen opened this issue Sep 28, 2024 · 24 comments · Fixed by #4213
Closed
1 task done

[Feature] Ability to suppress missing file checks for user-deleted files #4205

Sinomen opened this issue Sep 28, 2024 · 24 comments · Fixed by #4213
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Enhancement GUI Issues affecting the interactive GUI

Comments

@Sinomen
Copy link

Sinomen commented Sep 28, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Operating System

Windows 10 Pro

CKAN Version

1.35.0.24270

Games

KSP 1

Game Version

1.12.3

Did you make any manual changes to your game folder (i.e., not via CKAN)?

Yes! Changed some CFG-Files, removed some unnecessary files. On purpose.

I did on purpose change/delete some some files of those mods

Steps to reproduce

Changed some CFG-files, removed unnecessary files I don't need.
Example: Mod GregroxMultiColorSuitsPack. I only need the red, yellow, and blue suits. To save memory, I removed the other colored suits.
Example: Mod K2CommandPodCont. I only need the 1.25m version, hence deleted the 1.875m version, saving memory.
Example: Mod MissingHistory. I disabled the CFG file SpaceTuxIndustries.cfg, as I'll never have the KANDL engine from LLG.
Example: Mod ModularFuelTanks lacks configuration data for several fuel tanks, hence I added them to the respective CFG-files.

@HebaruSan HebaruSan added the Support Issues that are support requests label Sep 28, 2024
@Sinomen Sinomen changed the title [Bug]: Latest Version of CKAN 1.35.0.24270 complains about missing/changed files for several mods. I did on purpose change / delete some some files of those mods. How can I disable this nagging? [Bug]: Latest Version of CKAN 1.35.0.24270 complains about missing/changed files for several mods. I did on purpose change/delete some some files of those mods. How can I disable this nagging? Sep 28, 2024
@kretze
Copy link

kretze commented Sep 29, 2024

A label option comes to mind for this. With a type "no compare" or "no file check".
Just like it's currently done with "don't upgrade".

image

@Sinomen
Copy link
Author

Sinomen commented Sep 29, 2024

A label option comes to mind for this. With a type "no compare" or "no file check". Just like it's currently done with "don't upgrade".

No, that's not a solution at all. If there is a newer version of a mod, I want to be able to update it via CKAN, even if the update might add/replace files I deleted/modified, as I can easily rectify that later.

But I don't want to have an update notification nagging for a mod that is perfectly working fine, with the same version as already installed, just because I have modified/deleted some files.

@kretze
Copy link

kretze commented Sep 29, 2024

We talk past each other.
If a new label option "Do not check files" existed in CKAN, then you can create a label "modified mod" with this option and put it on a mod yourself. This way you have control over it yourself.

This option should only suspend file checking and nothing else.

@Sinomen
Copy link
Author

Sinomen commented Sep 29, 2024

I was hoping for global setting option, not a per-mod setting.

@HebaruSan HebaruSan changed the title [Bug]: Latest Version of CKAN 1.35.0.24270 complains about missing/changed files for several mods. I did on purpose change/delete some some files of those mods. How can I disable this nagging? Ability to suppress missing file checks Sep 29, 2024
@HebaruSan HebaruSan added Enhancement GUI Issues affecting the interactive GUI Core (ckan.dll) Issues affecting the core part of CKAN and removed Support Issues that are support requests labels Sep 29, 2024
@HebaruSan HebaruSan changed the title Ability to suppress missing file checks [Feature] Ability to suppress missing file checks Sep 29, 2024
@HebaruSan HebaruSan changed the title [Feature] Ability to suppress missing file checks [Feature] Ability to suppress missing file checks for intentionally deleted files Sep 29, 2024
@HebaruSan HebaruSan changed the title [Feature] Ability to suppress missing file checks for intentionally deleted files [Feature] Ability to suppress missing file checks for user-deleted files Sep 29, 2024
@Sinomen Sinomen changed the title [Feature] Ability to suppress missing file checks for user-deleted files [Feature] Ability to suppress missing / changed file checks for user-deleted / modified files Sep 29, 2024
@Sinomen
Copy link
Author

Sinomen commented Sep 29, 2024

@HebaruSan No offence, but I changed to the topic to
Ability to suppress missing / changed file checks for user-deleted / modified files
Hope that's ok with you?

@HebaruSan
Copy link
Member

Hi @Sinomen, no offense taken. But there is no changed or modified file check, so I'm going to go ahead and change it back. 👍

@HebaruSan HebaruSan changed the title [Feature] Ability to suppress missing / changed file checks for user-deleted / modified files [Feature] Ability to suppress missing file checks for user-deleted files Sep 29, 2024
@Sinomen
Copy link
Author

Sinomen commented Sep 29, 2024

I see, ok, my bad, beg your pardon.

@Sinomen
Copy link
Author

Sinomen commented Sep 29, 2024

Question: does the file check also cover additional files, or just deleted files?

@HebaruSan
Copy link
Member

Hi @Sinomen, user-created files are mostly ignored as they always have been. They don't trigger any re-install behavior.

If you want to review them, you can use the File → Unmanaged files... option.

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

@HebaruSan Thank you for your help, much appreciated! Btw, several "missing files" were pruned using the respective functionality of LGG's Editor Extensions Redux mod:

@HebaruSan
Copy link
Member

@Sinomen which files?

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

@Sinomen which files?

For example these here, the prune function of EditorExtensionsRedux adds .prune as extension, making the files invisible for KSP. Which was exactly what I wanted as I'll never use these parts.
The reason I mentioned this: maybe CKAN could not flag these files as missing when it finds the files with the .prune extension?
image

@HebaruSan
Copy link
Member

@Sinomen thanks. Taking a look at that mod, it does not seem to have any code to create files with a .prune extension. Are you sure that's the right mod? Could you maybe share a screenshot of the UI where you do this? I might be able to spot something that could lead to the right mod.

@HebaruSan
Copy link
Member

HebaruSan commented Sep 30, 2024

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

Ah ok, you're right, I mixed the two mods up. Sorry.

@HebaruSan
Copy link
Member

HebaruSan commented Sep 30, 2024

No problem. 👍

Investigating how Janitor's Closet works, it looks like another poor substitute for CKAN's installation filters (see #3458). If you ever upgrade or reinstall a mod containing a pruned file, the original file will be re-created by CKAN alongside the .prune file, and who knows how JC would react to that, on top of the parts being loaded again (it looks like JC has code to move such files again at startup). Whereas if the filename is added to CKAN's installation filters, it would never be installed, whether initially or at subsequent upgrades or reinstalls.

This might be another opportunity to advertise CKAN's installation filters in the abstract of the mod as we did in KSP-CKAN/NetKAN#10230. Or maybe a "copy pruned file list to clipboard" option could be added to Janitor's Closet that could then be pasted into the CKAN setting, as a form of easier integration between the two. I'll keeep this in mind and maybe contact @linuxgurugamer about it.

@HebaruSan
Copy link
Member

@Sinomen, does your KSP install have a file by this name?

  • GameData/JanitorsCloset/PluginData/RenamedFiles.dat

If so, can you share it? It might contain some strings that could be modified and then pasted into the CKAN installation filters.

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

Here you go (had the change the extension to txt)
RenamedFiles.txt

@HebaruSan
Copy link
Member

HebaruSan commented Sep 30, 2024

OK, I have submitted linuxgurugamer/JanitorsCloset#41 to automatically sync up that mod's pruned files into CKAN's filter list. In my testing, this successfully caused the pruned mods to no longer be shown as missing files in CKAN. I will contact @linuxgurugamer to see about getting those changes merged and a new version of Janitor's Closet released.

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

Sorry again for me mixing up JC and EER, my part mod base ist pretty stable for years, and it's been quite a while since I actively have used JC.

Regrading the pruned files, my idea how to deal with them:

CKAN finds that for a particular mod a specific file abc.xyz is missing.
Check if in the same directory there is a file with the name abc.xyz.prune
If yes, ignore the missing files warning, if no mark that mod as incomplete installed.

This way, no new dependency or even interaction between JC and CKAN arises, and JC could remain unchanged.

@HebaruSan
Copy link
Member

@Sinomen, no need to worry about that, the new Janitor's Closet release is out now.

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

Cool :)
If I read the description correctly, this change is not retroactive, i.e. already pruned files won't automatically create the respective CKAN filter?

@HebaruSan
Copy link
Member

The filters will be created when the user clicks the Permanent Prune button.

image

@Sinomen
Copy link
Author

Sinomen commented Sep 30, 2024

All right, thank you again for your help, and I'm glad that I could make a useful suggestion regarding the pruned files.
Good night.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Enhancement GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants