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

Use Amber.Mpris instead of org.nemomobile.mpris on newer SailfishOS releases #34

Closed
wants to merge 1 commit into from

Conversation

smokku
Copy link
Contributor

@smokku smokku commented Dec 17, 2023

Amber.Mpris is offered since SailfishOS 4.4.0.

Closes #98

@Olf0 Olf0 self-assigned this Feb 3, 2024
@Olf0 Olf0 added the enhancement New feature or request label Feb 3, 2024
@Olf0 Olf0 removed their assignment Feb 3, 2024
@Olf0
Copy link
Contributor

Olf0 commented Feb 3, 2024

@smokku, thank you very much for this contribution.

Currently, my first comment addressing your other contribution (PR #33) is also applicable here. I am sorry for the delay.

@Olf0
Copy link
Contributor

Olf0 commented Feb 24, 2024

@smokku, can you please consider implementing a switch as dcaliste did for resolving PR #33 (if technically feasible) or alternatively at least look up exactly which SailfishOS releases

  1. introduced Amber.Mpris
  2. omitted org.nemomobile.mpris

@smokku
Copy link
Contributor Author

smokku commented Feb 24, 2024

@smokku, can you please consider implementing

I tend to work in bursts, jumping from subject to subject.
I got bored with SailfishOS and I'm doing different stuff now.
I may get back and do some more stuff for Sailfish though…

@Olf0
Copy link
Contributor

Olf0 commented Feb 24, 2024

I tend to work in bursts, jumping from subject to subject.

Take your time, it is all spare time activity and should primarily be fun.

@Thaodan
Copy link

Thaodan commented May 8, 2024

I wonder if it is worth the effort to support Sailfish OS pre 4.0 when it adds so much overhead.
It doesn't look like it can be done easily (from a quick read).

@Olf0
Copy link
Contributor

Olf0 commented May 10, 2024

  1. Technically

    Well, the old, extant code is in this git repository, the new code is here in this PR, hence IMO it just takes a flag and a ifdef / ifndef construct to compile either code conditionally. The flag can be set based on sailfish-version in the spec-file. Examples for this construct are the SailfishOS:Chum GUI application and many of Slava Monich's apps, e.g. CodeReader.

    The exact SailfishOS release which brought this breaking change has to be determined any way in order to let FlowPlayer only install on appropriate SailfishOS releases. All release notes, one of which should document this change, are referenced here: Fix WebDAV + Thumbnail + Open + Share + port.text + another fix filecase#60 (comment)

    All in all this should be not be hard to implement, just a bit tedious to adapt this well known construct to FlowPlayer and to reference the original change in SailfishOS in Jolla's release notes or changelogs.

  2. Organisationally

    Jolla is free to regularly (with almost every minor SailfishOS release) break many third party SailfishOS apps (and sometimes also their own) with their disruptive changes (i.e. non backward compatible ones) and to destroy the remainder (or rather remnants) of the SailfishOS app ecosystem thereby. But I am tired of permanently adapting apps to these disruptive changes, although I have not ceased to maintain them completely as so many other developers already did; still I would rather enhance these apps instead of being kept busy by fixing the breakages Jolla causes.

    The biggest and worst fact in this context is that the most central of all software components to create apps nicely provides backward compatibility for all its minor releases: Qt But Jolla refuses to update exactly this core component and still deploys Qt 5.6, which was released 1. March 2016 and does not receive security updates any longer since 1. March 2019. Instead of updating Qt to 5.15, which should cause only minimal compatibility issues, Jolla loves to change many other, little things in SailfishOS in a disruptive manner, often without any necessity for disruptiveness or almost ever without announcing such breaking changes. Consequently SailfishOS has no future (since 2019) and I will not actively support Jolla's crazy course by fixing the breakages they cause, unless they switch to a Qt release which is still maintained (i.e. either 5.15 or 6.x); still I do review and integrate PRs which address any of these breakages in a backward compatible manner.

    The codebase of FlowPlayer currently supports every SailfishOS release since at least 2.0.0 (maybe also 1.1.9), I see no reason to destroy this feat. Do you?

@Thaodan
Copy link

Thaodan commented May 10, 2024 via email

@Olf0
Copy link
Contributor

Olf0 commented May 10, 2024

You received a clear answer to your question and additionally a detailed description of one way how to proceed: Feel free to pose a PR; I also promised that I will handle it.

Just read, don't rant.

@Thaodan

This comment was marked as abuse.

@Olf0
Copy link
Contributor

Olf0 commented May 12, 2024

I do no comprehend what you are trying to achieve with your personal attacks; obviously my comment hit a nerve.

Unfortunately you contribute nothing constructive in all your postings here, only try to command people outside of Jolla (here: me, again) what to do, as so often.
I hope you had your fun. 😉

@smokku smokku closed this by deleting the head repository May 28, 2024
@Olf0 Olf0 changed the title Use Amber.Mpris instead org.nemomobile.mpris Use Amber.Mpris instead of org.nemomobile.mpris on newer SailfishOS releases May 31, 2024
@Olf0
Copy link
Contributor

Olf0 commented May 31, 2024

From #34 (comment):

@smokku, can you please consider implementing a switch as dcaliste did for resolving PR #33 (if technically feasible) or alternatively at least look up exactly which SailfishOS releases

  1. introduced Amber.Mpris

  2. omitted org.nemomobile.mpris


  1. is SailfishOS 4.4.0.58, see https://forum.sailfishos.org/t/changelog-vanha-rauma-4-4-0/10655
    Edit: I updated the initial posting in this thread accordingly.
  2. org.nemomobile.mpris is not mentioned in the changelogs of SailfishOS 4.4.0.58, 4.5.0.16 and 4.6.0.11, hence it may be still valid and functional:
    This would need testing or a negative effect (missing functionality etc.) with FlowPlayer on SailfishOS ≥ 4.4.0 to be documented.

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

Successfully merging this pull request may close these issues.

[Suggestion] Use Amber.Mpris instead of org.nemomobile.mpris on newer SailfishOS releases
3 participants