So Castle.DynamicProxy.build should be deleted?

On Tue, Jan 12, 2010 at 12:13 PM, Jonathon Rossi <[email protected]> wrote:

> On Tue, Jan 12, 2010 at 9:09 PM, Roelof Blom <[email protected]>wrote:
>
>> Check DP for instance:
>>
>>    - default.build has <property name="project.version" value="2.2.0.0"
>>    />
>>    - Castle.DynamicProxy.build  has <property name="assembly.version"
>>    value="2.1.0.0" />
>>
>> We don't use Castle.DynamicProxy.build, its default.build even at the
> assembly level.
>
>
>>
>> Confusing (and wrong).
>>
>> On Tue, Jan 12, 2010 at 12:07 PM, Roelof Blom <[email protected]>wrote:
>>
>>> All the assembly.version instances should be removed and replaced by
>>> project.version in the top-level default.build for the project.
>>>
>>> On Tue, Jan 12, 2010 at 12:05 PM, John Simons <
>>> [email protected]> wrote:
>>>
>>>> This will be a breaking change, so if any project "default.build"
>>>> files do not have "<property name="assembly.version" value="X.Y.Z.W" /
>>>> >" defined (which btw are a lot of them, specially test projects), the
>>>> nant script will fail.
>>>> Is that ok?
>>>>
>>>>
>>>> On Jan 12, 9:59 pm, Jonathon Rossi <[email protected]> wrote:
>>>> > Oh I get Roelof now.
>>>> >
>>>> > Inside Castle.Core.dll's build script it has this:
>>>> > <property name="assembly.version" value="1.2.0.0" />
>>>> >
>>>> > When we already set this in the script for the project. Yes, I think
>>>> we
>>>> > should remove the assembly.version variable.
>>>> >
>>>> > On Tue, Jan 12, 2010 at 8:55 PM, John Simons <
>>>> [email protected]>wrote:
>>>> >
>>>> >
>>>> >
>>>> > > To set the version of an individual assembly, all you need to do is:
>>>> > > - Open the default.build file for the project  you want to set
>>>> > > - And add "<property name="assembly.version" value="x.y.z.w" />"
>>>> > > inside the "init" target, eg:
>>>> > >        <target name="init" depends="common.init">
>>>> > >                <property name="assembly.version" value="3.0.1.0" />
>>>> > >                <assemblyfileset id="project.references"
>>>> > > basedir="${build.dir}">
>>>> > >                </assemblyfileset>
>>>> > >        </target>
>>>> >
>>>> > > Cheers
>>>> > > John
>>>> >
>>>> > > On Jan 12, 9:39 pm, Jonathon Rossi <[email protected]> wrote:
>>>> > > > On Tue, Jan 12, 2010 at 8:22 PM, Roelof Blom <
>>>> [email protected]>
>>>> > > wrote:
>>>> > > > > The project.version property was originally for versioning the
>>>> entire
>>>> > > > > project, and assembly.version for versioning a specific project
>>>> > > assembly.
>>>> >
>>>> > > > I'm not sure I follow exactly. Are you saying we should get rid of
>>>> this:
>>>> >
>>>> > > >     <property overwrite="false" name="assembly.version"
>>>> > > > value="${project.major}.${project.minor}.${project.build}.0" />
>>>> > > >     <property overwrite="false"
>>>> name="assembly.version.informational"
>>>> >
>>>> > >
>>>> value="${assembly.major}.${assembly.minor}.${assembly.build}.${svn.revision}"
>>>> > > > />
>>>> >
>>>> > > >     <property name="assembly.major"
>>>> > > > value="${version::get-major(version::parse(assembly.version))}"
>>>> > > > dynamic="true" />
>>>> > > >     <property name="assembly.minor"
>>>> > > > value="${version::get-minor(version::parse(assembly.version))}"
>>>> > > > dynamic="true" />
>>>> > > >     <property name="assembly.build"
>>>> > > > value="${version::get-build(version::parse(assembly.version))}"
>>>> > > > dynamic="true" />
>>>> > > >     <property name="assembly.revision"
>>>> > > > value="${version::get-revision(version::parse(assembly.version))}"
>>>> > > > dynamic="true" />
>>>> >
>>>> > > > > With the schism we should probably drop assembly.version
>>>> >
>>>> > > > I'm not sure what "schism" means but we don't manually maintain
>>>> both, we
>>>> > > > only set project.version. I set the project version to 0.0.0.0 in
>>>> the
>>>> > > common
>>>> > > > build file because Krzysztof changed it from 1.1 to 1.2 because
>>>> the
>>>> > > logging
>>>> > > > services build script didn't have the line to change the project
>>>> version.
>>>> > > I
>>>> > > > don't think we should be using the common file to control any
>>>> specific
>>>> > > > project.
>>>> >
>>>> > > > Did I misunderstand you?
>>>> >
>>>> > > > > On Tue, Jan 12, 2010 at 11:19 AM, <[email protected]>
>>>> wrote:
>>>> >
>>>> > > > >>   *6637* <
>>>> > >http://fisheye2.atlassian.com/changelog/castleproject?cs=6637>
>>>> > > > >> *jonathon.rossi* Tue, 12 January 2010, 11:18:15 +0100
>>>> > > > >> ------------------------------
>>>> > > > >> Default the project version to 0.0.0.0 so that each project
>>>> needs to
>>>> > > set
>>>> > > > >> its own version number.
>>>> > > > >> ------------------------------
>>>> > > > >>   buildscripts/common.xml<
>>>> > >
>>>> http://fisheye2.atlassian.com/browse/castleproject/buildscripts/commo..
>>>> .>
>>>> > > > >> 6637<
>>>> > >
>>>> http://fisheye2.atlassian.com/browse/castleproject/buildscripts/commo..
>>>> .>
>>>> > > > >> (+1-1) diffs<
>>>> > >
>>>> http://fisheye2.atlassian.com/browse/castleproject/buildscripts/commo..
>>>> .>
>>>> >
>>>> > > > >> show watches <
>>>> http://fisheye2.atlassian.com/profile/showWatches.do>
>>>> > > > >> ------------------------------
>>>> >
>>>> > > > > --
>>>> > > > > You received this message because you are subscribed to the
>>>> Google
>>>> > > Groups
>>>> > > > > "Castle Project Development List" group.
>>>> > > > > To post to this group, send email to
>>>> > > [email protected]
>>>> > > > > .
>>>> > > > > To unsubscribe from this group, send email to
>>>> > > > > [email protected]<castle-project-devel%[email protected]>
>>>> <castle-project-devel%[email protected]<castle-project-devel%[email protected]>
>>>> >
>>>> > > <castle-project-devel%[email protected]<castle-project-devel%[email protected]>
>>>> <castle-project-devel%[email protected]<castle-project-devel%[email protected]>
>>>> >
>>>> >
>>>> > > > > .
>>>> > > > > For more options, visit this group at
>>>> > > > >http://groups.google.com/group/castle-project-devel?hl=en.
>>>> >
>>>> > > > --
>>>> > > > Jono
>>>> >
>>>> > > --
>>>> > > You received this message because you are subscribed to the Google
>>>> Groups
>>>> > > "Castle Project Development List" group.
>>>> > > To post to this group, send email to
>>>> [email protected]
>>>> > > .
>>>> > > To unsubscribe from this group, send email to
>>>> > > [email protected]<castle-project-devel%[email protected]>
>>>> <castle-project-devel%[email protected]<castle-project-devel%[email protected]>
>>>> >
>>>> > > .
>>>> > > For more options, visit this group at
>>>> > >http://groups.google.com/group/castle-project-devel?hl=en.
>>>> >
>>>> > --
>>>> > Jono
>>>>
>>>> --
>>>> You received this message because you are subscribed to the Google
>>>> Groups "Castle Project Development List" group.
>>>> To post to this group, send email to
>>>> [email protected].
>>>> To unsubscribe from this group, send email to
>>>> [email protected]<castle-project-devel%[email protected]>
>>>> .
>>>>
>>>> For more options, visit this group at
>>>> http://groups.google.com/group/castle-project-devel?hl=en.
>>>>
>>>>
>>>>
>>>>
>>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "Castle Project Development List" group.
>> To post to this group, send email to
>> [email protected].
>> To unsubscribe from this group, send email to
>> [email protected]<castle-project-devel%[email protected]>
>> .
>> For more options, visit this group at
>> http://groups.google.com/group/castle-project-devel?hl=en.
>>
>>
>
>
> --
> Jono
>
> --
> You received this message because you are subscribed to the Google Groups
> "Castle Project Development List" group.
> To post to this group, send email to [email protected]
> .
> To unsubscribe from this group, send email to
> [email protected]<castle-project-devel%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/castle-project-devel?hl=en.
>
>
--
You received this message because you are subscribed to the Google Groups "Castle Project Development List" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to [email protected].
For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.

Reply via email to