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-24830] (6_1_X) iOS: require cache improvements #9232

Merged
merged 7 commits into from Jul 27, 2017

Conversation

hansemannn
Copy link
Collaborator

…t to resolved path; consult cache BEFORE tryign to load files
…ath cache key generation, and storing resolved paths for the key; Add path/module cache logging so you can see what's going on.
…rrent dir segement as empty (basically share path cache keys from any dir if the request is absolute)
@sgtcoolguy
Copy link
Contributor

back port of #9158. We also will need a backport/cherry-pick to 6_2_X now, since the original PR is unmarked and targeted at master (which is now 7_0_X target).

@mukherjee2 mukherjee2 requested review from mukherjee2 and removed request for ewieberappc July 26, 2017 22:45
Copy link
Contributor

@mukherjee2 mukherjee2 left a comment

Choose a reason for hiding this comment

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

Node Version: 6.10.3
NPM Version: 3.10.10
Mac OS: 10.12.4
Appc CLI: 6.2.2
Appc CLI NPM: 4.2.9
Titanium SDK version: 6.1.2 (locally built)
Appcelerator Studio, build: 4.9.0.201705302345
Xcode 8.3.2
iOS 10

Passed FR with the above environment. I ran the demo app with debug flag. There were no compile-time or runtime errors. The debug messages that were shown in the description were not seen (or similar).

@hansemannn hansemannn merged commit 4670667 into tidev:6_1_X Jul 27, 2017
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

3 participants