-
Notifications
You must be signed in to change notification settings - Fork 4
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
U/fjammes/dm 750 #5
Conversation
…loy.py, in order install procedure become eups-compliant: - MySQL options management in scisql-deploy.py - scisql-deploy.py is installed in bin directory
- creation of deployment configuration (MySQL credential, sciSQL prefix, vsuffix and version) - use of templating system to create temporary deployment files/script
- removal of substitute.py - management of MySQL client binary, plugin directory and credentials removed from waf build script
Defaults to ${PREFIX}.''') | ||
ctx.add_option('--mysql-config', type='string', dest='mysql_config', | ||
help='''Path to the mysql_config script (e.g. /usr/local/bin/mysql_config). | ||
Used to obtain the location of MySQL header files and plugins.''') |
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.
Is it still used to get the location of plugins?
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.
No, I remove it.
logging.fatal('Invalid MySQL socket. Use --mysql-socket options.' | ||
.format(args.mysql_socket) | ||
) | ||
exit(1) |
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.
indentation, sys.exit(1)
?
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.
Done, thanks.
Overall, I think this looks OK, and I'm happy that you managed to keep all the testing intact. I trust that you tested everything, so I didn't actually try out this code myself (but let me know if you'd like me to). Thanks for getting this done on such short notice! |
92f2c44
to
0555747
Compare
- Better isolation of shared library and client builds in waf script
deb7632
to
483fa5f
Compare
- more robust client mode - undeployment of plugin
- moving run_command() from configure module to scisql-deploy.py - improving indentation - BUGFIX : replacing exit() with sys.exit()
483fa5f
to
f712a1c
Compare
- Adding python module named 'utils'
Merging U/fjammes/dm-750 from fjammes, reviewed by smonkewitz
Thanks :-) On 01/10/2014 20:59, Daniel L. Wang wrote:
|
Hi Serge,
Code is finished, but I still need to write documentation. (I may also had colored log if I've enough time.)
In order to test, you can use and update test/test-install-process.sh.
Thanks,
Fabrice