From: Jeremie Courreges-Anglas <jca@wxcvbn.org>
Subject: Re: devel/gas: update to 2.45
To: ports@openbsd.org, pascal@stumpf.co
Date: Tue, 5 Aug 2025 13:23:38 +0200

On Mon, Aug 04, 2025 at 09:00:12PM +0200, Kirill A. Korinsky wrote:
> ports@,
> 
> to move forward with update of lang/gcc I need fresh gas.
> 
> Changes:
>  - it needs GNU make;
>  - almost all patches not required anymore.
> 
> BTW a hunk for patches/patch-gas_doc_as_info looks wrong, but this patch
> contains some control codes. The good news that it should be removed.

Can you please explain why the patch can be dropped?  The package
name, the executable and more importantly the info file are named
"gas", for info file handling our install-info needs the info file
name to match:

  m2 /usr/ports/devel/gas$ make deinstall
  ===> Deinstalling for gas-2.45
  install-info: warning: no entries found for `/usr/local/info/gas.info'; nothing deleted
  gas-2.45: ok

Also please do not install

> +@info info/bfd.info
> +@info info/ctf-spec.info
>  @info info/gas.info
> +@info info/sframe-spec.info

since the port doesn't install those libraries.  See devel/binutils
for an example.

> Tested on -current amd64 and arm64.
> 
> Thus, I had sent it twice because the first time I forgot about subject.
> 
> Ok?

As you may have noticed, devel/gas is used by lang/gcc/8 and
lang/gcc/11 for machine code generation on all our platforms.  So this
update won't be good to commit until it's been tested on enough
architectures with lang/gcc/8, with "enough" being flexible but
containing at the very least amd64, arm64, i386, riscv64, sparc64 and
preferably also powerpc, powerpc64 and mips64.  *cough* armv7? *cough*

Build-testing all the lang/gcc/8 consumers + running the numpy
testsuite should be enough, but a full bulk would be nice on fast
archs.

Currently taking care of testing this on amd64, arm64, riscv64 and
sparc64.

-- 
jca