8000 Let swayfx-git depend on scenefx-git by MagneFire · Pull Request #35 · wlrfx/packages · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Let swayfx-git depend on scenefx-git #35

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MagneFire
Copy link

The swayfx git package depends on specific version of scenefx. While this generally works fine. This breaks whenever scenefx-git is installed. Because the provided library may change at any time. Causing issues with swayfx-git still depending on that specific version.

To solve this depend on the scenefx-git package instead of the provided library version. This fixes the issue where scenefx-git may install an updated library version that swayfx-git depends on in the same update cycle.

The only potential risk with this change is that swayfx-git will break when scenefx-git updates it's library version while not updating swayfx-git. This results in a runtime issue where swayfx-git can't find a specific version of the scenefx library.

ref: WillPower3309/swayfx#384

8000
The swayfx git package depends on specific version of scenefx.
While this generally works fine. This breaks whenever scenefx-git is
installed. Because the provided library may change at any time. Causing
issues with swayfx-git still depending on that specific version.

To solve this depend on the scenefx-git package instead of the provided
library version. This fixes the issue where scenefx-git may install an updated
library version that swayfx-git depends on in the same update cycle.

The only potential risk with this change is that swayfx-git will break
when scenefx-git updates it's library version while not updating
swayfx-git. This results in a runtime issue where swayfx-git can't find
a specific version of the scenefx library.

ref: WillPower3309/swayfx#384

Signed-off-by: Darrel Griët <dgriet@gmail.com>
@MagneFire MagneFire force-pushed the fix/depend-on-git branch from aa5abe5 to 9bd361b Compare July 5, 2025 17:29
@ErikReider
Copy link
Member

The only potential risk with this change is that swayfx-git will break when scenefx-git updates it's library version while not updating swayfx-git. This results in a runtime issue where swayfx-git can't find a specific version of the scenefx library.

That is exactly why we haven't done this before. If we update scenefx to a new wlroots version, swayfx won't start.

Me and Will have discussed this in the Discord and the alternatives are to either hold back scenefx wlroots version until swayfx gets updated (may take months), or do as we do now

@MagneFire
Copy link
Author

The only potential risk with this change is that swayfx-git will break when scenefx-git updates it's library version while not updating swayfx-git. This results in a runtime issue where swayfx-git can't find a specific version of the scenefx library.

That is exactly why we haven't done this before. If we update scenefx to a new wlroots version, swayfx won't start.

Is this really an issue though? Take the last two scenefx updates (0.3 and 0.4). They were updated in the following commits:

While swayfx updated its dependencies in the following commits:

The last two library updates to scenefx also required an update to swayfx. This update is automatically installed after scenefx-git is updated because swayfx-git just so happens to always build whenever a new commit is pushed to the swayfx repository.

Hence my rationale on why this may not be a huge issue in the real world 😄

Also also, the current approach results in update conflicts (circular dependency) right? This means that manual intervention is required anyway. My goal here is to make the update process more seamless for the -git packages. For the non git version of swayfx this may not be relevant (this is in contradiction to my comments in the main issue).

Me and Will have discussed this in the Discord and the alternatives are to either hold back scenefx wlroots version until swayfx gets updated (may take months), or do as we do now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0