JIRA: ABC-279 Fix for 2.3.X: Build fails with empty AlembicStreamPlay…

…er component (#488) * JIRA: ABC-279 Fix for 2.3.X: Build fails with empty AlembicStreamPlayer component Test * Fix for the bug * Changelog
Unity-Technologies · Oct 5, 2021 · 9fd74d9 · 9fd74d9
1 parent 174908d
commit 9fd74d9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/TestProjects/AlembicStandaloneBuild/Assets/Scene.unity b/TestProjects/AlembicStandaloneBuild/Assets/Scene.unity
diff --git a/com.unity.formats.alembic/CHANGELOG.md b/com.unity.formats.alembic/CHANGELOG.md
@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 ### Changed
 ### Fixed
+- Prevent build error when the Scene contains empty AlembicStreamPlayer components.
 
 ## [2.3.0-pre.2] - 2021-09-02
 ### Added

diff --git a/com.unity.formats.alembic/Editor/Importer/AlembicBuildPostProcess.cs b/com.unity.formats.alembic/Editor/Importer/AlembicBuildPostProcess.cs
@@ -41,6 +41,11 @@ public static void OnPostProcessBuild(BuildTarget target, string pathToBuiltProj
 
             foreach (var files in FilesToCopy)
             {
+                if (!File.Exists(files.Key))
+                {
+                    continue;
+                }
+
                 var dir = Path.GetDirectoryName(files.Value);
                 if (dir != null && !Directory.Exists(dir))
                 {
@@ -95,6 +100,10 @@ static void ProcessAlembicStreamPlayerAssets(AlembicStreamPlayer streamPlayer, s
         {
             streamPlayer.StreamDescriptor = streamPlayer.StreamDescriptor.Clone();// make a copy
             var srcPath = streamPlayer.StreamDescriptor.PathToAbc;
+            if (string.IsNullOrEmpty(srcPath))
+            {
+                return;
+            }
 
             // Avoid name collisions by hashing the full path
             var hashedFilename = HashSha1(srcPath) + ".abc";