-
Notifications
You must be signed in to change notification settings - Fork 1k
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
sqlite3 localdb: using single quotes for strings backport (#16123) #17938
base: develop
Are you sure you want to change the base?
Conversation
|
Hi @balazon Thanks for your contribution. The main problem with this is that Conan 1 development and release is now frozen in practice. Conan 2 is now more than 2 years old (almost 4 years counting with alphas and betas). The main issue is that we moved some time ago to GH actions for CI, but we didn't port the Conan 1.X pipeline. So it can be challenging to build, test and release such a new version for Conan 1. |
I've just seen an 1.67.0-dev version on the develop branch, which is only 3 months old 😂 |
Yes, the migration to GH actions happened in December: So yes, about 3 months since then, aligned with last Conan 1.66 release in December: https://docs.conan.io/1/changelog.html Isn't it possible to hold the update of sqlite3? That would be the recommendation, just stick with the working versions, isn't it? I'll check what is possible with the team, but this seems quite challenging, it might be that the migration of the CI to GH actions was the last nail on Conan 1. |
Yes it is. My options now seem to be:
The way I noticed this problem was quite subtle though, because I just created a new mamba environment, didn't change a thing, and my build still failed. Turns out new mamba environments installs the latest libsqlite 3.49.1 by default, so that's what broke it. Downgrading libsqlite inside an existing environment deletes pip as well though, so it's best to specify libsqlite version at the time of creating the environment with eg: Best longterm solution is I think to update the recipes to 2.x, doesn't look too bad |
Thanks for the feedback
Yes, please, this should be a priority:
The only issue is that it might not be that straightforward, depending on the current state of your recipes, if you have already modernized them into the Conan 1 new tools, or using legacy Conan 1 tools, the migration effort might be high. The thing is that it was planned as a long term effort, as commented, alphas and betas of Conan 2 started practically 4 years ago, and some of the features necessary to migrate (revisions, crosss-building, new generators) have been in Conan 1 for many, many years. |
A newer version of sqlite3 breaks double quotes on 1.x
https://www.reddit.com/r/freebsd/comments/1chb82b/sqlite3_pkg_just_became_stricter_with_quoting/
There are multiple issues relevant for this bug, for example:
#13446
#16115
The latter was fixed on v2.x here:
#16123
But 1.x still suffers from this, and some of us are still stuck on 1.x, so before we can update to 2.x, could we have this fix please?
Thanks