Skip to content

Conversation

@wks
Copy link
Collaborator

@wks wks commented Feb 17, 2023

No description provided.

@qinsoon
Copy link
Member

qinsoon commented Feb 20, 2023

@wks I had some attempts on the issues. I can't reproduce it on moma machines. So I just debug it with our CI. I haven't fixed it yet. Here are what I found.

I logged the error. It should be this (https://github.com/mmtk/mmtk-jikesrvm/actions/runs/4214308783/jobs/7314873252):

error[E0463]: can't find crate for `core`
  |
  = note: the `i686-unknown-linux-gnu` target may not be installed
  = help: consider downloading the target with `rustup target add i686-unknown-linux-gnu`

So I tried to use --target for cargo msrv. This is the new error message (https://github.com/mmtk/mmtk-jikesrvm/actions/runs/4218986281/jobs/7323931285).

= note: /usr/bin/ld: skipping incompatible /usr/lib/x86_64-linux-gnu/libz.so when searching for -lz
          /usr/bin/ld: skipping incompatible /usr/lib/x86_64-linux-gnu/libz.a when searching for -lz
          /usr/bin/ld: cannot find -lz: No such file or directory
          /usr/bin/ld: skipping incompatible /usr/lib/x86_64-linux-gnu/libz.so when searching for -lz
          collect2: error: ld returned 1 exit status

error: could not compile `mmtk` due to previous error

@wks
Copy link
Collaborator Author

wks commented Feb 20, 2023

I guess the i686 target depends on a 32-bit version of libz, which needs to be installed via apt-get.

@qinsoon
Copy link
Member

qinsoon commented Feb 21, 2023

I guess the i686 target depends on a 32-bit version of libz, which needs to be installed via apt-get.

Yeah. That fixed the problem. Basically, we need to supply --target i686-unknown-linux-gnu when running cargo msrv (I don't know why it worked without this before), and also we need to install the dependency.

@qinsoon
Copy link
Member

qinsoon commented Feb 21, 2023

We should merge this after mmtk/mmtk-core#756, and change the mmtk dependency to the new commit of mmtk/mmtk-core#756.

@qinsoon qinsoon merged commit 552b4a0 into mmtk:master Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants