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

Change "" to '' for compatibility with linux #213

Open
paolomassa opened this issue Jun 5, 2024 · 7 comments
Open

Change "" to '' for compatibility with linux #213

paolomassa opened this issue Jun 5, 2024 · 7 comments
Assignees
Labels
bug Something isn't working reviewed 2024-10
Milestone

Comments

@paolomassa
Copy link
Collaborator

There are some places in the code in which strings are defined within "" (see, e.g., here and ). This is incompatible with linux distributions. It should be changed to ' ' in the entire software.

@paolomassa paolomassa added the bug Something isn't working label Jun 5, 2024
@samaloney
Copy link
Collaborator

@paolomassa has any progress been made on this? Are you sure the issue is the 's or the encoding of the source files?

@paolomassa
Copy link
Collaborator Author

We still need to implement the modifications, I'll do that soon. For what I know it should be just a problem of changing " with ' , nothing more.

@samaloney
Copy link
Collaborator

samaloney commented Oct 15, 2024

Ok I guess I just don't understand the problem "linux" doesn't have an issue with " or ' so it must be IDL on linux and even then I don't really understand. I have had encoding issues before ascii / utf-8 / utf-16 or latin1 etc

I created these tiny test scripts and they work on linux?

test_func.pro

function test_func

a = 'a'
b = "b"

print, a , b

end

test_pro.pro

pro test_pro

a='a'
b="b"

print, a, b

end
IDL> test_func()
ab
       0
IDL> test_pro
ab
``

@paolomassa
Copy link
Collaborator Author

For what I understood, the problem is that the IDL installation on linux has troubles sometimes when strings are defined within ". If I remember correctly, @PaipaPsyche reported this issue to me.

@PaipaPsyche could you please tell us more if it was you who found this issue? Thanks!

@PaipaPsyche
Copy link

I'm running SSW on an Ubuntu distribution. For some reason I always have an error when running the demo because of these "" strings (the subroutines cannot recognize the specified date/UID ). The error disappears when replacing the " for ' in the scripts.

@samaloney
Copy link
Collaborator

Do you happen to know the encoding of the file is?

@PaipaPsyche
Copy link

it's ASCII

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reviewed 2024-10
Projects
None yet
Development

No branches or pull requests

4 participants