Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: "Helmut K. C. Tessarek" <tessarek@evermeet.cx>
To: Ridley Combs <rcombs@rcombs.me>, ffmpeg-devel <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] commits ecee6af8bd and 4ac869ca2a break build for macOS
Date: Mon, 20 Dec 2021 18:49:47 -0500
Message-ID: <fd22753b-35bf-6a3f-c82a-c48dab21dea0@evermeet.cx> (raw)
In-Reply-To: <B9999B73-0D3C-49B0-9C11-8F6134399907@rcombs.me>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Thanks for the rwply.

On 2021-12-20 18:25, Ridley Combs wrote:
> Could you try with this patch and let me know if it works for you then? 
> https://gist.github.com/7362df8a689c41b9174ac50fdeda3e43

I've applied your patch and get the following now:

CC	libavfilter/scale_eval.o
CC	libavfilter/scene_sad.o
libavfilter/metal/vf_yadif_videotoolbox.metal:47:1: error: constant sampler
must be declared constexpr
const sampler s(coord::pixel);
^
constexpr
libavfilter/metal/vf_yadif_videotoolbox.metal:122:14: error: use of
undeclared identifier 'max3'
    T diff = max3(tdiff0, tdiff1, tdiff2);
             ^
libavfilter/metal/vf_yadif_videotoolbox.metal:152:9: note: in instantiation
of function template specialization 'temporal_predictor<float>' requested here
        temporal_predictor(A.x, B.x, C.x, D.x, E.x, F.x,
        ^
libavfilter/metal/vf_yadif_videotoolbox.metal:125:18: error: use of
undeclared identifier 'max3'
        T maxi = max3(p2 - p3, p2 - p1, min(p0 - p1, p4 - p3));
                 ^
libavfilter/metal/vf_yadif_videotoolbox.metal:126:18: error: use of
undeclared identifier 'min3'
        T mini = min3(p2 - p3, p2 - p1, max(p0 - p1, p4 - p3));
                 ^
libavfilter/metal/vf_yadif_videotoolbox.metal:217:12: error: no matching
function for call to 'temporal_predictor'
    return temporal_predictor(A, B, C, D, E, F, G, H, I, J, K, L,
           ^~~~~~~~~~~~~~~~~~
libavfilter/metal/vf_yadif_videotoolbox.metal:233:16: note: in instantiation
of function template specialization 'yadif_compute_temporal<float>'
requested here
        return yadif_compute_temporal(cur, prev, cur, next, next,
spatial_pred, params.skip_spatial_check, pos);
               ^
libavfilter/metal/vf_yadif_videotoolbox.metal:265:23: note: in instantiation
of function template specialization 'yadif<float>' requested here
        pred = float2(yadif<float>(dst, prev, cur, next, params, pos));
                      ^
libavfilter/metal/vf_yadif_videotoolbox.metal:108:3: note: candidate
template ignored: substitution failure [with T = float]
T temporal_predictor(T A, T B, T C, T D, T E, T F,
  ^
5 errors generated.
make: *** [libavfilter/metal/vf_yadif_videotoolbox.metal.air] Error 1
make: *** Waiting for unfinished jobs....
libavfilter/metal/utils.m:33:21: warning: instance method '-supportsFamily:'
not found (return type defaults to 'id') [-Wobjc-method-access]
        if ([device supportsFamily:MTLGPUFamilyCommon3]) {
                    ^~~~~~~~~~~~~~
libavfilter/metal/utils.m:33:36: error: use of undeclared identifier
'MTLGPUFamilyCommon3'
        if ([device supportsFamily:MTLGPUFamilyCommon3]) {
                                   ^
1 warning and 1 error generated.
make: *** [libavfilter/metal/utils.o] Error 1
Error compiling ffmpeg...

> If you can test actual deployment on a 10.9 or 10.10 system that'd be
> great.

Unfortunately I don't have a 10.9 or 10.10 system. I compile on macOS 1.14
with export MACOSX_DEPLOYMENT_TARGET=10.9

Cheers,
  K. C.

- -- 
regards Helmut K. C. Tessarek              KeyID 0x172380A011EF4944
Key fingerprint = 8A55 70C1 BD85 D34E ADBC 386C 1723 80A0 11EF 4944

/*
   Thou shalt not follow the NULL pointer for chaos and madness
   await thee at its end.
*/
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEE191csiqpm8f5Ln9WvgmFNJ1E3QAFAmHBFpsACgkQvgmFNJ1E
3QDapA//WmxnoAqIGCqliaSIUpRkL/KxhSpj9ULJzL1h9yw6Ls0DIYT39Pa66hTk
IABKgLiTNxe1hNTC9kbXf6HdNqJw7KGGUlCGu777RHhh7wySq355g7b89f1wrSJa
1RwCkb9EoyC/DzeSnayDk6oOO7duJxCibkdTXEsYW9VgA9QGSriTHIRSG666UgXO
NC6EyjiOfGqlVkTr9Ott+r2t30H/0faihttbIhLwV6y/Dd/vx4mdG4y61B0PXLX1
rAQwDP9GYkAa302XLeLSTTJ/IHzsV7xD1eE+I3QcB63gbzKlujfbSBiOf9Czv6Vs
M1xoRbcqYkTi7zK45t7w+FuAMn/her/P5cCjmXbEQIHBZEfX3r2MKtkdIAhHsUPl
vDvR7Zmi90gRorQnyQ1fy/LaIcZt6E6CFPKQrIVkBsOnIuXvnrx4fxsvo4rYO32p
goyhtmBMyg9zIZy4eSUn/3M5NKMifgSQrYdUIIKrJZr4ENesKTGB/EZXS30v5VF4
7GrObWDT1oR+VoUiMSYrGPSBlF8R/NHJGU/BL7iVFtF7CfFRnHmfgzyII8RlS6G+
V5stD8eKNr/8fzI3sjQfiROEMtLFKJz4Zkf6qmBT7zeDizNJp/xaGc2egsC7LMzX
or7vCtN6YKPXhQimhsrxDvIXLdvt2ZsCwcbNpq5ZzEMi8mDgwCs=
=DJOy
-----END PGP SIGNATURE-----
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  reply	other threads:[~2021-12-20 23:49 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-20 21:30 Helmut K. C. Tessarek
2021-12-20 23:25 ` Ridley Combs
2021-12-20 23:49   ` Helmut K. C. Tessarek [this message]
2021-12-21  0:06     ` Ridley Combs
2021-12-21  0:13       ` Helmut K. C. Tessarek
2021-12-21  0:11 ` Christopher Degawa
2021-12-21  0:38   ` Aman Karmani
2021-12-21  0:43     ` Christopher Degawa
2021-12-21  0:50       ` Aman Karmani
2021-12-21  0:54         ` Helmut K. C. Tessarek
2021-12-21  0:52     ` Helmut K. C. Tessarek
2021-12-21  0:54       ` Aman Karmani
2021-12-21  1:18         ` Christopher Degawa
2021-12-21  1:24           ` Aman Karmani
2021-12-21  1:31             ` Christopher Degawa
2021-12-21  1:28           ` Ridley Combs
2021-12-21  1:35             ` Christopher Degawa
2021-12-21  1:03   ` Helmut K. C. Tessarek
2021-12-21  1:20     ` Aman Karmani
2021-12-21  1:35       ` Helmut K. C. Tessarek
2021-12-21  1:39         ` Ridley Combs
2021-12-21  2:31           ` Helmut K. C. Tessarek
2021-12-21  1:48       ` Christopher Degawa
2021-12-21  1:52         ` Aman Karmani
2021-12-21  6:04           ` Christopher Degawa
2021-12-21  7:39             ` Ridley Combs
2021-12-21 18:04               ` Christopher Degawa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=fd22753b-35bf-6a3f-c82a-c48dab21dea0@evermeet.cx \
    --to=tessarek@evermeet.cx \
    --cc=ffmpeg-devel@ffmpeg.org \
    --cc=rcombs@rcombs.me \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git