Overall it looks very good. The patched branch builds and I tested the armv7 apk on Android Q. Tor starts and I'm able to access an external site.
Note the following:
I provided the android-jsocks.patch which removes the jsocks.aar dependency. This is only used for VPN support, which we don't need. I broke out the tor-android-service VPN code into its own module, which we now exclude from browser builds. This patch can be removed once a similar change is made in firefox code.
--disable-tor-browser-update does not work in this build so I removed it from mozconfig until support is added.
network is enabled for the container but I added --offline flag to gradle so its can only use our maven cache. I confirmed that nothing gradle-related is downloading in the current build.
Overall it looks very good. The patched branch builds and I tested the armv7 apk on Android Q. Tor starts and I'm able to access an external site.
Compilation does not work for me if I build the x86 target at least. My tor-browser-build repo is at 571d14f36daa6dd1162803882475eae6ff20d3df. And I've pointed my firefox config to 2dbb7aefea for the nightly target and started the build with make nightly-android-x86. What I get is an error early in the browser step:
0:51.43 FAILURE: Build failed with an exception. 0:51.43 * What went wrong: 0:51.43 Could not resolve all files for configuration ':app:withoutGeckoBinariesDebugAndroidTestRuntimeClasspath'. 0:51.43 > Could not resolve net.freehaven.tor.control:jtorctl:0.2. 0:51.43 Required by: 0:51.43 project :app 0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode. 0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode. 0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode. 0:51.43 > Could not resolve org.slf4j:slf4j-api:1.7.25. 0:51.43 Required by: 0:51.43 project :app 0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode. 0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode. 0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode. 0:51.43 > Could not resolve org.slf4j:slf4j-android:1.7.25. 0:51.43 Required by: 0:51.43 project :app 0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode. 0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode. 0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode.
Those missing dependencies are in 0827-patched branch (not in 0827 branch, which doesn't require them). I think you need to clean out your gradle-dependencies-6 cache prior to building.
Overall it looks very good. The patched branch builds and I tested the armv7 apk on Android Q. Tor starts and I'm able to access an external site.
Compilation does not work for me if I build the x86 target at least. My tor-browser-build repo is at 571d14f36daa6dd1162803882475eae6ff20d3df. And I've pointed my firefox config to 2dbb7aefea for the nightly target and started the build with make nightly-android-x86. What I get is an error early in the browser step:
{{{
0:51.43 FAILURE: Build failed with an exception.
0:51.43 * What went wrong:
0:51.43 Could not resolve all files for configuration ':app:withoutGeckoBinariesDebugAndroidTestRuntimeClasspath'.
0:51.43 > Could not resolve net.freehaven.tor.control:jtorctl:0.2.
0:51.43 Required by:
0:51.43 project :app
0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode.
0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode.
0:51.43 > No cached version of net.freehaven.tor.control:jtorctl:0.2 available for offline mode.
0:51.43 > Could not resolve org.slf4j:slf4j-api:1.7.25.
0:51.43 Required by:
0:51.43 project :app
0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode.
0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode.
0:51.43 > No cached version of org.slf4j:slf4j-api:1.7.25 available for offline mode.
0:51.43 > Could not resolve org.slf4j:slf4j-android:1.7.25.
0:51.43 Required by:
0:51.43 project :app
0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode.
0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode.
0:51.43 > No cached version of org.slf4j:slf4j-android:1.7.25 available for offline mode.
}}}
While skimming over the patch: please don't enable networking during build. What is the error you get?
Additionally, while you remove the patch for 1527534 in the build script you still include it in the config file.
Looks better, but you still have the patch for 1527534 in the config file and not deleted while removing it from the build script. Additionally, I am wondering why we need the libclang.so.6 at all and then just for arm? Does Mozilla do the same? Could you add a comment on elaborating why this is needed for that architecture?
I moved forward to apply the patches you have so far to have at least some nightly builds before we need to release the alpha. But we should fix up as many loose ends before that as we can.