On my system, the generate source is perfectly valid :

[assembly:
AssemblyKeyFileAttribute("C:\\first\\project\\FCFL.NET\\build\\net-1.1.win32
\\FCFL.NET-4.0-debugbuild\\Licensing\\L" +
"icenseInstaller\\bin\\..\\..\\..\\..\\Key.snk")]

what version of the .NET Framework are you using ?

Gert

----- Original Message ----- 
From: "Kevin Miller" <[EMAIL PROTECTED]>
To: "Gert Driesen" <[EMAIL PROTECTED]>;
<[EMAIL PROTECTED]>
Sent: Monday, March 01, 2004 6:54 PM
Subject: RE: [nant-dev] AsmInfo patch


Another bug that I noticed is that if someone uses forward slashes in a

<attribute type="AssemblyKeyFileAttribute"
value="${build.dir}/../Key.snk" />
*Alternatively*
<attribute type="AssemblyKeyFileAttribute"
value="C:\first\project\FCFL.NET\build\net-1.1.win32\FCFL.NET-4.0-debugb
uild\Licensing\LicenseInstaller\bin\..\..\..\..\Key.snk" />

Result in an attribute like this:

[assembly:
AssemblyKeyFileAttribute("C:\\first\\project\\FCFL.NET/build/net-1.1.win
32/FCFL.NET-4.0-debugbuild/Licensing/L" +
"icenseInstaller/../Key.snk")]

I am guessing this is because the path is so long that the codedom is
trying to split it across lines. Which is unfortunate as this does not
seem to be legal syntax. Any CodeDOM-fu help here?

Kevin

-----Original Message-----
From: Gert Driesen [mailto:[EMAIL PROTECTED]
Sent: Saturday, February 28, 2004 5:18 PM
To: Kevin Miller
Subject: Re: [nant-dev] AsmInfo patch

well, clean and less error prone code is more important than performance
for tasks that are not called many times during a normal build cycle ...

keep the patches coming, we can use all help we can get :)

Gert

----- Original Message -----
From: "Kevin Miller" <[EMAIL PROTECTED]>
To: "Gert Driesen" <[EMAIL PROTECTED]>
Sent: Saturday, February 28, 2004 5:56 PM
Subject: RE: [nant-dev] AsmInfo patch


Gasp, well glad you enjoyed it so much it's performance didn't matter.

Thanks!
Kevin

-----Original Message-----
From: Gert Driesen [mailto:[EMAIL PROTECTED]
Sent: Sat 2/28/2004 10:30 AM
To: Kevin Miller; [EMAIL PROTECTED]
Subject: Re: [nant-dev] AsmInfo patch

Your implementation is indeed slower, but I've committed it to CVS
anyway
...

Thanks for the contribution !

Gert

----- Original Message ----- 
From: "Kevin Miller" <[EMAIL PROTECTED]>
To: "Gert Driesen" <[EMAIL PROTECTED]>;
<[EMAIL PROTECTED]>
Sent: Saturday, February 28, 2004 4:28 PM
Subject: RE: [nant-dev] AsmInfo patch


Ah interesting. I am sure your CodeDOM-fu is better than mine.

I am not sure how negative the "extra" generation of the assemblyinfo
file
when switching runtimes is. I am guessing if you are building multiple
runtime versions of your application you still need to compile all the
assemblies in the build for each runtime. Thus the only negative I see
in
that scenario is the extra time it takes to write the MemoryStream out
to
disk.




-----Original Message-----
From: Gert Driesen [mailto:[EMAIL PROTECTED]
Sent: Sat 2/28/2004 9:23 AM
To: Kevin Miller; [EMAIL PROTECTED]
Subject: Re: [nant-dev] AsmInfo patch

The problem is that the C# code provider automatically inserts the
comment
block I mentioned into the generated file.  Meaning the <asm> task will
generate different files when run on .NET 1.0 and .NET 1.1, as the
runtime
version that's output in that comment block will be different.

If there's any way of preventing that comment block from being
generated,
please let me know ...

We can still use your hash implementation, but in that case the <asm>
will
regenerate the AssemblyInfo file when switching runtimes (as the hash
will
be different because of the fact that the comment block that's
automatically
generated by the code provider will be different) ...

Gert

----- Original Message ----- 
From: "Kevin Miller" <[EMAIL PROTECTED]>
To: "Gert Driesen" <[EMAIL PROTECTED]>;
<[EMAIL PROTECTED]>
Sent: Saturday, February 28, 2004 3:58 PM
Subject: RE: [nant-dev] AsmInfo patch


Funny, oh well it was fun getting my hands dirty with NAnt stuff again.

I say use my implementation :)  Hmm, Not sure what which would be
better.

I don't understand the importance of the 1.0 vs 1.1 comment differences
when
the file generated will not have any comments. Unless I missed out on a
feature of asminfo that allows those comments to be retained.

I say if the comment feature is not explicitly necessary and the
performance
of hashing is better than line by line compare which is anyones guess
(my
gut tells my the line by line may be better) that we use the hash based
method.

Honestly I do not care too much either way as long as the feature makes
it
into NAnt :)

Kevin

-----Original Message-----
From: Gert Driesen [mailto:[EMAIL PROTECTED]
Sent: Sat 2/28/2004 2:01 AM
To: Kevin Miller; [EMAIL PROTECTED]
Subject: Re: [nant-dev] AsmInfo patch

Hi Kevin,

I just added support for this to the <asminfo> task in cvs yesterday :(

I actually compared each line of both files and skipped all comments ...
I
did this as files generated with .NET 1.0 or .NET 1.1 only differ in the
comment block that is insert into the generated file :

//----------------------------------------------------------------------
----
----
// <autogenerated>
//     This code was generated by a tool.
//     Runtime Version: 1.1.4322.940
//
//     Changes to this file may cause incorrect behavior and will be
lost if
//     the code is regenerated.
// </autogenerated>
//----------------------------------------------------------------------
----
----

However, I do like your approach a lot better.  So, I'm not sure if I
should
keep the current (quick and dirty) implementation or use yours ...

What do you think ?

Gert

----- Original Message ----- 
From: "Kevin Miller" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Friday, February 27, 2004 9:24 PM
Subject: [nant-dev] AsmInfo patch


I found the asminfo task today and fell in love. It solved a few
problems I was having with signing assemblies and paths to the key file.
I was frustrated by the fact that it always generates the
AssemblyInfo.cs file even if there are no changes. This causes the
csc/vbc task to build the project even if only the AssemblyInfo class
has been updated. This is clearly an annoyance during development
builds.

Attached is an updated  that compares the present AssemblyInfo.<x> to
the one being generated. If they are different it writes the new code
one out to disk.

This file is updating the .84 release. Sorry I did not compare to the
latest cvs. Also, I did not add a switch to make this behavior optional.
This post is just to feel out well this enhancement would be received. I
am willing to create a decent patch. Let me know if there is interest in
this feature.

Kevin Miller




















-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
_______________________________________________
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers

Reply via email to