RFR: 327: git webrev: Print the location of the generated webrev upon successful completion

Erik Joelsson erikj at openjdk.org
Fri Sep 23 14:38:27 UTC 2022


On Thu, 22 Sep 2022 16:04:44 GMT, Zhao Song <duke at openjdk.org> wrote:

> Originally, git webrev command would return nothing when executed successfully. It is not a good experience for user. 
> 
> Right now, when executed successfully, the location of index.html will be printed.
> 
> Also, added a --quiet flag if the current silent behavior is required.

cli/src/main/java/org/openjdk/skara/cli/GitWebrev.java line 463:

> 461:         if (!quiet) {
> 462:             System.out.println("Webrev executed successfully, details are in the link below:");
> 463:             System.out.println("file://" + new File(output.resolve("index.html").toUri()).getAbsoluteFile());

That looks like a weird way to generate a URL. Why not just:
Suggestion:

            System.out.println(output.resolve("index.html").toUri());

-------------

PR: https://git.openjdk.org/skara/pull/1379


More information about the skara-dev mailing list