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.

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

Compilation error with gdxrrw package on Linux #697

Open
lolow opened this issue Jul 9, 2024 · 3 comments
Open

Compilation error with gdxrrw package on Linux #697

lolow opened this issue Jul 9, 2024 · 3 comments

Comments

@lolow
Copy link

lolow commented Jul 9, 2024

An error occurs during the compilation of the gdxrrw package on Linux. This issue may be due to stricter rules enforced by the compiler.

rgdx.c: In function ‘checkRgdxList’:
rgdx.c:174:13: error: format not a string literal and no format arguments [-Werror=format-security]
  174 |       error(fieldErrorMsg);
      |             ^~~~~~~~~~~~~
rgdx.c:196:13: error: format not a string literal and no format arguments [-Werror=format-security]
  196 |       error(fieldErrorMsg);
      |             ^~~~~~~~~~~~~
rgdx.c: In function ‘rgdx’:
rgdx.c:528:14: error: format not a string literal and no format arguments [-Werror=format-security]
  528 |       error (buf);
      |              ^~~
rgdx.c:590:13: error: format not a string literal and no format arguments [-Werror=format-security]
  590 |       error(buf);
      |             ^~~
rgdx.c:602:15: error: format not a string literal and no format arguments [-Werror=format-security]
  602 |         error(buf);
      |               ^~~
cc1: some warnings being treated as errors

To resolve this, a few modifications can be made to the package. Please refer to the changes in the rgdx.c file at GAMS-dev/gdxrrw@master...lolow:gdxrrw:master

@tscheypidi
Copy link
Member

thanks for the hint!

@lolow
Copy link
Author

lolow commented Jul 22, 2024

Note that gdxrrw is not supported anymore by GAMS. They recommend to move to GAMS TRANSFERT, using the new package: https://github.com/GAMS-dev/transfer-r
I haven't tested it yet and I am not sure if it possible to do a smooth transition in our packages.

@tscheypidi
Copy link
Member

We are currently in the process of replacing gdxrrw with gamstransfer, but it is unfortunately not that easy as the output formats of both packages are quite different.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants