audio/lastfm-desktop: Update to 2.1.36

[woodsb02]

Summary

audio/lastfm-desktop: Update to 2.1.36

• Move to official GitHub repo, since the official CDN is out of date
• Patch of LAV_Source.cpp renamed to follow file move, and updated to fix build with FFmpeg 3.0.x - obtained from ArchLinx [1]. Note that multimedia/ffmpeg is currently still on the 2.8.x branch, however this patch is backwards compatible with ffmpeg 2.8.x.
• Patch of RadioService.cpp updated - obtained from ArchLinux [2].
• Remove patch of podDevice_linux.cpp, as the change was upstreamed
• Add additional dependencies to LIB_DEPENDS and USE_QT4 as found by new stage-qa script.
• Ensure program will appear in desktop manager menus by installing icon and adding DESKTOP_ENTRIES.
• Add the audio playback run dependencies, with the option of either VLC or GSTREAMER. WIthout one of these, it crashes upon run.
• Remove the LD_LIBRARY_PATH, as it is no longer needed now that the lastfm-desktop binaries are statically linked to it's private libraries.
• Also do not install the *.a static libraries, as they are not used by any other programs/ports and not needed by the lastfm-desktop binaries.
• Install all of the files required by for the .css stylesheet to work.

[1] https://aur.archlinux.org/cgit/aur.git/tree/LAV_Source_fix.patch?h=lastfm
[2] https://aur.archlinux.org/cgit/aur.git/tree/cast-bug.patch?h=lastfm

Test Plan

poudriere testport audio/lastfm-desktop

[adamw]

Approved, if vg is happy.

The switch to *.a seems odd. Are there other ports that depend on this one? If so, their LIB_DEPENDS lines will have to be changed. And they'll need PORTREVISION bumps every time this port gets updated so that they can statically relink against the new version.

If there are not any ports that depend on this one, then why install statically linked libraries at all?

[mat]

Change PORTNAME=lastfm-desktop, and cleanup all the things that you were coercing it into doing the right thing.

Also, I'm not sure going with .a is the right thing to do, it makes it impossible to know what software really uses it.

[mat]

As a rule of the thumb, when you use USE_GITHUB, and you have to set WRKSRC, it means you're doing it wrong.

[woodsb02]

As per comment from mat, change PORTNAME=lastfm-desktop, and cleanup all the
things that you were coercing it into doing the right thing.

Regarding the switch from *.so to *.a, this was not a change in the port, but
a change made by upstream here:
https://github.com/lastfm/lastfm-desktop/commit/a6d2225e368aaeb66ec5345717120013b40f2518

There are no other ports that depend on this one or use these shared libraries,
and the libraries are installed in DATADIR rather than lib/, meaning they are
intended to be private for lastfm-scrobbler binary.

[mat]

I don't understand the LD_LIBRARY_PATH thing, there are no .so installed any more, so nothing the dynamic linker will need to load.

[woodsb02]

• As per comment from mat, the LD_LIBRARY_PATH is no longer needed now that the lastfm-desktop binaries are statically linked to it's private libraries.
• Also do not install the *.a static libraries, as they are not used by any other programs/ports and not needed by the lastfm-desktop binaries.
• Lastly, install all of the files required by for the .css stylesheet to work.

[woodsb02]

vg: any comment on this one? It's been 2 weeks for maintainer timeout. Thanks :)

[adamw]

You're free to commit this under the maintainer timeout.