This is an automated email from the ASF dual-hosted git repository.

aw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yetus.git


The following commit(s) were added to refs/heads/master by this push:
     new bc5961a  YETUS-760. Update yetus-maven-plugin's releasedocmaker options
bc5961a is described below

commit bc5961af276bbdcbf4d5775dba3f4938d7e0c9d9
Author: Allen Wittenauer <a...@apache.org>
AuthorDate: Tue Apr 9 13:40:53 2019 -0700

    YETUS-760. Update yetus-maven-plugin's releasedocmaker options
    
    Signed-off-by: Allen Wittenauer <a...@apache.org>
---
 .../documentation/in-progress/releasedocmaker.md   |  4 ++
 .../in-progress/yetus-maven-plugin.md              |  9 ++---
 .../src/main/python/releasedocmaker/__init__.py    |  3 +-
 .../maven/plugin/rdm/ReleaseDocMakerMojo.java      | 44 +++++++++++++++++-----
 4 files changed, 44 insertions(+), 16 deletions(-)

diff --git a/asf-site-src/source/documentation/in-progress/releasedocmaker.md 
b/asf-site-src/source/documentation/in-progress/releasedocmaker.md
index 0a26337..13b3eb0 100644
--- a/asf-site-src/source/documentation/in-progress/releasedocmaker.md
+++ b/asf-site-src/source/documentation/in-progress/releasedocmaker.md
@@ -20,6 +20,8 @@
 releasedocmaker
 ===============
 
