Remove osx64 from pixi #1768
Closed
Remove osx64 from pixi #1768
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Explicit dependencies
Implicit dependencies
Footnotes |
|
TeamCity pixi cannot solve this: I wonder if this is because the pixi version is older on TeamCity. |
|
I ran Anyway, I asked Willem to update pixi version to v28.0 for all our Windows agents, and I will ask Maarten to do so for Linux ones |
|
@evetion do we still use osx-64, or can we remove it from our list? Since you are using the arm version. |
|
I'm fine with removing osx, Github now has free arm runners as well. |
|
@visr can you fix my pixi.lock file? pixi crashes every time on my computer. |
Did you try deleting the .pixi directory in the project? |
Yes removed the .pixi folder and the lock file, but still got it crashing after ~10 minutes |
|
Pixi doesn't seem to appreciate having to resolve with more than one channel, Now that we have some automatic testing of QGIS LTR via Docker in #1774, perhaps we should leave the QGIS in pixi to the latest release. As such we'll have some automatic testing for both versions. Would still be good to separately remove osx-64 if we don't use it. |
deltamarnix
force-pushed
the
fix/qgis_ltr
branch
from
ebfada1 to
69cce41
Compare
|
There's a conflict in pixi.lock. What commands did you use to run this? Does this only remove osx-64 or also update dependencies? Are you planning on fixing the Docker LTR tag in a separate issue? |
I talked with @evetion about the LTR version. But if I change it to qgis/ltr, then we will automatically upgrade to the latest LTR every x years. @evetion didn't know if that was desirable. I did a |
|
Yeah probably 3.34 is better than LTR. Our docs and plugin metadata also say 3.34. |
|
Closing in favor of #1777. |
visr
added a commit
that referenced
this pull request
Aug 30, 2024
Supersedes #1768. That had conflicts and I wanted to start fresh since `pixi update` takes 15 mins. So this removes `osx-64` from our platforms, as nobody is using it. This will make our lockfile smaller and hopefully a bit easier to resolve. Then I ran `pixi update` to update all our dependencies and remove osx-64.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Does not fix #1762 anymore, but we don't use osx64 anymore, so we can remove it.