try to generate bindings for other platforms in the CI, use target specific bindings dirs

Author: antonilol

.github/workflows/CI.yml

@@ -61,6 +61,32 @@ jobs:
           cargo build --examples --features "${CI_BUILD_FEATURES} ${{matrix.feature}}" ${{matrix.build_mode}}
           cargo test --features "${CI_BUILD_FEATURES} ${{matrix.feature}}" ${{matrix.build_mode}}
 
+  build-bindgen:
+    name: build and generate bindings
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [macos-latest, ubuntu-latest, windows-latest]
+    steps:
+      - uses: actions/checkout@v2
+      - name: Build SDL2
+        shell: bash
+        env:
+          RUSTFLAGS: "--cfg update_pregenerated_bindings"
+          CI_BUILD_FEATURES: "mixer,ttf,gfx,image,bindgen,bundled"
+          RUST_TEST_THREADS: 1
+        run: |
+          set -xeuo pipefail
+          rustc --version
+          cargo --version
+          cargo build --features "${CI_BUILD_FEATURES}"
+          cargo build --examples --features "${CI_BUILD_FEATURES}"
+          cargo test --features "${CI_BUILD_FEATURES}"
+          # dump generated bindings
+          git add sdl2-sys/bindings/
+          git diff --cached
+
   build-linux:
     name: build linux pkg-config
     runs-on: ubuntu-24.04

sdl2-sys/sdl_bindings.rs …x86_64-unknown-linux-gnu/sdl_bindings.rssdl2-sys/sdl_bindings.rs renamed to sdl2-sys/bindings/x86_64-unknown-linux-gnu/sdl_bindings.rs sdl2-sys/sdl_bindings.rs renamed to sdl2-sys/bindings/x86_64-unknown-linux-gnu/sdl_bindings.rs

@@ -16,6 +16,7 @@ extern crate pkg_config;
 ))]
 compile_error!("Enable 'bindgen' and 'bundled' features when using update_pregenerated_bindings");
 
+use std::io::ErrorKind;
 use std::path::{Path, PathBuf};
 use std::process::Command;
 use std::{env, fs, io};
@@ -558,7 +559,7 @@ fn main() {
 
     #[cfg(not(feature = "bindgen"))]
     {
-        copy_pregenerated_bindings();
+        copy_pregenerated_bindings(target.as_str());
         println!("cargo:include={}", sdl2_includes);
     }
 
@@ -574,10 +575,21 @@ fn main() {
     }
 }
 
+fn bindings_dir(mut base: PathBuf, target: &str) -> PathBuf {
+    base.push("bindings");
+    base.push(target);
+
+    base
+}
+
 #[cfg(any(not(feature = "bindgen"), update_pregenerated_bindings))]
-fn copy_pregenerated_bindings() {
+fn copy_pregenerated_bindings(target: &str) {
     let out_path = PathBuf::from(env::var("OUT_DIR").unwrap());
-    let crate_path = PathBuf::from(env::var("CARGO_MANIFEST_DIR").unwrap());
+    let crate_path = bindings_dir(
+        PathBuf::from(env::var("CARGO_MANIFEST_DIR").unwrap()),
+        target,
+    );
+
     fs::copy(
         crate_path.join("sdl_bindings.rs"),
         out_path.join("sdl_bindings.rs"),
@@ -642,7 +654,13 @@ fn generate_bindings(target: &str, host: &str, headers_paths: &[String]) {
     #[cfg(not(update_pregenerated_bindings))]
     let out_path = PathBuf::from(env::var("OUT_DIR").unwrap());
     #[cfg(update_pregenerated_bindings)]
-    let out_path = PathBuf::from(".");
+    let out_path = bindings_dir(PathBuf::from("."), target);
+
+    #[cfg(update_pregenerated_bindings)]
+    match fs::create_dir(&out_path) {
+        Err(err) if err.kind() == io::ErrorKind::AlreadyExists => {}
+        res => res.expect("unable to create bindings dir"),
+    }
 
     let mut all_bindings = bindgen::builder()
         // enable no_std-friendly output by only using core definitions
@@ -836,7 +854,7 @@ fn generate_bindings(target: &str, host: &str, headers_paths: &[String]) {
     }
 
     #[cfg(update_pregenerated_bindings)]
-    copy_pregenerated_bindings();
+    copy_pregenerated_bindings(target);
 }
 
 fn get_os_from_triple(triple: &str) -> Option<&str> {