search

fd00 / yacp

Yet Another Cygwin Ports
MIT License
109 stars 58 forks source link

OpenCoarrays version bump & upstreaming of patches #25

Open zbeekman opened 4 years ago

zbeekman commented 4 years ago

Hi, OpenCoarrays maintainer here 👋

A few questions:

  1. Could you take a look at master or the latest release of OpenCoarrays and let me know if there are changes we should be making so that you don't have to apply patches?
  2. We're about to mint a new release and it would be great if a newer version of OpenCoarrays could be packaged with YACP. I am not a cygwin user, but for minor version bumps I'm happy to submit PRs, especially if we can get to a point where patching is no longer required.
  3. Why is ruby needed?
fd00 commented 4 years ago
  1. YACP is experimental. I think my patch is still dirty therefore it isn't appliable to official source repository.
  2. Same as above.
  3. Because my utility script (https://github.com/fd00/xezat) detects ruby file in source tarball. (However the result is not appropriate...)
zbeekman commented 4 years ago

Ok, great. It looks to me like the biggest patch is just to force dynamic linking only, and not package/distribute static libraries. Some other distros (Fedora) are doing this too, so my plan is to add an option like the one in your patch to force static or dynamic libs only. I'll see if I can port over other stuff too, then submit a rough-draft PR for the next version of OpenCoarrays.