#3 pulled v1.0.1 fails to build

Opened by dubiousjim on February 22, 2021
dubiousjim on February 22, 2021

Pulled the latest changesets as of 22 Feb. pijul log begins with:

Change 5LSYTRQ6IOVUW26VJW5SWGFEIB7T2N4PVEB6VMNMR5ZHQ75MFOQAC
Author: [Author { name: "pmeunier", full_name: Some("Pierre-Étienne Meunier"), email: None }]
Date: 2021-02-21 23:16:03.223821651 UTC

    More docs, example, and fixing the free page diagnostic function for mutable transactions

Happy that you’ve hit 1.0 on these changes. But when I try to build, with cargo clean; cargo build, I get:

   Compiling sanakirja-core v1.0.1 (/Users/jim/repo/rusty/pijul/sanakirja-1.0/sanakirja-core)
error[E0423]: expected value, found module `page`
   --> sanakirja-core/src/btree/page.rs:530:17
    |
530 |                 page,
    |                 ^^^^ not a value

error[E0425]: cannot find value `freed` in this scope
   --> sanakirja-core/src/btree/page.rs:531:17
    |
531 |                 freed,
    |                 ^^^^^ not found in this scope

warning: unreachable statement
   --> sanakirja-core/src/btree/page.rs:529:13
    |
524 | /             return if m.modified.c0.is_leaf {
525 | |                 super::page_unsized::merge::<_, _, _, _, Leaf>(txn, m)
526 | |             } else {
527 | |                 super::page_unsized::merge::<_, _, _, _, Internal>(txn, m)
528 | |             };
    | |_____________- any code following this expression is unreachable
529 | /             return Ok(Op::Merged {
530 | |                 page,
531 | |                 freed,
532 | |                 marker: core::marker::PhantomData,
533 | |             });
    | |_______________^ unreachable statement
    |
    = note: `#[warn(unreachable_code)]` on by default

error: aborting due to 2 previous errors; 1 warning emitted

Some errors have detailed explanations: E0423, E0425.
For more information about an error, try `rustc --explain E0423`.
error: could not compile `sanakirja-core`

Is this the same version that was pushed to crates.io? If not, how do I get those sources?

pmeunier on February 22, 2021

Looks like a bug in Pijul.

pmeunier on February 22, 2021

If you just want to get going, remove the second return statement.