<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 27, 2016 at 5:06 PM Hans-Christoph Steiner <<a href="mailto:hans@at.or.at">hans@at.or.at</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Checked out the new repo, two things:<br>
<br>
* when working on a package, you should only modify things in the<br>
debian/ folder, the upstream sources need to be modified via patches.<br>
There is a 'copyright' file in the root of the project<br>
<br>
* .gitignore should be moved to debian/.gitignore and then "debian/"<br>
stripped from the paths in that file<br></blockquote><div>I have had this question for a while, and was also thinking of adding gitignores to all our repos.<br>Is it not preferable to add it to the root of the repo?<br>Some packages leave artifacts there, and there's some generic debian build related things too.<br>So would it be okay if I added a .gitignore to android-platform-frameworks-base, with it's contents being:</div><div>aapt</div><div>aidl</div><div>libaapt.so<br><br>etc.<br><br>If not, then what should we do to ignore these files? Move them to another directory perhaps?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
.hc<br>
<br>
Mouaad Aallam:<br>
> Hello,<br>
><br>
> After updating, I pushed the project to alioth[1]<br>
><br>
> [1]:<br>
> <a href="http://anonscm.debian.org/cgit/android-tools/google-android-m2repository-installer.git/commit/" rel="noreferrer" target="_blank">http://anonscm.debian.org/cgit/android-tools/google-android-m2repository-installer.git/commit/</a><br>
><br>
> Cheers,<br>
> *Mouaad Aallam*<br>
><br>
> *Mouaad Aallam*<br>
><br>
> On Fri, Jun 24, 2016 at 5:13 PM, Mouaad Aallam <<a href="mailto:mouaad@aallam.com" target="_blank">mouaad@aallam.com</a>> wrote:<br>
><br>
>> PS: forgot to mention the TIP bug for the package[1]<br>
>><br>
>> [1]: <a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=828059" rel="noreferrer" target="_blank">https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=828059</a><br>
>><br>
>> *Mouaad Aallam*<br>
>><br>
>> On Fri, Jun 24, 2016 at 5:05 PM, Mouaad Aallam <<a href="mailto:mouaad@aallam.com" target="_blank">mouaad@aallam.com</a>> wrote:<br>
>><br>
>>> I have updated the package and follow your comments :)<br>
>>><br>
>>> Please check it again. [1]<br>
>>><br>
>>> [1]:<br>
>>> <a href="https://github.com/Aallam/debian_google-android-m2repository-installer" rel="noreferrer" target="_blank">https://github.com/Aallam/debian_google-android-m2repository-installer</a><br>
>>><br>
>>> *Mouaad Aallam*<br>
>>><br>
>>> On Fri, Jun 24, 2016 at 9:16 AM, Hans-Christoph Steiner <<a href="mailto:hans@at.or.at" target="_blank">hans@at.or.at</a>><br>
>>> wrote:<br>
>>><br>
>>>><br>
>>>> Oops, one last one:<br>
>>>><br>
>>>> The Architecture should be "all" since this binary package will work on<br>
>>>> all Debian architectures:<br>
>>>><br>
>>>> <a href="https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Architecture" rel="noreferrer" target="_blank">https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Architecture</a><br>
>>>><br>
>>>> .hc<br>
>>>><br>
>>>> Hans-Christoph Steiner:<br>
>>>>><br>
>>>>> Also, the debian/source/format needs to be "3.0 (native)"<br>
>>>>><br>
>>>>> .hc<br>
>>>>><br>
>>>>> Hans-Christoph Steiner:<br>
>>>>>><br>
>>>>>> Hey Mouaad,<br>
>>>>>><br>
>>>>>> That was fast :) Here are some comments:<br>
>>>>>><br>
>>>>>> * please file an ITP bug for this package, you can use any of the<br>
>>>>>> existing ITP bug reports for android-tools packages as a template.,<br>
>>>> e.g.<br>
>>>>>> <a href="https://bugs.debian.org/795173" rel="noreferrer" target="_blank">https://bugs.debian.org/795173</a><br>
>>>>>><br>
>>>>>> * google-android-build-tools-installer.lintian-overrides should be<br>
>>>>>> renamed both in filename and contents using the source package name:<br>
>>>>>> google-android-m2repository-installer<br>
>>>>>><br>
>>>>>> * the Description in debian/control should be filled in more<br>
>>>> describing<br>
>>>>>> what the "android support m2 repository" is. I'm sure you can find<br>
>>>>>> description text from Google about what Android Support is, then also<br>
>>>>>> say that this structured as a maven m2 repository of all the versions<br>
>>>> of<br>
>>>>>> the library.<br>
>>>>>><br>
>>>>>><br>
>>>>>> .hc<br>
>>>>>><br>
>>>>>> Mouaad Aallam:<br>
>>>>>>> Hello,<br>
>>>>>>><br>
>>>>>>> Here is my first attempt for the package that downloads android<br>
>>>> support m2<br>
>>>>>>> repo[1]. I past it in the channel too. (not sure if the package name<br>
>>>> is<br>
>>>>>>> correct)<br>
>>>>>>><br>
>>>>>>> I created this based on two existing packages as templates [2][3].<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> [1]:<br>
>>>> <a href="https://github.com/Aallam/debian_google-android-m2repository-installer" rel="noreferrer" target="_blank">https://github.com/Aallam/debian_google-android-m2repository-installer</a><br>
>>>>>>> [2]:<br>
>>>>>>><br>
>>>> <a href="https://packages.qa.debian.org/g/google-android-build-tools-installer.html" rel="noreferrer" target="_blank">https://packages.qa.debian.org/g/google-android-build-tools-installer.html</a><br>
>>>>>>> [3]:<br>
>>>> <a href="https://packages.qa.debian.org/g/google-android-sdk-docs-installer.html" rel="noreferrer" target="_blank">https://packages.qa.debian.org/g/google-android-sdk-docs-installer.html</a><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> Cheers,<br>
>>>>>>> *Mouaad Aallam*<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> _______________________________________________<br>
>>>>>>> Android-tools-devel mailing list<br>
>>>>>>> <a href="mailto:Android-tools-devel@lists.alioth.debian.org" target="_blank">Android-tools-devel@lists.alioth.debian.org</a><br>
>>>>>>><br>
>>>> <a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel</a><br>
>>>>>>><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> Android-tools-devel mailing list<br>
>>>> <a href="mailto:Android-tools-devel@lists.alioth.debian.org" target="_blank">Android-tools-devel@lists.alioth.debian.org</a><br>
>>>><br>
>>>> <a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel</a><br>
>>>><br>
>>><br>
>>><br>
>><br>
><br>
><br>
><br>
> _______________________________________________<br>
> Android-tools-devel mailing list<br>
> <a href="mailto:Android-tools-devel@lists.alioth.debian.org" target="_blank">Android-tools-devel@lists.alioth.debian.org</a><br>
> <a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel</a><br>
><br>
<br>
_______________________________________________<br>
Android-tools-devel mailing list<br>
<a href="mailto:Android-tools-devel@lists.alioth.debian.org" target="_blank">Android-tools-devel@lists.alioth.debian.org</a><br>
<a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel</a><br>
</blockquote></div></div>