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

fix(android)(8_1_X): improved merge of <uses-feature/> elements #11111

Merged
merged 2 commits into from Aug 20, 2019

Conversation

jquick-axway
Copy link
Contributor

- [TIMOB-27304] Fixed so that Android setting <uses-feature/> required "true" in "tiapp.xml" should override "false" setting in libraries.
  * Merge is supposed to do a logical-OR on the "android:required" value.
- Added "tools:replace" attribute support to <uses-feature/>.
  * Allows overriding "android:required" to "false" if set "true" by library. (Only means of doing this.)
@build
Copy link
Contributor

build commented Aug 3, 2019

Fails
🚫 Tests have failed, see below for more information.
Messages
📖

💾 Here's the generated SDK zipfile.

📖 ❌ 4 tests have failed There are 4 tests failing and 466 skipped out of 3643 total tests.

Tests:

ClassnameNameTimeError
android.emulator.Titanium.UI.LayoutTIMOB-23372 #35.008
Error: timeout of 5000ms exceeded
at Titanium.<anonymous> (/ti-mocha.js:4326:18)
android.emulator.Titanium.UI.Window.eventsblur event is fired when closed20.006
Error: timeout of 20000ms exceeded
at Titanium.<anonymous> (/ti-mocha.js:4326:18)
ios.Titanium.Database#executeAllAsync() does not allow DB to be GC'd60.005
Error: timeout of 60000ms exceeded
file:///Users/build/Library/Developer/CoreSimulator/Devices/29E1514E-93D8-4A4B-A306-F232955340BA/data/Containers/Bundle/Application/C665E8C0-4F69-4517-9F61-D1D5CA550812/mocha.app/ti-mocha.js:4290:32
ios.Titanium.UI.WebViewuserAgent30.003
Error: timeout of 30000ms exceeded
file:///Users/build/Library/Developer/CoreSimulator/Devices/29E1514E-93D8-4A4B-A306-F232955340BA/data/Containers/Bundle/Application/C665E8C0-4F69-4517-9F61-D1D5CA550812/mocha.app/ti-mocha.js:4326:27

Generated by 🚫 dangerJS against 75626f2

Copy link
Contributor

@ypbnv ypbnv left a comment

Choose a reason for hiding this comment

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

LGTM!

@ssekhri
Copy link

ssekhri commented Aug 5, 2019

FR passed. Correct value of elements in manifest file based on OR(ing) of values from all the involved xml files.
Env:
Mac OS 10.14.3
Ti SDK: 8.1.0.v20190802182019
Appc CLI: 7.1.0-24
Node: 8.16.0
JDK: 10.0.2

@sgtcoolguy sgtcoolguy merged commit 710083a into tidev:8_1_X Aug 20, 2019
rlustemberg added a commit to inzdr/titanium_mobile that referenced this pull request Aug 21, 2019
* 8_1_X: (46 commits)
  fix(ios): exclude all frameworks from optimizing files step (8_1_X) (tidev#10996)
  fix(android): fix asset encryption on windows 32bit java (tidev#11155)
  fix(ios): add script message handler for local files only (tidev#11086)
  fix(android): improve merge of <uses-feature/> elements (tidev#11111)
  fix(android): splash open animation sometimes stutters on cold start (tidev#11140)
  fix(android): fix dialog without selectedIndex reusage
  fix(android): resuming with intent "FLAG_ACTIVITY_MULTIPLE_TASK" can hang the app (tidev#11081)
  fix(android): regression where closing root window from child window causes app exit issues as of 8.0.1 (tidev#11093)
  fix(android): fix reusing a dialog with a new "parent" window (tidev#11096)
  chore(release): bump version
  fix(ios): console.log does not log properly if it has multiple arguments (tidev#11125)
  fix(android): update titanium_prep windows binaries
  chore(android)(8_1_X): update titanium_prep binaries (tidev#11100)
  fix(android)(8_1_X): ListView's SearchBar/SearchView wrongly overlaps rows as of 8.0.1 (tidev#11105)
  chore(android): update aps-analytics (tidev#11101)
  fix(ios): fix crashes in calendar upon giving permission (tidev#11078)
  fix(ios,android)(8_1_X): implement exception signal handler (tidev#11077)
  fix(cli): add deprecation notice for node < 10.13 (tidev#11063)
  test(db): bump up timeout
  test(db): drop long-running query count to 5000 rows
  ...
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

5 participants