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

Error unzip pod with folder with a special character #177

Open
rquesada opened this issue Jun 14, 2021 · 0 comments · May be fixed by #179 or #178
Open

Error unzip pod with folder with a special character #177

rquesada opened this issue Jun 14, 2021 · 0 comments · May be fixed by #179 or #178

Comments

@rquesada
Copy link

If I try to add a pod that a have a special character in its name (emoji o Chinese character) the command make pods or make pods_debug fail.
The issue is caused by unzip command.

Example:
https://github.com/danielgindi/Charts/
In the path:
Test/ChartsTests/Snapshots__Silicon

What did you do?

I add the following pod to Pods.WORSPACE
new_pod_repository( name = "Charts", url = "https://github.com/danielgindi/Charts/archive/refs/tags/v4.0.1.zip", )

What did you expect to happen?

The Library added correctly.

What happened instead?

I got the following error:
checkdir error: cannot create OUT/Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__
Illegal byte sequence
unable to process Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__/.
checkdir error: cannot create OUT/Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__
Illegal byte sequence
unable to process Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__/BarChartTests/.
checkdir error: cannot create OUT/Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__
Illegal byte sequence
unable to process Charts-4.0.1/Tests/ChartsTests/Snapshots??+Silicon__/BarChartTests/testDefaultBarDataSetLabels-iOS_390-0_844-0.1.png.
....

@rquesada rquesada linked a pull request Jun 15, 2021 that will close this issue
rquesada pushed a commit to rquesada/PodToBUILD that referenced this issue Jun 22, 2021
@rquesada rquesada linked a pull request Jun 22, 2021 that will close this issue
This was linked to pull requests Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant