Page MenuHomeFreeBSD

Update multimedia/emby-server to 4.0.2.0
ClosedPublic

Authored by driesm on Feb 8 2019, 7:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 20, 8:53 PM
Unknown Object (File)
Tue, Mar 12, 8:08 AM
Unknown Object (File)
Tue, Mar 12, 8:08 AM
Unknown Object (File)
Tue, Mar 12, 8:07 AM
Unknown Object (File)
Tue, Mar 12, 8:07 AM
Unknown Object (File)
Tue, Mar 12, 8:04 AM
Unknown Object (File)
Tue, Mar 12, 8:04 AM
Unknown Object (File)
Tue, Mar 12, 8:04 AM

Details

Summary

This patch updates emby-server to the latest and greatest version which brings multiple improvements.
Release Notes: https://emby.media/community/index.php?/blog/1/entry-447-announcing-emby-server-40/

This patch also unbreaks the current emby-server port which experiences shared library errors.
Shared library errors: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=232966

Test Plan

QA checked:
portlint -ac
poudriere testport -j head
poudriere testport -j stable12
poudriere testport -j stable11

Emby-server 4.0.2.0 has been running fine on my system based on the provided patch.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jbeich added subscribers: multimedia, jbeich.

multimedia doesn't maintain this port. Redirecting to the larger pool of ports/ committers.

tcberner added inline comments.
multimedia/emby-server/Makefile
8

^ you could probably get away with setting DISTVERSIONPREFIX=netframework_

10

^can't you use the ffmpeg from ports?

multimedia/emby-server/files/ImageMagickSharp.dll.config.in
3

^ this should probably be called %%LOCALBASE%%

multimedia/emby-server/files/SQLitePCLRaw.provider.sqlite3.dll.config.in
3

^same

multimedia/emby-server/pkg-message
1–2

^ why not just add lame as a fixed dependency / make it default on?

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

multimedia/emby-server/Makefile
8

Mmm, but PORTNAME is emby-server, and DISTNAME as embyserver, it would not change a lot, except defining one more variable.

52

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

(Should go in the USES block)

73–104

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

(Should go before options)

108–110

For single files, please use the INSTALL_* macros.

multimedia/emby-server/pkg-message
1–2

Probably because of license. Either the port has a package, or it has LAME on and has no package, so it still has to be built manually.

driesm marked an inline comment as done.

Address comments from mat & tcberner

driesm added inline comments.
multimedia/emby-server/Makefile
10

Emby uses a customized ffmpeg with local patches to provide features and tailor it for Emby. For example, transcoding throttling which is a big feature for this release. I asked if they can upstream as much as possible, as it would be a lot cleaner if we could depend on the ffmpeg in our tree. Although, I don't see this happening in the short term.

FYI: https://emby.media/community/index.php?/topic/12906-freebsd-package/?p=677067

multimedia/emby-server/pkg-message
1–2

Exactly this!

multimedia/emby-server/Makefile
9

This should be ${DISTNAME}.zip.

12

This file name is too generic, please use DIST_SUBDIR.

13

DISTNAME here too.

61

Are you sure about this one?

63–64

Why +=?

  • Address mat comments
  • Remove unneeded clutter in rc script
  • Streamline ffmpeg configure args

Rechecked QA with poudriere testport and portlint.

multimedia/emby-server/Makefile
61

I have tried building the ffmpeg without this set but then it fails to build.

lwhsu added a subscriber: lwhsu.
lwhsu added inline comments.
multimedia/emby-server/Makefile
1

@woodsb02 Ben, are you OK for removing this line?

multimedia/emby-server/Makefile
47

Is tar:xz needed?

multimedia/emby-server/Makefile
1

Yes

Remove unneeded use of tar:xz.

driesm added inline comments.
multimedia/emby-server/Makefile
47

Good catch! Its not needed anymore; updated diff accordingly.

multimedia/emby-server/files/emby-server.in
26

PREFIX (Actually, you copied the typo.)

driesm marked an inline comment as done.

Fix typo's in RC script.

multimedia/emby-server/Makefile
66

No need for += either.

Remove unneeded += for configure args.

driesm retitled this revision from Update multimedia/emby-server to 4.0.1.0 to Update multimedia/emby-server to 4.0.2.0.
  • Upstream released Emby-server 4.0.2.0 so update to latest version
  • Link ffmpeg with x265
  • Remove X11 as option (as it defaulted to on anyway and USE_XORG=x11 is set)

Retested QA with poudriere testport and portlint -ac.

driesm edited the test plan for this revision. (Show Details)

Re-add X11 option as there is interest for this in the community and an extra option doesn't hurt anyway.
Also handy for people wanting to keep their system clean of X11 and extra libraries.

This is the final diff from my part. If a committer could go over it one last time :).
Thanks to everyone who has helped getting this reviewed!

QA checked:
Poudriere testport
Portlint -ac

Remove trailing slash in LIB_DEPENDS

I think this is fine. @mat , I'm going to commit this patch, are you ok with this?

This revision is now accepted and ready to land.Feb 26 2019, 12:48 PM
This revision was automatically updated to reflect the committed changes.