geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vamsavardhana Reddy" <c1vams...@gmail.com>
Subject Re: Cleanup of directory server code in branches\2.0 and branches\2.0.0
Date Mon, 06 Aug 2007 15:23:07 GMT
Even better if the code is moved to plugins and removed from server tree at
the same time.

--vamsi


On 8/6/07, David Jencks <david_jencks@yahoo.com> wrote:
>
> I was hoping to get to this today.  I need to move the commented out stuff
> to the plugins area and I'd like to make it so the code removal on all
> branches is linked in svn.
> thanks
> david jencks
>
> On Aug 6, 2007, at 5:46 AM, Vamsavardhana Reddy wrote:
>
> As part of "GERONIMO-3340 Take apache directory plugin out of the main
> build, put it in plugins.", the directory server code has been disabled from
> the main build in rev 558213 (prior to creating branches\2.0), by commenting
> out the entries in pom.xml, but the code has been left in the trunk as it
> has not been moved to plugins.  Now that the code is in trunk, the copy of
> this unused code is not required in branches\2.0 and branches\2.0.0.  I
> thought I would cleanup this code from the branches.  I checked with Matt
> and he is ok with it.  But, he asked me to check on the dev-list.  If there
> are no objections, I propose to knock this unused code from branches (from
> branches\2.0.0 only if an RC has not been spun already).  Here is an excerpt
> from geronimo IRC.
>
>
> <vamsic007 >hogstrom: just wanted to findout if some unused directories in
> modules and configs can be removed from branches\2.0.0. The directory server
> related suff.
> <vamsic007> not that it is a big thing... but just some cleanup...
> <vamsic007 >hogstrom: Removing those dirs won't make a difference because
> the build is not touching those directories anyway as the corresponding
> entries have been commented out in pom.xml files
> <hogstrom >if you want to clean it up that's ok with me...however, we're
> dealing fall out from some other little changes :)
> < hogstrom>ask on the dev list ... I gotta run ... got a plane to catch
>
> hogstrom has left #geronimo
> <kevan> vamsic007: what are the dirs?
> <vamsic007> hogstrom: that's why I didn't propose any thing new to add at
> this stage :)
> <vamsic007> the directory server related stuff that has been disabled in
> the build for now...
> < vamsic007>hogstrom: ttyl
> <vamsic007>kevan: https://issues.apache.org/jira/browse/GERONIMO-3340
> <vamsic007> keavn: the directory server related stuff that has been
> disabled in the build for now...
> < vamsic007>kevan: the directory server related stuff that has been
> disabled in the build for now...
> <vamsic007>kevan: modules\geronimo-directory configs\ldap-realm
> configs\ldap-demo-jetty configs\ldap-demo-tomcat
> <vamsic007> kevan: The JIRA is on "Take apache directory plugin out of the
> main build, put it in plugins"
> <vamsic007>kevan: djencks disabled these in the main build in rev 558213
> by commenting out the pom.xml entries, but left the code in there as it
> has not been moved to plugins.
> <vamsic007> kevan: now that a copy of the code is in trunk, we may not
> need it in branches\2.0 and branches\2.0.0
>
>
> --vamsi
>
>
>

Mime
View raw message