Uploaded image for project: 'Maven'
  1. Maven
  2. MNG-5975

Use Java 7's SimpleDateFormat in CLIReportingUtils#formatTimestamp

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 3.3.9
        • Fix Version/s: 3.5.0-alpha-1, 3.5.0
        • Component/s: Embedding
        • Labels:
          None

          Description

          The current code manually formats the timestamp because Commons Lang 3 was not in the dependencies list. Now there is, drop custom code and use builtin features.

            Issue Links

            mentioned in

            Page Loading...

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              michael-o Michael Osipov added a comment -

              Commons Lang 3.4 cannot be used because it contains a bug. it does not properly detect UTC timezone. Using Java 7 builtin.

              Show
              michael-o Michael Osipov added a comment - Commons Lang 3.4 cannot be used because it contains a bug. it does not properly detect UTC timezone. Using Java 7 builtin.
              Hide
              Permalink
              michael-o Michael Osipov added a comment -

              Fixed with 250d2545ea35cdb81883d6e9eb5519c3bbd8abcd.

              Show
              michael-o Michael Osipov added a comment - Fixed with 250d2545ea35cdb81883d6e9eb5519c3bbd8abcd .
              Hide
              Permalink
              hudson Hudson added a comment -

              SUCCESS: Integrated in maven-3.x #1217 (See https://builds.apache.org/job/maven-3.x/1217/)
              MNG-5975 Use Java 7's SimpleDateFormat in (michaelo: rev 250d2545ea35cdb81883d6e9eb5519c3bbd8abcd)

              • maven-embedder/src/main/java/org/apache/maven/cli/CLIReportingUtils.java
              Show
              hudson Hudson added a comment - SUCCESS: Integrated in maven-3.x #1217 (See https://builds.apache.org/job/maven-3.x/1217/ ) MNG-5975 Use Java 7's SimpleDateFormat in (michaelo: rev 250d2545ea35cdb81883d6e9eb5519c3bbd8abcd) maven-embedder/src/main/java/org/apache/maven/cli/CLIReportingUtils.java
              Hide
              Permalink
              stephenc Stephen Connolly added a comment -

              Maven 3.4.0 has been dropped. See this thread for more details.

              This issue will need to be re-scheduled for a Maven release in the (hopefully near) future.

              Show
              stephenc Stephen Connolly added a comment - Maven 3.4.0 has been dropped. See this thread for more details. This issue will need to be re-scheduled for a Maven release in the (hopefully near) future.
              Hide
              Permalink
              michael-o Michael Osipov added a comment - - edited

              Fixed with 355f4dff03cea8cab0d7bdcb00fffa4e43b2c10b.

              Show
              michael-o Michael Osipov added a comment - - edited Fixed with 355f4dff03cea8cab0d7bdcb00fffa4e43b2c10b .
              Hide
              Permalink
              hudson Hudson added a comment -

              SUCCESS: Integrated in Jenkins build maven-3.x #1500 (See https://builds.apache.org/job/maven-3.x/1500/)
              MNG-5975 Use Java 7's SimpleDateFormat in (michaelo: http://git-wip-us.apache.org/repos/asf/?p=maven.git&a=commit&h=355f4dff03cea8cab0d7bdcb00fffa4e43b2c10b)

              • (edit) maven-embedder/src/main/java/org/apache/maven/cli/CLIReportingUtils.java
              Show
              hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build maven-3.x #1500 (See https://builds.apache.org/job/maven-3.x/1500/ ) MNG-5975 Use Java 7's SimpleDateFormat in (michaelo: http://git-wip-us.apache.org/repos/asf/?p=maven.git&a=commit&h=355f4dff03cea8cab0d7bdcb00fffa4e43b2c10b ) (edit) maven-embedder/src/main/java/org/apache/maven/cli/CLIReportingUtils.java

                People

                • Assignee:
                  michael-o Michael Osipov
                  Reporter:
                  michael-o Michael Osipov
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development