[GitHub] [commons-compress] PeterAlfredLee commented on pull request #127: COMPRESS-550 : add writePreamble to ZipArchiveInputStream

2020-08-27 Thread GitBox


PeterAlfredLee commented on pull request #127:
URL: https://github.com/apache/commons-compress/pull/127#issuecomment-682278860


   Hi all. I'm merging this PR these days. WDYT? @garydgregory @bodewig 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Work logged] (COMPRESS-550) Support writing self-extracting preamble data to a zip archive

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/COMPRESS-550?focusedWorklogId=475636=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475636
 ]

ASF GitHub Bot logged work on COMPRESS-550:
---

Author: ASF GitHub Bot
Created on: 28/Aug/20 01:48
Start Date: 28/Aug/20 01:48
Worklog Time Spent: 10m 
  Work Description: PeterAlfredLee commented on pull request #127:
URL: https://github.com/apache/commons-compress/pull/127#issuecomment-682278860


   Hi all. I'm merging this PR these days. WDYT? @garydgregory @bodewig 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475636)
Time Spent: 1h 20m  (was: 1h 10m)

> Support writing self-extracting preamble data to a zip archive
> --
>
> Key: COMPRESS-550
> URL: https://issues.apache.org/jira/browse/COMPRESS-550
> Project: Commons Compress
>  Issue Type: New Feature
>  Components: Archivers
>Affects Versions: 1.20
>Reporter: Scott Frederick
>Priority: Major
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> On Linux, it is possible to create a self-extracting zip file by prepending a 
> script or other executable to the zip file. For example, this can be done 
> from the command line with the following commands: 
> {code:bash}
> $ echo "test file text" > test.txt
> $ zip test.zip test.txt
>   adding: test.txt (stored 0%)
> $ cat /usr/bin/unzipsfx test.zip > test
> $ zipinfo test
> Archive:  test
> Zip file size: 101568 bytes, number of entries: 1
> warning [test]:  101392 extra bytes at beginning or within zipfile
>   (attempting to process anyway)
> -rw-r--r--  3.0 unx   10 tx stor 20-Jul-30 15:43 test.txt
> 1 file, 10 bytes uncompressed, 10 bytes compressed:  0.0%
> $ zip --adjust-sfx test.zip
> Zip entry offsets appear off by 101392 bytes - correcting...
> $ zipinfo test
> Archive:  test
> Zip file size: 101568 bytes, number of entries: 1
> -rw-r--r--  3.0 unx   10 tx stor 20-Jul-30 15:43 test.txt
> 1 file, 10 bytes uncompressed, 10 bytes compressed:  0.0%
> $ chmod 755 test 
> {code}
> Note that the first invocation of "zipinfo" reports that the central 
> directory is invalid because of extra bytes at the beginning of the zip file. 
> "zip --adjust-sfx" can be used to adjust the central directory to account for 
> the self-extracting preamble bytes. 
> It would be nice to be able to prepend this kind of self-extracting preamble 
> to a zip using Apache Commons Compress. 
> A method like the following on ZipArchiveOutputStream seems to provide this 
> behavior: 
> {code:java}
> public void writePreamble(byte[] preamble) throws IOException {
> if (entry != null) {
> throw new IllegalStateException("Preamble must be written before 
> creating an entry");
> }
> this.streamCompressor.writeCounted(preamble);
> }
> {code}
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Resolved] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread Alex Herbert (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Herbert resolved NUMBERS-149.
--
Resolution: Implemented

Added to master in commit

da53215163ac0e53f16e0d70756d00815602cb87

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Affects Versions: 1.0-beta1
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 5h 20m
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread Alex Herbert (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Herbert updated NUMBERS-149:
-
Affects Version/s: 1.0-beta1

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Affects Versions: 1.0-beta1
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 5h 20m
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475472=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475472
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 18:39
Start Date: 27/Aug/20 18:39
Worklog Time Spent: 10m 
  Work Description: asfgit merged pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475472)
Time Spent: 5h 20m  (was: 5h 10m)

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 5h 20m
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-numbers] asfgit merged pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


asfgit merged pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475412=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475412
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 16:55
Start Date: 27/Aug/20 16:55
Worklog Time Spent: 10m 
  Work Description: coveralls edited a comment on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-680561210


   
   [![Coverage 
Status](https://coveralls.io/builds/33050215/badge)](https://coveralls.io/builds/33050215)
   
   Coverage remained the same at 99.636% when pulling 
**da53215163ac0e53f16e0d70756d00815602cb87 on 
XenoAmess:port_commons_lang_Fraction** into 
**674f6da2d495031d3ea054dddad2818500b9728c on apache:master**.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475412)
Time Spent: 5h 10m  (was: 5h)

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 5h 10m
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-numbers] coveralls edited a comment on pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


coveralls edited a comment on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-680561210


   
   [![Coverage 
Status](https://coveralls.io/builds/33050215/badge)](https://coveralls.io/builds/33050215)
   
   Coverage remained the same at 99.636% when pulling 
**da53215163ac0e53f16e0d70756d00815602cb87 on 
XenoAmess:port_commons_lang_Fraction** into 
**674f6da2d495031d3ea054dddad2818500b9728c on apache:master**.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475389=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475389
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 16:12
Start Date: 27/Aug/20 16:12
Worklog Time Spent: 10m 
  Work Description: XenoAmess commented on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-682046367


   @aherbert 
   sqruashed.
   please wait for travis-ci & recheck.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475389)
Time Spent: 5h  (was: 4h 50m)

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 5h
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475388=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475388
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 16:11
Start Date: 27/Aug/20 16:11
Worklog Time Spent: 10m 
  Work Description: XenoAmess commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478535081



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1161 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+

[GitHub] [commons-numbers] XenoAmess commented on pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


XenoAmess commented on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-682046367


   @aherbert 
   sqruashed.
   please wait for travis-ci & recheck.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-numbers] XenoAmess commented on a change in pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


XenoAmess commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478535081



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1161 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+assertEquals(15, f.getDenominator());
+
+// improper
+f = Fraction.of(22, 7);
+assertEquals(22, f.getNumerator());
+assertEquals(7, f.getDenominator());
+
+// negatives
+f = Fraction.of(-6, 10);
+assertEquals(-3, f.getNumerator());
+assertEquals(5, f.getDenominator());
+
+f = Fraction.of(6, -10);
+assertEquals(3, f.getNumerator());
+

