diff options
author | Lluis Sanchez <lluis@xamarin.com> | 2014-01-24 15:47:17 +0400 |
---|---|---|
committer | Lluis Sanchez <lluis@xamarin.com> | 2014-01-24 15:47:17 +0400 |
commit | 64a45d96f39d4714ec85adf0fe04b68ec7273ae1 (patch) | |
tree | 92a8f6de3b9cce051cf9add9f64c50b059a7362d | |
parent | f2e9e9c991188cbaa7942da865636a31d80b755a (diff) | |
parent | 82460570998db86d5201acd1523fda44c93cd3d3 (diff) |
Merge branch 'sm-with-config'mono-addins-1.1
-rw-r--r-- | Mono.Addins/Mono.Addins.Database/AddinScanner.cs | 82 | ||||
-rw-r--r-- | Mono.Addins/Mono.Addins.Description/AddinDescription.cs | 3 | ||||
-rw-r--r-- | Mono.Addins/Mono.Addins.Description/ModuleDescription.cs | 6 |
3 files changed, 68 insertions, 23 deletions
diff --git a/Mono.Addins/Mono.Addins.Database/AddinScanner.cs b/Mono.Addins/Mono.Addins.Database/AddinScanner.cs index 36de3d2..923eb94 100644 --- a/Mono.Addins/Mono.Addins.Database/AddinScanner.cs +++ b/Mono.Addins/Mono.Addins.Database/AddinScanner.cs @@ -551,22 +551,8 @@ namespace Mono.Addins.Database // Get the config file from the resources, if there is one - foreach (string res in reflector.GetResourceNames (asm)) { - if (res.EndsWith (".addin") || res.EndsWith (".addin.xml")) { - using (Stream s = reflector.GetResourceStream (asm, res)) { - AddinDescription ad = AddinDescription.Read (s, Path.GetDirectoryName (filePath)); - if (config != null) { - if (!config.IsExtensionModel && !ad.IsExtensionModel) { - // There is more than one add-in definition - monitor.ReportError ("Duplicate add-in definition found in assembly: " + filePath, null); - return false; - } - config = AddinDescription.Merge (config, ad); - } else - config = ad; - } - } - } + if (!ScanEmbeddedDescription (monitor, filePath, reflector, asm, out config)) + return false; if (config == null || config.IsExtensionModel) { // In this case, only scan the assembly if it has the Addin attribute. @@ -596,6 +582,29 @@ namespace Mono.Addins.Database } } + static bool ScanEmbeddedDescription (IProgressStatus monitor, string filePath, IAssemblyReflector reflector, object asm, out AddinDescription config) + { + config = null; + foreach (string res in reflector.GetResourceNames (asm)) { + if (res.EndsWith (".addin") || res.EndsWith (".addin.xml")) { + using (Stream s = reflector.GetResourceStream (asm, res)) { + AddinDescription ad = AddinDescription.Read (s, Path.GetDirectoryName (filePath)); + if (config != null) { + if (!config.IsExtensionModel && !ad.IsExtensionModel) { + // There is more than one add-in definition + monitor.ReportError ("Duplicate add-in definition found in assembly: " + filePath, null); + return false; + } + config = AddinDescription.Merge (config, ad); + } + else + config = ad; + } + } + } + return true; + } + bool ScanDescription (IProgressStatus monitor, IAssemblyReflector reflector, AddinDescription config, object rootAssembly, AddinScanResult scanResult) { // First of all scan the main module @@ -669,14 +678,14 @@ namespace Mono.Addins.Database if (!config.IsRoot) { foreach (ModuleDescription mod in config.OptionalModules) { try { - assemblies.Clear (); + var asmList = new List<Tuple<string,object>> (); for (int n=0; n<mod.Assemblies.Count; n++) { string s = mod.Assemblies [n]; if (mod.IgnorePaths.Contains (s)) continue; string asmFile = Path.Combine (config.BasePath, s); object asm = reflector.LoadAssembly (asmFile); - assemblies.Add (asm); + asmList.Add (new Tuple<string,object> (asmFile,asm)); scanResult.AddPathToIgnore (Path.GetFullPath (asmFile)); ScanAssemblyImports (reflector, mod, asm); } @@ -691,9 +700,9 @@ namespace Mono.Addins.Database scanResult.AddPathToIgnore (Path.GetFullPath (path)); } - foreach (object asm in assemblies) - ScanAssemblyContents (reflector, config, mod, asm, scanResult); - + foreach (var asm in asmList) + ScanSubmodule (monitor, mod, reflector, config, scanResult, asm.Item1, asm.Item2); + } catch (Exception ex) { ReportReflectionException (monitor, ex, config, scanResult); } @@ -704,6 +713,37 @@ namespace Mono.Addins.Database return true; } + bool ScanSubmodule (IProgressStatus monitor, ModuleDescription mod, IAssemblyReflector reflector, AddinDescription config, AddinScanResult scanResult, string assemblyName, object asm) + { + AddinDescription mconfig; + ScanEmbeddedDescription (monitor, assemblyName, reflector, asm, out mconfig); + if (mconfig != null) { + if (!mconfig.IsExtensionModel) { + monitor.ReportError ("Submodules can't define new add-ins: " + assemblyName, null); + return false; + } + if (mconfig.OptionalModules.Count != 0) { + monitor.ReportError ("Submodules can't define nested submodules: " + assemblyName, null); + return false; + } + if (mconfig.ConditionTypes.Count != 0) { + monitor.ReportError ("Submodules can't define condition types: " + assemblyName, null); + return false; + } + if (mconfig.ExtensionNodeSets.Count != 0) { + monitor.ReportError ("Submodules can't define extension node sets: " + assemblyName, null); + return false; + } + if (mconfig.ExtensionPoints.Count != 0) { + monitor.ReportError ("Submodules can't define extension points sets: " + assemblyName, null); + return false; + } + mod.MergeWith (mconfig.MainModule); + } + ScanAssemblyContents (reflector, config, mod, asm, scanResult); + return true; + } + void ReportReflectionException (IProgressStatus monitor, Exception ex, AddinDescription config, AddinScanResult scanResult) { scanResult.AddFileToWithFailure (config.AddinFile); diff --git a/Mono.Addins/Mono.Addins.Description/AddinDescription.cs b/Mono.Addins/Mono.Addins.Description/AddinDescription.cs index 3799751..00eaaf2 100644 --- a/Mono.Addins/Mono.Addins.Description/AddinDescription.cs +++ b/Mono.Addins/Mono.Addins.Description/AddinDescription.cs @@ -1193,8 +1193,7 @@ namespace Mono.Addins.Description desc1.MainModule.Assemblies.Add (s); foreach (string s in desc2.MainModule.DataFiles) desc1.MainModule.DataFiles.Add (s); - desc1.MainModule.Dependencies.AddRange (desc2.MainModule.Dependencies); - desc1.MainModule.Extensions.AddRange (desc2.MainModule.Extensions); + desc1.MainModule.MergeWith (desc2.MainModule); return desc1; } diff --git a/Mono.Addins/Mono.Addins.Description/ModuleDescription.cs b/Mono.Addins/Mono.Addins.Description/ModuleDescription.cs index f57a33c..adc04f0 100644 --- a/Mono.Addins/Mono.Addins.Description/ModuleDescription.cs +++ b/Mono.Addins/Mono.Addins.Description/ModuleDescription.cs @@ -66,6 +66,12 @@ namespace Mono.Addins.Description { } + internal void MergeWith (ModuleDescription module) + { + Dependencies.AddRange (module.Dependencies); + Extensions.AddRange (module.Extensions); + } + /// <summary> /// Checks if this module depends on the specified add-in. /// </summary> |