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

Install-script not working completely #36

Closed
natrius opened this issue Sep 16, 2019 · 4 comments
Closed

Install-script not working completely #36

natrius opened this issue Sep 16, 2019 · 4 comments
Labels
bug Something isn't working

Comments

@natrius
Copy link
Contributor

natrius commented Sep 16, 2019

When trying to install it over an old installation

user@server:~/hauk/Hauk$ sudo ./install.sh -c /var/www/hauk/
./install.sh: 31: [: /var/www/hauk/: unexpected operator
./install.sh: 31: [: /var/www/hauk/: unexpected operator
WARNING! Target directory is not empty. If you proceed with the
installation, all files in the directory will be deleted.
./install.sh: 100: read: Illegal option -e
./install.sh: 101: ./install.sh: [[: not found
Aborting...

after moving the folder and reinstalling the followind appears

user@server:~/hauk/Hauk$ sudo ./install.sh -c /var/www/hauk/
./install.sh: 31: [: /var/www/hauk/: unexpected operator
./install.sh: 31: [: /var/www/hauk/: unexpected operator
./install.sh: 77: read: Illegal option -e
./install.sh: 78: ./install.sh: [[: not found

-e Installation complete!
Before you use Hauk, make sure to change Hauk's configuration.
The configuration file can be found at:
/etc/hauk/config.php

Best regards

@bilde2910 bilde2910 added the bug Something isn't working label Sep 17, 2019
@bilde2910
Copy link
Owner

Apparently my shell installation uses bash when I told it not to. This is a bug; I'll work on a fix promptly

@natrius
Copy link
Contributor Author

natrius commented Sep 17, 2019

One further suggestion:
Mark the "The configuration file can be found at:" as bold because i was to blind to read everything and tried for some minutes to change the config in the /var/www/hauk/include/ folder :)

Reading "Installation cmplete!" going "Yeah!" and not reading more. When someone is installing it the first time, i don't think the mistake will happen.

@bilde2910
Copy link
Owner

Good suggestion. If the $EDITOR environment variable is set, the installer will usually also ask if you want to open it for editing. But having it be bold, especially in cases where it doesn't do that, is also useful.

@bilde2910
Copy link
Owner

The changes have been pushed, so you should have a working shell script now if you pull the latest code from the repo.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants