You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by pa...@apache.org on 2017/10/14 13:27:36 UTC

[1/2] [lang] update easymock test dependency to version 3.5

Repository: commons-lang
Updated Branches:
  refs/heads/master 0dc5b94a0 -> 66226ec1c


update easymock test dependency to version 3.5


Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/2a8187fd
Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/2a8187fd
Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/2a8187fd

Branch: refs/heads/master
Commit: 2a8187fdf9bdc8ab4315a1a5e9ac7d9dde31fc68
Parents: 0dc5b94
Author: pascalschumacher <pa...@gmx.net>
Authored: Sat Oct 14 15:25:05 2017 +0200
Committer: pascalschumacher <pa...@gmx.net>
Committed: Sat Oct 14 15:25:05 2017 +0200

----------------------------------------------------------------------
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-lang/blob/2a8187fd/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 81f2d30..6ba161e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -532,7 +532,7 @@
     <dependency>
       <groupId>org.easymock</groupId>
       <artifactId>easymock</artifactId>
-      <version>3.4</version>
+      <version>3.5</version>
       <scope>test</scope>
     </dependency>
 


Re: [2/2] [lang] remove unused bcel test dependency

Posted by sebb <se...@gmail.com>.
On 16 October 2017 at 08:54, Benedikt Ritter <br...@apache.org> wrote:
> Hi Chas,
>
>> Am 15.10.2017 um 20:29 schrieb Chas Honton <ch...@honton.org>:
>>
>> You can use dependency:analyze to find used and undeclared or unused and declared dependencies.  Perhaps we should add dependency:analyze-only to release profile of parent pom to make sure dependencies are clean.
>
> Good idea! We could even add it to our default goal, so that it will be run on every Travis build.

I'm not keen on adding more to the default goal.
Depends on how long it takes to run, and does it use network resources?

> Benedikt
>
>>
>> Chas
>>
>>> On Oct 14, 2017, at 9:56 AM, Gary Gregory <ga...@gmail.com> wrote:
>>>
>>> boom! :-)
>>>
>>> too bad Checkstyle does not do that for you!
>>>
>>> Gary
>>>
>>>> On Oct 14, 2017 10:54, "Benedikt Ritter" <br...@apache.org> wrote:
>>>>
>>>> Nice one!
>>>>
>>>> <pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:
>>>>
>>>>> remove unused bcel test dependency
>>>>>
>>>>>
>>>>> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
>>>>> Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/
>>>>> 66226ec1
>>>>> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
>>>>> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>>>>>
>>>>> Branch: refs/heads/master
>>>>> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
>>>>> Parents: 2a8187f
>>>>> Author: pascalschumacher <pa...@gmx.net>
>>>>> Authored: Sat Oct 14 15:27:27 2017 +0200
>>>>> Committer: pascalschumacher <pa...@gmx.net>
>>>>> Committed: Sat Oct 14 15:27:27 2017 +0200
>>>>>
>>>>> ----------------------------------------------------------------------
>>>>> pom.xml | 7 -------
>>>>> 1 file changed, 7 deletions(-)
>>>>> ----------------------------------------------------------------------
>>>>>
>>>>>
>>>>> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
>>>>> ----------------------------------------------------------------------
>>>>> diff --git a/pom.xml b/pom.xml
>>>>> index 6ba161e..6db828a 100644
>>>>> --- a/pom.xml
>>>>> +++ b/pom.xml
>>>>> @@ -523,13 +523,6 @@
>>>>>    </dependency>
>>>>>
>>>>>    <dependency>
>>>>> -      <groupId>org.apache.bcel</groupId>
>>>>> -      <artifactId>bcel</artifactId>
>>>>> -      <version>6.0</version>
>>>>> -      <scope>test</scope>
>>>>> -    </dependency>
>>>>> -
>>>>> -    <dependency>
>>>>>      <groupId>org.easymock</groupId>
>>>>>      <artifactId>easymock</artifactId>
>>>>>      <version>3.5</version>
>>>>>
>>>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> For additional commands, e-mail: dev-help@commons.apache.org
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [2/2] [lang] remove unused bcel test dependency

Posted by Benedikt Ritter <br...@apache.org>.
Hi Chas,

> Am 15.10.2017 um 20:29 schrieb Chas Honton <ch...@honton.org>:
> 
> You can use dependency:analyze to find used and undeclared or unused and declared dependencies.  Perhaps we should add dependency:analyze-only to release profile of parent pom to make sure dependencies are clean. 

Good idea! We could even add it to our default goal, so that it will be run on every Travis build.

Benedikt

