Comment # 1 on bug 4361 from Nicolas Chauvet

Few comments:
- Does it really uses the bundled fdk-aac instead of the internal ffmpeg aac
encoder ?
# Use system libraries in place of bundled ones
for module in a52dec %{?_with_fdk:fdk-aac}
...
- We cannot rely on this currently (we might need to restart the discussion)
At least it should be dlopenable.
Requires:       libdvdcss%{_isa}

- What is the point to have -gui -tui separation without any main package ?
I would use handbrake (main) as the default (-gui) package and use -core or
keep -tui as a sub-package.
(even using a virtual provide for lower case handbrake compatibility if not
handled by dnf already).

- Using , %make_build, %make_install, missing BR gcc, gcc-c++

- Remove dependency that are in the default BR (m4, patch, Makefile, etc)

- No appdata ?

Thx for this submit.