Make sound asset dependency listing behavior more consistent.

kevinhartman · May 7, 2020 · 2decef6 · 2decef6
1 parent 3717968
commit 2decef6
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/scala/BuildDependencyInfo.scala b/src/main/scala/BuildDependencyInfo.scala
@@ -32,8 +32,9 @@ object BuildDependencyInfo {
         externStates = deps.externStates ++ obj.states.filterNot(socScript.states.contains).filterNot(_ == "0"),
         externSounds = deps.externSounds ++ obj.sounds.filterNot(socScript.sounds.contains).filterNot(_ == "0"),
 
-        // even if the sound is not declared locally, we can know its name if not hard-coded
-        soundsFiles = deps.soundsFiles ++ obj.sounds.filterNot(isHardcoded).map(soundName)
+        // We mention the sound file if it has a local declaration, as long as it's not hardcoded
+        // TODO: support hard coded sound names.
+        soundsFiles = deps.soundsFiles ++ obj.sounds.filter(socScript.sounds.contains).filterNot(isHardcoded).map(soundName)
       )
     })