From: Kacper Michajlow <kasper93@gmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] configure: don't force specific C++ standard library linking Date: Thu, 7 Sep 2023 23:38:38 +0200 Message-ID: <CABPLASRHbAoFPWt=sHWfNotT584nkNFxJ82_Xa6MbwDPDzP3-A@mail.gmail.com> (raw) In-Reply-To: <ee4fb8be-9831-e3db-fa56-84ba97fae955@gmail.com> On Thu, 7 Sept 2023 at 15:12, Derek Buitenhuis <derek.buitenhuis@gmail.com> wrote: > > On 9/6/2023 6:31 PM, Kacper Michajlow wrote: > > What would be a downside of preferring CXX always if it exists? > > FFmpeg runs in a multitude of environments with a multitude of portability > requirements. Needlessly linking a C++ runtime is not OK. This does not answer my question. Let me rephrase. Do we know the case where using C++ compiler driver rather than C would degrade the quality of the resulting build? Using C++ driver would indeed append the (correct) runtime library to the linker command, but if nothing references any symbols from it it would not be linked. It is also why the current way of forcing `lstdc++` kinda works, because it is silently ignored when not needed. Implementing logic to use C++ only when necessary is possible, but I'm not a big fan of such automation. And in practice not sure how well it would work, because it would require trying to link twice every dependency in configure. Also the fact that "FFmpeg runs in a multitude of environment" is precisely why I really don't like the current unconditional including `-lstdc++`. > Further, we do not generally automatically change build output based on > what is available on the system. I must kindly disagree. configure has 35 components marked as `[autodetect]` and they are certainly changing the build output based on what's available on the system. Whether it is intended or historical behaviour is another question. But I'm also not proposing to auto detect anything, quite the opposite with this silly manual list of libraries. - Kacper _______________________________________________ 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".
next prev parent reply other threads:[~2023-09-07 21:39 UTC|newest] Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-09-05 23:26 Kacper Michajłow 2023-09-06 10:15 ` Timo Rothenpieler 2023-09-06 16:54 ` Kacper Michajlow 2023-09-06 17:09 ` Timo Rothenpieler 2023-09-06 17:31 ` Kacper Michajlow 2023-09-07 13:12 ` Derek Buitenhuis 2023-09-07 21:38 ` Kacper Michajlow [this message] 2023-09-07 22:10 ` Kieran Kunhya 2023-09-08 18:41 ` Kacper Michajlow 2023-09-08 18:46 ` Kieran Kunhya 2023-09-08 18:59 ` Kacper Michajlow 2023-09-07 23:34 ` Timo Rothenpieler via ffmpeg-devel 2023-09-08 18:55 ` Kacper Michajlow 2023-09-08 18:58 ` Timo Rothenpieler 2023-09-09 13:29 ` Rémi Denis-Courmont
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='CABPLASRHbAoFPWt=sHWfNotT584nkNFxJ82_Xa6MbwDPDzP3-A@mail.gmail.com' \ --to=kasper93@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /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