Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

[Cpp] Don't use outputPath for results file, ticket:4773 #2813

Closed
wants to merge 2 commits into from
Closed

[Cpp] Don't use outputPath for results file, ticket:4773 #2813

wants to merge 2 commits into from

Conversation

rfranke
Copy link
Member

@rfranke rfranke commented Dec 3, 2018

The results file has its own call argument -F (or -r for the C runtime)
that is typically used with an absolute file name, including path.
A results file with relative name should be placed in the current
directory according to discussions at dev meeting on 12/03, 2018.

Moreover raise an error if open of the results file fails.

The results file has its own call argument -F (or -r for the C runtime)
that is typically used with an absolute file name, including path.
A results file with relative name should be placed in the current
directory according to discussions at dev meeting on 12/03, 2018.

Moreover raise an error if open of the results file fails.
@OpenModelica-Hudson
Copy link
Member

The tests run correctly according to OpenModelica_TEST_PULL_REQUEST 2018-12-03_20-25-28.

OpenModelica-Hudson pushed a commit that referenced this pull request Dec 3, 2018
The results file has its own call argument -F (or -r for the C runtime)
that is typically used with an absolute file name, including path.
A results file with relative name should be placed in the current
directory according to discussions at dev meeting on 12/03, 2018.

Moreover raise an error if open of the results file fails.

Belonging to [master]:
  - #2813
OpenModelica-Hudson pushed a commit that referenced this pull request Dec 3, 2018
@rfranke rfranke deleted the outres branch December 4, 2018 06:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants