Bug#909949: RFS: android-platform-system-extras/8.1.0+r23-1 (experimental)

[殷啟聰 | Kai-Chung Yan]

Hello Roger,

> 
> The problem is you didn't mention +ds or +repack in version in d/changelog.
> I guess the version need to be changed to 8.1.0+r23+ds-1 or 
> 8.1.0+r23+repack-1.
> 
> Cheers,
> 

Truth to be told, almost all packages in Debian are "repacked" either by 
applying a different compression (ZIP to TARXZ) and/or stripping out 
incompatible/unnecessary files. And as a result, almost all packages would have 
had a "+repackXX" in the version strings, which is quite unnecessary. The 
Android Tools team does not have such tradition of adding that suffix.

I repacked it manually because `uscan` (or specifically `mk-origtargz`) failed 
to do so with an error, which happens on many of our packages. It was reported 
[1] but seems the fault is on `tar`. Hans probably didn't bother to do it so he 
simply imported the original tarball anyway...

Additionally, 8.1.0+r23 is never uploaded to the archive, so adding a 
"+repackXX" suffix isn't really mandatory.

[1]: https://bugs.debian.org/831870



signature.asc
Description: OpenPGP digital signature

Bug#909949: RFS: android-platform-system-extras/8.1.0+r23-1 (experimental)

[Roger Shimizu]

Dear Kai-Chung,

On Wed, Oct 3, 2018 at 2:05 AM 殷啟聰 | Kai-Chung Yan  wrote:
>
> Thanks for the sponsor again, Roger!
>
> I did push the changes already, check 
> .
>
> Or do you mean you want a dedicated merge request so you can leave comments 
> their so that in a more sophisticated way? That's probably not viable...

I did a quick review.
Since you removed a few stuff from upstream, such as unused test data,
jar, etc, the upstream source need to be called "repack", or debian
source [1].
And all removed stuff need to be listed in debian/copyright. (I see
you already handled this.)
In this way, the repack can be done automatically by uscan. (nice to
do, but not must)

[1] https://wiki.debian.org/BenFinney/software/repack

The problem is you didn't mention +ds or +repack in version in d/changelog.
I guess the version need to be changed to 8.1.0+r23+ds-1 or 8.1.0+r23+repack-1.

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 4096R/6C6ACD6417B3ACB1

Bug#909949: RFS: android-platform-system-extras/8.1.0+r23-1 (experimental)

[殷啟聰 | Kai-Chung Yan]

Thanks for the sponsor again, Roger!

I did push the changes already, check 
.

Or do you mean you want a dedicated merge request so you can leave comments 
their so that in a more sophisticated way? That's probably not viable...



signature.asc
Description: OpenPGP digital signature

Bug#909949: RFS: android-platform-system-extras/8.1.0+r23-1 (experimental)

[Roger Shimizu]

On Sun, Sep 30, 2018 at 10:45 PM 殷啟聰 | Kai-Chung Yan
 wrote:
>
> Package: sponsorship-requests
> Severity: normal
>
> Dear mentors,
>
> I am looking for a sponsor for my package "android-platform-system-extras"

Thanks for your work!
Could you push your commits to salsa? It will make the review easier.
You can push to a tentative branch if you're not quite sure.

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 4096R/6C6ACD6417B3ACB1

Bug#909949: RFS: android-platform-system-extras/8.1.0+r23-1 (experimental)

[殷啟聰 | Kai-Chung Yan]

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "android-platform-system-extras"

* Package name: android-platform-system-extras
  Version : 8.1.0+r23-1
  Upstream Author : Android Open Source Project
* URL : https://android.googlesource.com/platform/system/extras
* License : Apache-2.0
  Section : devel

It builds those binary packages:

* android-libext4-utils - Android ext4 utility library
* android-libext4-utils-dev - Android ext4 utility library - Development files
* android-libf2fs-utils - Android F2FS utility library
* android-libf2fs-utils-dev - Android F2FS utility library - Development files
* android-sdk-ext4-utils - Android ext4-utils tools (new package, hence the 
need for sponsor)

To access further information about this package, please visit the following 
URL:

  https://mentors.debian.net/package/android-platform-system-extras

Alternatively, one can download the package with dget using this command:

  dget -x 
https://mentors.debian.net/debian/pool/main/a/android-platform-system-extras/android-platform-system-extras_8.1.0+r23-1.dsc

This upload will enable us to update fastboot to 8.1.0.

Changes since the last upload:

[ Hans-Christoph Steiner ]
* New upstream release
* d/copyright: Use HTTPS URLs
* Switch to Salsa

[ Saif Abdul Cassim ]
* New package android-sdk-ext4-utils providing:
  * mkuserimg
  * ext4fixup
  * make_ext4fs
  * This package is replacing the obsolete android-tools-fsutils

[ Kai-Chung Yan ]
* Debhelper level => 11
* d/rules: Use dh-auto-build instead of plain make thanks to Helmut Grohne
* Fix the conflicting types errors (Closes: #892170)
* Standards-Version => 4.2.1

Regards,
殷啟聰 | Kai-Chung Yan



signature.asc
Description: OpenPGP digital signature