Re: [nant-dev] AsmInfo patch

2004-03-01 Thread Gert Driesen
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 :


[nant-dev] [ nant-Bugs-907825 ] Precompiled header for C++ Projects

2004-03-01 Thread SourceForge.net
Bugs item #907825, was opened at 2004-03-01 15:39
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detailatid=402868aid=907825group_id=31650

Category: Tasks
Group: 0.85
Status: Open
Resolution: None
Priority: 5
Submitted By: Maxime Thibeault (zixthree)
Assigned to: Nobody/Anonymous (nobody)
Summary: Precompiled header for C++ Projects

Initial Comment:
Support for precompiled header in C++ projects pour 
Solution Task is incomplete. At least, none of my simple 
C++ projects get to compile telling me pre-compiled 
header is missing even though it is on generate, even 
when it is not using pre-compiled headers at all.

--

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detailatid=402868aid=907825group_id=31650


---
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=1356alloc_id=3438op=click
___
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers


[nant-dev] Current target again...

2004-03-01 Thread Giuseppe Greco
Hi all,

In my previous email there was a copy-and-paste
problem, and my explanation was incomplete!

I'll try again...

I use to structure projects like this:

/myProject
+ default.build
+ /src
+ default.build
+ /subproject1
|   + default.build
|   + mySource.cs
+ /subproject2
+ default.build
+ mySource.cs

This structure let me build subprojects from any
level. The build file located in the /src directory
is just a gateway build file, and looks like this:

?xml version=1.0?

project
  name=myProject
  default=recurse

  target
name=recurse
description=Builds recursively all subprojects
foreach
  item=Folder
  property=foldername
  in
items
  includes name=*/
  excludes name=CVS/
/items
  /in
  do
nant
  buildfile=${foldername}/default.build
  target=${project.config} ${target}/
  /do
/foreach
  /target

/project

The 'target' property of the nant task specifies
two targets: ${project.config} and ${target}. Both
are inherited from the calling build file (the one
located in the project's root directory).

${project.config} contains 'release' or 'debug', while
${target} contains the target that should be executed
in each subproject.

My proposal is to introduce the concept of generic target.
Let's go back to our gateway build file described above,
and rewrite the recurse target as a generic target:

?xml version=1.0?

project
  name=myProject
  default=*

  target
name=*
description=Builds recursively all subprojects
foreach
  item=Folder
  property=foldername
  in
items
  includes name=*/
  excludes name=CVS/
/items
  /in
  do
nant
  buildfile=${foldername}/default.build
  target=${project.config} ${nant.current.target}/
  /do
/foreach
  /target

/project

Generic targets would specify * as their name, and NAnt
would always execute them. At that point we also need a way
to get the target name specified by the calling build file,
and the answer is a built-in property named ${nant.current.target}.

What's your opinion?

j3d.


Giuseppe Greco

::agamura::

phone:  +41 (0)91 604 67 65
mobile: +41 (0)76 390 60 32
email:  [EMAIL PROTECTED]
web:www.agamura.com



---
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=1356alloc_id=3438op=click
___
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers