-
Notifications
You must be signed in to change notification settings - Fork 484
feat(install): provide easy install script #859
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
Conversation
install/easy-install
Outdated
make | ||
sudo make install && sudo /usr/local/lib/fossology/fo-postinstall | ||
sudo cp install/src-install-apache-example.conf /etc/apache2/sites-available/fossology.conf | ||
sudo cp install/src-install-apache-example.conf /etc/apache2/sites-enabled/fossology.conf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use a2ensite
, which creates the corresponding symlink
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if that's the right way, because a2ensite only works on debian based systems.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then please symlink via
ln -s /etc/apache2/sites-available/fossology.conf /etc/apache2/sites-enabled/fossology.conf
That is the expected way of placing the config in sites-enabled
.
This will only work on some systems (e.g. the debian based ones), other systems (e.g. RHEL) might use /etc/httpd/conf.d/
instead of /etc/apache2/sites-enabled/
.
install/easy-install
Outdated
make clean | ||
sudo utils/fo-installdeps | ||
make | ||
sudo make install && sudo /usr/local/lib/fossology/fo-postinstall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here is the &&
unnecessary, just use a newline
c1272cb
to
90e77ff
Compare
This uses the instruction in the fossology wiki https://github.com/fossology/fossology/wiki/Install-from-Source
90e77ff
to
5576025
Compare
(followup from discussion): actually it's okay how it is. No user-data is deleted and the offical wiki also references fo-cleanold without parameters as first step. |
# Description: Easy install script for fossology. Goes through the steps on the Fossology wiki | ||
set -e | ||
cd "$(dirname $(readlink -f "${BASH_SOURCE[0]}"))"/.. | ||
sudo utils/fo-cleanold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that is maybe dangerous - I am not sure users would expect that deletion of the database, just not cleaning the DB would be OK
This uses the instruction in the fossology wiki
https://github.com/fossology/fossology/wiki/Install-from-Source