openwhisk-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ying Chun Guo" <guoyi...@cn.ibm.com>
Subject Re: Release policy compliance: Adding license headers and the excluded files
Date Fri, 02 Mar 2018 02:51:56 GMT
Hi, Matt

I tested. The answer is NO. Apache Rat cannot recognize short header format.
That will cause a problem because the large size of action files will slow down the tests
running.
We can ignore these files till Rat supports short header format.

Best regards
Ying Chun Guo (Daisy)


-----"Matt Rutkowski" <mrutkows@us.ibm.com> wrote: -----

>To: dev@openwhisk.apache.org
>From: "Matt Rutkowski" <mrutkows@us.ibm.com>
>Date: 02/12/2018 11:53PM
>Subject: Re: Release policy compliance: Adding license headers and
>the excluded files
>
>Is there any way for the RAT tool to acknowledge the "minified" ASF
>header 
>that is approved for use in .js (and other files)?
>
>See "short form" approved here: 
>https://urldefense.proofpoint.com/v2/url?u=https-3A__www.apache.org_l
>egal_src-2Dheaders.html&d=DwIFAg&c=jf_iaSHvJObTbx-siA1ZOg&r=V_NQebMEs
>ahq0wRsMMLN8VHG-pcqPRpdHygvo4rmK4o&m=V06q35OiWjYJAWe2jq-3fSk7CreRb6Ah
>wORWCs2KSJw&s=iNii7yGWwSTIBwziwicIhtKB91LCc5XgQpmAu_bTb9k&e=
>
>Kind regards,
>Matt 
>
>
>
>
>From: "Ying Chun Guo" <guoyingc@cn.ibm.com>
>To: dev@openwhisk.apache.org
>Date: 02/12/2018 08:18 AM
>Subject: Re: Release policy compliance: Adding license headers
>and 
>the excluded files
>
>
>
>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://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apach
>e_incubator-2Dopenwhisk-2Drelease_blob_master_README.md&d=DwIFaQ&c=jf
>_iaSHvJObTbx-siA1ZOg&r=6zQLM7Gc0Sv1iwayKOKa4_SFxRIxS478q2gZlAJj4Zw&m=
>tuRJI4eVUHlqHACnIGwvLgDqy1D9dIaSeLjOME3KJJg&s=RqfPKNiDPwlE1yljo6i5Rct
>CRB6jXLLbQ5gickneycE&e=
>. I think these information can be documented here: 
>https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apach
>e_incubator-2Dopenwhisk-2Drelease_blob_master_docs_work-5Fitems.md-23
>2-2Dverify-2Dthe-2Dcompliance-2Dof-2Dthe-2Dsource-2Dcode-2Dopen-2Diss
>ue&d=DwIFaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=6zQLM7Gc0Sv1iwayKOKa4_SFxRIxS4
>78q2gZlAJj4Zw&m=tuRJI4eVUHlqHACnIGwvLgDqy1D9dIaSeLjOME3KJJg&s=t34yGdw
>cAE-qLfsKcS0ueeVqUAffslBtsP_kxQmK2fk&e=
>.
>
>Best regards
>Ying Chun Guo (Daisy)
>
>
>-----Bertrand Delacretaz <bdelacretaz@apache.org> wrote: -----
>To: dev@openwhisk.apache.org
>From: Bertrand Delacretaz <bdelacretaz@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 <guoyingc@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_apach
>e_incubator-2Dopenwhisk-2Dwskdeploy_issues_716&d=DwIBaQ&c=jf_iaSHvJOb
>Tbx-siA1ZOg&r=V_NQebMEsahq0wRsMMLN8VHG-pcqPRpdHygvo4rmK4o&m=m3ygIAacP
>w2tdx9MhWLfpLzsW1V32RO3z5g6A2YFWCo&s=zY3fWnnLVXjHDVQvYS73Db0ljWoQg4Oa
>YlqNm1mbjWE&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)
>
>
>
>
>
>
>


Mime
View raw message