* [gentoo-dev] TIDAL ebuild
@ 2024-12-24 14:11 Daniel Buschke
2024-12-24 14:26 ` Filip Kobierski
2024-12-24 15:17 ` Lucio Sauer
0 siblings, 2 replies; 4+ messages in thread
From: Daniel Buschke @ 2024-12-24 14:11 UTC (permalink / raw
To: gentoo-dev
Hi all,
TIDAL is a music streaming service. As I missed a TIDAL desktop app on
gentoo I created an ebuild based on
https://github.com/Mastermindzh/tidal-hifi.
It's a node js application which drove me a bit crazy. I was today years
old when I realized that the gentoo emerge sandbox is cutting network
access :)
But the node js application, as every node js app, heavily relies on
dependencies, so I decided to go with the binary version.
Anyway, as I am not very firm with ebuilds it would be nice if someone
could do a review before I try to commit this ebuild to the official
portage via bugs.gentoo.org. Anyone? Thanks in advance.
https://source.devloop.de/damage/gentoo/src/branch/main/media-sound/tidal-hifi-bin
Stay healthy, everyone out there.
regards
Daniel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [gentoo-dev] TIDAL ebuild
2024-12-24 14:11 [gentoo-dev] TIDAL ebuild Daniel Buschke
@ 2024-12-24 14:26 ` Filip Kobierski
2024-12-24 14:31 ` Filip Kobierski
2024-12-24 15:17 ` Lucio Sauer
1 sibling, 1 reply; 4+ messages in thread
From: Filip Kobierski @ 2024-12-24 14:26 UTC (permalink / raw
To: gentoo-dev
[-- Attachment #1.1: Type: text/plain, Size: 1748 bytes --]
Hi Daniel,
I am no ebuild-master myself but I think I have some feedback;
- since it's a -bin package make sure to have -* in KEYWORDS
- use desktop.eclass[1]:
- domenu for the desktop file
- doicon for the icon
- also I would use dobin for installing binaries
- I would use ${PN} where possible (besides SRC_URI & HOMEPAGE
On the name though, is the -bin really nescessary if there is no
source option? On one hand we have net-im/discord but on the other
there's net-im/discord-canary-bin.
Another thing is the -hifi part. For me it implies that before
there was some non-hifi version. If that's the case than that is
fine to me.
I however would name the package tidal-desktop or just tidal.
I hope that this helps
Filip Kobierski
[1]: https://devmanual.gentoo.org/eclass-reference/desktop.eclass/
On Tuesday, December 24th, 2024 at 15:11, Daniel Buschke <damage@devloop.de> wrote:
> Hi all,
> TIDAL is a music streaming service. As I missed a TIDAL desktop app on
> gentoo I created an ebuild based on
> https://github.com/Mastermindzh/tidal-hifi.
>
> It's a node js application which drove me a bit crazy. I was today years
> old when I realized that the gentoo emerge sandbox is cutting network
> access :)
> But the node js application, as every node js app, heavily relies on
> dependencies, so I decided to go with the binary version.
>
> Anyway, as I am not very firm with ebuilds it would be nice if someone
> could do a review before I try to commit this ebuild to the official
> portage via bugs.gentoo.org. Anyone? Thanks in advance.
>
> https://source.devloop.de/damage/gentoo/src/branch/main/media-sound/tidal-hifi-bin
>
> Stay healthy, everyone out there.
>
> regards
> Daniel
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [gentoo-dev] TIDAL ebuild
2024-12-24 14:26 ` Filip Kobierski
@ 2024-12-24 14:31 ` Filip Kobierski
0 siblings, 0 replies; 4+ messages in thread
From: Filip Kobierski @ 2024-12-24 14:31 UTC (permalink / raw
To: gentoo-dev
[-- Attachment #1.1: Type: text/plain, Size: 2188 bytes --]
Errata:
I have just looked up the upstream and I think the name is fine.
However I would mention somewhere that it's a third party app.
(AFAIR discord also is an electron app and 3rd party client
note that they are not official)
On Tuesday, December 24th, 2024 at 15:26, Filip Kobierski <fkobi@pm.me> wrote:
> Hi Daniel,
>
> I am no ebuild-master myself but I think I have some feedback;
> - since it's a -bin package make sure to have -* in KEYWORDS
> - use desktop.eclass[1]:
> - domenu for the desktop file
> - doicon for the icon
> - also I would use dobin for installing binaries
> - I would use ${PN} where possible (besides SRC_URI & HOMEPAGE
>
> On the name though, is the -bin really nescessary if there is no
> source option? On one hand we have net-im/discord but on the other
> there's net-im/discord-canary-bin.
> Another thing is the -hifi part. For me it implies that before
> there was some non-hifi version. If that's the case than that is
> fine to me.
> I however would name the package tidal-desktop or just tidal.
>
>
> I hope that this helps
> Filip Kobierski
>
> [1]: https://devmanual.gentoo.org/eclass-reference/desktop.eclass/
>
> On Tuesday, December 24th, 2024 at 15:11, Daniel Buschke damage@devloop.de wrote:
>
> > Hi all,
> > TIDAL is a music streaming service. As I missed a TIDAL desktop app on
> > gentoo I created an ebuild based on
> > https://github.com/Mastermindzh/tidal-hifi.
> >
> > It's a node js application which drove me a bit crazy. I was today years
> > old when I realized that the gentoo emerge sandbox is cutting network
> > access :)
> > But the node js application, as every node js app, heavily relies on
> > dependencies, so I decided to go with the binary version.
> >
> > Anyway, as I am not very firm with ebuilds it would be nice if someone
> > could do a review before I try to commit this ebuild to the official
> > portage via bugs.gentoo.org. Anyone? Thanks in advance.
> >
> > https://source.devloop.de/damage/gentoo/src/branch/main/media-sound/tidal-hifi-bin
> >
> > Stay healthy, everyone out there.
> >
> > regards
> > Daniel
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [gentoo-dev] TIDAL ebuild
2024-12-24 14:11 [gentoo-dev] TIDAL ebuild Daniel Buschke
2024-12-24 14:26 ` Filip Kobierski
@ 2024-12-24 15:17 ` Lucio Sauer
1 sibling, 0 replies; 4+ messages in thread
From: Lucio Sauer @ 2024-12-24 15:17 UTC (permalink / raw
To: gentoo-dev
[-- Attachment #1: Type: text/plain, Size: 1126 bytes --]
On Tue, Dec 24, 2024 at 03:11:40PM +0100, Daniel Buschke wrote:
> Hi all,
Hi,
thanks for your contribution!
> Anyway, as I am not very firm with ebuilds it would be nice if someone could
> do a review before I try to commit this ebuild to the official portage via
> bugs.gentoo.org. Anyone? Thanks in advance.
>
media-sound/tidal-hifi-bin has already been packaged since last year in
the GURU overlay[1]. The linked webpage outlines how to install
packages offered in GURU and how to request access to the server to help
with development, if you're interested.
When looking to get your ebuilds reviewed, you're better off asking in
the #gentoo-dev-help channel on IRC or on the gentoo-devhelp mailing
list. Issues raised on the gentoo-dev ML generally affect a larger user
base, e.g. eclass changes, changes to global USE flags, new tools for
ebuild development.
> https://source.devloop.de/damage/gentoo/src/branch/main/media-sound/tidal-hifi-bin
>
> Stay healthy, everyone out there.
>
> regards
> Daniel
>
[1]: https://wiki.gentoo.org/wiki/Project:GURU
Best,
--
Lucio Sauer
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 618 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-12-24 15:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-24 14:11 [gentoo-dev] TIDAL ebuild Daniel Buschke
2024-12-24 14:26 ` Filip Kobierski
2024-12-24 14:31 ` Filip Kobierski
2024-12-24 15:17 ` Lucio Sauer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox