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.

Sign up for GitHub

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

Jump to bottom

ModelicaStrings.c does not compile on dSPACE targets #1273

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 10 comments
Closed

ModelicaStrings.c does not compile on dSPACE targets #1273

modelica-trac-importer opened this issue Jan 14, 2017 · 10 comments
Assignees
@beutlich
Labels
bug Critical/severe issue L: C-Sources Issue addresses Modelica/Resources/C-Sources
Milestone
MSL3.2.1+build.3

Comments

@modelica-trac-importer
Copy link

Reported by beutlich on 18 Sep 2013 13:35 UTC
After a78ddd1 for #1151 ModelicaStrings.c does no longer compile for dSPACE DS1006. I asked dSPACE Support about the other targets I cannot test.

Attached is a hotfix for the DS1006.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1273
@modelica-trac-importer modelica-trac-importer added this to the MSL3.2.2 milestone Jan 14, 2017
@modelica-trac-importer modelica-trac-importer added bug Critical/severe issue L: C-Sources Issue addresses Modelica/Resources/C-Sources labels Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Comment by beutlich on 24 Sep 2013 12:47 UTC
Resolved by 5167762.

@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 24 Sep 2013 13:19 UTC
That patch looks like a very bad hack. Isn't there a better way of doing this without having to specify all future devices of different manufacturers?

@modelica-trac-importer
Copy link
Author

Comment by beutlich on 24 Sep 2013 14:43 UTC
Replying to [comment:2 dietmarw]:

That patch looks like a very bad hack. Isn't there a better way of doing this without having to specify all future devices of different manufacturers?

Your are right. We tried to find a generic solution but did not succeed. The DS1006 is the only dSPACE board where __linux__ (or __gnu_linux__ if you like) and __GLIBC__ are defined but the locale-dependent versions of strtod and strtol are missing (even if compiled with -std=gnu99).

@modelica-trac-importer
Copy link
Author

Comment by beutlich on 17 Oct 2013 08:22 UTC
I checked several glibc releases from ftp://ftp.gnu.org/gnu/glibc/ and noticed that the locale-dependent versions of strtod and strtol first occur in \include\stdlib.h of glibc 2.3. The dSPACE DS1006 GCC environment uses older glibc 2.2. I propose to replace the bad hack for _DS1006 by appropriate checks for __GLIBC__ and __GLIBC_MINOR__.

@modelica-trac-importer
Copy link
Author

Comment by beutlich on 17 Oct 2013 09:51 UTC
Resolved by 1f08824 and 3fa1b52.

@modelica-trac-importer
Copy link
Author

Comment by beutlich on 17 Oct 2013 14:24 UTC
Last commit breaks compilation on Cygwin and MinGW.

@modelica-trac-importer
Copy link
Author

Comment by beutlich on 17 Oct 2013 14:33 UTC
Resolved by 8a6439a and 6d32054.

@modelica-trac-importer
Copy link
Author

Modified by dietmarw on 15 Nov 2013 13:37 UTC

@modelica-trac-importer
Copy link
Author

Modified by beutlich on 11 Jun 2014 08:39 UTC

@modelica-trac-importer
Copy link
Author

Changelog modified by beutlich on 11 Jun 2014 08:39 UTC
Fixed compilation of ModelicaStrings on dSPACE DS1006

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@beutlich beutlich

Labels
bug Critical/severe issue L: C-Sources Issue addresses Modelica/Resources/C-Sources
Projects
None yet
Milestone
MSL3.2.1+build.3
Development

No branches or pull requests
2 participants
@beutlich @modelica-trac-importer