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

Update dependencies, fix finish args & update appstream metadata #329

Merged
merged 3 commits into from
Feb 2, 2024

Conversation

flathubbot
Copy link
Contributor

🤖 This pull request was automatically generated by flathub-infra/flatpak-external-data-checker. Please open an issue if you have any questions or complaints. 🤖

@flathubbot
Copy link
Contributor Author

Started test build 96702

@flathubbot
Copy link
Contributor Author

Build 96702 failed

@travier
Copy link
Member

travier commented Jan 30, 2024

{
    "errors": [
        "appstream-missing-developer-name"
    ],
    "warnings": [
        "appstream-screenshot-missing-caption",
        "finish-args-contains-both-x11-and-wayland"
    ]
}

@flathubbot
Copy link
Contributor Author

Started test build 96710

@flathubbot
Copy link
Contributor Author

Build 96710 failed

@flathubbot
Copy link
Contributor Author

Started test build 96718

@flathubbot
Copy link
Contributor Author

Build 96718 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/79477/org.kde.kdenlive.flatpakref

flathubbot and others added 2 commits February 2, 2024 02:12
Update certifi-2023.11.17-py3-none-any.whl to 2024.2.2
Update urllib3-2.1.0-py3-none-any.whl to 2.2.0
Update intel-onevpl-24.1.1.tar.gz to 24.1.2
Update libmediainfo_23.11.tar.xz to 24.01
Update mediainfo_23.11.tar.xz to 24.01
@flathubbot
Copy link
Contributor Author

Started test build 97386

@travier travier changed the title Update intel-onevpl-24.1.1.tar.gz to 24.1.2 Update dependencies, fix finish args & update appstream metadata Feb 2, 2024
@flathubbot
Copy link
Contributor Author

Started test build 97387

@flathubbot
Copy link
Contributor Author

Build 97386 was cancelled

@flathubbot
Copy link
Contributor Author

Build 97387 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/80163/org.kde.kdenlive.flatpakref

@flathubbot
Copy link
Contributor Author

Started test build 97419

@flathubbot
Copy link
Contributor Author

Build 97419 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/80195/org.kde.kdenlive.flatpakref

@travier travier merged commit ca976c8 into master Feb 2, 2024
1 check passed
@travier travier deleted the update-abac658 branch February 2, 2024 19:01
@jakubklos77
Copy link

There is a regression in this build on flathub:

  1. It shows "INVALID" for any title clip
  2. It does not render effects (fade in)
    It seems the render is much faster than before which is suspicious.

If I revert on flatpak to the previous version it all works.

I assume this is this very pull request which does not work:

    Commit: 42442e91b668dd75c05b3fc202d9d6dc18dbd7af9cf1227535ae26b10bb89072
   Subject: Add AppStream metadata patch (ca976c86)
      Date: 2024-02-02 21:40:04 +0000

And this is the previous one which works properly:

    Commit: 82dab8d18d7706a9197d301acf21da9cbaafe82a36fa3512dc931028386467b8
   Subject: Update r57.tar.gz to 58 (25e3eab9)
      Date: 2024-01-25 23:29:58 +0000

Although the changes in this pull request are little and do not indicate anything redarding rendering, the issue is there 100%.

Reproduction:
Place any video and add a 1sec fade in effect. Render. The video will not have any effect.

Thank you

@travier
Copy link
Member

travier commented Feb 5, 2024

OK, thanks for the report. I'll revert the dependency updates. Can you file an issue so that we can track testing this? Thanks

@travier
Copy link
Member

travier commented Feb 5, 2024

Once this is built, can you test the build from #334 ? Thanks

@jakubklos77
Copy link

OK, thanks for the report. I'll revert the dependency updates. Can you file an issue so that we can track testing this? Thanks

Done, thank you
#335

@jakubklos77
Copy link

jakubklos77 commented Feb 5, 2024

Once this is built, can you test the build from #334 ? Thanks

Tested and it still does not work, strange huh? If I update to the previous commit, it all works properly...

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.

3 participants