-
Notifications
You must be signed in to change notification settings - Fork 1.1k
SR-14933: Implement copy()
in ISO8601DateFormatter
.
#3009
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
Conversation
@swift-ci test |
So interestingly I ran the
@millenomi Is @YOCKOW I think this PR is correct I just want to see Lily's response first. I thinks its also worth adding a |
@spevans |
@YOCKOW I don't think your PR is wrong, it might be that there is a bug in Darwin's Foundation. I just wanted confirmation that the bug exists on macOS as well. So I would hold off on making any changes for now. |
@spevans |
|
@millenomi Could you check my earlier comment in #3009 (comment) it looks like Darwin's Foundation doesn't properly implement |
It is indeed a bug. Do you happen to have a FB number handy for it? |
The sync system seemed to link https://bugs.swift.org/browse/SR-14108 to rdar://problem/73742491, not sure if that is the best bug to cover it, otherwise could link to https://bugs.swift.org/browse/SR-14933 which covers this fix. |
Should I file a separate report about that using an other bug tracker such as Feedback Assistant? |
@YOCKOW When you get a chance could you possibly backport this fix to the 5.5 branch (and also 5.4 if it can be done before Friday), thanks. |
As with #3008,
ISO8601DateFormatter
should also implement its own override ofcopy()
.Resolves SR-14933.