[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475381=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475381
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 16:03
Start Date: 27/Aug/20 16:03
Worklog Time Spent: 10m 
  Work Description: aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478530080



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1161 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+

[GitHub] [commons-numbers] aherbert commented on a change in pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478530080



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1161 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+assertEquals(15, f.getDenominator());
+
+// improper
+f = Fraction.of(22, 7);
+assertEquals(22, f.getNumerator());
+assertEquals(7, f.getDenominator());
+
+// negatives
+f = Fraction.of(-6, 10);
+assertEquals(-3, f.getNumerator());
+assertEquals(5, f.getDenominator());
+
+f = Fraction.of(6, -10);
+assertEquals(3, f.getNumerator());
+

[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475303=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475303
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 13:36
Start Date: 27/Aug/20 13:36
Worklog Time Spent: 10m 
  Work Description: coveralls edited a comment on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-680561210


   
   [![Coverage 
Status](https://coveralls.io/builds/33044971/badge)](https://coveralls.io/builds/33044971)
   
   Coverage increased (+0.002%) to 99.636% when pulling 
**80efbb7975580909da6e3977d110836db9c464aa on 
XenoAmess:port_commons_lang_Fraction** into 
**c3d173ae9cf25329644b87b8e064f56a2c31ea53 on apache:master**.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475303)
Time Spent: 4.5h  (was: 4h 20m)

> port tests in commons-lang for Fraction.
> 
>
> Key: NUMBERS-149
> URL: https://issues.apache.org/jira/browse/NUMBERS-149
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: fraction
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> https://github.com/apache/commons-numbers/pull/82



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-numbers] coveralls edited a comment on pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


coveralls edited a comment on pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#issuecomment-680561210


   
   [![Coverage 
Status](https://coveralls.io/builds/33044971/badge)](https://coveralls.io/builds/33044971)
   
   Coverage increased (+0.002%) to 99.636% when pulling 
**80efbb7975580909da6e3977d110836db9c464aa on 
XenoAmess:port_commons_lang_Fraction** into 
**c3d173ae9cf25329644b87b8e064f56a2c31ea53 on apache:master**.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-numbers] XenoAmess commented on a change in pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


XenoAmess commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478417612



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+assertEquals(15, f.getDenominator());
+
+// improper
+f = Fraction.of(22, 7);
+assertEquals(22, f.getNumerator());
+assertEquals(7, f.getDenominator());
+
+// negatives
+f = Fraction.of(-6, 10);
+assertEquals(-3, f.getNumerator());
+assertEquals(5, f.getDenominator());
+
+f = Fraction.of(6, -10);
+assertEquals(3, f.getNumerator());
+

[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475300=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475300
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 13:26
Start Date: 27/Aug/20 13:26
Worklog Time Spent: 10m 
  Work Description: XenoAmess commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478417612



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+

[jira] [Updated] (NET-687) FTP data connection error Unsupported or unrecognized SSL message. Probably another ssl_reuse session error ?

2020-08-27 Thread Mikael (Jira)


 [ 
https://issues.apache.org/jira/browse/NET-687?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mikael updated NET-687:
---
Description: 
After adding the self signed polynesie.cer certificate to JVM security 
(_jdk-x.x.x/lib/security_) :
{code:java}
keytool.exe -import -storepass "changeit" -keystore "./cacerts" -alias 
polynesie.cer -file ./polynesie.cer -noprompt{code}
 polynesie.cer obtained by copying certificate part from this command line 
result :

{code:java}
openssl s_client -connect ftp0.gov.pf:21 -starttls ftp{code}
Trying to retrieve a file with ftpes :
{code:java}
java -cp commons-net-examples-3.5.jar;commons-net-3.5.jar 
examples/ftp/FTPClientExample -A -p TLS,false -e -b ftp0.gov.pf 
DataVRS/fiche_Station_VRS_VAI1.pdf fiche_Station_VRS_VAI1.pdf{code}
Produce this exception :
{code:java}
javax.net.ssl.SSLException: Unsupported or unrecognized SSL message
at 
java.base/sun.security.ssl.SSLSocketInputRecord.handleUnknownRecord(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketInputRecord.decode(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at 
java.base/sun.security.ssl.SSLSocketImpl.performInitialHandshake(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at 
org.apache.commons.net.ftp.FTPSClient._openDataConnection_(FTPSClient.java:642)
at 
org.apache.commons.net.ftp.FTPClient._retrieveFile(FTPClient.java:1907)
at 
org.apache.commons.net.ftp.FTPClient.retrieveFile(FTPClient.java:1893)
at testFTP2.FTPClientExample.main(FTPClientExample.java:513)
{code}
It is probably the same error of ssl_reuse session as NET-408.

Same try with ftp4j library reports this error :
{code:java}
code=522, message= SSL connection failed; session reuse required: see 
require_ssl_reuse option in vsftpd.conf man page
{code}
 

  was:
After adding the self signed polynesie.cer certificate to JVM security 
(_jdk-x.x.x/lib/security_) :
{code:java}
keytool.exe -import -storepass "changeit" -keystore "./cacerts" -alias 
polynesie.cer -file ./polynesie.cer -noprompt{code}
 polynesie.cer obtained by copying certificate part from this command line 
result :

{{}}
{code:java}
openssl s_client -connect ftp0.gov.pf:21 -starttls ftp{code}
Trying to retrieve a file with ftpes :
{code:java}
java -cp commons-net-examples-3.5.jar;commons-net-3.5.jar 
examples/ftp/FTPClientExample -A -p TLS,false -e -b ftp0.gov.pf 
DataVRS/fiche_Station_VRS_VAI1.pdf fiche_Station_VRS_VAI1.pdf{code}
Produce this exception :
{code:java}
javax.net.ssl.SSLException: Unsupported or unrecognized SSL message
at 
java.base/sun.security.ssl.SSLSocketInputRecord.handleUnknownRecord(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketInputRecord.decode(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at 
java.base/sun.security.ssl.SSLSocketImpl.performInitialHandshake(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at 
org.apache.commons.net.ftp.FTPSClient._openDataConnection_(FTPSClient.java:642)
at 
org.apache.commons.net.ftp.FTPClient._retrieveFile(FTPClient.java:1907)
at 
org.apache.commons.net.ftp.FTPClient.retrieveFile(FTPClient.java:1893)
at testFTP2.FTPClientExample.main(FTPClientExample.java:513)
{code}
It is probably the same error of ssl_reuse session as NET-408.

Same try with ftp4j library reports this error :
{code:java}
code=522, message= SSL connection failed; session reuse required: see 
require_ssl_reuse option in vsftpd.conf man page
{code}
 


> FTP data connection error Unsupported or unrecognized SSL message. Probably 
> another ssl_reuse session error ?
> -
>
> Key: NET-687
> URL: https://issues.apache.org/jira/browse/NET-687
> Project: Commons Net
>  Issue Type: Bug
>  Components: FTP
>Affects Versions: 3.7
> Environment: Tested with JDK 8, 11, 13, 14
>Reporter: Mikael
>Priority: Major
>
> After adding the self signed polynesie.cer certificate to JVM security 
> (_jdk-x.x.x/lib/security_) :
> {code:java}
> keytool.exe -import -storepass "changeit" -keystore "./cacerts" -alias 
> polynesie.cer -file ./polynesie.cer -noprompt{code}
>  polynesie.cer obtained by copying certificate part from this command line 
> result :
> {code:java}
> openssl s_client 

[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475278=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475278
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 12:34
Start Date: 27/Aug/20 12:34
Worklog Time Spent: 10m 
  Work Description: aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478379873



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+

[GitHub] [commons-numbers] aherbert commented on a change in pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478379873



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+assertEquals(15, f.getDenominator());
+
+// improper
+f = Fraction.of(22, 7);
+assertEquals(22, f.getNumerator());
+assertEquals(7, f.getDenominator());
+
+// negatives
+f = Fraction.of(-6, 10);
+assertEquals(-3, f.getNumerator());
+assertEquals(5, f.getDenominator());
+
+f = Fraction.of(6, -10);
+assertEquals(3, f.getNumerator());
+

[jira] [Work logged] (NUMBERS-149) port tests in commons-lang for Fraction.

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-149?focusedWorklogId=475272=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475272
 ]

ASF GitHub Bot logged work on NUMBERS-149:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 12:21
Start Date: 27/Aug/20 12:21
Worklog Time Spent: 10m 
  Work Description: aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478376559



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+

[GitHub] [commons-numbers] aherbert commented on a change in pull request #82: [NUMBERS-149] port tests in commons-lang for Fraction.

2020-08-27 Thread GitBox


aherbert commented on a change in pull request #82:
URL: https://github.com/apache/commons-numbers/pull/82#discussion_r478376559



##
File path: 
commons-numbers-fraction/src/test/java/org/apache/commons/numbers/fraction/CommonsLangPortedFractionTest.java
##
@@ -0,0 +1,1148 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.numbers.fraction;
+
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Test cases for the {@link Fraction} class.
+ *
+ * This class is ported from commons lang to demonstrate interoperability of
+ * the Fraction class in numbers.
+ */
+public class CommonsLangPortedFractionTest {
+
+private static final int SKIP = 500;  //53
+
+
//--
+@Test
+public void testConstants() {
+assertEquals(0, Fraction.ZERO.getNumerator());
+assertEquals(1, Fraction.ZERO.getDenominator());
+
+assertEquals(1, Fraction.ONE.getNumerator());
+assertEquals(1, Fraction.ONE.getDenominator());
+
+/*
+ *  All these constants need not be supported.
+ *  Users can create whatever constants they require.
+ *  The special constants ZERO and ONE are for the additive and 
multiplicative identity.
+ *
+ *  assertEquals(1, Fraction.ONE_HALF.getNumerator());
+ *  assertEquals(2, Fraction.ONE_HALF.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_THIRD.getNumerator());
+ *  assertEquals(3, Fraction.ONE_THIRD.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_THIRDS.getNumerator());
+ *  assertEquals(3, Fraction.TWO_THIRDS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_QUARTER.getNumerator());
+ *  assertEquals(4, Fraction.ONE_QUARTER.getDenominator());
+ *
+ *  assertEquals(1, Fraction.TWO_QUARTERS.getNumerator());
+ *  assertEquals(2, Fraction.TWO_QUARTERS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_QUARTERS.getNumerator());
+ *  assertEquals(4, Fraction.THREE_QUARTERS.getDenominator());
+ *
+ *  assertEquals(1, Fraction.ONE_FIFTH.getNumerator());
+ *  assertEquals(5, Fraction.ONE_FIFTH.getDenominator());
+ *
+ *  assertEquals(2, Fraction.TWO_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.TWO_FIFTHS.getDenominator());
+ *
+ *  assertEquals(3, Fraction.THREE_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.THREE_FIFTHS.getDenominator());
+ *
+ *  assertEquals(4, Fraction.FOUR_FIFTHS.getNumerator());
+ *  assertEquals(5, Fraction.FOUR_FIFTHS.getDenominator());
+ */
+}
+
+@Test
+public void testFactory_int_int() {
+Fraction f = null;
+
+// zero
+f = Fraction.of(0, 1);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(0, 2);
+assertEquals(0, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+// normal
+f = Fraction.of(1, 1);
+assertEquals(1, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(2, 1);
+assertEquals(2, f.getNumerator());
+assertEquals(1, f.getDenominator());
+
+f = Fraction.of(23, 345);
+assertEquals(1, f.getNumerator());
+assertEquals(15, f.getDenominator());
+
+// improper
+f = Fraction.of(22, 7);
+assertEquals(22, f.getNumerator());
+assertEquals(7, f.getDenominator());
+
+// negatives
+f = Fraction.of(-6, 10);
+assertEquals(-3, f.getNumerator());
+assertEquals(5, f.getDenominator());
+
+f = Fraction.of(6, -10);
+assertEquals(3, f.getNumerator());
+

[jira] [Work logged] (NUMBERS-151) Performance refinement for ArithmeticUtils.pow

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-151?focusedWorklogId=475269=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475269
 ]

ASF GitHub Bot logged work on NUMBERS-151:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 12:12
Start Date: 27/Aug/20 12:12
Worklog Time Spent: 10m 
  Work Description: asfgit merged pull request #84:
URL: https://github.com/apache/commons-numbers/pull/84


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475269)
Time Spent: 2.5h  (was: 2h 20m)

> Performance refinement for ArithmeticUtils.pow
> --
>
> Key: NUMBERS-151
> URL: https://issues.apache.org/jira/browse/NUMBERS-151
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: core
>Affects Versions: 1.0-beta1
>Reporter: Jin Xu
>Priority: Minor
>  Time Spent: 2.5h
>  Remaining Estimate: 0h
>
> Hi.
> right now ArithmeticUtils.pow(0,0) = 1, but not throw an Exception, which I 
> think not quite right...
> Is it a bug?
> Anyway, we found a way to faster the calculate, so at least we can get an 
> performance refine pr.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Resolved] (NUMBERS-151) Performance refinement for ArithmeticUtils.pow

2020-08-27 Thread Alex Herbert (Jira)


 [ 
https://issues.apache.org/jira/browse/NUMBERS-151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Herbert resolved NUMBERS-151.
--
Fix Version/s: 1.0
   Resolution: Fixed

Added in commit 42ebc644ff46e0cef75a59512390c9b78fa16a04

> Performance refinement for ArithmeticUtils.pow
> --
>
> Key: NUMBERS-151
> URL: https://issues.apache.org/jira/browse/NUMBERS-151
> Project: Commons Numbers
>  Issue Type: Improvement
>  Components: core
>Affects Versions: 1.0-beta1
>Reporter: Jin Xu
>Priority: Minor
> Fix For: 1.0
>
>  Time Spent: 2.5h
>  Remaining Estimate: 0h
>
> Hi.
> right now ArithmeticUtils.pow(0,0) = 1, but not throw an Exception, which I 
> think not quite right...
> Is it a bug?
> Anyway, we found a way to faster the calculate, so at least we can get an 
> performance refine pr.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-numbers] asfgit merged pull request #84: [NUMBERS-151] performance refine for ArithmeticUtils.pow

2020-08-27 Thread GitBox


asfgit merged pull request #84:
URL: https://github.com/apache/commons-numbers/pull/84


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Created] (NET-687) FTP data connection error Unsupported or unrecognized SSL message. Probably another ssl_reuse session error ?

2020-08-27 Thread Mikael (Jira)
Mikael created NET-687:
--

 Summary: FTP data connection error Unsupported or unrecognized SSL 
message. Probably another ssl_reuse session error ?
 Key: NET-687
 URL: https://issues.apache.org/jira/browse/NET-687
 Project: Commons Net
  Issue Type: Bug
  Components: FTP
Affects Versions: 3.7
 Environment: Tested with JDK 8, 11, 13, 14
Reporter: Mikael


After adding the self signed polynesie.cer certificate to JVM security 
(_jdk-x.x.x/lib/security_) :
{code:java}
keytool.exe -import -storepass "changeit" -keystore "./cacerts" -alias 
polynesie.cer -file ./polynesie.cer -noprompt{code}
 polynesie.cer obtained by copying certificate part from this command line 
result :

{{}}
{code:java}
openssl s_client -connect ftp0.gov.pf:21 -starttls ftp{code}
Trying to retrieve a file with ftpes :
{code:java}
java -cp commons-net-examples-3.5.jar;commons-net-3.5.jar 
examples/ftp/FTPClientExample -A -p TLS,false -e -b ftp0.gov.pf 
DataVRS/fiche_Station_VRS_VAI1.pdf fiche_Station_VRS_VAI1.pdf{code}
Produce this exception :
{code:java}
javax.net.ssl.SSLException: Unsupported or unrecognized SSL message
at 
java.base/sun.security.ssl.SSLSocketInputRecord.handleUnknownRecord(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketInputRecord.decode(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.readRecord(Unknown Source)
at 
java.base/sun.security.ssl.SSLSocketImpl.performInitialHandshake(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown 
Source)
at 
org.apache.commons.net.ftp.FTPSClient._openDataConnection_(FTPSClient.java:642)
at 
org.apache.commons.net.ftp.FTPClient._retrieveFile(FTPClient.java:1907)
at 
org.apache.commons.net.ftp.FTPClient.retrieveFile(FTPClient.java:1893)
at testFTP2.FTPClientExample.main(FTPClientExample.java:513)
{code}
It is probably the same error of ssl_reuse session as NET-408.

Same try with ftp4j library reports this error :
{code:java}
code=522, message= SSL connection failed; session reuse required: see 
require_ssl_reuse option in vsftpd.conf man page
{code}
 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-766?focusedWorklogId=475150=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475150
 ]

ASF GitHub Bot logged work on COLLECTIONS-766:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 06:11
Start Date: 27/Aug/20 06:11
Worklog Time Spent: 10m 
  Work Description: XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-680709554


   the third and later commit solved a problem that `--no-transfer-progress` is 
not usable on openjdk8 at ppc64le .
   so I added a condition select in travis.yml.
   will see if it can work correctly.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475150)
Time Spent: 20m  (was: 10m)

> fix ci build (or to say, fix everything that wrongly blocked the ci build.)
> ---
>
> Key: COLLECTIONS-766
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
> Project: Commons Collections
>  Issue Type: Bug
>Reporter: Jin Xu
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Hi.
> The current ci build fails for 3 reasons.
> I fixed them all in https://github.com/apache/commons-collections/pull/178
> please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-766?focusedWorklogId=475151=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475151
 ]

ASF GitHub Bot logged work on COLLECTIONS-766:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 06:11
Start Date: 27/Aug/20 06:11
Worklog Time Spent: 10m 
  Work Description: XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-681398369


   @kinow 
   Hi.
   After search in docs / try by self, I can find NO way to get ARCH variable 
from travis-ci.
   So we MIGHT either use if condition to JDK_NAME (see c302eea above),
   or delete `--no-transfer-progress`.  (see 379e4b5 above).
   Both can pass ci, and acceptable.
   
   Personally I slightly prefer deleting `--no-transfer-progress`, because 
seems it is added not to solve an build error, but to slightly reduce 
travis-ci's pressure.
   
   Aynway, please choose one for the final choice, and then I will rebase / 
sqruash.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475151)
Time Spent: 0.5h  (was: 20m)

> fix ci build (or to say, fix everything that wrongly blocked the ci build.)
> ---
>
> Key: COLLECTIONS-766
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
> Project: Commons Collections
>  Issue Type: Bug
>Reporter: Jin Xu
>Priority: Major
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Hi.
> The current ci build fails for 3 reasons.
> I fixed them all in https://github.com/apache/commons-collections/pull/178
> please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-collections] XenoAmess edited a comment on pull request #178: [COLLECTIONS-766] fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread GitBox


XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-681398369


   @kinow 
   Hi.
   After search in docs / try by self, I can find NO way to get ARCH variable 
from travis-ci.
   So we MIGHT either use if condition to JDK_NAME (see c302eea above),
   or delete `--no-transfer-progress`.  (see 379e4b5 above).
   Both can pass ci, and acceptable.
   
   Personally I slightly prefer deleting `--no-transfer-progress`, because 
seems it is added not to solve an build error, but to slightly reduce 
travis-ci's pressure.
   
   Aynway, please choose one for the final choice, and then I will rebase / 
sqruash.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-collections] XenoAmess edited a comment on pull request #178: [COLLECTIONS-766] fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread GitBox


XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-680709554


   the third and later commit solved a problem that `--no-transfer-progress` is 
not usable on openjdk8.
   so I added a condition select in travis.yml.
   will see if it can work correctly.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-collections] XenoAmess edited a comment on pull request #178: [COLLECTIONS-766] fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread GitBox


XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-680709554


   the third and later commit solved a problem that `--no-transfer-progress` is 
not usable on openjdk8 at ppc64le .
   so I added a condition select in travis.yml.
   will see if it can work correctly.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[jira] [Work logged] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread ASF GitHub Bot (Jira)


 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-766?focusedWorklogId=475149=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-475149
 ]

ASF GitHub Bot logged work on COLLECTIONS-766:
--

Author: ASF GitHub Bot
Created on: 27/Aug/20 06:10
Start Date: 27/Aug/20 06:10
Worklog Time Spent: 10m 
  Work Description: XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-680709554


   the third and later commit solved a problem that `--no-transfer-progress` is 
not usable on openjdk8.
   so I added a condition select in travis.yml.
   will see if it can work correctly.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 475149)
Remaining Estimate: 0h
Time Spent: 10m

> fix ci build (or to say, fix everything that wrongly blocked the ci build.)
> ---
>
> Key: COLLECTIONS-766
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
> Project: Commons Collections
>  Issue Type: Bug
>Reporter: Jin Xu
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi.
> The current ci build fails for 3 reasons.
> I fixed them all in https://github.com/apache/commons-collections/pull/178
> please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread Jin Xu (Jira)


 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jin Xu updated COLLECTIONS-766:
---
Priority: Major  (was: Critical)

> fix ci build (or to say, fix everything that wrongly blocked the ci build.)
> ---
>
> Key: COLLECTIONS-766
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
> Project: Commons Collections
>  Issue Type: Bug
>Reporter: Jin Xu
>Priority: Major
>
> Hi.
> The current ci build fails for 3 reasons.
> I fixed them all in https://github.com/apache/commons-collections/pull/178
> please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Updated] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread Jin Xu (Jira)


 [ 
https://issues.apache.org/jira/browse/COLLECTIONS-766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jin Xu updated COLLECTIONS-766:
---
Priority: Critical  (was: Major)

> fix ci build (or to say, fix everything that wrongly blocked the ci build.)
> ---
>
> Key: COLLECTIONS-766
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
> Project: Commons Collections
>  Issue Type: Bug
>Reporter: Jin Xu
>Priority: Critical
>
> Hi.
> The current ci build fails for 3 reasons.
> I fixed them all in https://github.com/apache/commons-collections/pull/178
> please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Created] (COLLECTIONS-766) fix ci build (or to say, fix everything that wrongly blocked the ci build.)

2020-08-27 Thread Jin Xu (Jira)
Jin Xu created COLLECTIONS-766:
--

 Summary: fix ci build (or to say, fix everything that wrongly 
blocked the ci build.)
 Key: COLLECTIONS-766
 URL: https://issues.apache.org/jira/browse/COLLECTIONS-766
 Project: Commons Collections
  Issue Type: Bug
Reporter: Jin Xu


Hi.
The current ci build fails for 3 reasons.
I fixed them all in https://github.com/apache/commons-collections/pull/178
please have a look.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [commons-collections] XenoAmess edited a comment on pull request #178: fix ci builds

2020-08-27 Thread GitBox


XenoAmess edited a comment on pull request #178:
URL: 
https://github.com/apache/commons-collections/pull/178#issuecomment-681398369


   @kinow 
   Hi.
   After search in docs / try by self, I can find NO way to get ARCH variable 
from travis-ci.
   So we MIGHT either use if condition to JDK_NAME (see c302eea above),
   or delete `--no-transfer-progress`.  (see 379e4b5 above).
   Both can pass ci, and acceptable.
   
   Personally I slightly prefer deleting `--no-transfer-progress`, because 
seems it is added not to solve an build error, but to slightly reduce 
travis-ci's pressure.
   
   So please choose one for the final choice, and then I will rebase / sqruash.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-lang] coveralls commented on pull request #613: Bump jmh.version from 1.21 to 1.25.1

2020-08-27 Thread GitBox


coveralls commented on pull request #613:
URL: https://github.com/apache/commons-lang/pull/613#issuecomment-681587755


   
   [![Coverage 
Status](https://coveralls.io/builds/33035255/badge)](https://coveralls.io/builds/33035255)
   
   Coverage remained the same at 94.704% when pulling 
**fef0707a5335a5b739d89fa3a74c3edd5ce598fc on 
dependabot/maven/jmh.version-1.25.1** into 
**31dc5c700e1a08f4af33d94301ff06790533099b on master**.
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org