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.

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

way-cooler: Remove #77625

Closed
wants to merge 1 commit into from
Closed

Conversation

@matthiasbeyer
Copy link
Contributor

@matthiasbeyer matthiasbeyer commented Jan 13, 2020

Motivation for this change

As of 2020-01-09, way-cooler is officially dead:

http://way-cooler.org/blog/2020/01/09/way-cooler-post-mortem.html

hence, remove the package and the module.


Pinging maintainers from the packages touched in this PR:

@gnidorah @miltador @primeos

As of 2020-01-09, way-cooler is officially dead:

    http://way-cooler.org/blog/2020/01/09/way-cooler-post-mortem.html

hence, remove the package and the module.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
@matthiasbeyer matthiasbeyer requested a review from Infinisil as a code owner Jan 13, 2020
@matthiasbeyer
Copy link
Contributor Author

@matthiasbeyer matthiasbeyer commented Jan 13, 2020

We should, of course, discuss whether this is what we want. But as we removed 'rtv' just a few days ago because it is no longer maintained, I'd guess that this is the way (not so cool) forward.

@Valodim
Copy link
Contributor

@Valodim Valodim commented Jan 13, 2020

The maintainer doesn't seem confident that anyone will step up to take over the project in that postmortem. The module looks extensive though - I wonder how many people's break if we remove it? Is there a way to display deprecation warnings for a grace period, and give it some time to see if someone steps up to maintain the project? It does have 2.5k github stars.

@matthiasbeyer
Copy link
Contributor Author

@matthiasbeyer matthiasbeyer commented Jan 13, 2020

From what I experienced during the removal (I do not say deprecation process here, please notice that!) of rtv, we do not have a process. I'd love to have one to follow, but we do not.

@gnidorah
Copy link
Contributor

@gnidorah gnidorah commented Jan 13, 2020

@Valodim
I'm for to remove it completely, since we now have a retirement letter from developer. It was an immature product, no one used it seriously. I already dropped orbment and velox wayland compositors for the same reason.
Also wlc library on which this product is based is deprecated for years now.
If someone will revive it and make it at least not worser quality than Sway, then I will re-add it.

Copy link
Contributor

@gnidorah gnidorah left a comment

Finally! Thank you! :)

@Valodim
Copy link
Contributor

@Valodim Valodim commented Jan 13, 2020

Sounds reasonable, users who use this will be forced to move somewhere else sooner or later regardless. Go for it 👍

@adisbladis
Copy link
Member

@adisbladis adisbladis commented Jan 13, 2020

@matthiasbeyer Can you please add a change log entry?

@matthiasbeyer
Copy link
Contributor Author

@matthiasbeyer matthiasbeyer commented Jan 13, 2020

Sure, where?

@matthiasbeyer
Copy link
Contributor Author

@matthiasbeyer matthiasbeyer commented Jan 13, 2020

Ah. No, I refuse to work with that XML garbage. Someone else can add it, I will cherry-pick the commit onto this PR.

@grahamc
Copy link
Member

@grahamc grahamc commented Jan 14, 2020

Closing until somebody does that.

@grahamc grahamc closed this Jan 14, 2020
@gnidorah gnidorah mentioned this pull request Jan 14, 2020
0 of 10 tasks complete
@matthiasbeyer matthiasbeyer deleted the matthiasbeyer:remove-way-cooler branch Jan 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.