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

[Giorgos Tsiapaliokas]

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

(Updated May 3, 2013, 5:58 p.m.)


Review request for Bodega.


Changes
-------

fix the name of the variable


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 (updated)
-----

  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/c511ed43/attachment.html>