-
Notifications
You must be signed in to change notification settings - Fork 8
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
Various improvements [breaks build under Haiku R1B4] #34
Conversation
- Add .gitignore - Add .clang-format and run
Still builds and works, though some changes to use BNumberFormat appear to be done just for changes sake? |
Build failed... |
I know what you mean :). Ideally, all user facing numbers should be formatted to account for different locale settings. Other OS usually have this baked into their translation macros, but we have to do it manually for the time being… |
Hmm, works here with hrev57672, 64bit. Did a "jam clean && jam -qj8"... |
It failed on r1beta4 |
That's normal, it uses API's not available on Beta 4. |
What's the benefit of this PR then? |
What do you mean what's the benefit of this PR? Can you elaborate? |
Trying to understand what you want with this PR? It obviously can't be merged. Wait until the next release for a merge? |
Well obviously? Is there a time-cap or something for submitting PR's? You could just indicate something like "to be merged after next Haiku release", and be done with it. I have trouble understanding the reasons for these pointless questions. |
Seems all clear now... Thanks. |
Sorry if it sounded rude, I hope you understand what I mean. |
Let's merge then and I'll add a reminder issue not to update the haikuports recipe (or require some nightly hrev) because it breaks the build under R1B4. |
So we can't fix anything for r1beta4 because of this, how nice. Please stop doing this. |
Sorry, I apprently misinterpreted your "Seems all clear now"... |
No description provided.