Ryan Foster

Results 601 comments of Ryan Foster

> > Also switches curl to being a static library. > > Unfortunately, OBS Studio is not the only binary relying on curl, some plugins do. So building statically is...

> Just to leave some notes on where this PR is at: > * libcurl's cmake is annoying because it exports with absolute paths to dependent libraries (nghttp2/brotli) > *...

Can this be merged before #190?

Let's cut down CI's workload then.

> This needs to be rebased and merge #227 in. @PatTheMav and I work closely together off-thread. Rest assured, that we discuss this regularly. However, do note that this is...

> > together off-thread. Rest assured, that we discuss this regularly. > > May I join the conversation? I appreciate the enthusiasm, but these are conversations that we have either...

Was just skimming back over this. > * `x264` requires an update of the `gas-preprocessor` it ships with - luckily `FFmpeg` provides a very recent version of it (the version...

As an aside, I noticed that this PR adds `deps.ffmpeg/patches/libpng/0001-enable-ARM-NEON-optimisations-windows.patch`, but this patch already exists at `deps.ffmpeg/patches/libpng/0001-enable-ARM-NEON-optimisations.patch`, though it is not currently used in the libpng script.

> Was just skimming back over this. > > > * `x264` requires an update of the `gas-preprocessor` it ships with - luckily `FFmpeg` provides a very recent version of...

> > As an aside, I noticed that this PR adds `deps.ffmpeg/patches/libpng/0001-enable-ARM-NEON-optimisations-windows.patch`, but this patch already exists at `deps.ffmpeg/patches/libpng/0001-enable-ARM-NEON-optimisations.patch`, though it is not currently used in the libpng script. >...