Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-9856

Add rat exclude files to excludes those files that does not need license header

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      As recently we added Apache Rat in Buildbot, Rat audits software distributions, with a special interest in headers.
      Here is the Rat report for ofbiz
      https://ci.apache.org/projects/ofbiz/rat-output.html

      In this reports there are several files that does not need license header like *.svg etc.
      Rat provide a way to exclude files while auditing
      http://creadur.apache.org/rat/apache-rat/

      It is common to use the Rat with the Maven or Ant plugins and specify a series of files to exclude (such as a README or version control files). If you are using the Rat application instead of a plugin you can specify a series of regex excludes in a file and specify that with the -E option.

      We can add .rat-excludes.txt files at ofbiz root for this purpose.

        Activity

        Hide
        jacques.le.roux Jacques Le Roux added a comment - - edited

        I have added a WIP rat-excludes.txt files in OFBiz root dir but it does not work yet.

        I have asked infra about it https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-15395

        Show
        jacques.le.roux Jacques Le Roux added a comment - - edited I have added a WIP rat-excludes.txt files in OFBiz root dir but it does not work yet. I have asked infra about it https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-15395
        Hide
        deepak.dixit Deepak Dixit added a comment -

        Hi Jacques Le Roux

        I think we need to add rat-exclude.txt path in buildboat.

        http://creadur.apache.org/rat/apache-rat/

        {info}

        Using multiple excludes from a file

        It is common to use the Rat with the Maven or Ant plugins and specify a series of files to exclude (such as a README or version control files). If you are using the Rat application instead of a plugin you can specify a series of regex excludes in a file and specify that with the -E option.

        java -jar apache-rat-0.13-SNAPSHOT.jar
        -E /path/to/project/.rat-excludes
        -d /path/to/project{info}
        Show
        deepak.dixit Deepak Dixit added a comment - Hi Jacques Le Roux I think we need to add rat-exclude.txt path in buildboat. http://creadur.apache.org/rat/apache-rat/ {info} Using multiple excludes from a file It is common to use the Rat with the Maven or Ant plugins and specify a series of files to exclude (such as a README or version control files). If you are using the Rat application instead of a plugin you can specify a series of regex excludes in a file and specify that with the -E option. java -jar apache-rat-0.13-SNAPSHOT.jar -E /path/to/project/.rat-excludes -d /path/to/project{info}
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Hi Deepak,

        I answered you also on HipChat

        I'll wait infra return on INFRA-15395 because it's not clear to me how to do that in Buildbot

        Show
        jacques.le.roux Jacques Le Roux added a comment - Hi Deepak, I answered you also on HipChat I'll wait infra return on INFRA-15395 because it's not clear to me how to do that in Buildbot
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Gavin fixed INFRA-15395, so I could go on. I have completed the rat exclude file, please review...

        We now need to check if the "SIL Open Font License" has been correctly handled: http://www.apache.org/legal/resolved.html

        Show
        jacques.le.roux Jacques Le Roux added a comment - Gavin fixed INFRA-15395 , so I could go on. I have completed the rat exclude file, please review... We now need to check if the "SIL Open Font License" has been correctly handled: http://www.apache.org/legal/resolved.html

          People

          • Assignee:
            Unassigned
            Reporter:
            deepak.dixit Deepak Dixit
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development