Re: [PATCH v4 03/21] ci/lib.sh: encapsulate Travis-specific things

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:

> No, not really. Actually, not at all.
>
>> It would be, under this new arrangement of the code, most natural for
>> Those who want to use gnu-time to arrange it to be somehow added to
>> $BREW_INSTALL_PACKAGES, no?
>
> The purpose of BREW_INSTALL_PACKAGES is to list the packages necessary to
> build Git and run its test suite, and the only reason why this is no
> longer a hard-coded list of packages is that it depends on the CI platform
> (or more concretely, on the available macOS agents of said CI platform)
> which packages need to be installed to do so.
>
> The gnu-time package is not such a package, and it is unlikely to be
> dependent on the particular CI you want to use.

Those who want to do perf tests in the current setup would need to
install gnu-time because the current setup is only Travis, whose
macOS agent does not have it preinstalled.  Other CI platforms'
macOS agents may already have it, they may not want to get an error
by trying to install it there.  I am not sure how that is different
from the situation for gettext etc.?








[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux