FS#42015 - [ffmpeg] cleanup flags in PKGBUILD

Attached to Project: Arch Linux
Opened by ux (ubitux) - Thursday, 18 September 2014, 18:40 GMT
Last edited by Bartłomiej Piotrowski (Barthalion) - Friday, 19 September 2014, 15:33 GMT
Task Type General Gripe
Category Packages: Extra
Status Closed
Assigned To Bartłomiej Piotrowski (Barthalion)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

• FFmpeg has hundreds of codecs, there is -and there never was- any reason to explicit the atrac3[p] decoders since it is not an external library. These 2 lines can be dropped:
--enable-decoder=atrac3 \
--enable-decoder=atrac3p \

• Archlinux probably doesn't care about DirectX Video Acceleration. The following line can be dropped:
--enable-dxva2 \

• Enabling explicit Position-independent code is not useful. FFmpeg does that by default when it matters, typically when --enable-shared is set. The following line can be dropped:
--enable-pic \

• libpostproc is enabled by default since a very long time, the following line can be dropped:
--enable-postproc \

By the way, FFmpeg 2.4 has been released, so you will probably want to add --enable-fribidi so the drawtext filter can be used to write correctly Arabic text.
This task depends upon

Closed by  Bartłomiej Piotrowski (Barthalion)
Friday, 19 September 2014, 15:33 GMT
Reason for closing:  Fixed
Additional comments about closing:  ffmpeg 1:2.4-2
Comment by ux (ubitux) - Thursday, 18 September 2014, 18:44 GMT
(I meant --enable-libfribidi sorry)
Comment by Doug Newgard (Scimmia) - Thursday, 18 September 2014, 18:49 GMT
You want to restate your request in light of the new ffmpeg 2.4 PKGBUILD from 3 days ago?

https://projects.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/ffmpeg&id=f917515cce4a19096faa4f33d92b735a6f6d6fb1
Comment by ux (ubitux) - Thursday, 18 September 2014, 18:58 GMT
Ah, I'm sorry. I actually had a talk with Bartłomiej Piotrowski and he asked me to open a task for this, and I didn't check the Git since then.

I guess, only --enable-pic and --enable-libfribidi potentially stand now.
Comment by Bartłomiej Piotrowski (Barthalion) - Thursday, 18 September 2014, 19:50 GMT
I'll remove them during next rebuild/upgrade, thanks.

Loading...