flathub / com.slack.Slack

https://flathub.org/apps/details/com.slack.Slack
36 stars 36 forks source link

"Launch on login" does not work #170

Closed pgalbraith closed 2 years ago

pgalbraith commented 2 years ago

The "launch on login" option can be selected, but it does not work. Upon logging in the app does not start, and upon starting the app again the setting for launch on login has been lost.

FakeShemp commented 2 years ago

Are you using the flatpak version? This option cannot work on the flatpak version.

WhyNotHugo commented 2 years ago

Slack would need to ship a systemd unit for this to work, but it cannot do this via flatpak: https://github.com/flatpak/flatpak/issues/2787

FWIW, this doesn't work on non-flatpak versions either, due to the lack of a standard mechanism that would work on Linux desktop in general. Some DEs use autostart desktop entries, while many services rely on systemd's user units.

Just configure this via your DE settings (or your init script). Slack should probably remove the checkbox entirely and let users configure it via their desktop anyway.

pgalbraith commented 2 years ago

@WhyNotHugo this works fine on my desktop if I install the deb from the slack website. While you may not like they way they wrote their app, and you may have very justifiable reasons for that opinion, I don't see how that translates into a supportable position that their implementation is defective and you can arbitrarily change the functionality you don't like and still label it as the same application.

WhyNotHugo commented 2 years ago

Like I said, there's no single standard way to implement "auto start on login" on Linux desktop. Whatever mechanism Slack's deb implements is also supported by your desktop, but it's no supported by Flatpak.

We can agree to disagree on opinions on the topic, but there's nothing that can be done for this package.You need to ask the guys over at https://github.com/flatpak/flatpak to implement a way for applications to auto-start. Like https://github.com/flatpak/flatpak/issues/2787

pgalbraith commented 2 years ago

@WhyNotHugo when there's a functionality issue that's blocked by an upstream issue, the usual way to handle that is to acknowledge the problem with a flagged issue, and refer to the upstream changes that are needed to fix the problem. Simply denying that there's a problem doesn't help anyone.

pgalbraith commented 2 years ago

I'm definitely getting the impression that there's some nefarious practices going on somewhere, this is not the first flatpak app I've had issues with, nor the first where a dev has tried to pretend that a difficult-to-fix problem must not be a problem at all. All this pretending that there aren't issues with flatpak apps is really painting a bad picture of the whole thing.

FakeShemp commented 2 years ago

We're not saying it's not an issue. We're saying it's an issue that is literally impossible to fix.

pgalbraith commented 2 years ago

If you acknowledge that this is functionality that works in the upstream app, and does not work in this package, then it's a packaging issue that should be acknowledged formally by documenting it here and pointing out that it is difficult to fix because it requires upstream work and collaboration, as @WhyNotHugo has already pointed out. It is not impossible, it's just difficult.

Closing this issue very loudly says that you don't believe it's an issue, and that attitude speaks very poorly of this entire project. If the upstream developers realized that there were 150 issues caused that required this upstream functionality, don't you think that's important information? Closing this issue is dishonest, and unhelpful.

FakeShemp commented 2 years ago

You are correct. I don't believe this is an issue. I believe you are the issue.

Feel free to be the one taking over maintaining this Slack package cause I sure don't want to do it any more.

pgalbraith commented 2 years ago

Then you probably shouldn't be doing it.

GerardoGR commented 2 years ago

Sorry for Necro-bumping this issue. But I stumbled upon this issue looking for an answer and the solution I found might work for someone else: Create a com.slack.Slack.desktop file under the autostart directory:

Thank you @FakeShemp and @WhyNotHugo for maintaining this flatpak!