ReasonNativeProject icon indicating copy to clipboard operation
ReasonNativeProject copied to clipboard

Published 20 hours ago • verified publisher icon reasonml

Port to jbuilder

Open rgrinberg opened this issue 8 years ago • 8 comments

This PR requires a minimum version of jbuilder: 1.0+beta9. This is the first version when reason was introduced.

rgrinberg avatar May 13 '17 01:05 rgrinberg

Wow thanks for providing an example so I can get a sense for how to use jBuilder. I don't have a lot of experience setting up native opam projects, so this is helpful to see.

  • I think you mis-named the opam file (it should say reason-native not react-native).
  • How does the README change? Does anything else get simpler?

In addition, I think what people would really want to see in a follow up diff:

  1. How you depend on another opam package. Supposed you wanted to depend on @bordoley's Immutable opam package. Maybe in a follow up diff you could show what that would look like?
  2. How you would export a consumable library, and a binary.

I can create a playground branch if you want to push your work to it. We probably wouldn't want to encourage using jBuilder until it stabilizes, but a playground branch would be an easy way for you to try things out while jBuilder stabilizes.

jordwalke avatar May 13 '17 02:05 jordwalke

Sounds reasonable.

rgrinberg avatar May 13 '17 02:05 rgrinberg

The pin is no longer necessary! Might want to update and we can start pointing people to this PR so they can test it out and get some feedback. Thanks again, @rgrinberg !

jordwalke avatar May 20 '17 09:05 jordwalke

@rgrinberg It would also be really great if we could use this as an opportunity to teach people how to use opam effectively for local projects. For example, would you be able to suggest a workflow to teach newcomers:

  • How to create a switch just for this project.
  • How to install just the dependencies for the project into the switch in one command.

I notice people mess these steps up frequently - especially when they have existing packages already installed in the global switch, which conflict.

jordwalke avatar May 20 '17 09:05 jordwalke

Bug report: When I clone down this PR and run make build, I get the following error:

Executable test in _build/default/reason-native-project doesn't have a corresponding .ml file

It seems it is overlooking the .re within that folder.

gilbert avatar May 22 '17 01:05 gilbert

@gilbert Which version of jbuilder are you using? I hear the latest does not require you to pin that git URL - you merely need to install beta v9.

jordwalke avatar May 22 '17 05:05 jordwalke

@jordwalke I thought I was using the latest, but it turns out I wasn't. Using jbuilder 1.0+beta9 worked, thank you!

gilbert avatar May 23 '17 00:05 gilbert

Hopefully this gets merged soon!

kennetpostigo avatar Jun 09 '17 13:06 kennetpostigo