[Mono-bugs] [Bug 57612][Nor] Changed - [PATCH] Mono ignores privatePath in appconfig

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by urilith at gentoo.org.

http://bugzilla.ximian.com/show_bug.cgi?id=57612

--- shadow/57612	2005-07-06 06:06:25.000000000 -0400
+++ shadow/57612.tmp.18437	2005-07-06 06:08:49.000000000 -0400
@@ -10,13 +10,13 @@
 Component: misc
 AssignedTo: mono-bugs at ximian.com                            
 ReportedBy: gert.driesen at pandora.be               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: V1
 URL: 
-Summary: Mono ignores privatePath in appconfig
+Summary: [PATCH] Mono ignores privatePath in appconfig
 OtherBugsDependingOnThis: 57602[DUPLICATE], 57710[FIXED]
 
 Currently Mono does not use the semicolon delimited list of subdirectories 
 (of the application base) in the privatePath attribute of the 
 configuration/runtime/assemblyBinding/probing element to search for 
 assemblies.
@@ -81,6 +81,20 @@
 it'll work out of the box using the library's search path on Windows.
 
 ------- Additional Comments From urilith at gentoo.org  2005-07-06 06:06 -------
 Created an attachment (id=15437)
 Patch to fix PrivatePath issues.
 
+
+------- Additional Comments From urilith at gentoo.org  2005-07-06 06:08 -------
+The above patch fixes the problem for me.  It definitely works for the
+Load (byte[]) case, and should work for others as well.
+
+What I did was add a function update_path_env that is nearly identical
+to check_path_env in assembly.c.  Instead of checking the environment,
+update_path_env takes a MonoString containing the search dirs and adds
+those to assemblies_path.  In appdomain.c I added the update_path_env
+call in ves_icall_System_AppDomain_LoadAssemblyRaw if
+ad->data->setup->private_path is set.  This definitely works for me. 
+Out of my 34 NModule tests, 21 pass with a simple nunit-console
+NModule.Core.Test.dll, and the 13 that fail are bad test cases, not
+the loader.