New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TIMOB-24446](6_1_X) Allow replacing bundled Android Support Libraries #9070
Conversation
android/cli/commands/_build.js
Outdated
@@ -3065,7 +3073,13 @@ AndroidBuilder.prototype.copyModuleResources = function copyModuleResources(next | |||
resPkgFile = jarFile.replace(/\.jar$/, '.respackage'); | |||
|
|||
if (fs.existsSync(resPkgFile) && fs.existsSync(resFile)) { | |||
this.resPackages[resFile] = fs.readFileSync(resPkgFile).toString().split('\n').shift().trim(); | |||
var packageName = fs.readFileSync(resPkgFile).toString().split('\n').shift().trim(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe support both line-endings? split(/\r?\n/)
android/cli/commands/_buildModule.js
Outdated
@@ -422,8 +443,13 @@ AndroidModuleBuilder.prototype.processResources = function processResources(next | |||
var resArchivePathAndFilename = path.join(modulesPath, file.replace(/\.jar$/, '.res.zip')); | |||
var respackagePathAndFilename = path.join(modulesPath, file.replace(/\.jar$/, '.respackage')); | |||
if (fs.existsSync(resArchivePathAndFilename) && fs.existsSync(respackagePathAndFilename)) { | |||
extraPackages.push(fs.readFileSync(respackagePathAndFilename).toString().split('\n').shift().trim()); | |||
resArchives.push(resArchivePathAndFilename); | |||
var packageName = fs.readFileSync(respackagePathAndFilename).toString().split('\n').shift().trim(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR: PASS (support both line endings)
👍
Fixes an issue with Hyperloop 2.1.0 which causes a crash when trying to override the Android Support Libraries with local .aar files. Hyperloop 2.1.0 will always manually add our bundled Android Support Libraries to the dexer paths even if they were replaced by the builder. To fix this we check the altered dexer paths again and remove any replaced libraries.
FR Passed. Used the steps in the ticket to do the FR. Studio Ver: 4.9.0.201705180402 |
JIRA: https://jira.appcelerator.org/browse/TIMOB-24446
Optional Description:
6_1_X backport of #9008