Expose fallibility of individual SQL results

`LocalGeneration::sql::files()` runs an SQL query, iterates over the
results and collects the rows into a `Vec`. This can fail at any step:
the query might fail to run, or one of the rows might fail to be fetched
or processed.

Right now, we lump all those failures into a `Result` that wraps the
whole return value. This is only possible because we process each row
before returning. Once `Vec` is replaced by an iterator, we won't have
that luxury anymore, so we now wrap each individual element into its own
`Result` (as well as wrapping the whole vector into a `Result` of its
own).

1 files changed, 1 insertions, 0 deletions

diff --git a/src/cmd/list_files.rs b/src/cmd/list_files.rs
index 22b102e..7f0cfed 100644
--- a/src/cmd/list_files.rs
+++ b/src/cmd/list_files.rs
@@ -23,6 +23,7 @@ impl ListFiles {
 
         let gen = client.fetch_generation(&gen_id, temp.path())?;
         for file in gen.files()? {
+            let file = file?;
             println!("{}", format_entry(&file.entry(), file.reason()));
         }