Fix UPM package generation issues for UWP

.travis.yml

@@ -8,8 +8,9 @@ before_script:
 - sudo chmod +x ./scripts/export-upm-package.sh
 script:
 - msbuild ./GLTFSerialization/GLTFSerialization.sln /t:GLTFSerialization /p:TargetFramework=net35
-- "./scripts/export-upm-package.sh"
 - "./scripts/export-unity-package.sh"
+- msbuild ./GLTFSerialization/GLTFSerialization.sln /t:GLTFSerialization
+- "./scripts/export-upm-package.sh"
 deploy:
   provider: releases
   file: 

UnityGLTF/Assets/UnityGLTF/Scripts/Editor/UnityGLTFEditor.asmdef

@@ -0,0 +1,16 @@
+{
+    "name": "UnityGLTFEditor",
+    "references": [
+        "UnityGLTFScripts"
+    ],
+    "optionalUnityReferences": [],
+    "includePlatforms": [
+        "Editor"
+    ],
+    "excludePlatforms": [],
+    "allowUnsafeCode": false,
+    "overrideReferences": false,
+    "precompiledReferences": [],
+    "autoReferenced": true,
+    "defineConstraints": []
+}

UnityGLTF/Assets/UnityGLTF/Scripts/Tests/UnityGLTFTests.asmdef

@@ -0,0 +1,17 @@
+{
+    "name": "UnityGLTFTests",
+    "references": [
+        "UnityGLTFScripts",
+        "UnityGLTFEditor"
+    ],
+    "optionalUnityReferences": [
+        "TestAssemblies"
+    ],
+    "includePlatforms": [],
+    "excludePlatforms": [],
+    "allowUnsafeCode": false,
+    "overrideReferences": false,
+    "precompiledReferences": [],
+    "autoReferenced": true,
+    "defineConstraints": []
+}

UnityGLTF/Assets/UnityGLTF/Scripts/UnityGLTFScripts.asmdef

@@ -0,0 +1,3 @@
+{
+	"name": "UnityGLTFScripts"
+}

scripts/export-upm-package.sh

@@ -1,4 +1,4 @@
-#! /bin/sh
+#! /bin/bash
 
 project_path=$(pwd)/UnityGLTF
 log_file=$(pwd)/build/unity-mac.log
@@ -8,9 +8,22 @@ upm_name=org.khronos.UnityGLTF
 upm_src_folder_path=$(pwd)/UnityGLTF/Assets/UnityGLTF
 upm_manifest_path=$(pwd)/scripts/package.json
 upm_staging_path=$(pwd)/current-package/$upm_name
+upm_staging_UWP_plugins_path=$upm_staging_path/UnityGLTF/Plugins/uap10.0.10586
 upm_zip_export_path=$(pwd)/current-package/$upm_name.zip
 upm_targz_export_path=$(pwd)/current-package/$upm_name.tar.gz
 
+# msbuild spits out every single dependency dll for UWP
+# These are the only files that are needed by the UPM package for Unity 2018.3+
+# Including all the files in the UWP plugin directory causes name collision errors when 
+# building for UWP in Unity
+upm_UWP_Plugins=(
+	"GLTFSerialization.dll"
+	"GLTFSerialization.dll.meta"
+	"GLTFSerialization.pdb"
+	"Newtonsoft.Json.dll"
+	"Newtonsoft.Json.dll.meta"
+)
+
 error_code=0
 echo $upm_name
 echo $upm_src_folder_path
@@ -28,14 +41,20 @@ cp -r $upm_src_folder_path $upm_staging_path
 echo "Changing to $upm_staging_path folder"
 cd $upm_staging_path
 
+echo "Cleaning out UWP plugin DLLs that are not needed for Unity2018.3+"
+echo "Only keeping the following files:"
+printf '%s\n' "${upm_UWP_Plugins[@]}"
+find $upm_staging_UWP_plugins_path -maxdepth 1 -type f | grep -vE "$(IFS=\| && echo "${upm_UWP_Plugins[*]}")" | xargs rm
+
 echo "Creating .zip of UPM package"
-sudo zip -r $upm_zip_export_path ./
+sudo zip -q -r $upm_zip_export_path ./
 
 echo "Creating .tar.gz of UPM package"
-tar -zcvf $upm_targz_export_path ./
+tar -zcf $upm_targz_export_path ./
 
 echo "Changing back to original folder $cached_folder"
 cd $cached_folder
 
 echo "Finishing with code $error_code"
-exit $error_code
+exit $error_code
+