Re: [nant-dev] ILASM Task

[Ian MacLean]

Gert Driesen wrote:
Giuseppe Greco wrote:
   

Ach... I knew, in fact I've used writer.Write everywhere,
but not in the WriteOptions method because I've copied
the 3-lines 'foreach' statement from CompilerBase...
By the way, attached to this email you'll find the
definitive source file.
     

thanks. I'll look at adding this to the code base shortly.
   

Will you be adding it to NAnt or NAntContrib ?  Guess we should discuss what
procedure to follow when adding new tasks, or do decide on a task-by-task
basis ?
 

Whatever. Obviously it belongs in NAnt.DotNet.Tasks. Its a fairly 
straightforward command line app wrapper so theres not likely to be any 
issues putting it straight into core. I'm not convinced that we have to 
put everything in contrib first if it has an obvious fit in core - ie 
its a piece of previously missing functionality - I would put any other 
.net command line tools in this category. Others may have different 
opinions though.

Ian

-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com
_______________________________________________
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers