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.

Sign up for GitHub

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

Jump to bottom

Bug with RealPlume setup #1

Open
jsolson opened this issue Mar 4, 2018 · 1 comment
Open

Bug with RealPlume setup #1

jsolson opened this issue Mar 4, 2018 · 1 comment

Comments

@jsolson
Copy link

jsolson commented Mar 4, 2018

NAR_MEM/Config/RealismConfigs.cfg

Line 642 in bf515f6

@PART[MEMDeOrbiter]:FOR[RealPlume]:NEEDS[SmokeScreen]

Instances of :FOR[RealPlume]:NEEDS[SmokeScreen] need to be changed to :NEEDS[RealPlume,SmokeScreen]

NEEDS[SmokeScreen] passes when SmokeScreen is installed and RealPlume is not, and the FOR[RealPlume] part is causing other mods that use the NEEDS[RealPlume,SmokeScreen] test to believe RealPlume is installed as well. Engine effects get screwed up everywhere as a result. The RealPlume stuff is a bit scattered and should probably be moved to it's own file for ease of maintenance.

Nice mod, please keep up the good work :-)
@TiktaalikDreaming
Copy link
Owner

True. I'll fix that up, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TiktaalikDreaming @jsolson