wasm-builder: Use riscv32emac-unknown-none-polkavm.json target

[jarkkojs]

Description

Closes #6335.

Integration

N/A

Review Notes

RuntimeTarget is converted to return path to the custom target JSON file

[jarkkojs]

How do I pass check-labels?

[athei]

Looks good. Just asking for a little bit of refactoring.

[jarkkojs]

@koute possible to approve this?

[github-actions]

Review required! Latest push from author must always be reviewed

[jarkkojs]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

[athei]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

This is because you opened the PR from your fork AFAIK. Create a branch on this repo to avoid this.

[jarkkojs]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

This is because you opened the PR from your fork AFAIK. Create a branch on this repo to avoid this.

Hmm...I might not fully understand what ur saying. I.e. I have branch in my fork and I opened PR. Isn't this the normal procedure.

Please fix my process and I will follow anything i.e. not challenging status quo ;-

[jarkkojs]

I fixup that error causing 51 tests in total to fail. My bad caused by my afternoon changes..

[athei]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

This is because you opened the PR from your fork AFAIK. Create a branch on this repo to avoid this.

Hmm...I might not fully understand what ur saying. I.e. I have branch in my fork and I opened PR. Isn't this the normal procedure.

Please fix my process and I will follow anything i.e. not challenging status quo ;-

What I am saying is: Don't use a fork. You can create a branch directly on this repository since you are part of the Parity organization.

[jarkkojs]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

This is because you opened the PR from your fork AFAIK. Create a branch on this repo to avoid this.

Hmm...I might not fully understand what ur saying. I.e. I have branch in my fork and I opened PR. Isn't this the normal procedure.
Please fix my process and I will follow anything i.e. not challenging status quo ;-

What I am saying is: Don't use a fork. You can create a branch directly on this repository since you are part of the Parity organization.

OK got it. I'll use that process in future PR's.

[jarkkojs]

Review required! Latest push from author must always be reviewed

@athei @koute was just a merge

This is because you opened the PR from your fork AFAIK. Create a branch on this repo to avoid this.

Hmm...I might not fully understand what ur saying. I.e. I have branch in my fork and I opened PR. Isn't this the normal procedure.
Please fix my process and I will follow anything i.e. not challenging status quo ;-

What I am saying is: Don't use a fork. You can create a branch directly on this repository since you are part of the Parity organization.

OK got it. I'll use that process in future PR's.

OK I fixed the assert and now it is giving that same 5 errors as it has been two'ish weeks :-)

[jarkkojs]

@athei, @koute: no other changes expect solved merge conflicts with Cargo.lock.