-
Notifications
You must be signed in to change notification settings - Fork 63
some (?) distributions define GDL_PATH. They should NOT AT ALL! #1998
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
Comments
Moreover we should add in share/gnudatalanguage a "bin/idl" link to the installed gdl so that:
I'm sure @acoulais who also was involved in having NASA's solarsoft suite working with gdl will jump on these new infos! 😄 |
Sorry @olebole I wanted to attract the attention of the fedora manager, so it should be @opoplawski |
@opoplawski apparently the openSuse is derived from fedora, and, apparently again, the 'gdl.sh' in the openSuse distro defining GDL_PATH comes from fedora... |
What problems does setting Without setting |
I'm the current maintainer of gdl for openSUSE. I was contacted by GillesDuvert about the GDL_PATH issue and I went looking back in the changelog history of our package to identify when and why GDL_PATH was set. The openSUSE packages dates back to 2010 and when it was created was based on the fedora spec file. The fedora package started to set GDL_PATH in 2007 (fedora 5). I assume GDL_PATH was set in the openSUSE package because it was set in the fedora one. When I took over as package maintainer, I did try to figure out why GDL_PATH was set. Since I could not find a direct reference to why it was set, I assumed it was because it was necessary to find the functions in /usr/share/gnudatalanguage/lib. After GillesDuvert mentioned that setting GDL_PATH was not required, I did the following simple check: |
Ah, thanks for that - it pointed out that I was not testing properly. I do see now that it works without GDL_PATH (and IDL_PATH) not set. I'll make the change in the Fedora package. |
See #1992
The suse gdl installer creates a
/etc/profile.d/gdl.csh
that is not needed, AFAIK, and has adverse effects in the case of the solarsoft suite install.Originally posted by @GillesDuvert in #1992 (comment)
The text was updated successfully, but these errors were encountered: