Agree for 1) or 2) but not on 3) as if you put the tests in api, you will
not have one of the implementation in the classpath and the tests will fail.
Regards
Jeff MAURY
On 8/26/08, Jeremy Hughes <[EMAIL PROTECTED]> wrote:
>
> Hi John, I agree with all the changes you propose.
>
> Cheers,
> Jeremy
>
> 2008/8/18 John Kaputin (gmail) <[EMAIL PROTECTED]>
>
>> I have some more thoughts on the source tree restructuring. I'm happy to
>> make the changes in the woden211 branch if people agree.
>>
>> 1) Could we change 'woden-commons' to 'woden-common' as this directory
>> just holds common woden code (not multiple common projects as per
>> ws-commons).
>>
>
> +1
>
>
>
>>
>>
>> 2) Could we change 'woden-tool' to 'woden-tools' as this directory does
>> hold multiple tools (the converter and the wsdl-viewer).
>>
>
> +1
>
>
>>
>>
>> 3) I think some of the test cases under woden-test/src/test can be moved
>> to woden-api/src/test or woden-commons/src/test. The ones in the testcase.*
>> packages should probably stay under woden-test, but the following ones can
>> be moved:
>>
>> woden-tests/src/test/java/org/apache/woden to
>> woden-api/src/test/java/org/apache/woden
>>
>>
>
>> woden-tests/src/test/java/org/apache/woden/internal/wsdl20/assertions
>> woden-commons/src/test/java/org/apache/woden/internal/wsdl20/assertions
>>
>> woden-tests/src/test/java/org/apache/woden/internal/wsdl20/validation
>> woden-commons/src/test/java/org/apache/woden/internal/wsdl20/validation
>>
>> woden-tests/src/test/java/org/apache/woden/resolver
>> woden-commons/src/test/java/org/apache/woden/internal/resolver ('internal'
>> added)
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20
>> woden-api/src/test/java/org/apache/woden/wsdl20
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20/extensions
>> woden-api/src/test/java/org/apache/woden/wsdl20/extensions
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20/extensions/http
>> woden-api/src/test/java/org/apache/woden/wsdl20/extensions/http
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20/extensions/soap
>> woden-api/src/test/java/org/apache/woden/wsdl20/extensions/soap
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20/fragids
>> woden-api/src/test/java/org/apache/woden/wsdl20/fragids
>>
>> woden-tests/src/test/java/org/apache/woden/wsdl20/xml
>> woden-api/src/test/java/org/apache/woden/wsdl20/xml
>>
>>
>
> +1 to all these too
>
>
>
>>
>> regards,
>> John Kaputin.
>>
>>
>
>
>
--
La mélancolie c'est communiste
Tout le monde y a droit de temps en temps
La mélancolie n'est pas capitaliste
C'est même gratuit pour les perdants
La mélancolie c'est pacifiste
On ne lui rentre jamais dedans
La mélancolie oh tu sais ça existe
Elle se prend même avec des gants
La mélancolie c'est pour les syndicalistes
Il faut juste sa carte de permanent
Miossec (2006)
http://www.jeffmaury.com
http://riadiscuss.jeffmaury.com
http://www.lastfm.fr/listen/user/jeffmaury/personal
Mes CDs à récupérer:
http://spreadsheets.google.com/ccc?key=pNeg4Doa_oCsh7CepKPaPTA&hl=en
