Panic when building on FreeBSD: "grpcwrap_batch_context_(unnamed_struct_at_grpc_wrap_cc_82_3)" is not a valid Ident about grpc-rs HOT 5 CLOSED

Does it work if using the master version?

from grpc-rs.

@BusyJay Thanks a lot for your swift reply, unfortunately I am getting the same error when using the master version

from grpc-rs.

I may have located the issue, maybe "freebsd" must be added to the following places:

grpc-rs/grpc-sys/build.rs:

    #[cfg(any(
        feature = "_gen-bindings",
        not(all(
            any(target_os = "linux", target_os = "macos"),
            any(target_arch = "x86_64", target_arch = "aarch64")
        ))
    ))]
    {
        // On some system (like Windows), stack size of main thread may
        // be too small.
        let f = file_path.clone();
        std::thread::Builder::new()
            .stack_size(8 * 1024 * 1024)
            .name("bindgen_grpc".to_string())
            .spawn(move || bindgen_grpc(&f))
            .unwrap()
            .join()
            .unwrap();
    }

grpc-rs/grpc-sys/Cargo.toml:

[target.'cfg(not(all(any(target_os = "linux", target_os = "macos"), any(target_arch = "x86_64", target_arch = "aarch64"))))'.build-dependencies]
bindgen = { version = "0.69.0", default-features = false, features = ["runtime"] }

A problem with the stack size would explain why a panic is triggered during the build process. If freebsd is added to the target_os array of the Cargo file, my understanding is that bindgen would not be included as a dependency on FreeBSD, and the code section in build.rs would not be compiled and executed. But I do not have enough understanding of the domain/project to understand the implications.

from grpc-rs.

No, this is an issue with old rust-bindgen and new clang. It's expected to be fixed in bindgen >= 0.61. And master has upgraded bindgen to 0.69, so I asked to check whether master can be compiled.

from grpc-rs.

@BusyJay You were right, I have isolated the repo and can confirm that the build completes without problem when building against master. Sorry for the confusion.

Do you plan to push a release some time or maybe create a tag to ensure that we are building against a relatively stable version?

from grpc-rs.