Move Compiler - set stdlib root

Move
#1

Hi ALL,

The Move compiler can not run on other computer because of the stdlib root has been set when rust gens the compiler.

I make a PR to set stdlib root, then compiler can run on the other computer with stdlib folder.

test

  1. cargo build --bin compiler
  2. run compiler on other computer
$ ./compiler ~/hello.mvir
thread 'main' panicked at 'Unable to read file: "/Volumes/backup/repos/libra/language/stdlib/modules/hash.mvir"', language/compiler/src/util.rs:18:29
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
  1. with this PR
$ ./compiler --stdlib-root ~/repos/bcx/libra/language/stdlib/ ~/hello.mvir

Thanks,

3 Likes
#2

Very nice! :ok_hand:

#3

https://github.com/sdkbox/vscode-libra-move/tree/dev has support compile Move contract locally.

#4

0.0.9 has been released.

#5

Hi @yinjimmy,

Thanks for bringing this up! I had a discussion with the team and we all welcome this change. Right now we do not merge pull requests unless they are tiny, but I’ll port/reimplement your changes manually and credit you – most likely in the commit message. (Taking my words back for now. Checking with the team on how we’ll do this.) There’s also some related cleanup we’d like to do together with this.

Again good catch and thanks for your contribution!

1 Like
#6

Hi Victor, glad to help! Please let me know how the clean up goes.

#7

Hi vgao,
how is the cleanup going ?

I think we can use -L to add library searching path. such as gcc -L option.

Then compiler can search stdlib/modules/hash.mvir with every library path.

function find_mvirs(mvirs):
    ret = []
    for mvir in mvirs:
        for path in lib_paths:
            if path.exists: ret.append(path)
    return ret

# build stdlib
function build_stdlib:
    for mvir in find_mvirs(["stdlib/modules/hash.mvir", ...]):
        do_compile_module(mvir)

// build stdio
function build_stdio:
    for mvir in find_mvirs(["stdio/modules/hash.mvir", ...]):
        do_compile_module(mvir)

usage:

./compiler -L ~/repos/bcx/libra/language/stdlib/ -L ~/repos/bcx/libra/language/others_lib ~/hello.mvir

# or 
./compiler -L~/repos/bcx/libra/language/stdlib/ -L~/repos/bcx/libra/language/others_lib ~/hello.mvir

is it ok?

UPDATE
maybe -I is better.

#8

Hi @vgao, how is it going?
Does it support set stdlib root path?