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

build: can't build in a path that has a space in it #12084

Closed
3 tasks
refack opened this issue Mar 28, 2017 · 9 comments
Closed
3 tasks

build: can't build in a path that has a space in it #12084

refack opened this issue Mar 28, 2017 · 9 comments
Labels
build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation. tools Issues and PRs related to the tools directory.

Comments

@refack
Copy link
Contributor

refack commented Mar 28, 2017

After trying to solve this in #11217 I gave up. It's deeper rooted and it's in the icu tooling, sorry @srl295.

since we can't add a /test/know_issues/ for this I'd rather keep this one open at least until it's mitigated significantly.
Arbitrary exit criteria - should work on:

  • macOS
  • Ubuntu
  • Windows

old irrelevant venting, I'm not deleting this since I did write it, but I do retract.
[cynicism]
As a native Hebrew speaker (which is even RTL), I just wanna say internationalization is stupid! 😝

@mscdex mscdex added build Issues and PRs related to build files or the CI. windows Issues and PRs related to the Windows platform. i18n-api Issues and PRs related to the i18n implementation. labels Mar 28, 2017
@refack refack changed the title build: on windows can't build in a path that has a space in it build: can't build in a path that has a space in it May 1, 2017
@refack refack removed the windows Issues and PRs related to the Windows platform. label May 1, 2017
@addaleax addaleax added wontfix Issues that will not be fixed. and removed i18n-api Issues and PRs related to the i18n implementation. labels May 1, 2017
@addaleax
Copy link
Member

addaleax commented May 1, 2017

@refack See #11713 (comment), I’m closing this too. Also, again, please try to be respectful of other’s work and try to keep unstructured critique to a minimum, even if you tag it as a joke/“cynicism”.

@addaleax addaleax closed this as completed May 1, 2017
@refack
Copy link
Contributor Author

refack commented May 1, 2017

Also, again, please try to be respectful of other’s work and try to keep unstructured critique to a minimum, even if you tag it as a joke/“cynicism”.

It's the old me :(

@refack
Copy link
Contributor Author

refack commented May 1, 2017

@srl295 if you read the old comment above. I meant no disrespect to you or the ICU project.
What I did mean was that i10n is hard and frustrating, and something you forget the goal, which is basically accessibility.

P.S. I even have a start-up in the field https://www.bablic.com/

@refack
Copy link
Contributor Author

refack commented May 1, 2017

@addaleax I'm gonna reopen and self assign. I think I can solve this.

@refack refack reopened this May 1, 2017
@refack refack self-assigned this May 1, 2017
@refack refack added i18n-api Issues and PRs related to the i18n implementation. tools Issues and PRs related to the tools directory. and removed wontfix Issues that will not be fixed. labels May 1, 2017
@bzoz bzoz marked this as a duplicate of #14337 Jul 20, 2017
@Trott
Copy link
Member

Trott commented Aug 18, 2017

@refack Are you still working on this? If not, should we unassign and label help wanted?

@addaleax
Copy link
Member

I still think solving this is a bit of an unrealistic expectation. :) Nothing against @refack, but this is something that far too many people have struggled with for far too many software projects to be optimistic here.

@refack
Copy link
Contributor Author

refack commented Aug 19, 2017

I have an idea about how to overcome the issue with the ICU toolchain. I'm not sure if it will solve the entire build, but it's a one step.
Even after that, since we can't add a /test/know_issues/ for that I'd rather keep this one open, until it's at least mitigated significantly, say works on macOS Ubuntu and Windows (adding that to the first comment)

@srl295
Copy link
Member

srl295 commented Aug 20, 2017

@refack sure no problem. Anyway my frustration here is gyp.

@bnoordhuis
Copy link
Member

This has been open for over a year without movement so I'm going to assume it's not happening.

@refack refack removed their assignment Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

No branches or pull requests

6 participants