-
Notifications
You must be signed in to change notification settings - Fork 21
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
Issues and errors in install script output in Ubuntu 18.10 (Ubuntu 18.04 LTS affected too) #36
Comments
|
Don't know whether this is a separate issue, but install fails on 18.04 as well. To be more exact - something is wrong with awp installation package -
Reinstall using
|
https://gist.github.com/teadur/12c1eccee063384d3b88eb46acb0cbf9 for example it assumes package is installed via sudo (usage of SUDO_USER variable) - very bad assumption for any script. I would think that adding the plugin to some system level place would make a bit more sense because there could be more then one user using the computer. It seems this package has had zero testing before released, it does zero validations in the script about the env, so many bad practices a bit dissapointing for so important software. |
Is it even common practice to do something like
in post-install script? I thought its purpose is to add users/groups/modules/whatnot, not install other packages. That's why there's probably also a warning
Very disappointed, this shouldn't have been released at all. Also I don't seem to have the possibility of rolling back to previous version of this software, can anyone confirm this? |
What is going on? Idemia shipped this half-assed software to RIA and RIA gladly accepted this without any QA?! Also as I understand the Idemia software contains binary only blobs with no source code available? |
Thank you for all that information. Best Regards |
Signed-off-by: Raul Metsma <raul@metsma.ee> #36
#36 Signed-off-by: Raul Metsma <raul@metsma.ee>
#36 Signed-off-by: Raul Metsma <raul@metsma.ee>
Thanks for reporting this, I almost tried testing the scripts again in Debian and you saved me this time! @tynisr can you share this link with your team and can you add a note indicating the installer doesn't work for GNU/Linux in the README ? It should also include requirements and a clear indication that sudo is required while warning users that downloading a script to execute with admin permissions is bad practice, one should carefully read its code before execution. |
@dkorzhevin can you please update the issue description to indicate 18.04 LTS support is broken too ? |
@MagicFab Hello. I've changed title. Hope this will help to get more attention on this problem. |
@tynisr, is there any updates on this? |
"awp" dependency is removed now: https://github.com/open-eid/linux-installer/pull/49/files |
Hello, I met next errors and warnings in output:
The text was updated successfully, but these errors were encountered: