/etc/profile.d/qt5.csh has incorrect syntax for setenv command

rkomar · 03-20-2022, 12:05 PM

The qt5.csh file in qt5-5.15.3_20220312_33a3f16f-i586-1 has the following line:

setenv QTWEBENGINE_CHROMIUM_FLAGS="--disable-seccomp-filter-sandbox"

but the '=' is not valid for csh, so it generates an error and no further processing is done within the /etc/csh.login script. The line should be:

setenv QTWEBENGINE_CHROMIUM_FLAGS "--disable-seccomp-filter-sandbox"

Petri Kaukasoina · 03-21-2022, 04:57 AM

Quote:

Originally Posted by rkomar

setenv QTWEBENGINE_CHROMIUM_FLAGS="--disable-seccomp-filter-sandbox"

No one has used (t)csh for decades, so this kind of error is left unnoticed. There is also a commented line '#setenv MALLOC_CHECK_=1' in glibc.csh and a few similar commented lines in libreoffice.csh of Alien's libreoffice package.

drumz · 03-21-2022, 08:28 AM

Quote:

Originally Posted by Petri Kaukasoina

No one has used (t)csh for decades, so this kind of error is left unnoticed. There is also a commented line '#setenv MALLOC_CHECK_=1' in glibc.csh and a few similar commented lines in libreoffice.csh of Alien's libreoffice package.

That doesn't mean they shouldn't be fixed!

marav · 03-21-2022, 08:34 AM

Fixed

Code:

Mon Mar 21 07:31:06 UTC 2022
l/qt5-5.15.3_20220318_e507d3e5-x86_64-1.txz:  Upgraded.
  Pulled from git again to fix missing liblocationlabsplugin.so.
  Fixed syntax error in qt5.csh. Thanks to rkomar.

rkomar · 03-21-2022, 10:34 AM

Quote:

Originally Posted by Petri Kaukasoina

No one has used (t)csh for decades, so this kind of error is left unnoticed...

I have been using tcsh for decades, so maybe not "no one". However, I agree that it isn't common, and I'm not surprised that the odd error comes up. No hard feelings on my side.