[vlc-devel] [PATCH] apple: build: avoid configure if already done
david.fuhrmann at gmail.com
david.fuhrmann at gmail.com
Fri Feb 14 15:38:37 CET 2020
I would also say let’s wait for meson before doing improvements in makefiles.
The patch itself is fine from my point of view, and I think as long as there is agreement from the developers who actually regularly use the script it’s ok to merge.
Von meinem iPhone gesendet
> Am 14.02.2020 um 09:25 schrieb Alexandre Janniaux <ajanni at videolabs.io>:
>> On Fri, Feb 14, 2020 at 08:12:29AM +0100, Steve Lhomme wrote:
>>> On 2020-02-13 22:43, Alexandre Janniaux wrote:
>>>> On Thu, Feb 13, 2020 at 10:52:13PM +0200, Rémi Denis-Courmont wrote:
>>>> Le torstaina 13. helmikuuta 2020, 20.46.48 EET Alexandre Janniaux a écrit :
>>>>> Actually, it's iOS. I don't know how and how often you build
>>>>> for it but I use --config-cache locally and it takes 27 seconds
>>>> 27 seconds is peanuts when configure needs to be rerun, as opposed to the time
>>>> it takes without caching.
>>> I don't think new contributors should endorse this time, or
>>> the time to rewrite iOS build system.
>>>> The fact that you run configure everytime
>>> Actually I don't, I have this patch.
>>>> is a self-inflicted problem of using a
>>>> shell script instead of calling make directly. make anyway reruns configure, or
>>>> even autoconf, when needed.
>>> Are you suggesting you would rewrite the build system for iOS?
>> Not really, a make.sh alongside the build.sh could do.
> Unfortunately it's not enough, this script is currently
> mainly run from VLCKit buildscript, which makes it a much
> more time demanding task than «copy and paste the code to
> a build.sh». The reasons it's so it than you may build
> multiple architecture at once.
> The makefile integration is the way to go to solve this
> correctly, but it needs time to integrate with Android too
> without the current flaws (eg, symbol conflicts and exposure)
> and taking into account differences between linkage for
> darwin platforms and linkage for android platform, and
> I don't think this time investment is worthy as autotools
> buildsystem will be replaced with meson.
> Meanwhile, about this patch, Android and Macosx have been
> using this for long (android is using config.h btw) and
> I don't remember having people complaining about this.
> Alexandre Janniaux
>> Most build systems have a configure.sh next to the build.sh so it can be run
>> independently (and maybe other reasons).
>> If your make process needs many commands you could put them in a separate
>> Looking at the apple/build.sh it seems we could have a "static" target in
>> the Makefile to do some of the common things static builds have to do. Then
>> you just run "make static".
>>> Or maybe you have a better way to compile for it? Please share
>>> if so, it would be a better argument than telling me what I'm
>>> doing. I recall that you didn't care about this platform but
>>> maybe things have changed since that time.
>>> I personnally can't spent that time and I'm just sharing what
>>> I did to workaround the issue.
>>> Alexandre Janniaux
>>> vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel