[ci] Update InterOp builds to Ubuntu 24.04

[aaronj0]

Meant to fix #265

The CI seems to be picking up cppyy-cling from master when the backend install fails and uses an include RTypes.h used for ROOT. Specifically strlcpy.h which was copied to fix issues with Clang parsing gcc headers. These are BSD only string functions and not part of any standard, and declared in cling/src/core/clib:

#ifndef ROOT_strlcpy
#define ROOT_strlcpy

#include <ROOT/RConfig.hxx>
#ifdef __cplusplus
extern "C" {
#endif

size_t strlcpy(char *dst, const char *src, size_t siz);
size_t strlcat(char *dst, const char *src, size_t siz);

#ifdef __cplusplus
}

Ideally we should update the header guards for ubuntu 24 but in our case we don't use cppyy-cling, nor does our fork have those files. pip defaults to downloading and using the tarball for upstream cppyy-backend which causes this mess. Defaulting to cmake could fix this

[mcbarton]

@maximusron If this works you'll want to wipe the cache the cache this PR made before merging, and delete the Ubuntu cache on the main too.

[mcbarton]

@maximusron can you link this PR to #265 ?

[aaronj0]

@maximusron can you link this PR to #265 ?

Yeah sure, this PR was meant to just fix the cppyy-backend issue but if this works then maybe we can just upgrade the ubuntu versions with this. Ideally we shouldn't have to change anything on the cppyy-backend side but I am still investigating that

[mcbarton]

@maximusron Is the commit where you removed -DLLVM_ENABLE_WERROR=On temporary while you debug this PR?

[aaronj0]

@mcbarton yep its temporary, but we have another set of warnings that emerge on ubuntu 24 that we need to fix. On this PR, I removed it since we would error out on the build CppInterOp step. This may happen everytime we change the os/platform/gcc/clang

[vgvassilev]

@mcbarton yep its temporary, but we have another set of warnings that emerge on ubuntu 24 that we need to fix. On this PR, I removed it since we would error out on the build CppInterOp step. This may happen everytime we change the os/platform/gcc/clang

Yes, and as part of this change we will fix them. Otherwise people commit stuff with warnings which are actually real issues...

[aaronj0]

@vgvassilev yes I have only removed it fix the pypi issue, theres something in the pyproject.toml file that needs to be changed for no-pep to work. Once that worked I thought we can fix the new warnings before merging

[mcbarton]

@maximusron The ci won't run on this PR until you have resolve the merge conflicts

[github-actions]

clang-tidy made some suggestions

[github-actions]

warning: typedef 'FILE' cannot be referenced with a struct specifier [clang-diagnostic-error]

      void operator()(struct FILE* d) const {
                             ^

Additional context

/usr/include/x86_64-linux-gnu/bits/types/FILE.h:6: declared here

typedef struct _IO_FILE FILE;
                        ^

[github-actions]

clang-tidy made some suggestions

[github-actions]

warning: no matching function for call to 'pclose' [clang-diagnostic-error]

        pclose(d);
        ^

Additional context

/usr/include/stdio.h:896: candidate function not viable: cannot convert argument of incomplete type 'struct DIR *' to 'FILE *' (aka '_IO_FILE *') for 1st argument

extern int pclose (FILE *__stream) __nonnull ((1));
           ^

[mcbarton]

Closing as this has been done now for CppInterOp.