> 
> Chas
> 
>> On Oct 14, 2017, at 9:56 AM, Gary Gregory <ga...@gmail.com> wrote:
>> 
>> boom! :-)
>> 
>> too bad Checkstyle does not do that for you!
>> 
>> Gary
>> 
>>> On Oct 14, 2017 10:54, "Benedikt Ritter" <br...@apache.org> wrote:
>>> 
>>> Nice one!
>>> 
>>> <pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:
>>> 
>>>> remove unused bcel test dependency
>>>> 
>>>> 
>>>> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
>>>> Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/
>>>> 66226ec1
>>>> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
>>>> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>>>> 
>>>> Branch: refs/heads/master
>>>> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
>>>> Parents: 2a8187f
>>>> Author: pascalschumacher <pa...@gmx.net>
>>>> Authored: Sat Oct 14 15:27:27 2017 +0200
>>>> Committer: pascalschumacher <pa...@gmx.net>
>>>> Committed: Sat Oct 14 15:27:27 2017 +0200
>>>> 
>>>> ----------------------------------------------------------------------
>>>> pom.xml | 7 -------
>>>> 1 file changed, 7 deletions(-)
>>>> ----------------------------------------------------------------------
>>>> 
>>>> 
>>>> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
>>>> ----------------------------------------------------------------------
>>>> diff --git a/pom.xml b/pom.xml
>>>> index 6ba161e..6db828a 100644
>>>> --- a/pom.xml
>>>> +++ b/pom.xml
>>>> @@ -523,13 +523,6 @@
>>>>    </dependency>
>>>> 
>>>>    <dependency>
>>>> -      <groupId>org.apache.bcel</groupId>
>>>> -      <artifactId>bcel</artifactId>
>>>> -      <version>6.0</version>
>>>> -      <scope>test</scope>
>>>> -    </dependency>
>>>> -
>>>> -    <dependency>
>>>>      <groupId>org.easymock</groupId>
>>>>      <artifactId>easymock</artifactId>
>>>>      <version>3.5</version>
>>>> 
>>>> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [2/2] [lang] remove unused bcel test dependency

Posted by Chas Honton <ch...@honton.org>.
You can use dependency:analyze to find used and undeclared or unused and declared dependencies.  Perhaps we should add dependency:analyze-only to release profile of parent pom to make sure dependencies are clean. 

Chas

> On Oct 14, 2017, at 9:56 AM, Gary Gregory <ga...@gmail.com> wrote:
> 
> boom! :-)
> 
> too bad Checkstyle does not do that for you!
> 
> Gary
> 
>> On Oct 14, 2017 10:54, "Benedikt Ritter" <br...@apache.org> wrote:
>> 
>> Nice one!
>> 
>> <pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:
>> 
>>> remove unused bcel test dependency
>>> 
>>> 
>>> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
>>> Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/
>>> 66226ec1
>>> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
>>> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>>> 
>>> Branch: refs/heads/master
>>> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
>>> Parents: 2a8187f
>>> Author: pascalschumacher <pa...@gmx.net>
>>> Authored: Sat Oct 14 15:27:27 2017 +0200
>>> Committer: pascalschumacher <pa...@gmx.net>
>>> Committed: Sat Oct 14 15:27:27 2017 +0200
>>> 
>>> ----------------------------------------------------------------------
>>> pom.xml | 7 -------
>>> 1 file changed, 7 deletions(-)
>>> ----------------------------------------------------------------------
>>> 
>>> 
>>> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
>>> ----------------------------------------------------------------------
>>> diff --git a/pom.xml b/pom.xml
>>> index 6ba161e..6db828a 100644
>>> --- a/pom.xml
>>> +++ b/pom.xml
>>> @@ -523,13 +523,6 @@
>>>     </dependency>
>>> 
>>>     <dependency>
>>> -      <groupId>org.apache.bcel</groupId>
>>> -      <artifactId>bcel</artifactId>
>>> -      <version>6.0</version>
>>> -      <scope>test</scope>
>>> -    </dependency>
>>> -
>>> -    <dependency>
>>>       <groupId>org.easymock</groupId>
>>>       <artifactId>easymock</artifactId>
>>>       <version>3.5</version>
>>> 
>>> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: [2/2] [lang] remove unused bcel test dependency

Posted by Gary Gregory <ga...@gmail.com>.
boom! :-)

too bad Checkstyle does not do that for you!

Gary

On Oct 14, 2017 10:54, "Benedikt Ritter" <br...@apache.org> wrote:

