Respect the standards of Windows application installation

[anonymous]

(English translation)

Thank you for respecting the standards of installation of Windows and to allow the installation under c:\Program Files or c:\Program Files (x86). This is to allow a standard deployment on enterprise systems where these directories have appropriate security settings.

The non-conformancy w.r.t. good practices and standars is a blocker for a correct deployment of the program.

============================
merci de respecter les standards d'installation de Windows et d'autorise l'installation dans c:\program Fiels ou c:\program Files (x86). Et ce pour permettre un déploiement standard sur des postes d'entreprises ou ces répertoires comporte les sécurités adéquates.
Le non respect de ces recommandations et bonnes pratiques est == bloquant == pour un déploiement correct.

[Jan Kokert]

Google translate:

Please respect the Windows installation standards and allow installation in c: \ program Fiels or c: \ program Files (x86). And this to allow a standard deployment on corporate stations or these directories includes the appropriate security.
Failure to adhere to these recommendations and good practices is == blocking == for proper deployment.

[Adrian Pop]

Adeel, can you make it a warning not an error to install in a path with spaces so we can properly test if is working?

[Francesco Casella]

Tried it right now with the latest nightly and OMEdit, installed in C:\Program Files\. This is what I get

C:/Program Files/OpenModelica1.13.0-dev-64bit//share/omc/scripts/Compile.bat Modelica.Blocks.Examples.Filter gcc mingw64 parallel 4 0
PATH = "C:\Program Files\OpenModelica1.13.0-dev-64bit\tools\msys\mingw64\bin;C:\Program Files\OpenModelica1.13.0-dev-64bit\tools\msys\mingw64\bin\..\..\usr\bin;"
"C:/Program" non è riconosciuto come comando interno o esterno,
 un programma eseguibile o un file batch.
Compilation process failed. Exited with code 9009.

The OS message is in Italian but it basically seems that "C:/Program" is not a valid command. Not sure when this is invoked, but it seems it is quite early in the compilation process.

[Francesco Casella]

Another issue (orginally reported in #3769 and #3554) is that some users cannot run simulations because they lack admin rights and OMEdit wants to run executable files in %Localappdata%, %Localappdata%\Temp, and sub directories.

We should make sure that the default working directory chosen by OMEdit doesn't have this problem.

[Francesco Casella]

See also #1752 and the Compile.c file therein

[Francesco Casella]

This issue is important but not critical, and we have much more critical fixes to take care of. I'll retarget this to 2.0.0

[Francesco Casella]

See also the comments in #5192: the installation on a Windows 7 PC with a Cyrillic username causes OMEdit to crash. We should check with Cyrillic usernames, as well as usernames with Swedish, German, and Italian accented letter, which are probably quite common among European users.

[Francesco Casella]

Thanks to recent work by @adeas31, we should be able to install OpenModelica in the proper directories in 1.14.0 already.

[Adeel Asghar]

A 64bit nightly build is already available for testing now. ​https://build.openmodelica.org/omc/builds/windows/nightly-builds/64bit/

[Adrian Pop]

This is not working properly yet, I will remove the builds for now as they do not work as they should. See:
​https://openmodelica.org/forum/default-topic/2721-error-process-failed-to-start-no-such-file-or-directory#p9205

[Francesco Casella]

@adrpo, @adeas31, I'm not sure what was the latest Windows nightly build that still used the c:\OpenModelica path - I'd recommend that you copy that here ​https://build.openmodelica.org/omc/builds/windows/nightly-builds/64bit/ and remove all past nightly builds (if any) which are known to be broken.

The same applies to 32-bit installations.

[Francesco Casella]

As I reported during today's webmeeting, the new Windows installation of OMEdit works fine for me. However, OMNotebook fails with this error message:

record SimulationResult
    resultFile = "",
    messages = "Simulation execution failed for model: TestSuperheater
\"\"C:/Users/FRANCE~1/AppData/Local/Temp/OpenModelica/TestSuperheater.exe\"\" non � riconosciuto come comando interno o esterno,
 un programma eseguibile o un file batch.
"
end SimulationResult;
false

when I try to simulate a model. I guess the damn 6-character short Windows name are to blame, FRANCE~1 should be Francesco Casella.

This is too bad, because I'm using OMNotebook for teaching, so I need to install an older version to use it.

[Adeel Asghar]

FRANCE~1 is expected.
Can you run getErrorString() afterwards to see if there are anymore error messages.
What will happen if you run this via a mos-script.

[Francesco Casella]

Replying to adeas31:

FRANCE~1 is expected.

Good to hear. I'm having a bout of nostalgia for early '80s home computing, MS-DOS and Windows 3.1. Or maybe not. It's incredible we still have to deal with this crap after so many years :)

Can you run getErrorString() afterwards to see if there are anymore error messages.

getErrorString() returns ""

What will happen if you run this via a mos-script.

I put the model in a .mos script (attached) and ran it from the command line provided by OMEdit with omc test.mos > log.txt. The result is more or less the same

true
true
record SimulationResult
    resultFile = "",
    simulationOptions = "startTime = 0.0, stopTime = 1.0, numberOfIntervals = 500, tolerance = 1e-006, method = 'dassl', fileNamePrefix = 'TestSuperheater', options = '', outputFormat = 'mat', variableFilter = '.*', cflags = '', simflags = ''",
    messages = "Simulation execution failed for model: TestSuperheater
\"\"D:/Temp/OMEdit/TestSuperheater.exe\"\" non � riconosciuto come comando interno o esterno,
 un programma eseguibile o un file batch.
",
    timeFrontend = 0.019118,
    timeBackend = 0.0572006,
    timeSimCode = 0.0103997,
    timeTemplates = 0.1271479,
    timeCompile = 0.1424594
end SimulationResult;
""

[Adeel Asghar]

I think your model is failing for some other known reason. Do you have the file TestSuperheater.exe in D:/Temp/OMEdit/?

Guessing from the above two results it should also fail in OMEdit.

[Francesco Casella]

Replying to adeas31:

I think your model is failing for some other known reason. Do you have the file TestSuperheater.exe in D:/Temp/OMEdit/?

In fact, I haven't. I have all the .c and .h source code files, but no .o files and no .exe. It looks like the compilation didn't take place for some reason.

Guessing from the above two results it should also fail in OMEdit.

The same models run fine in OMEdit.

It also runs fine in OMNotebook if I use older versions installed in c:\openmodelica.

[Adeel Asghar]

That is very weird since all of them use the same compile script.

I will install the nightly build and test it myself tomorrow.

[Francesco Casella]

I tried a variant of test.mos which calls translateModel instead of simulate. It runs fine, getErrorString() returns "", but at the end of the day all source files are found in the working directory, but the C-compiler hasn't really run, nor has the linker, and their outputs are not in there.

BTW, running the GNU linker takes several seconds on Windows, so I would have noticed it running, while the script is actually executed in much less than a second. Too fast to be true :)

[Adeel Asghar]

c2275ed/OpenModelica should work fine.

[Francesco Casella]

Replying to adeas31:

c2275ed/OpenModelica should work fine.

OMNotebook now also works fine with the latest nightly build. I think we can close this ticket for good, if anybody still experiences problems, please reopen it.