On Tue, 17 Jan 2023 17:37:01 +0300 Valery Ushakov <u...@stderr.spb.ru> wrote:
> On Tue, Jan 17, 2023 at 05:37:05 +0000, matthew green wrote: > > > Module Name: xsrc > > Committed By: mrg > > Date: Tue Jan 17 05:37:05 UTC 2023 > > > > Modified Files: > > xsrc/external/mit/xorg-cf-files/dist: Imake.tmpl > > > > Log Message: > > pull over fix from pkgsrc xorg-cf-files and avoid ar's "l" flag. > > That doesn't seem to handle all the cases where "l" flag is used > (ArAddCmd, ArExtCmd). > > Might be easier to either define HasLargeTmp which seems to be *the* > knob to control "l" flag to ar to begin with, or to factor out the > test for "supports l flag" so that the unwieldy > > SystemV4 || LinuxBinUtilsMajorVersion || defined(NetBSDArchitecture) > > is not repeated multiple times. In interest of reducing the diff to upstream I would suggest to use their fix and if that is not sufficient we should talk to them. https://gitlab.freedesktop.org/xorg/util/cf/-/commit/90b66dc73deb9ef303690370395cf831425547c7