Skip to content

js-tests\res\cocosui file paths are too long for Windows Phone #1556

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

Closed
stammen opened this issue Mar 17, 2015 · 8 comments
Closed

js-tests\res\cocosui file paths are too long for Windows Phone #1556

stammen opened this issue Mar 17, 2015 · 8 comments
Labels
Milestone

Comments

@stammen
Copy link
Contributor

stammen commented Mar 17, 2015

We need to reduce the paths to some of the resource files in js-test/res. They are too long for Windows Phone. All of the file paths in js-tests\res\cocosui probably need to be shortened.

When installing js-tests on Windows Phone 8.1 I get the following error because the path is too long for the phone.

2>Error : DEP2100 : Cannot copy the file "D:\GitHub\cocos2d-js\samples\js-tests\res\cocosui\UIEditorTest\UILayout_Editor\UILayout_Linear_Horizontal_Layout_Editor\crossplatform_UILayout_Linear_Horizontal_Layout_Editor_1.ExportJson" to the target machine "169.254.51.115".

For example:
Can we remove UILayout_ and _Editor and other things from the folder names?
Can crossplatform_UILayout_Linear_Horizontal_Layout_Editor_1.ExportJson become simply crossplatform1.ExportJson?

@ricardoquesada
Copy link
Contributor

@minggo @linshun could you please give this issue high priority? thanks.

@minggo
Copy link

minggo commented Mar 18, 2015

@pandamicro
Could you please take a look?

@pandamicro
Copy link
Contributor

Hi, @stammen we will change the resources name this week.

@stammen
Copy link
Contributor Author

stammen commented Mar 18, 2015

@pandamicro Thanks for your help!

@pandamicro
Copy link
Contributor

Fix PR: #1564
Please pull the repo and test if it works for you

@pandamicro pandamicro added the bug label Mar 19, 2015
@pandamicro pandamicro added this to the v3.4 milestone Mar 19, 2015
@pandamicro
Copy link
Contributor

@stammen Have you tested ?

@stammen
Copy link
Contributor Author

stammen commented Mar 30, 2015

@pandamicro This works fine now. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants
@ricardoquesada @minggo @pandamicro @stammen and others