On 3/1/18 10:34 AM, Myrle Krantz wrote: > Hey Isaac, > > At a first glance, it looks good. May I ask though why you removed > the license parameter strictCheck? > > Regards, > Myrle
The copyright statement should be removed entirely from the source header files and placed instead in NOTICE.txt. See http://www.apache.org/legal/src-headers.html Phil > > On Thu, Mar 1, 2018 at 5:27 PM, Isaac Kamga <isaac.ka...@mifos.org> wrote: >> Hi Myrle, >> >> I've just updated copyright information in fineract-cn-lang and created a >> new pull request <https://github.com/apache/fineract-cn-lang/pull/4>. >> >> I patiently await your review and possible merger. >> >> At Your Service, >> Isaac Kamga. >> >> On Thu, Mar 1, 2018 at 5:05 PM, Isaac Kamga <isaac.ka...@mifos.org> wrote: >> >>> Hi Myrle, >>> >>> Thanks a million for your advice and guidance on this. >>> >>> I've closed the Pull Request, will do appropriate changes and send in >>> another for review. >>> >>> At Your Service, >>> Isaac Kamga. >>> >>> On Thu, Mar 1, 2018 at 4:56 PM, Myrle Krantz <my...@apache.org> wrote: >>> >>>> Hey Isaac, >>>> >>>> Replies inline: >>>> >>>> On Thu, Mar 1, 2018 at 1:20 PM, Isaac Kamga <isaac.ka...@mifos.org> >>>> wrote: >>>>> I have just updated the copyright information and package name on the >>>>> fineract-cn-lang repository and sent in another pull request >>>>> <https://github.com/apache/fineract-cn-lang/pull/3> for review. >>>> Please make a pull request with *just* the copyright information >>>> adjusted. Changing package names is a backwards incompatible change. >>>> By changing them, you break everything that depends on lang. And all >>>> of the other fineract cn repositories depend on lang. Package names >>>> will have to be changed in all the repositories at once. >>>> >>>> We will have to continue to be careful about backwards compatible >>>> changes until we have three things: >>>> * signature checking >>>> * our artifacts in an artifactory >>>> * an established process of incrementing versions. >>>> >>>> Because changing package names is a global change and changing >>>> copyright information can be done locally, one repository at a time, I >>>> believe we should adjust the copyright information first. >>>> >>>>> On a related note, I'd like to ask experienced developers on Fineract >>>> CN if >>>>> it would be necessary to change the project's name ( from *lang* to >>>>> *fineract-cn-lang* ) in the settings.gradle >>>>> <https://github.com/apache/fineract-cn-lang/blob/develop/set >>>> tings.gradle> >>>>> file. >>>> I don't see any reason to change those names. the project's name is >>>> appended to the artifact id. If we did this, the artifact id in this >>>> case would be org.apache.fineract.cn.fineract-cn-lang. It would >>>> contain duplicated information. Of course we could adjust the build >>>> elsewhere to change the artifact id back to lang. But i don't see a >>>> benefit in changing the project name. >>>> >>>> But perhaps I'm missing something here? >>>> >>>>> Also, in the bintray.pkg section of the build.gradle file >>>>> <https://github.com/apache/fineract-cn-lang/blob/develop/build.gradle>, >>>>> should the `repo` , `userOrg` and `vcsUrl` variables be changed too ? >>>> These >>>>> will help with subsequent updates to fineract-cn-api, >>>>> fineract-cn-cassandra, etc. >>>> Oops! Good catch. We can delete the bintray section in that >>>> build.gradlel file until we're ready to introduce the use of the >>>> apache artifactory. I don't think you'll find a section like that in >>>> any other fineract cn project. I was experimenting there. >>>> >>>> Thank you for taking this on Isaac, >>>> Myrle >>>> >>>