> Nice one!
>
> <pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:
>
>> remove unused bcel test dependency
>>
>>
>> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
>> Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/
>> 66226ec1
>> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
>> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>>
>> Branch: refs/heads/master
>> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
>> Parents: 2a8187f
>> Author: pascalschumacher <pa...@gmx.net>
>> Authored: Sat Oct 14 15:27:27 2017 +0200
>> Committer: pascalschumacher <pa...@gmx.net>
>> Committed: Sat Oct 14 15:27:27 2017 +0200
>>
>> ----------------------------------------------------------------------
>>  pom.xml | 7 -------
>>  1 file changed, 7 deletions(-)
>> ----------------------------------------------------------------------
>>
>>
>> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
>> ----------------------------------------------------------------------
>> diff --git a/pom.xml b/pom.xml
>> index 6ba161e..6db828a 100644
>> --- a/pom.xml
>> +++ b/pom.xml
>> @@ -523,13 +523,6 @@
>>      </dependency>
>>
>>      <dependency>
>> -      <groupId>org.apache.bcel</groupId>
>> -      <artifactId>bcel</artifactId>
>> -      <version>6.0</version>
>> -      <scope>test</scope>
>> -    </dependency>
>> -
>> -    <dependency>
>>        <groupId>org.easymock</groupId>
>>        <artifactId>easymock</artifactId>
>>        <version>3.5</version>
>>
>>

Re: [2/2] [lang] remove unused bcel test dependency

Posted by Benedikt Ritter <br...@apache.org>.
Nice one!

<pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:

> remove unused bcel test dependency
>
>
> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
> Commit:
> http://git-wip-us.apache.org/repos/asf/commons-lang/commit/66226ec1
> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>
> Branch: refs/heads/master
> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
> Parents: 2a8187f
> Author: pascalschumacher <pa...@gmx.net>
> Authored: Sat Oct 14 15:27:27 2017 +0200
> Committer: pascalschumacher <pa...@gmx.net>
> Committed: Sat Oct 14 15:27:27 2017 +0200
>
> ----------------------------------------------------------------------
>  pom.xml | 7 -------
>  1 file changed, 7 deletions(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
> ----------------------------------------------------------------------
> diff --git a/pom.xml b/pom.xml
> index 6ba161e..6db828a 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -523,13 +523,6 @@
>      </dependency>
>
>      <dependency>
> -      <groupId>org.apache.bcel</groupId>
> -      <artifactId>bcel</artifactId>
> -      <version>6.0</version>
> -      <scope>test</scope>
> -    </dependency>
> -
> -    <dependency>
>        <groupId>org.easymock</groupId>
>        <artifactId>easymock</artifactId>
>        <version>3.5</version>
>
>

Re: [2/2] [lang] remove unused bcel test dependency

Posted by Benedikt Ritter <br...@apache.org>.
Nice one!

<pa...@apache.org> schrieb am Sa. 14. Okt. 2017 um 15:27:

> remove unused bcel test dependency
>
>
> Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
> Commit:
> http://git-wip-us.apache.org/repos/asf/commons-lang/commit/66226ec1
> Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
> Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1
>
> Branch: refs/heads/master
> Commit: 66226ec1c2ff33e138189463001c649dbb404f56
> Parents: 2a8187f
> Author: pascalschumacher <pa...@gmx.net>
> Authored: Sat Oct 14 15:27:27 2017 +0200
> Committer: pascalschumacher <pa...@gmx.net>
> Committed: Sat Oct 14 15:27:27 2017 +0200
>
> ----------------------------------------------------------------------
>  pom.xml | 7 -------
>  1 file changed, 7 deletions(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
> ----------------------------------------------------------------------
> diff --git a/pom.xml b/pom.xml
> index 6ba161e..6db828a 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -523,13 +523,6 @@
>      </dependency>
>
>      <dependency>
> -      <groupId>org.apache.bcel</groupId>
> -      <artifactId>bcel</artifactId>
> -      <version>6.0</version>
> -      <scope>test</scope>
> -    </dependency>
> -
> -    <dependency>
>        <groupId>org.easymock</groupId>
>        <artifactId>easymock</artifactId>
>        <version>3.5</version>
>
>

[2/2] [lang] remove unused bcel test dependency

Posted by pa...@apache.org.
remove unused bcel test dependency


Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/66226ec1
Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/66226ec1
Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/66226ec1

Branch: refs/heads/master
Commit: 66226ec1c2ff33e138189463001c649dbb404f56
Parents: 2a8187f
Author: pascalschumacher <pa...@gmx.net>
Authored: Sat Oct 14 15:27:27 2017 +0200
Committer: pascalschumacher <pa...@gmx.net>
Committed: Sat Oct 14 15:27:27 2017 +0200

----------------------------------------------------------------------
 pom.xml | 7 -------
 1 file changed, 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-lang/blob/66226ec1/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 6ba161e..6db828a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -523,13 +523,6 @@
     </dependency>
 
     <dependency>
-      <groupId>org.apache.bcel</groupId>
-      <artifactId>bcel</artifactId>
-      <version>6.0</version>
-      <scope>test</scope>
-    </dependency>
-
-    <dependency>
       <groupId>org.easymock</groupId>
       <artifactId>easymock</artifactId>
       <version>3.5</version>