mqlidea icon indicating copy to clipboard operation
mqlidea copied to clipboard

Building

Open lischetzke opened this issue 7 years ago • 5 comments

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.

lischetzke avatar Jun 23 '17 21:06 lischetzke

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

mfursov avatar Jun 23 '17 22:06 mfursov

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?

mfursov avatar Jun 23 '17 22:06 mfursov

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.

lischetzke avatar Jun 24 '17 09:06 lischetzke

Good catch! Will be fixed next week.

mfursov avatar Jun 24 '17 15:06 mfursov

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

laendle avatar Jan 18 '18 17:01 laendle