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

Building #3

Open
lischetzke opened this issue Jun 23, 2017 · 5 comments
Open

Building #3

lischetzke opened this issue Jun 23, 2017 · 5 comments

Comments

@lischetzke
Copy link

lischetzke commented Jun 23, 2017

I don't know, if I am too stupid, but I can't compile my .mq4 file. I've set the metaeditor.exe as SDK and choose the file. When I run, it exits with code 0, but no file where compiled and the log file is empty. The command-line, which executes by IntelliJ is:
"C:/Program Files (x86)/MetaTrader 4/metaeditor.exe" "/compile:MACD Sample.mq4" /log
Compiling manually over command-line works fine, but then I would use the full path to the .mq4 file. Hopefully someone can help.
I'm using version 171.1.16 of the plugin.

@mfursov
Copy link
Member

mfursov commented Jun 23, 2017

Looks like file name is missed in the command line. What is the actual mq4 file name?

@mfursov
Copy link
Member

mfursov commented Jun 23, 2017

Other options to check:

  1. Could you install and use metatrader from some folder inside your user dir?
    or
  2. Could you specify build folder inside your user dir?

@lischetzke
Copy link
Author

The name was removed by github, because I used "< my file >" for it.

I have tried several over things, which then solved the problem.
The problem was with "MACD Sample.mq4", because the name contains spaces and then the command-line isn't right. It shouldn't use quotation marks before and after /compile: and it doesn't if the name does not contain any spaces.

@mfursov
Copy link
Member

mfursov commented Jun 24, 2017

Good catch! Will be fixed next week.

@laendle
Copy link

laendle commented Jan 18, 2018

Good catch! Will be fixed next week.

Hello @mfursov - The last master branch commit (20 Jun 2017) predates this achnowledged issue of failing to compile files containing spaces due to misplaced quotation marks around cmdline parameters...

It would be great if a change to fix this issue is..

  1. uploaded to EAP which is still at version 171.1.16
  2. made available via the releases tab here on GitHub

After 7 months of inactivity, i'm sure others will agree with me in the hope that this is not abandonware.

Thank you

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