Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Change all references from sway, to swayfx #191

Open
sonjiku opened this issue Jun 28, 2023 · 4 comments · May be fixed by #257
Open

Change all references from sway, to swayfx #191

sonjiku opened this issue Jun 28, 2023 · 4 comments · May be fixed by #257
Labels
enhancement New feature or request

Comments

@sonjiku
Copy link

sonjiku commented Jun 28, 2023

Hey,

I wanna try this fork on fedora 38. Installing this though, means I have to uninstall sway, sway-config-upstream, and a lot of other dependencies and unused packages that the official sway pkg relies on, and I need.

The official sway maintainers have stated multiple times that they don't plan on implementing the features you have added, and don't plan to merge a fork like this.

I suggest just renaming all references in code from "sway" to "swayfx", or any name of your choosing. This will mean that people using sway as their daily driver, don't have to uninstall it just to try this project out. and both compositors can live happily in a system.
@sonjiku sonjiku added the enhancement New feature or request label Jun 28, 2023
@ErikReider
Copy link
Collaborator

What you could do in the meantime is to build it manually and run that executable in a tty while still having sway installed.

git clone https://github.com/WillPower3309/swayfx.git swayfx && cd swayfx
meson build -Dwerror=false
meson compile -C build

And run ./build/sway/sway in a tty.

@werdahias
Copy link

werdahias commented Sep 23, 2023 •
edited

Probably related: I looked into building a debian package for swayfx, but the binaries and manpages conflict with the regular sway package. I could hack around this by renaming /usr/bin/sway to /usr/bin/swayfx and so on but it would really simplify adaption if all refererences to sway were renamed. See also the official debian reference here.

@MrPenguin07
Copy link

I concur with renaming all references to sway, allowing both to be installed at the same time - with separate configs for both.

Sway have made their bed re. hard refusal to modernize and accomodate a growing number of users wanting simple effects, this fork would benefit from being more independent.

@bhepple
Copy link

bhepple commented Nov 17, 2023

I came here to say exactly this - thanks to OP for the fluent report.

@notpeelz notpeelz linked a pull request Dec 26, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change all sway references to swayfx notpeelz/swayfx
5 participants
@bhepple @werdahias @ErikReider @sonjiku @MrPenguin07