Skip to content
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-25749] Android: Fix loading of commonjs modules #9821

Merged
merged 2 commits into from Feb 17, 2018

Conversation

garymathews
Copy link
Contributor

@garymathews garymathews commented Feb 14, 2018

TEST CASE
  • Include com.foo.bar in a Titanium project
  • Build project

JIRA Ticket

Copy link
Contributor

@jquick-axway jquick-axway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR: Pass

@lokeshchdhry
Copy link
Contributor

@garymathews , I am getting this different error with the fix.

[ERROR] :  Failed to compile Java source files:
[ERROR] :  
[ERROR] :  /Users/lchoudhary/Desktop/workspaces/workspace_2018/TIMOB-25749/build/android/gen/com/app/timob25749/Timob25749Application.java:113: error: cannot find symbol
[ERROR] :  						(Class<KrollSourceCodeProvider>) Class.forName(className));
[ERROR] :  						       ^
[ERROR] :    symbol:   class KrollSourceCodeProvider
[ERROR] :    location: class Timob25749Application
[ERROR] :  Note: Some input files use unchecked or unsafe operations.
[ERROR] :  Note: Recompile with -Xlint:unchecked for details.
[ERROR] :  1 error
[ERROR] Application Installer abnormal process termination. Process exit value was 1

@garymathews garymathews changed the title [TIMOB-25769] Android: Fix loading of commonjs modules [TIMOB-25749] Android: Fix loading of commonjs modules Feb 14, 2018
@garymathews garymathews changed the base branch from master to 7_1_X February 14, 2018 21:28
@garymathews garymathews changed the title [TIMOB-25749] Android: Fix loading of commonjs modules [7_1_X][TIMOB-25749] Android: Fix loading of commonjs modules Feb 14, 2018
@garymathews
Copy link
Contributor Author

@lokeshchdhry Updated PR

@build build added the android label Feb 14, 2018
@lokeshchdhry
Copy link
Contributor

@garymathews , Now I get this error:

[ERROR] :  Failed to compile Java source files:
[ERROR] :  
[ERROR] :  /Users/lchoudhary/Desktop/workspaces/workspace_2018/TIMOB-25749/build/android/gen/com/app/timob25749/Timob25749Application.java:11: error: cannot find symbol
[ERROR] :  import org.appcelerator.kroll.KrollSourceCodeProvider;
[ERROR] :                               ^
[ERROR] :    symbol:   class KrollSourceCodeProvider
[ERROR] :    location: package org.appcelerator.kroll
[ERROR] :  /Users/lchoudhary/Desktop/workspaces/workspace_2018/TIMOB-25749/build/android/gen/com/app/timob25749/Timob25749Application.java:114: error: cannot find symbol
[ERROR] :  						(Class<KrollSourceCodeProvider>) Class.forName(className));
[ERROR] :  						       ^
[ERROR] :    symbol:   class KrollSourceCodeProvider
[ERROR] :    location: class Timob25749Application
[ERROR] :  Note: Some input files use unchecked or unsafe operations.
[ERROR] :  Note: Recompile with -Xlint:unchecked for details.
[ERROR] :  2 errors
[ERROR] Application Installer abnormal process termination. Process exit value was 1

@garymathews garymathews changed the base branch from 7_1_X to master February 15, 2018 20:28
@garymathews garymathews changed the title [7_1_X][TIMOB-25749] Android: Fix loading of commonjs modules [TIMOB-25749] Android: Fix loading of commonjs modules Feb 15, 2018
@build build added the android label Feb 15, 2018
@build
Copy link
Contributor

build commented Feb 15, 2018

Messages
📖

💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

@lokeshchdhry
Copy link
Contributor

FR Passed.

No error seen when using commonjs modules.

Studio Ver: 5.0.0.201712081732
SDK Ver: 7.2.0 local build
OS Ver: 10.13.2
Xcode Ver: Xcode 9.2
Appc NPM: 4.2.12
Appc CLI: 7.0.2
Daemon Ver: 1.0.1
Ti CLI Ver: 5.0.14
Alloy Ver: 1.11.0
Node Ver: 8.9.1
NPM Ver: 5.5.1
Java Ver: 1.8.0_101
Devices: ⇨ google Nexus 6P --- Android 8.0.0
⇨ google Nexus 5 --- Android 6.0.1

@lokeshchdhry lokeshchdhry merged commit 9a6f501 into tidev:master Feb 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants