Build breaks with v0.24.6

[tronical]

PR #820 changed the config struct in a way that breaks existing code:

@@ -1003,6 +1003,8 @@ pub struct Config {
     pub only_target_dependencies: bool,
     /// Configuration options specific to Cython.
     pub cython: CythonConfig,
+    #[serde(skip)]
+    pub(crate) config_path: Option<StdPathBuf>,
 }

 impl Default for Config {

Our code initialises Config not via ::default() but like this:

cbindgen::Config {
        pragma_once: true,
        include_version: true,
        // some more fields here
        ..Default::default()
    }

and that causes

error[E0451]: field `config_path` of struct `Config` is private
  --> api/cpp/cbindgen.rs:83:11
   |
83 |         ..Default::default()
   |           ^^^^^^^^^^^^^^^^^^ field `config_path` is private

I can work around this of course by using Config::default(), but unfortunately the crate release with this change breaks our existing crate on crates.io that relies on this :-(

[konsti219]

Same issue here trying to build https://github.com/fzyzcjy/flutter_rust_bridge .
The change in that PR is actually a breaking change but was shipped in a patch version.

https://github.com/rust-lang/rfcs/blob/master/text/1105-api-evolution.md

Major change: adding a private field when all current fields are public.

[konsti219]

@emilio This seems quite urgent.

[emilio]

Yep, thanks. Yanked 0.24.6. Sorry for not catching that during review. Will publish 0.25 later this week. Let me know if a 0.24.7 with that change reverted would make someone's life easier.

[tronical]

Thanks for the quick reaction and fix!

[sporksmith]

I'm confused - it looks like this is still broken (or maybe broken again)? I guess based on the CHANGELOG it was decided to keep this breaking change?

80526e7

• Breaking changes: The Config struct now has a private member.