[vlc-devel] [PATCH v2 08/13] contrib: use consistent names between the package name and the extracted dir

Steve Lhomme robux4 at ycbcr.xyz
Fri Jul 10 15:04:23 CEST 2020


On 2020-07-10 14:12, Rémi Denis-Courmont wrote:
> Le jeudi 9 juillet 2020, 15:33:00 EEST Steve Lhomme a écrit :
>> The fact that all extracted targets can be addressed via $(PKG) is an
>> added bonus to this change.
> 
> No. That can't work ever since there is mostly but *not* exactly a 1:1 mapping
> between packages and directories.

You always say "can't work" when people come up with patches that work. 
That makes a lot of your patch reviews less trustworthy.

>> Right now the inconsistencies make it
>> impossible. And I don't see a good reason not to have consistency.
> 
> There is a consistent rule that is mostly followed. Again, if there are cases

"*consistent* rule *mostly* followed". Thanks for the oxomoron.

> not following the rule, you can fix them, but that's not what this patch seems
> to do.

There is no written rule nor any consistent rule. The closest rule there 
is to use a the target name, the same as the .target name. This is 
exactly what I followed and what allows to address PKGS just like you 
can address .PKGS and .sum-PKGS which is otherwise impossible since 
there is no easy correspondence between the tarball name and the 
.package and .sum-package name. So I am bring a missing consistency in 
this patch.

And since you mention some churn in the contrib directories, it's better 
*NOT* to rely on the tarball names ever but use our consistent names. So 
in the future if a target completely changes their output name, we still 
have our own consistency.


More information about the vlc-devel mailing list