Bug 1965672 - Review Request: heroic-games-launcher - Heroic is an Open Source Game Launcher for Linux
Summary: Review Request: heroic-games-launcher - Heroic is an Open Source Game Launche...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-05-28 19:42 UTC by Rémi Lauzier
Modified: 2021-05-29 18:52 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-05-29 18:52:10 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Comment 1 Rémi Lauzier 2021-05-28 19:47:06 UTC
i have done the maximum i was able to do but the package as some flaws i can't correct for now.
It need internet access during build and the program crash immediately.
I am not even sure if it's really a good idea to package it for fedora.
If somebody with better experience packaging yarn program can help thanks.

Comment 3 Robert-André Mauchin 🐧 2021-05-29 18:44:09 UTC
 - Provides: bundled(libffmpeg)

This is not allowed. ffmpeg is covered by patents which makes it not packageable in Fedora. FFMPEG is packaged in RPMFusion instead. If you want to package it, propose it to RPMFusion.

Also, requiring internet access during build is not ok, it is not available in Koji.


 - You would need to unbundle all of this:

Provides: bundled(libffmpeg)
Provides: bundled(libEGL)
Provides: bundled(libGLESv2)
Provides: bundled(libvk_swiftshader)


 - Generally, electron apps are not packaged in Fedora due to the huge mess it is.

 - Atim packaged it in https://copr.fedorainfracloud.org/coprs/atim/heroic-games-launcher/package/heroic-games-launcher-bin/ based off the official RPM package, so not bothering with electron in its spec. However since it includes FFMPEG, it is not allowed in COPR too.

Comment 4 Rémi Lauzier 2021-05-29 18:52:10 UTC
I have tried making it work without ffmpeg and it didn't work, and it need internet access to download a thousand of dependencies, so i will closed the review. packaging electron apps is too hard for me.
Will use flatpak or something else instead.
Thanks for your review and your time.


Note You need to log in before you can comment on or make changes to this bug.