Hi, Bertrand Thank you for the inputs.
My original thought was to maintain different Rat exclusions for different repo because they may have different files. If we want to maintain a unique version, we may need to collect all exclusions in all the repo together and maintain a set. Sometimes it's difficult to tell whether a file shall be excluded or not. Hope you can help review after the collection is ready. OpenWhisk has a repo in github to track release documents and scripts. Here it is: https://github.com/apache/incubator-openwhisk-release/blob/master/README.md. I think these information can be documented here: https://github.com/apache/incubator-openwhisk-release/blob/master/docs/work_items.md#2-verify-the-compliance-of-the-source-code-open-issue. Best regards Ying Chun Guo (Daisy) -----Bertrand Delacretaz <bdelacre...@apache.org> wrote: ----- To: dev@openwhisk.apache.org From: Bertrand Delacretaz <bdelacre...@apache.org> Date: 02/09/2018 06:41PM Subject: Re: Release policy compliance: Adding license headers and the excluded files Hi Daisy, On Thu, Feb 8, 2018 at 1:24 PM, Ying Chun Guo <guoyi...@cn.ibm.com> wrote: > ...I use Apache Rat to audit license headers. I use wskdeploy repo as the > trial repo. > I reported an issue > (https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dopenwhisk-2Dwskdeploy_issues_716&d=DwIBaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=V_NQebMEsahq0wRsMMLN8VHG-pcqPRpdHygvo4rmK4o&m=m3ygIAacPw2tdx9MhWLfpLzsW1V32RO3z5g6A2YFWCo&s=zY3fWnnLVXjHDVQvYS73Db0ljWoQg4OaYlqNm1mbjWE&e=) > to wskdeploy including a report generated by Apache Rat. In the report, all > the files > with unapproved licenses will be listed... This sounds great. What you want basically is that for any build that's meant to create a release: -Rat should run -The build should fail if Rat reports any anomalies -The Rat exclusions should ideally be defined in the same way for all modules, for consistency -The Rat exclusions should be commented to indicate why each exclusion (or family of exclusions) is here I think this should be documented in a "coding standards" or "release management" page, does OpenWhisk have this already? -Bertrand (with my incubation mentor hat on)
