Ian,

I think we need to add a <resource> child element that would allow the
manifest resource name of a given resources file to be explicitly set.

We actually need this ourselves in several places :

- <al> : right now, we're forced to pass embedded resources from
CompilerBase to the <al> task as command line arguments
- NAntContrib : just found a major issue with the msi task / schema
validated tasks, to fix it I have to embed the msi xsd file directly into
the assembly, but in order to match the manifest resource name that the
schema validator expects I actually have to copy the xsd to give it a file
name that matches the manifest resource name, which is kinda of a hack.
Would be better if the <resourcefileset> type allowed explicitly manifest
resource names to be set for a given resource.

Now, I'm not quite finished with the details of the implementation :

- should we extend ResourceFileSet ? Not sure about this, as it doesn't
always make sense to have manifest resource names : ResourceFileSet is also
used for the <resgen> task, which does not involve manifest resource names.
So, we might actually need a different type for the <resgen> and the
compiler tasks.

- <al> currently uses a normal fileset, it does not try to automatically
determine manifest resource names for the resources it embeds. So for <al>
we'd need a new type (which derives from FileSet) that also supports the
nested <resource> element. Right ?

- should we extend any fileset at all ? Should this perhaps be a completely
separate collection of elements ? Not sure I like this...

- should the file be resolved to a full path using the base directory of the
containing fileset (or resourcefileset, or whatever) ? Guess so, right ?

- do we automatically compile the file if its a resx/licx file ? Don't think
we should do this ...

What do you think ?

Gert



-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=4721&alloc_id=10040&op=click
_______________________________________________
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers

Reply via email to