rdedup icon indicating copy to clipboard operation
rdedup copied to clipboard

Crash on init

Open kevincox opened this issue 5 years ago • 4 comments

% rdedup -v
rdedup 3.1.1
...
% RUST_BACKTRACE=1 RDEDUP_PASSPHRASE=test rdedup -d ~/d init
Using passphrase set in RDEDUP_PASSPHRASE
thread 'main' panicked at 'attempted to leave type `linked_hash_map::Node<yaml_rust::yaml::Yaml, yaml_rust::yaml::Yaml>` uninitialized, which is invalid', /build/rustc-1.51.0-src/library/core/src/mem/mod.rs:671:9
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::panic
   3: linked_hash_map::LinkedHashMap<K,V,S>::insert
   4: <serde_yaml::ser::SerializeStruct as serde::ser::SerializeStruct>::serialize_field
   5: rdedup_lib::config::Repo::write
   6: rdedup::run
   7: rdedup::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

kevincox avatar Apr 20 '21 23:04 kevincox

WAT? :D

That's weird. Looks like ... serialization error? A bug in yaml_rust maybe? I'll try to investigate if I find time.

dpc avatar Apr 23 '21 17:04 dpc

It seems to be related to https://github.com/contain-rs/linked-hash-map/pull/100

dywedir avatar Apr 24 '21 21:04 dywedir

So, cargo update should fix it?

dpc avatar Apr 25 '21 03:04 dpc

Oops, it totally slipped my mind, sorry! This should already be fixed in https://github.com/dpc/rdedup/commit/5b4b5fd4a5955c749c6bd4cd87dfcb9275ce7765 (2020) Unfortunately, the last version of rdedup was released before that time.

dywedir avatar Aug 09 '21 20:08 dywedir