Review Request 110287: fix some issues in the sql script and move away from the npm scripts functionality

Giorgos Tsiapaliokas terietor at gmail.com
Fri May 3 17:46:35 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110287/
-----------------------------------------------------------

Review request for Bodega.


Description
-------

Hello,

this patch does

1. it requires 3 parameters now instead of 1.
The new parameters are the config.json filepath, the sql dir and the sql command.
2. it removes the npm scripts functionality

check this issue https://github.com/isaacs/npm/issues/3370#issuecomment-16892283

3. it adds the sql functionality into the makefile.


Diffs
-----

  server/Makefile a35b309 
  server/package.json 3ee556d 
  sql/do_sql 93bf764 

Diff: http://git.reviewboard.kde.org/r/110287/diff/


Testing
-------


Thanks,

Giorgos Tsiapaliokas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130503/4ef2edb4/attachment.html>


More information about the Active mailing list