On Tue, 28 Aug 2018 17:23:00 +0000 Joonas Kylmälä <joonas.kylm...@iki.fi> wrote:
> Hi, Hi, > I have made bunch of little modifications to merge latest updates and > security fixes from upstream sources for the next Replicant 6.0 0004 > release. Thanks for working on this. > Please review them: On my side I should improve reviews time. I should also look into installing a patchwork instance to keep track of the patches I forgot about. > [0001-Adapt-to-recent-LineageOS-forks-LineageOS-mirror.patch > [0001-Adapt-to-recent-LineageOS-forks-replicant-6.0-dev.patch Both commit messages don't give much details about why this is needed, but in another hand it might take some time to understand why upstream (lineageOS) might have done that too. Acked-By: Denis 'GNUtoo' Carikli <gnu...@no-log.org> > [0001-Revert-Point-ResXMLTree-to-a-copy-of-the-asset-buffe.patch The commit reverted by the patch above is this one: > commit 953d2a3493aaceb3209afc7eef1ff1e7f7242908 > (replicant/replicant-6.0, m/replicant-6.0-dev) Author: Wolfgang > Wiedmeyer <wolf...@wiedmeyer.de> Date: Thu Nov 2 23:01:34 2017 +0100 > > Point ResXMLTree to a copy of the asset buffer in > AssetManager::getPkgName() > This ensures that there is no dangling pointer when the asset is > closed. Without this fix, a device ends up in a boot loop when an app > is installed that was created using aapt2. > > Reference: > > https://github.com/TwidereProject/Twidere-Android/issues/963#issuecomment-335616312 > > Signed-off-by: Wolfgang Wiedmeyer <wolf...@wiedmeyer.de> Your commit message could give a bit more context on the issue, for instance with something like that: > This reverts commit 953d2a3493aaceb3209afc7eef1ff1e7f7242908. > > The 953d2a3493aaceb3209afc7eef1ff1e7f7242908 commit (Point ResXMLTree > to a copy of the asset buffer in AssetManager::getPkgName()) was > a fix for preventing devices from ending in a boot loop when an > application that was created with aapt2 was installed. > > Since upstream has now fixed it we can now drop this patch to use > upstream's fix instead. > [0002-Update-F-Droid-Privileged-Extension-to-0.2.8-replicant-6.0-dev.patch This looks good, keeping F-Droid privileged extension doesn't harm. Acked-By: Denis 'GNUtoo' Carikli <gnu...@no-log.org> Note that I didn't have the ability to test the patches (I was at a conference with limited time to do remote builds). Denis.
pgpH1nYd1Uxot.pgp
Description: OpenPGP digital signature
_______________________________________________ Replicant mailing list Replicant@osuosl.org https://lists.osuosl.org/mailman/listinfo/replicant