Skip to content

fix: update spawn#5593

Merged
rgrinberg merged 1 commit intomainfrom
ps/rr/fix__update_spawn
Apr 19, 2022
Merged

fix: update spawn#5593
rgrinberg merged 1 commit intomainfrom
ps/rr/fix__update_spawn

Conversation

@rgrinberg
Copy link
Copy Markdown
Member

imports nojb's windows fixes

@rgrinberg rgrinberg added this to the 3.1.1 milestone Apr 19, 2022
@rgrinberg rgrinberg requested a review from nojb April 19, 2022 18:35
@rgrinberg rgrinberg force-pushed the ps/rr/fix__update_spawn branch from 3e1cea1 to 58e3b30 Compare April 19, 2022 18:36
Copy link
Copy Markdown
Collaborator

@nojb nojb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't check the commit for update-spawn.sh, but LGTM otherwise! Thanks.

imports nojb's windows fixes

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

ps-id: 33181C51-CD5A-4C51-B0FE-C28512450D02
@rgrinberg rgrinberg force-pushed the ps/rr/fix__update_spawn branch from 58e3b30 to 09eac98 Compare April 19, 2022 21:05
@rgrinberg rgrinberg merged commit 09eac98 into main Apr 19, 2022
@rgrinberg rgrinberg deleted the ps/rr/fix__update_spawn branch April 19, 2022 21:05
rgrinberg added a commit to rgrinberg/opam-repository that referenced this pull request Apr 19, 2022
…ne-site, dune-rpc, dune-rpc-lwt, dune-private-libs, dune-glob, dune-configurator, dune-build-info and dune-action-plugin (3.1.1)

CHANGES:

- Fix build on Cygwin. (ocaml/dune#5593, fixes 5577, @nojb)

- Fix execution of `(system ..)` actions on Windows. (ocaml/dune#5593, fixes ocaml/dune#5523,
  @nojb)
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

Successfully merging this pull request may close these issues.

Can't build dune on Cygwin (sys/syscall.h: No such file or directory) "system" behaves differently since dune 3 (Windows only)

2 participants