Skip to content

Changed the riscv64 compilation path to the wrong one #46154

Description

@luyahan

Version

18.x-staging

Platform

riscv64

What steps will reproduce the bug?

48cf890
Changed the riscv64 compilation path to the wrong one
18.x don't include the RISC-V 32-bit support in V8 the directory

What is the expected behavior?

build pass

What do you see instead?

Build failed

Activity

  1. added
    buildIssues and PRs related to Node.js builds or CI infrastructure.
    on Jan 11, 2023
  2. F3n67u commented on Jan 11, 2023

    @F3n67u
    Contributor

    @targos Seems that we should also add "don't land on v18.x" label to #45149 because in v18.x branch src/heap/base/asm/riscv64 has not renamed to riscv yet. See https://github.com/nodejs/node/tree/v18.x/deps/v8/src/heap/base/asm.

  3. targos commented on Jan 11, 2023

    @targos
    Member

    It's too late. The commit is already on v18.x. That's why this issue was opened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    buildIssues and PRs related to Node.js builds or CI infrastructure.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions