FX31337 / MQL-Compile-Action

Compiles MQL programs via GitHub Actions
MIT License
7 stars 6 forks source link

Support for Linux/macOS #9

Closed kenorb closed 4 years ago

kenorb commented 4 years ago

Issuehunt badges

On non-Windows system, run EXE via Wine.

Currently it doesn't work:

/bin/sh: 1: MetaTrader 5/metaeditor64.exe: Permission denied

    at /__w/MQL-Compile-Action/MQL-Compile-Action/dist/index.js:9488:21
    at ChildProcess.exithandler (child_process.js:302:5)
    at ChildProcess.emit (events.js:210:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)

So if OS is non-Windows, we need to prefix command with wine.

Acceptance criteria.


IssueHunt Summary #### [bunysae bunysae](https://issuehunt.io/u/bunysae) has been rewarded. ### Backers (Total: $20.00) - [ea31337 ea31337](https://issuehunt.io/u/ea31337) ($20.00) ### Submitted pull Requests - [#20 Support linux](https://issuehunt.io/r/FX31337/MQL-Compile-Action/pull/20) --- ### Tips - Checkout the [Issuehunt explorer](https://issuehunt.io/r/FX31337/MQL-Compile-Action/) to discover more funded issues. - Need some help from other developers? [Add your repositories](https://issuehunt.io/r/new) on IssueHunt to raise funds.
issuehunt-oss[bot] commented 4 years ago

@ea31337 has funded $20.00 to this issue.


issuehunt-oss[bot] commented 4 years ago

@kenorb has rewarded $18.00 to @bunysae. See it on IssueHunt