[vlc-devel] [PATCH 0/3] Expand contrib

[Steve Lhomme]

On Mon, Jun 12, 2017 at 10:56 PM, Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le maanantaina 12. kesäkuuta 2017, 11.57.28 EEST Steve Lhomme a écrit :
>> Another use case is when you know you want to edit a contrib. The only
>> way to get the code is to build it and then edit and then build again,
>> but you have to remove the .target file for that. It would be easier
>> to just decompress and edit and then build.
>
> And that won´t be fixed by this patch. The decompression targets also don´t
> rerun by default, if they are written correctly.
>
> To "fix" that, you would is to add dependencies (and I don´t think we should
> even try).
>
>> It's also a good way to test if the patches merge properly after you
>> modify one without having to build after that.
>
> No, it isn´t. You have to erase the expansion directories, which is even less
> straightforward than erasing the dot-files.

It may be specific to my use-case but when doing a MSVC build (for ARM
at least but also to debug) I either have the choice of doing it in
parallel in a reasonable time (20-30 minutes) or not in parallel to
catch errors when they occur (they *always* do). In that case I have
to sit for a few hours waiting for things to fail. And the failure is
most of the time because a contrib changed and patches need to be
updated. Separating the part when it fails (expand) from the
compilation would make my life a lot easier. I could even do the
decompression in a single thread, performance wise it shouldn't be
much slower. And do the compilation always with multiple threads.

> --
> 雷米‧德尼-库尔蒙
> https://www.remlab.net/
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel