Skip to content
Snippets Groups Projects

Draft: main/rust: rebuild with llvm17

Closed Patrycja Rosa requested to merge ptrcnull/aports:rust-73-llvm17 into master
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
29 29 # several times. For this reason we build cargo with vendored libgit2.
30 30 makedepends_build="
31 31 !libgit2-dev
32 32 cargo-bootstrap=>$_bootver
  • Patrycja Rosa added 1 commit

    added 1 commit

    • 35168ba0 - main/rust: rebuild with llvm17

    Compare with previous version

  • omni added 1 commit

    added 1 commit

    • d9ab6744 - 1 commit from branch alpine:master

    Compare with previous version

  • Developer

    oh, uhm!

  • Developer

    It was only a rebase I accidentally reverted, right?

  • Developer

    Does it now fail because rust-1.73.0-r0 was built with revert-pr-114382.patch?

  • Contributor

    you can do the rebuild without removing the patch

    not sure about after that tho

  • omni mentioned in merge request !60301 (merged)

    mentioned in merge request !60301 (merged)

  • Developer

    @Celeste thinks we need "half a patch", I'll try and find out what that may look like.

  • Contributor

    half a patch

    keeping the last 2 file diffs (cmp.rs/intrinsics.rs) and removing the rest of the patch works it seems

  • Contributor

    works it seems

    (as in it doesn't crash like this, have to see about arm* of course)

  • Developer

    If it does, should we then rebuild again completely without the patch or try to move on to 1.74.0?

  • Contributor

    then completely without the patch, then 1.74

  • Developer

    I'm one who doesn't understand why we can't jump ahead to the latest release (currently 1.75.0) and skip mid-releases, but my rust "knowledge" is very shallow.

  • Developer

    Closing since !60301 (merged) and moving on to !60307 (merged)

  • closed

  • Please register or sign in to reply
    Loading