-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
Near Future Solar Core split should probably be removed #9922
Comments
what split? |
NearFutureSolar is actually two packages in CKAN: https://github.com/KSP-CKAN/NetKAN/blob/2992710bec6c0d1b6785ca939a4c0b886438cccb/NetKAN/NearFutureSolar.netkan |
Interesting, they both point to the same download. The core only installs the dll. |
Yes, it's the same situation as CryoTanks. Nertea apparently decided to split the plugin from the content, with the forethought that perhaps the DLL could be used by other mods without the content being installed. That's perfectly reasonable. But no other mods are actually doing this, and the splits are causing issues. I think there's a clue here: https://www.reddit.com/r/KerbalSpaceProgram/comments/1akzguu/comment/kpcicut/?utm_source=share&utm_medium=web2x&context=3
|
Going to verify... |
Similar to #9911 , but I haven't seen any examples of a mod doing
NEEDS[NearFutureSolar]
yet. However (due to CKAN not recognizing when mods are manually removed), this split does contribute to user frustration because it's a common culprit of B9PS errors: https://www.reddit.com/r/KerbalSpaceProgram/comments/1akzguu/endless_list_of_issues_reported_by_b9_part_switch/I think that kind of problem would be less likely to occur if the split didn't exist - the user might notice that CKAN says they have Near Future Solar but it's not in their GameData, so they'd use CKAN to reinstall it and everything would be fine.
Also sorta related to KSP-CKAN/CKAN#4017
The text was updated successfully, but these errors were encountered: