From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 4EC8144216 for ; Wed, 2 Nov 2022 01:08:20 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 891B168BE15; Wed, 2 Nov 2022 03:08:17 +0200 (EET) Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-oln040092069045.outbound.protection.outlook.com [40.92.69.45]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3484868BD4D for ; Wed, 2 Nov 2022 03:08:11 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=isuNF01H/mtAYuKPpZA9qUDRN6AwLvUN0ZmiEGP0zJuv7o00PlggG2GuFdFL0ifMKHzy7cECyo4yhixAXUqvEleTMsFSlQ0QOmgINQAQv6PR6mQILReRaZtBUFRXn/lCIeRLBokE9rPpbS79yL+fRd+uLk+/AN+e3pPAkpPH+58NNskB/Yysxfuhf8eFfrWt65aSiy4Pb0hB8ukSsMIHO5EbKdeTndXHiVmYQ+EiX+LXjogT+sa4bVoj8QCW3xm92Hry2Kr43BiHHYMJXfd4/oprrM7TCC3Wv8F66zUhdNM24Ukqvkbc9jtizB+XoJn42QjmJ5Ni+C+k99ya4WGO5A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=sRPAPUeWX9asBrs0Vr0lmr1UXbSupgb42XzYr96d0tw=; b=G6/Bq9LMyoKIQh3Tft7pM8CZgT5YVKSekRO4inflom2IsGbJFtuskK4Z1ohBiStL/UzCx//fdqc9RxvEhKAxBG0Nw7ipiacgExgDbDNTIMakSiz2PXCSme5fP4q3r7bjgwlJ7+HP8tcRPsi7QzVlJMhvS0jlg9BZmUqXh47Jd5nBCWAXIR4n8UeAA0Obr3iygwzgp1hZwQGIeCMVeQNFwr1hBbTV1R2FjZwBuZ20k73wSdH1nL8Qt0xEVy4icbUckEGdo6a9QuQkYYwIUSNNkN2kAhC+t4THOrYCCj5IqswBIJoqk5v819Fg6OmTeZ/HoIFeJI2cXBZzSdBwvh1LDQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=sRPAPUeWX9asBrs0Vr0lmr1UXbSupgb42XzYr96d0tw=; b=hTSTA+7XQmXv4ODw9w9+mkKA4U+oNVES4bJm4MteDcyq/FpCHfv5Hb3ZXQND0xhyCko1FNT844qLcx9/9Evo1YpphaMo2q9uMCfBNvRcxxjGL8zAh2UZBDlJwOBsxw+DqqVaIzNX5EZL4y8QhEJC1DlhdKd4Jig36OcJKEL6Xgcdr+LRRarw734ZUWqI99IOHOMfNyq7UdA0iA/VNGwKkvIhyKuykGBmFlVE9aE+NYdLpOXVd6Az161tZibkCNGo6coKGcA8XbZNlkV+6WuCAS/oUaF/Mx4HU8KNMCbEKdyw8/Ne/cjXrQL+W3Pxu9/I50IIEqiiGXVu5JO9090AEQ== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DB4P250MB1030.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3ce::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5769.15; Wed, 2 Nov 2022 01:08:09 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::739a:8740:19c5:925c]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::739a:8740:19c5:925c%5]) with mapi id 15.20.5769.019; Wed, 2 Nov 2022 01:08:09 +0000 Message-ID: Date: Wed, 2 Nov 2022 02:08:20 +0100 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [m7dAY2vOGQWLIXIcD3o1FS0zH1xFvBJV/5av3KL4DiQ=] X-ClientProxiedBy: FR3P281CA0041.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4a::17) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <32ccac75-22f0-ae18-4feb-f1096d39b8ae@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DB4P250MB1030:EE_ X-MS-Office365-Filtering-Correlation-Id: 51386834-702b-4eca-353a-08dabc6eb4c2 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: zlgB0++Cz0CfoHgbSkpUM6BS6EOl4j+Y2262oTKSsX+ocpGbDuHP3beYX7vXBHG2qCim50XYFeWSZKMFWiHmARl7E7okstoVee12/zkPh40Eoy/EXPmYZHyBfEXpWAIS7+Y9TDTq/RK0JhPhFyl7U6vJN6zMOh2sx63tHCDqg9k8hay+LDQBpnksH0HhUWlqLtzIQ2bhA3vFHQaZUxP9YjFu+K6pwVyBmertGsCLZ1LuOh7ewuoRbgmTXWdnzyfY1qZ4o8daBGHba2uNthz7v0E5HkXG+fs5Y6ZZLLMqMt4qa75QU6B8XguG0aEez/TMjvWw9zMLZvIoCfz4N9Vfn5yUhd3ANDBtJ7C9jobwrAweoMSBgegqrctIuQcwGEUTTHg0HmrK5LT6JcYQDa8g4w5JRkg45vOjvJY8gddWmR+zpb6cqemxOdNIvtMBE62r6TfhHhAvfMPkIXjUCZX6tOH3LI0Yh9bPEEC0opicUQomaQ9Ui5ZmITP+8S4KeHAjS66m1/aOS4r6D0HOKaNCMjvuRHsnIM0teitvLWMhQJhE/bTvRcd/KxI4WXQy8yMNw8u4rIMrryiefk9MjY8ZtwbuWfavqc/5yRr0CLMG9LivRbUfXmsXwlTF2E7VT8UUYVMh+sZu29ZknfZdijtubE90zAbg70TAQWrhaugm7UI9/ct5kXYK6vQm4nH3/qSR X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SWNKbXlmNllBRTRnUlZna1VuQ0pySjVQTDJSV2xFdjQ0TEFvOVJSRWlqcFlh?= =?utf-8?B?ZzN3OHhEdUpiczN5Y2ZnMkJyZGtVakhoSS93SDZ4QnNtWW9mMFlCUlVVUjNn?= =?utf-8?B?eVh4Yi8weXN3NHBCWDFLVVpQSStXWG5IVHhQSFZMNFZuNk5Zd3lrU1FZVHNz?= =?utf-8?B?Nk9KdmVVVXNmV3R1S0NQNWJydFFGSFIyN0VCdFFoRVBnRHZkNWNGQ2dUTFFR?= =?utf-8?B?bkUxd2hCc1ZCR2pUMEpDaHFmM3JiRU50ekMwQllVWk5wZzE1bHpBRVJjUG5a?= =?utf-8?B?VTc4WnZSMmorWlp3VWMvT0U2ejVSL2RLc2U1MzF5dWFldkwyL1lOVUdzOXAr?= =?utf-8?B?Q21zVTZhSHNIbkFIYThHcU9xMFFGL2NMQUhTUStYVzZJTVB2QzNPeWw1U0xz?= =?utf-8?B?ckJkbXdabHN1dUtIUGpoL05oZGFNelRBaEI2cHYveThkS09oaE5pRW50RVhN?= =?utf-8?B?SVIxc3FzT3c0YWpPd3F5dVR0T0xlOURFWkVXYWRFVWtZQ3dOQ3ZIbGxTWDFp?= =?utf-8?B?dVppc05LcHBqODJRK3ZsNDBHVUlVanhWM01YcHZWeUdKOU5adERFQkYrbjls?= =?utf-8?B?WGFEN3VnSFQ4ai96eXhSU1l2b1J4L0tZZ00wWHgxK29mMjFrSEIvU1haYXZH?= =?utf-8?B?Z0I5UGhiU0ZZVGdLK3RLdlVVR3pNZEROemlkWTJ0cnhRWlVnUGR4N1VxZ0lN?= =?utf-8?B?Umk0RDBDakxZajlHZmgrMTNrVlF5ODQySEFyKzUwMEhzYkxrSWlpQU1kZDVH?= =?utf-8?B?eTZlMFRybjZIUU5tVy9ZU05oU1VSSDNXK0diYnB6dm9LeVhxRld0TWIrSlVL?= =?utf-8?B?Y05qS1YydTVpZ25JZDhJOE91d2lUeHpIdE1rSHd6ZlhSUEhWMi9hb2N5Z2pO?= =?utf-8?B?dXNBSFdvZUhqYURLcUY5OTV3U2E0Z3RNNE1jM0FaeG5KVVc3bFIyRFZubVF2?= =?utf-8?B?ZDgwUVdhNC9KL3NsdTVyZTY0bnd2S0VmT2dwZTRvdThOaTNZc091YW82Si82?= =?utf-8?B?cHBwWDFUK05GZEgwZzVIUlhtaU0vTVdVeFF0WU1WZFFYYmtaTDBFcmpoK284?= =?utf-8?B?QmZRaEhtcnFhTVdTdmRHMWQvMHZIVGNpREhKVDRBYlA5VWwxTWdEWHRhaUxN?= =?utf-8?B?bUpkSFpuYlFxb1RuWkRSb2JrV2loZFYrR0tNa1JCTzZLaEFFemNYbHlEUmEx?= =?utf-8?B?VGhHNkZmcVU3c0VMR25UVS9QZ1ByM2tKbVl5ZG5iaVNPZ3pVQ3BZN1prd3Jt?= =?utf-8?B?RWw5QXpmRXcwazJPNmJsT0F6TUI4SjBkd3Rxd0ZVZElTK01Mb1ZOZ05MN2d4?= =?utf-8?B?VWFvMVVZTW1sVElWWnhaV2dZamhMWWx0OXZSemthY3JIb3h6YnphZkFoajFZ?= =?utf-8?B?R3M2ekpmUEI1ZVY3QWZSREpaR29oaDdaVDNvRVZiWDBUQVdqWWdwaHlZY2RY?= =?utf-8?B?bGZrODJuN2U2SmlDZnhKMUVRVjdBKzN4UnJaTC9ZMjVJcEtIdmNpRDBsNFJY?= =?utf-8?B?VjRUMzkyWXlhVS94M3JtdlZSR3NYS3ZQTkF5UStvUUJuNC9NWUY2TmQ2Q2NQ?= =?utf-8?B?RTRtaEx4VFE5TE1YdDN2VERnckdJekdkejFVN28zZXRiS3p2OHRKSGd0d3Ir?= =?utf-8?B?NWFJUHd1UW5MSnY1SStDWE1VQUJyVWI4SGdKR2J0WHcxeU5EMGZkdWVLWVN1?= =?utf-8?B?aHZyd2JidFFkK29FOWhpZHRqZFNsTThKUWJRdWY1WWJOekEySHdlU25rdm0y?= =?utf-8?Q?e1/t6YqDtjkLn69FDNOsCRwRHkiw3bOJohkkeoO?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 51386834-702b-4eca-353a-08dabc6eb4c2 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Nov 2022 01:08:09.3272 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4P250MB1030 Subject: Re: [FFmpeg-devel] [PATCH 0/4] Fix FFmpeg compilation without DCE X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: L. E. Segovia: > Hi all, > > This is a patch to fix building FFmpeg without having DCE enabled. > This was previously attempted by Andreas Rheinhardt in commit > 40e6575aa3eed64cd32bf28c00ae57edc5acb25a. However, this is not remotely > enough for MSVC; one can quickly check for this by configuring the build > without any optimizations, and then overriding OPTFLAGS with /Od: > You seem to misunderstand the goal of 40e6575aa3eed64cd32bf28c00ae57edc5acb25a: It was not meant for --disable-optimizations builds; it was instead meant for full LTO builds (with optimizations enabled), where MSVC wanted to take a look at arch-specific code that was not available although it was under a "if (0)". Builds without asm or where components have been disabled (which can easily lead to linking failures if your compiler does not perform DCE) are not considered "full" builds in the preceding sentence. (I actually do not even know whether it has achieved said aim.) > ../configure --disable-asm --disable-optimizations --optflags=-Od --enable-shared --disable-static --toolchain=msvc > > Issuing make should yield the first of many LNK2019 errors: > > LD libavformat/avformat-59.dll > Creating library libavformat/avformat.lib and object libavformat/avformat.exp > rtmpproto.o : error LNK2019: unresolved external symbol ff_rtmpe_gen_pub_key referenced in function rtmp_handshake > rtmpproto.o : error LNK2019: unresolved external symbol ff_rtmpe_compute_secret_key referenced in function rtmp_handshake > rtmpproto.o : error LNK2019: unresolved external symbol ff_rtmpe_encrypt_sig referenced in function rtmp_handshake > rtmpproto.o : error LNK2019: unresolved external symbol ff_rtmpe_update_keystream referenced in function rtmp_handshake > libavformat\avformat-59.dll : fatal error LNK1120: 4 unresolved externals > make: *** [/c/Users/Amalia/Desktop/ffmpeg/ffbuild/library.mak:119: libavformat/avformat-59.dll] Error 96 > > The proposed patchset comprehensively converts all instances of ARCH_FOO, > CONFIG_FOO, INLINE_FOO and EXTERNAL_FOO to the equivalent preprocessor > guards. > You are not converting all of them; you seem to only convert those that caused linker errors in your build (which is fine by me). E.g. mpegvideo_enc.c is full of CONFIG checks that would lead to linker errors in case DCE is disabled if some of the components whose existence is checked are actually disabled. > L. E. Segovia (4): > all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 > all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO > all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM > all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM > > fftools/ffprobe.c | 22 +- > fftools/opt_common.c | 18 +- > libavcodec/x86/aacencdsp_init.c | 2 + > libavcodec/x86/aacpsdsp_init.c | 2 + > libavcodec/x86/ac3dsp_init.c | 4 + > libavcodec/x86/audiodsp_init.c | 2 + > libavcodec/x86/bswapdsp_init.c | 2 + > libavcodec/x86/cavsdsp.c | 2 + > libavcodec/x86/celt_pvq_init.c | 2 + > libavcodec/x86/cfhddsp_init.c | 2 + > libavcodec/x86/cfhdencdsp_init.c | 2 + > libavcodec/x86/dcadsp_init.c | 4 + > libavcodec/x86/dct_init.c | 2 + > libavcodec/x86/dnxhdenc_init.c | 2 + > libavcodec/x86/exrdsp_init.c | 2 + > libavcodec/x86/fdctdsp_init.c | 2 + > libavcodec/x86/fft_init.c | 2 + > libavcodec/x86/flacdsp_init.c | 8 +- > libavcodec/x86/g722dsp_init.c | 2 + > libavcodec/x86/h263dsp_init.c | 2 + > libavcodec/x86/h264_intrapred_init.c | 2 + > libavcodec/x86/h264chroma_init.c | 2 + > libavcodec/x86/hevcdsp_init.c | 465 +++++++++++---------- > libavcodec/x86/hpeldsp_init.c | 2 + > libavcodec/x86/hpeldsp_vp3_init.c | 2 + > libavcodec/x86/huffyuvdsp_init.c | 2 + > libavcodec/x86/huffyuvencdsp_init.c | 2 + > libavcodec/x86/idctdsp_init.c | 11 +- > libavcodec/x86/jpeg2000dsp_init.c | 2 + > libavcodec/x86/lossless_videodsp_init.c | 2 + > libavcodec/x86/lossless_videoencdsp_init.c | 2 + > libavcodec/x86/mdct15_init.c | 2 + > libavcodec/x86/me_cmp_init.c | 2 + > libavcodec/x86/mlpdsp_init.c | 8 +- > libavcodec/x86/mpegvideoencdsp_init.c | 2 + > libavcodec/x86/opusdsp_init.c | 2 + > libavcodec/x86/pixblockdsp_init.c | 2 + > libavcodec/x86/pngdsp_init.c | 2 + > libavcodec/x86/proresdsp_init.c | 2 + > libavcodec/x86/rv34dsp_init.c | 2 + > libavcodec/x86/sbcdsp_init.c | 2 + > libavcodec/x86/sbrdsp_init.c | 2 + > libavcodec/x86/svq1enc_init.c | 2 + > libavcodec/x86/utvideodsp_init.c | 2 + > libavcodec/x86/v210enc_init.c | 2 + > libavcodec/x86/vc1dsp_init.c | 6 +- > libavcodec/x86/vorbisdsp_init.c | 2 + > libavcodec/x86/vp3dsp_init.c | 2 + > libavcodec/x86/vp6dsp_init.c | 2 + > libavfilter/x86/af_afir_init.c | 2 + > libavfilter/x86/af_anlmdn_init.c | 2 + > libavfilter/x86/af_volume_init.c | 2 + > libavfilter/x86/avf_showcqt_init.c | 2 + > libavfilter/x86/colorspacedsp_init.c | 6 +- > libavfilter/x86/vf_atadenoise_init.c | 8 +- > libavfilter/x86/vf_blend_init.c | 2 + > libavfilter/x86/vf_bwdif_init.c | 2 + > libavfilter/x86/vf_convolution_init.c | 2 + > libavfilter/x86/vf_framerate_init.c | 2 + > libavfilter/x86/vf_fspp_init.c | 2 + > libavfilter/x86/vf_gblur_init.c | 2 + > libavfilter/x86/vf_hflip_init.c | 2 + > libavfilter/x86/vf_limiter_init.c | 2 + > libavfilter/x86/vf_maskedclamp_init.c | 2 + > libavfilter/x86/vf_maskedmerge_init.c | 2 + > libavfilter/x86/vf_overlay_init.c | 2 + > libavfilter/x86/vf_pp7_init.c | 2 + > libavfilter/x86/vf_psnr_init.c | 2 + > libavfilter/x86/vf_removegrain_init.c | 2 + > libavfilter/x86/vf_ssim_init.c | 8 +- > libavfilter/x86/vf_stereo3d_init.c | 2 + > libavfilter/x86/vf_threshold_init.c | 2 + > libavfilter/x86/vf_tinterlace_init.c | 2 + > libavfilter/x86/vf_transpose_init.c | 2 + > libavfilter/x86/vf_v360_init.c | 2 + > libavfilter/x86/vf_w3fdif_init.c | 6 +- > libavfilter/x86/vf_yadif_init.c | 2 + > libavformat/rtmpproto.c | 24 +- > libavutil/x86/fixed_dsp_init.c | 2 + > libavutil/x86/float_dsp_init.c | 2 + > libavutil/x86/imgutils_init.c | 2 + > libavutil/x86/lls_init.c | 2 + > libavutil/x86/pixelutils_init.c | 2 + > libswresample/x86/audio_convert_init.c | 2 + > libswresample/x86/resample_init.c | 6 + > libswscale/x86/rgb2rgb.c | 2 + > libswscale/x86/swscale.c | 2 + > 87 files changed, 493 insertions(+), 255 deletions(-) > _______________________________________________ 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".