Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#2857 closed defect (fixed)

loadFile does not handle '\\' equivalent to '/'

Reported by: Lennart Ochel Owned by: somebody
Priority: high Milestone: 1.9.4
Component: Command Prompt Environment Version: trunk
Keywords: Cc: Mahder Alemseged Gebremedhin

Description

There are some calls in there that *say* the function converts \ to /. But maybe it doesn't for some reason.

This can be reproduced on Windows using the following snippet:

loadFile("D:\\Modelica\\OpenModelica1.9.0\\lib\\omlibrary\\Modelica 3.2.1\\package.mo"); getErrorString();
list(Modelica.Math); getErrorString();

Change History (9)

comment:1 by Martin Sjölund, 10 years ago

See also #2381

comment:2 by Mahder Alemseged Gebremedhin, 10 years ago

Works for me. With \\ on Windows.

Maybe I have missed something here.

comment:3 by Adrian Pop, 10 years ago

Maybe it doesn't work with \\ if you have a " " in the path.
I'll have a look sometime.

Last edited 10 years ago by Adrian Pop (previous) (diff)

in reply to:  3 comment:4 by Mahder Alemseged Gebremedhin, 10 years ago

Cc: Mahder Alemseged Gebremedhin added
Component: UnknownCommand Prompt Environment

Replying to adrpo:

Maybe it doesn't work with \\ if you have a " " in the path.
I'll have a look sometime.

You mean space by " " right?
Modelica 3.2.1 has a space. Still works :)

comment:5 by Martin Sjölund, 10 years ago

Can anyone reproduce this on a recent omc version?

comment:6 by Lennart Ochel, 10 years ago

No, I cannot reproduce it (r22551).

comment:7 by Lennart Ochel, 10 years ago

Resolution: fixed
Status: newclosed

Probably fixed some time ago.

comment:8 by Dietmar Winkler, 9 years ago

Milestone: Futurepre1.9.4

It doesn't make sense to keep closed ticket in the "Future" milestone that were simply forgotten to assign to the correct milestone in the past.

comment:9 by Martin Sjölund, 7 years ago

Milestone: pre1.9.41.9.4

Removing the pre1.9.4 milestone in favor of 1.9.4.

Note: See TracTickets for help on using tickets.