+<!-- MarkdownTOC levels="1,2" autolink="true" -->
+
 * [Purpose](#purpose)
 * [Requirements](#requirements)
 * [Basic Usage](#basic-usage)
@@ -32,6 +34,8 @@ releasedocmaker
 * [Index Mode](#index-mode)
 * [Release Version](#release-version)
 
+<!-- /MarkdownTOC -->
+
 # Purpose
 
 Building changelog information in a form that is human digestible but still 
containing as much useful information is difficult.  Many attempts over the 
years have resulted in a variety of methods that projects use to solve this 
problem:
diff --git 
a/asf-site-src/source/documentation/in-progress/yetus-maven-plugin.md 
b/asf-site-src/source/documentation/in-progress/yetus-maven-plugin.md
index 58b6920..8cfa6d9 100644
--- a/asf-site-src/source/documentation/in-progress/yetus-maven-plugin.md
+++ b/asf-site-src/source/documentation/in-progress/yetus-maven-plugin.md
@@ -178,8 +178,6 @@ This goal runs releasedocmaker without the need to download 
or build an Apache Y
           </execution>
         </plugin>
 
-
-
 The configuration options generally map 1:1 to the `releasedocmaker` 
executable's options.  Unless otherwise specified, defaults are set by the 
actual executable.
 
 | Option | Description | Default |
@@ -190,14 +188,15 @@ The configuration options generally map 1:1 to the 
`releasedocmaker` executable'
 | `incompatibleLabel` | --incompatiblelabel | |
 | `index` | boolean; --index | false  |
 | `license` | boolean; --license | false |
-| `lint` | boolean; --lint | false |
+| `lintFilters` | ArrayList; --lint | |
 | `outputDir` | --outputdir | 
`${project.build.directory}/generated-site/markdown` |
 | `projects` | ArrayList; --projects | `${project.name}` |
 | `projectTitle` | --projecttitle | |
 | `range` | boolean; --range | false |
+| `retries` | integer; --retries | |
 | `skipcredits` | boolean; --skipcredits | false |
-| `sortorder` | --sortorder | older |
-| `sorttype` | --sorttype | resolutiondate |
+| `sortorder` | --sortorder | |
+| `sorttype` | --sorttype | |
 | `useToday` | --usetoday | false |
 | `versions` | ArrayList; --versions | `${project.version}` |
 
diff --git a/releasedocmaker/src/main/python/releasedocmaker/__init__.py 
b/releasedocmaker/src/main/python/releasedocmaker/__init__.py
index 47f104c..c536b24 100755
--- a/releasedocmaker/src/main/python/releasedocmaker/__init__.py
+++ b/releasedocmaker/src/main/python/releasedocmaker/__init__.py
@@ -892,7 +892,8 @@ def main(): # pylint: disable=too-many-statements, 
too-many-branches, too-many-l
             print(linter.message())
             if linter.had_errors():
                 haderrors = True
-                shutil.rmtree(vstr)
+                if os.path.exists(vstr):
+                    shutil.rmtree(vstr)
                 continue
 
         reloutputs.write_all("\n\n")
diff --git 
a/yetus-maven-plugin/src/main/java/org/apache/yetus/maven/plugin/rdm/ReleaseDocMakerMojo.java
 
b/yetus-maven-plugin/src/main/java/org/apache/yetus/maven/plugin/rdm/ReleaseDocMakerMojo.java
index fde7a49..c1c9810 100644
--- 
a/yetus-maven-plugin/src/main/java/org/apache/yetus/maven/plugin/rdm/ReleaseDocMakerMojo.java
+++ 
b/yetus-maven-plugin/src/main/java/org/apache/yetus/maven/plugin/rdm/ReleaseDocMakerMojo.java
@@ -15,7 +15,6 @@ package org.apache.yetus.maven.plugin.rdm;
 
 import java.io.File;
 import java.util.ArrayList;
-import org.apache.yetus.maven.plugin.utils.Utils;
 
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -68,7 +67,7 @@ public final class ReleaseDocMakerMojo extends AbstractMojo {
   private Boolean fileversions;
 
   /**
-   * label for incompatible issues
+   * label for incompatible issues.
    */
   @Parameter
   private String incompatibleLabel;
@@ -88,10 +87,17 @@ public final class ReleaseDocMakerMojo extends AbstractMojo 
{
   /**
    * Run in --lint mode.
    */
+  @Deprecated
   @Parameter(defaultValue = "false")
   private Boolean lint;
 
   /**
+   * lint filters.
+   */
+  @Parameter
+  private String[] lintFilters;
+
+  /**
    * Location of output.
    */
   @Parameter(defaultValue = 
"${project.build.directory}/generated-site/markdown")
@@ -116,23 +122,30 @@ public final class ReleaseDocMakerMojo extends 
AbstractMojo {
   private Boolean range;
 
   /**
-   * Drop reporter/assignee
+   * Number of times to retry the HTTPS connection.
+   */
+  @Parameter
+  private Integer retries;
+
+  /**
+   * Drop reporter/assignee.
    */
   @Parameter(defaultValue = "false")
   private Boolean skipcredits;
 
   /**
-   * Set the sort order
+   * Set the sort order.
    */
-  @Parameter(defaultValue = "older")
+  @Parameter
   private String sortorder;
 
   /**
-   * Set the type order
+   * Set the type order.
    */
-  @Parameter(defaultValue = "resolutiondate")
+  @Parameter
   private String sorttype;
 
+
   /**
    * Use today.
    */
@@ -159,9 +172,9 @@ public final class ReleaseDocMakerMojo extends AbstractMojo 
{
   public void execute() throws MojoExecutionException {
 
     buildArgs();
-    String [] args = argList.toArray(new String[0]);
+    String[] args = argList.toArray(new String[0]);
 
-    ReleaseDocMaker rdm=new ReleaseDocMaker();
+    ReleaseDocMaker rdm = new ReleaseDocMaker();
     rdm.main(args);
   }
 
@@ -171,7 +184,7 @@ public final class ReleaseDocMakerMojo extends AbstractMojo 
{
    */
   private void buildArgs() {
 
-    if (baseUrl != null ) {
+    if (baseUrl != null) {
       argList.add("--baseurl");
       argList.add(baseUrl);
     }
@@ -203,6 +216,12 @@ public final class ReleaseDocMakerMojo extends 
AbstractMojo {
 
     if (lint) {
       argList.add("--lint");
+      argList.add("all");
+    }
+
+    for (String f: lintFilters) {
+      argList.add("--lint");
+      argList.add(f);
     }
 
     argList.add("--outputdir");
@@ -222,6 +241,11 @@ public final class ReleaseDocMakerMojo extends 
AbstractMojo {
       argList.add("--range");
     }
 
+    if (retries != null) {
+      argList.add("--retries");
+      argList.add(retries.toString());
+    }
+
     if (skipcredits) {
       argList.add("--skipcredits");
     }

Reply via email to