Mailing-List: contact dev-help@felix.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@felix.apache.org
Received-SPF: pass (nike.apache.org: domain of heavy@ungoverned.org designates
 67.222.39.38 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=ungoverned.org;
	h=Received:Message-ID:Date:From:User-Agent:MIME-Version:To:Subject:References:In-Reply-To:Content-Type:Content-Transfer-Encoding:X-Identified-User;
	b=W0QaNWJ2+3ggUOdM+sJXyZB7ilEXYfOHWM36OKojzVSWwCvXrP7DWofrcduGzXW4uk7Me7gDelqaLUD88R/l2GW/dLXJQhSNh/1hwGDasWRwOflOhzxyoLZNoq6wHGDx;
Message-ID: <4BA13C87.9010102@ungoverned.org>
Date: Wed, 17 Mar 2010 13:33:11 -0700
From: "Richard S. Hall" <heavy@ungoverned.org>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US;
 rv:1.9.1.8) Gecko/20100227 Thunderbird/3.0.3
MIME-Version: 1.0
To: dev@felix.apache.org
Subject: Re: Refactor bundlerepositorty and obr related code from
 	maven-bundle-plugin
 ?
References: <b23ecedc1003170232g8308428w2a3b6aa495ea04ba@mail.gmail.com>
	 <b23ecedc1003171221y4f980710rb50c945476a3b0ed@mail.gmail.com>
	 <4BA12C4C.8020503@ungoverned.org>
 <b23ecedc1003171312s7e5c7aa7r7ece5d22a9957ae@mail.gmail.com>
In-Reply-To: <b23ecedc1003171312s7e5c7aa7r7ece5d22a9957ae@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 3/17/10 13:12, Guillaume Nodet wrote:
> Yeah, what I mean is that we have a lot of different models to represent the
> same data.
>    * the one in bundlerepository
>    * the one included in the maven-bundle-plugin with the bindex code
>    * another one from the maven-bundle-plugin inherited from the
> obr-maven-plugin
> Those three models are slightly different but really represent the same
> data, so I'm planning to enhance the one from bundlerepository to support
> all the use cases.
>
> In addition, i'll extract some bits (header parser, version range, etc...)
> in a util subproject which can be reused (simply by embedding the classes
> for now) in the other subprojects.
>    

Ok.

-> richard

> On Wed, Mar 17, 2010 at 20:23, Richard S. Hall<heavy@ungoverned.org>  wrote:
>
>    
>> On 3/17/10 12:21, Guillaume Nodet wrote:
>>
>>      
>>> One thing I'm struggling with is the number of other projects included in
>>> the maven-bundle-plugin.
>>> It seems we include the whole bindex code, obr plugins, some maven bits.
>>> There are a lot of duplication here.
>>> For bindex, why do we include it ? It is mostly redundant with the
>>> bundlerepository.
>>>
>>>
>>>        
>> We don't include bindex in OBR, so I am not sure how that is redundant. I
>> believe bindex is included in maven-bundle-plugin because it needs to
>> generate repository.xml files from the generated bundle files.
>>
>> ->  richard
>>
>>
>>   I think I'm going to refactor the whole thing unless there is something
>>      
>>> obvious I'm missing here.
>>>
>>> On Wed, Mar 17, 2010 at 10:32, Guillaume Nodet<gnodet@gmail.com>   wrote:
>>>
>>>
>>>
>>>        
>>>> Both bundlerepository and maven-bundle-plugin contain similar code
>>>> like the RepositoryImpl, BundleInfo (quite similar to the
>>>> LocalResourceImpl from bundlerepository), repository xml parser,
>>>> etc...
>>>> I think it would make more sense to enhance the ones in
>>>> bundlerepostiory and reuse those from the maven-bundle-plugin rather
>>>> than keep the duplication.
>>>>
>>>> Thoughts ?
>>>>
>>>> --
>>>> Cheers,
>>>> Guillaume Nodet
>>>> ------------------------
>>>> Blog: http://gnodet.blogspot.com/
>>>> ------------------------
>>>> Open Source SOA
>>>> http://fusesource.com
>>>>
>>>>
>>>>
>>>>          
>>>
>>>
>>>
>>>        
>>      
>
>