compiler error message shows the error path containing parent directory identifier ( `/../` )

[fbernier]

error[E0599]: no method named `map` found for type `std::iter::Map<postgres::rows::Iter<'_>, for<'r, 's> fn(postgres::rows::Row<'r>, &'s extract::CmdOptions) -> std::result::Result<<Q as extract::query::Query>::Message, postgres::Error> {<Q as extract::query::Query>::map}>` in the current scope
  --> extract/tests/query/../helper.rs:40:10
   |
40 |         .map(Result::unwrap)

the actual error is in extract/tests/helper.rs and that's what I would expect to be displayed instead of extract/tests/query/../helper.rs.

binary: rustc
commit-hash: 827013a31b88e536e85b8e6ceb5b9988042ec335
commit-date: 2018-05-25
host: x86_64-unknown-linux-gnu
release: 1.26.1
LLVM version: 6.0

[estebank]

Path::canonicalize could be used for <FileName as Display>, which would get rid of the /../ (which you can't just do with the text of the path because of symlinks), but we should probably do so only in non-windows environments, as canonicalize uses UNC paths on windows.

[basavesh]

Can I work on this?

[estebank]

@basavesh of course! Looking at the closed PR should give you a good idea of what needs to be done.

[23Marbles]

It seems this issue has not been completed, can I still work on this?

[arferreira]

@rustbot claim