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

%relay ppx: This function is a curried function where an uncurried function is expected #464

Closed
DZakh opened this issue Aug 31, 2023 · 12 comments

Comments

@DZakh
Copy link
Contributor

DZakh commented Aug 31, 2023

I'm migrating to ReScript 11 and got the error:
image

Unfortunately, I couldn't reproduce it in a sandbox, I am still trying... https://codesandbox.io/p/sandbox/rescript-sandbox-forked-6mr964
Maybe you know how it can be fixed?

@zth
Copy link
Owner

zth commented Aug 31, 2023

You're not trying to run in uncurried, right? Did you try cleaning?

@DZakh
Copy link
Contributor Author

DZakh commented Aug 31, 2023

Yep.

@DZakh
Copy link
Contributor Author

DZakh commented Aug 31, 2023

Can you try to compile the code in the curried folder https://github.com/DZakh/rescript-relay-curried-repro
It doesn't work for me.
At the same time the uncurried works fine, even though I didn't pass -uncurried to ppx.

@DZakh
Copy link
Contributor Author

DZakh commented Sep 1, 2023

Hm, I've forked the package, checkouted to 2.x and built ppx. Than I've added ../../../DZakh-forks/rescript-relay/packages/rescript-relay/rescript-relay-ppx/_build/default/bin/RescriptRelayPpxApp.exe to the flags and it worked correctly. Hmmm...

@DZakh
Copy link
Contributor Author

DZakh commented Sep 1, 2023

Can the ppx be cached on my machine somehow?

@zth
Copy link
Owner

zth commented Sep 1, 2023

Are you on an m1/2?

@DZakh
Copy link
Contributor Author

DZakh commented Sep 1, 2023

m1

@zth
Copy link
Owner

zth commented Sep 1, 2023

Then that's probably it. I commit the built binary to the repo for m1 because there's no GitHub runner for arm. So it might be that 2.0.2 has a binary that's not updated.

@DZakh
Copy link
Contributor Author

DZakh commented Sep 1, 2023

Could you fix it, please 😅 I'm stuck with updating our project to rescript V11

@zth
Copy link
Owner

zth commented Sep 4, 2023

Out in 2.0.3, reopen if it doesn't work.

@zth zth closed this as completed Sep 4, 2023
@DZakh
Copy link
Contributor Author

DZakh commented Sep 5, 2023

I've forgotten to write back. It works now, thanks

@mishaszu
Copy link

mishaszu commented May 21, 2024

I have the same issue with:

    "@rescript/core": "^1.3.0",
    "rescript": "^11.1.0",
    "rescript-relay": "^2.3.0"

On Ubuntu 22.04.4 LTS (WSL)

If I run install from rescript-relay dir I got:

Error: ENOENT: no such file or directory, copyfile '/home/user/project/node_modules/rescript-relay/ppx-linux' -> '/home/user/project/node_modules/rescript-relay/ppx'

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

No branches or pull requests

3 participants