Improve save_history

[gwenn]

See https://github.com/danburkert/ksql/blob/master/src/main.rs#L199

[Yamakaky]

I don't understand, it's expected because the two calls don't output the same Error. BTW, since ReadlineError can contain a io::Error, you can do

fs::create_dir_all(history_path.parent().unwrap())
   .map_err(Into::into)
   .and_then(|_| readline.save_history(&history_path))

A better improvement would be to use https://github.com/brson/error-chain. Waiting for rust-lang-deprecated/error-chain#53.

[Yamakaky]

Waiting for rust-lang-deprecated/error-chain#55 to be merged.

[walles]

@Yamakaky both rust-lang-deprecated/error-chain#53 and rust-lang-deprecated/error-chain#55 are merged.

Is this ticket still relevant?

[Yamakaky]

error-chain doesn't seem to be used anymore so no !

[walles]

@gwenn, @Yamakaky says this ticket isn't relevant any more.

Can it be closed?