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

Reply via email to