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

          Activity

          Hide
          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
          michael-o Michael Osipov added a comment -
          Show
          michael-o Michael Osipov added a comment - Fixed with 250d2545ea35cdb81883d6e9eb5519c3bbd8abcd .
          Hide
          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
          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
          michael-o Michael Osipov added a comment - - edited
          Show
          michael-o Michael Osipov added a comment - - edited Fixed with 355f4dff03cea8cab0d7bdcb00fffa4e43b2c10b .
          Hide
          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