-
-
Notifications
You must be signed in to change notification settings - Fork 6k
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
[Feature request] Release 3.6.1 version supporting Swift 5.7 (Xcode 14) #5001
Comments
@danielgindi @liuxuan30 @pmairoldi is there anything else I can do to help in order to have https://github.com/corteggo/Charts/releases/tag/v3.6.1 moved into this repo? If so, please let me know and I'll do it This is the required change: corteggo@ed5f554 Thanks for all your effort and your great contribution to the iOS community by building |
@corteggo sorry I cannot look at every issue every day. however,
this seems something we cannot help because the legacy is not maintained any more. We quite lack of people to maintain different releases. Since the are multiple PRs, I would prefer fix them all in one PR. Could you pull these together once we merge #5009 and make a new PR, then we can move forward? The target is make new 4.x release work on Xcode 14, and perhaps solve your problem, but for 3.x releases, IMO you can keep it in your own repo. |
@liuxuan30 thank you for your answer. We'll keep 3.6.1 in our own repo then. To make 4.x compatible with Xcode 14 when I'll work on it if it's fine with you and will get back to you here once I have a pull request ready for it. Thank you for your input. |
removing |
What would you do to address this issue then? Any other idea or suggestion that could work? Because having I can help to work on this and contribute to the repo but if 3.x is not maintained anymore and |
the problem is not 100% on us.. We tried our best to make it compatible and follow the trend of swift. When we adopted swift-algorithms for good, we think it's the future, however, we cannot forcase that today we see issues caused by it. We try to solve this, but Apple is not replying at all. Currently, the problem is not swift-algorithms itself, it's the swift 5.7 that fail to build for our Objective-C demo. If everyone is totally ok with removing Objective-C demo support, we are ok too. However, we still treat Objc as a must-support. So we stuck on making the demo work. I filed radar and asked help on swift foum(https://forums.swift.org/t/bug-report-unable-to-build-demo-project-if-imported-swift-framework-turned-on-build-library-for-distribution/63904/3), no update all. If you could help solve this build issue, we are reaching a release date. |
Sorry, I can't help with the Obj-C issue. I saw your post in Swift forums but don't know how to help. From my point of view, it can be a known issue for the 5.0.0 release, at least to provide a fix for naming conflict with Swift Charts. Going back to the original topic, I tried
Unless
I'd still suggest removing |
I’m not opposed to removing it but we would need to analyze how long it work take. |
Feature request
with code provided belowWhat did you do?
I need to update a project to build with Xcode 14, so I tried
Charts 3.6.0
andCharts 4.1.0
(the latest available versions when creating this issue) withXcode 14.1
andXcode 14.2
.This project is an SDK distributed to third parties so it has
BUILD_LIBRARY_FOR_DISTRIBUTION
=YES
(which infers it for dependencies likeCharts
).What did you expect to happen?
Either Charts 3.6.0 or Charts 4.1.0 would work with Xcode 14.1 or 14.2 having
BUILD_LIBRARY_FOR_DISTRIBUTION
set toYES
.What happened instead?
Charts 3.6.0
failed with build errors due toChartDataSet
not conforming toRangeReplaceableDataSet
.Charts 4.1.0
failed with build errors due to transitive dependencySwiftAlgorithms
not building withBUILD_LIBRARY_FOR_DISTRIBUTION
set toYES
(missing some@frozen
attributes).Charts Environment
Charts version/Branch/Commit Number:
Charts 3.6.0
Charts 4.1.0
Xcode version:
Xcode 14.1
Xcode 14.2
Swift version:
Swift 5.7
Platform(s) running Charts:
iOS
macOS version running Xcode:
macOS 12
Demo Project
Unnecessary. The
Charts
repo itself can be used as a demo project.Related Issues
Related Pull Requests
Solution
Currently, there's no possible way to use
Charts
withXcode 14
withBUILD_LIBRARY_FOR_DISTRIBUTION
set toYES
. This is a blocking issue for those creating libraries. SettingBUILD_LIBRARY_FOR_DISTRIBUTION
toNO
is not an option.To fix the build issue with
Charts 4.1.0
havingBUILD_LIBRARY_FOR_DISTRIBUTION
set toYES
,SwiftAlgorithms
requires some changes to add@frozen
to all itsstruct
definitions. However, according to swiftlang/swift#62507, the build errors seem to be intentional so I don't think a pull request would be merged in there and it could take some time. Also, after merging, it would requireSwiftAlgorithms
custom podspec (https://github.com/jshier/swift-pods/blob/main/SwiftAlgorithms.podspec) to be versioned and a newCharts
version to be released.Fixing the build issue with
Charts 3.6.0
is easier. ConformingChartDataSet
toRangeReplaceableCollection
is as easy as adding one method (as was already done in this repo forCharts 4.x
).I forked this repository and released version
3.6.1
working fine withXcode 14.1
and14.2
. Please find it here: https://github.com/corteggo/Charts/releases/tag/v3.6.1. However, I can't open a pull request to any of the existing branches in this repo becausemaster
already contains4.x.x
and there're no3.x.x
release branches.I'd kindly ask maintainers to move the
3.6.1
tag from my fork to this repo, release3.6.1
officially and push the updatedCharts.podspec
file to the public podspec repo.With this proposed solution, users can still use 3.6.x with the latest Xcode available if they're building a library. Users building regular applications can use 4.x.
The text was updated successfully, but these errors were encountered: