Mailing-List: contact dev-help@taverna.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@taverna.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <CAOWBHntykM5XTuvwYwiM0gn72m7bGqo4BdVHsfELiggSv=Vu4A@mail.gmail.com>
References: <CAOWBHntAQ4QvfqVr-QzdMtuwtdg_ti=i7BO5zzFQs69yGAk0pA@mail.gmail.com>
 <CAC2qDMJWUD-9CS3+opy+4ndNf_b8D=4wmWumY5L9h1_T1WenZA@mail.gmail.com>
 <CAC2qDMK-epQUtsK_6kSOmOFiCq6N2CQB67TnhNO+hVNx20Gd9g@mail.gmail.com>
 <CAOMUzGe8-wgPDYXj0MSbAVA=dewQHmt7oq9Woo4yUJ1sKUgOKQ@mail.gmail.com>
 <CAC2qDMKn=-GoK+q7ivGN0v4LvJRwrmaXUF2idba7cUJBtEFe9w@mail.gmail.com>
 <CAOMUzGd1TniLZJ9vUo-r5A=QdVLDmUUCOE4g=tPU3PN2Swgp5w@mail.gmail.com>
 <CAOWBHnu5_6UsOuEv7JzmEKuvbp341SCgJZr=KQb9OA7nYmvvxw@mail.gmail.com>
 <CAMBJEmWR66b4f-4AW0==23oiB_wm+TSoeVoN-=AVBnOvTpeRAQ@mail.gmail.com>
 <CAC2qDM+wQbu3OoBscGHR9Xbb1AXxzXdHFC=0R-t1v7BrvS=KfQ@mail.gmail.com> <CAOWBHntykM5XTuvwYwiM0gn72m7bGqo4BdVHsfELiggSv=Vu4A@mail.gmail.com>
From: Stian Soiland-Reyes <stain@apache.org>
Date: Mon, 13 Jun 2016 14:13:57 +0100
Message-ID: <CAMBJEmWqNK+W7d481g7r+wXSGex=ZurEdLzAZVwMx3F+FJdk4g@mail.gmail.com>
Subject: Re: Taverna Mobile Application
To: dev@taverna.incubator.apache.org
Content-Type: text/plain; charset=UTF-8
archived-at: Mon, 13 Jun 2016 13:14:27 -0000

Hi,

Thanks for the pull request - it has been merged.

https://github.com/apache/incubator-taverna-mobile/pull/15


Let's start a new thread for each thing rather than continue on this
long thread :) I'm loosing a bit track of what has been replied to or
not..

While the GitHub integration sends a separate email., it's convenient
if you include the link to the pull request in question.


Also I'm not the only one who can review/merge your patches, so let's
address the whole of Taverna community rather than Stian. :)

I would suggest something like:


"Hi folks,

Could you review the pull request
https://github.com/apache/incubator-taverna-mobile/pull/15 ?

This fixes licensing headers.
"


On 8 June 2016 at 13:32, Sagar <kumarsagar15795@gmail.com> wrote:
> Hi Stain,
>
> I have added ASF file header in all newly added file .Please check and
> merge it so that I work on next feature because this is the base of my next
> PRs.
>
> Thanks
>
> On Mon, Jun 6, 2016 at 9:24 PM, Rajan Maurya <rajanmaurya154@gmail.com>
> wrote:
>
>> Actually, the good way to manage and list the dependencies is :
>>
>>
>> https://github.com/therajanmaurya/android-client/blob/MVP-Phase-1/mifosng-android/build.gradle
>>  and
>>
>> https://github.com/therajanmaurya/android-client/blob/MVP-Phase-1/build.gradle
>>
>> But I think, It has to be in separate #PR. So, Sagar will open his next PR
>> of managing the build.gradle.
>>
>> And About the licenses of the dependencies, we don't need to mention here
>> we will add all dependencies licenses in Application (All Libraries used
>> license).
>> As a Feature.
>>
>> Thanks
>>
>>
>> On Monday, June 6, 2016, Stian Soiland-Reyes <stain@apache.org> wrote:
>>
>> > On 5 June 2016 at 22:03, Sagar <kumarsagar15795@gmail.com> wrote:
>> > > Hi Rajan,
>> > >
>> > > I have fixed all your input. Please  review it again and suggests me
>> > > something if there is something needed to change .
>> > >
>> > > If anyone else wants to work on it or review my work, it'd be great.
>> >
>> > Thanks both! I've commented on the pull request that we just need some
>> > ASF headers on the new files, so we know they are licensed correctly.
>> >
>> >
>> >
>> https://github.com/apache/incubator-taverna-mobile/pull/14#issuecomment-223956980
>> >
>> > (Note the license headers are not quite consistent in the rest of
>> > Taverna Mobile, see also
>> > https://issues.apache.org/jira/browse/TAVERNA-976 )
>> >
>> >
>> > I checked the licenses of the new dependencies manually - they are all
>> > ASF 2.0 and so compabible. But do you know if there is a good Gradle
>> > way to list all dependencies (including transitives) and their
>> > licenses?
>> >
>> >
>> >
>> https://github.com/sagar15795/incubator-taverna-mobile/blob/Announcement/app/build.gradle#L31
>> >
>> > --
>> > Stian Soiland-Reyes
>> > Apache Taverna (incubating), Apache Commons
>> > http://orcid.org/0000-0001-9842-9718
>> >
>>
>>
>> --
>> *Thanks*
>> *Namaste*
>>
>> Rajan Maurya
>> Contact Number : +91 8010665248
>> Github : https://github.com/therajanmaurya
>> College : Cluster Innovation Centre, University of Delhi
>> Student : B.Tech. (Information Technology & Mathematical Innovations)
>>
>
>
>
> --
>
> *With warm regards  *
> *Sincerely yours*
> Sagar
>
>
> *Student ,(6th Semester )B.Tech IT and mathematical innovation*
> *Cluster Innovation Centre*
> *University of Delhi *


-- 
Stian Soiland-Reyes
Apache Taverna (incubating), Apache Commons
http://orcid.org/0000-0001-9842-9718