search

hdl / containers

Building and deploying container images for open source electronic design automation (EDA)
https://hdl.github.io/containers/
Apache License 2.0
107 stars 24 forks source link

prjoxide: add debian/buster dockerfile #37

Closed umarcor closed 2 years ago

umarcor commented 3 years ago

Ref #35.

@tcal-x, this is an attempt at building prjoxide on a Debian Buster container. I used https://github.com/gatecat/prjoxide#building-the-prjoxide-tool as a reference.

The build is failing: undefined reference to llvm.x86.addcarry.64 (https://github.com/hdl/containers/runs/3093067566?check_suite_focus=true#step:4:296). In https://github.com/librespot-org/librespot/issues/555, they fixed it by adding a Cargo.lock file, but I don't know if that can be done in prjoxide.

/cc @gatecat

mithro commented 3 years ago

Ping - @tcal

tcal-x commented 3 years ago

Hi @umarcor , thank you for this PR and the investigation into the error! I tried adding --locked to the cargo install command, but it did not change the outcome for me. But maybe I was misunderstanding the fixes suggested in the links you found.

umarcor commented 3 years ago

@tcal-x my understanding is that --locked in Cargo might be similar to requirements.txt files in Python, go.mod in Golang, etc. That is, it requires a file upstream (in prjoxide) where the dependencies are pinned. See https://github.com/librespot-org/librespot/blob/dev/Cargo.lock in the project referenced above.

umarcor commented 3 years ago

There seems not to be problems on Debian Bullseye, so I pushed that to main. This PR is now for tracking the addition of the Debian Buster dockerfile.

tcal-x commented 3 years ago

Thanks @umarcor !

umarcor commented 3 years ago

FTR, nextpnr/nexus and nextpnr/prjoxide images are available now.

umarcor commented 2 years ago

Collection Debian Buster was deprecated.