Bug#831870: devscripts: [mk-origtargz] Fails to create tar with Files-Excluded

[Chirayu Desai]

Hi,

On Wed, Jul 20, 2016 at 8:21 PM, Osamu Aoki <osamuaoki at e01.itscom.net> wrote:
> Hi,
>
> Is this a bug or lack of documentation ...
Apologies, I should've included more info.
>
> On Wed, Jul 20, 2016 at 05:10:01PM +0530, Chirayu Desai wrote:
>> Package: devscripts
>> Version: 2.16.6
>> Severity: normal
>>
>> Dear Maintainer,
>>
>> I'm working with the Android Tools team. We're seeing a failure with mk-origtargz.
>> The package is src:android-platform-frameworks-base [1]. The output can be viewed at [2].
>> The output is basically:
>> mk-origtargz --repack --compression xz android-6.0.1_r43.tar.gz
>> No files matched excluded pattern as the last matching glob: *.zip
>> No files matched excluded pattern as the last matching glob: core/tests
>> No files matched excluded pattern as the last matching glob: media/tests
>> tar: tools/preload/sorttable.js: Not found in archive
>> tar: <files matching Files-Excluded>: Not found in archive
>> tar: Exiting with failure status due to previous errors
>> mk-origtargz: error: tar --delete tools/preload/sorttable.js <more files matching Files-Excluded>
>> gave error exit status 2
>> The file android-6.0.1_r43.tar.gz can be grabbed from [3]. It is big, 930M.
>
> Yah ... It would have been nice if you tool time to check the situation
> over the messages of mk-origtargz and actual file contents
>
> Let me requote:
>> No files matched excluded pattern as the last matching glob: *.zip
There are zip files under the docs dir.
>> No files matched excluded pattern as the last matching glob: core/tests
This directory exists.
>> No files matched excluded pattern as the last matching glob: media/tests
And so does this.
>> tar: tools/preload/sorttable.js: Not found in archive
Same here.
>> tar: <files matching Files-Excluded>: Not found in archive
All of the files listed here do exist in the tar. If you look at the
output, it looks like the list is the actual content we want to
exclude. i.e. it lists all files under core/tests, media/tests, etc,
when we only specify the directory at [1]. So it all does exist.

[1]: http://anonscm.debian.org/cgit/android-tools/android-platform-frameworks-base.git/tree/debian/copyright#n4
>
> So did you check these exists?  If not you need to update
> debian/copyright entries first.
They do exist in the original tar, and we don't want them in the repo.
>
>> tar: Exiting with failure status due to previous errors
>
> Quick work around is remove debian/copyright entry and run uscan.  Then
> carefully resotore its contents with updated situation.
I think this is not what we want to do, given all of what we list does
exist in the archive and we don't want any of that in debian, as it
isn't used, and needlessly increases the size of the repo.
>
> Osamu
>
> (Still downloading android-6.0.1_r43.tar.gz so I have not checked the
> real situation.
You can also view it's contents here:
https://android.googlesource.com/platform/frameworks/base/+/android-6.0.1_r43/
>
> Osamu
>
>> Regards,
>> Chirayu Desai
>>
>> [1]: http://anonscm.debian.org/cgit/android-tools/android-platform-frameworks-base.git
>> [2]: http://sprunge.us/RCIR
>> [3]: https://android.googlesource.com/platform/frameworks/base/+archive/android-6.0.1_r43.tar.gz
>>