Re: subversion ruby bindings and clang 15 (was: Re: amd64-clang bulk build report (llvm 15))

On Tue, Jan 24, 2023 at 12:15 PM Stefan Sperling <stsp@openbsd.org> wrote:

> On Tue, Jan 24, 2023 at 07:22:24PM +0100, Jeremie Courreges-Anglas wrote:
> > On Sun, Jan 08 2023, Jeremie Courreges-Anglas <jca@wxcvbn.org> wrote:
> >
> > [...]
> >
> > >>
> https://wxcvbn.org/~jca/build-failures/amd64-clang/2023-01-03/devel/subversion.txt
> > > subversion/bindings/swig/ruby/svn_client.c:6473:17: error:
> incompatible pointer to integer conversion passing 'apr_hash_t *' (aka
> 'struct apr_hash_t *') to parameter of type 'VALUE' (aka 'unsigned long')
> [-Wint-conversion]
> > > if (NIL_P(arg2)) {
> > > ^~~~
> >
> > Lots of similar failures in here. I didn't try to fix it, but an
> > alternate way to handle it would be to drop the ruby bindings for the
> > time being (they're not used elsewhere in the tree). Something like the
> > diff below. Thoughts?
>
> This error is in generated code. I guess this means any port that
> depends on devel/swig to generate code is potentially failing with
> clang 15?
>
> Anyway, fine with me to disable SVN ruby bindings by default if it helps.
>

OK jeremy@ for disabling the SVN ruby bindings as well.

If devel/swig + ruby is an issue with clang 15, audio/ruby-taglib may also
be worth testing.

Thanks,
Jeremy