You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Michael Osipov (Jira)" <ji...@apache.org> on 2022/04/22 12:53:00 UTC

[jira] [Commented] (MSHARED-1067) Improve the Reproducible Builds methods in Maven-Archiver

    [ https://issues.apache.org/jira/browse/MSHARED-1067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17526403#comment-17526403 ] 

Michael Osipov commented on MSHARED-1067:
-----------------------------------------

What is the benefit here of {{Optional}} compared to {{null}}?

> Improve the Reproducible Builds methods in Maven-Archiver
> ---------------------------------------------------------
>
>                 Key: MSHARED-1067
>                 URL: https://issues.apache.org/jira/browse/MSHARED-1067
>             Project: Maven Shared Components
>          Issue Type: Dependency upgrade
>          Components: maven-archiver
>    Affects Versions: maven-archiver-3.5.2
>            Reporter: Jorge Solórzano
>            Priority: Major
>
> Maven Archiver 3.5.0 introduces some methods to parse the timestamp and configure the reproducible builds in plexus-archiver.
> The proposal for improvement is to leverage Java 8 APIs and deprecate older methods.
>  # Deprecate the method
> {code:java}
> public Date parseOutputTimestamp( String outputTimestamp )
> {code}
> and introduce a new static method:
> {code:java}
> public static Optional<Instant> parseBuildOutputTimestamp( String outputTimestamp )
> {code}
> #  Deprecate the method
> {code:java}
> public Date configureReproducible( String outputTimestamp )
> {code}
> and introduce the method
> {code:java}
> public void configureReproducibleBuild( String outputTimestamp )
> {code}
> The new API return an Optional<Instant> parsing the timestamp using java.time.format.DateTimeFormatter#ISO_INSTANT.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)