Review Request 109975: various fixes to sql scripts and package.json

Aleix Pol Gonzalez aleixpol at gmail.com
Fri Apr 12 17:19:49 UTC 2013


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



sql/setup.sh
<http://git.reviewboard.kde.org/r/109975/#comment22986>

    Shouldn't these come from the config.json file?


- Aleix Pol Gonzalez


On April 12, 2013, 2:40 p.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109975/
> -----------------------------------------------------------
> 
> (Updated April 12, 2013, 2:40 p.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> This patch does,
> 
>     * adds a sql/updates dir
>     * impoves the setup.sh file
>     * executes the sql/updates scripts before npm updates
>     * populates the package.json
>     ** adds a restriction to node
>     ** overrides the default functionality of npm start
> 
> 
> Diffs
> -----
> 
>   README 7515b65 
>   server/package.json e68c27c 
>   sql/setup.sh 82cf22e 
>   sql/updates/README PRE-CREATION 
>   sql/updates/do_updates.sh PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/109975/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130412/28e973d3/attachment.html>


More information about the Active mailing list