update for latest nant

[Ian MacLean]

Hi all,
I'm a developer on the nant project. We just had a user query about VB.doc and in the process of answering it I rebuilt the vb.doc nant task against the latest cvs of nant. To do so I needed to make a couple of small changes so I've attached the patch to do that. You may not want to apply this until nant releases a new version but it may be useful in the meantime for those of your users using the bleeding edge nant builds. This was the first I encountered VB.Doc - great project. I don't do much vb.net coding but if I did I'd be stoked with this.

Oh one other thing - is there any reason you're using
<arg value="/reference:${bin}/Nant.Core.dll"/> in your build files to add references instead of using the <reference> fileset ?

Ian

VBDOCTask.cs.diff follows:

--- VBDOCTask.cs    2003-12-29 12:40:08.000000000 +0900
+++ VBDOCTaskupdated.cs    2004-04-14 11:03:32.000000000 +0900
@@ -198,11 +198,11 @@

                 for(int i=0;i<sources.FileNames.Count;i++)
                 {
-                    options.Files[i] = Path.Combine(sources.BaseDirectory, sources.FileNames[i]);
+                    options.Files[i] = Path.Combine(sources.BaseDirectory.FullName, sources.FileNames[i]);
                 }
                
                 for(int i=0;i<referencedirectories.DirectoryNames.Count;i++) {
-                    options.ReferenceDirectories[i] = Path.Combine(referencedirectories.BaseDirectory, referencedirectories.DirectoryNames[i]);
+                    options.ReferenceDirectories[i] = Path.Combine(referencedirectories.BaseDirectory.FullName, referencedirectories.DirectoryNames[i]);
                 }
                
                 if(rootnamespace != null) {

[Markus Palme]

Hi Ian !

Thanks for the patch, I will incorporate it when the next stable version of NAnt is released.

>is there any reason you're using
><arg value="/reference:${bin}/Nant.Core.dll"/>
> [...]

No. Once, I was using a version of NAnt where the reference tag was broken and I didn't revert this. It's fixed now in CVS.

Thanks
Markus