Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Topiary v0.6.0 #27367

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Topiary v0.6.0 #27367

wants to merge 2 commits into from

Conversation

Niols
Copy link
Contributor

@Niols Niols commented Feb 4, 2025

@mseri
Copy link
Member

mseri commented Feb 5, 2025

It builds wherever it can. Is the update to cargo lock v4 necessary?


#=== ERROR while compiling topiary.0.6.0 ======================================#
# context              2.4.0~alpha1~dev | linux/x86_64 | ocaml-base-compiler.4.03.0 | pinned(https://github.com/tweag/topiary-opam/releases/download/v0.6.0/source-code-with-submodules.tar.xz)
# path                 ~/.opam/4.03/.opam-switch/build/topiary.0.6.0
# command              ~/.opam/opam-init/hooks/sandbox.sh build cargo build --release --package topiary-cli
# exit-code            101
# env-file             ~/.opam/log/topiary-7-ed0a71.env
# output-file          ~/.opam/log/topiary-7-ed0a71.out
### output ###
# error: failed to parse lock file at: /home/opam/.opam/4.03/.opam-switch/build/topiary.0.6.0/Cargo.lock
# 
# Caused by:
#   lock file version `4` was found, but this version of Cargo does not understand this lock file, perhaps Cargo needs to be updated?

@Niols
Copy link
Contributor Author

Niols commented Feb 5, 2025

It looks like Topiary now requires Rust 1.83.0 anyway, and probably it comes with Cargo lock v4?

Co-authored-by: Marcello Seri <[email protected]>
Niols added a commit to tweag/topiary-opam that referenced this pull request Feb 5, 2025
Recommanded by @mseri in
ocaml/opam-repository#27367; should help with us
requiring a more recent version of Rust.
@mseri
Copy link
Member

mseri commented Feb 5, 2025

I have been playing around with this and since 2024 is still unstable, topiary would be completely uninstallable. We may revisit this in the future. For now I think it is better to stick with 2021 until the next release, when 2024 will be stable. What do you think?

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