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

Giorgos Tsiapaliokas terietor at gmail.com
Fri Apr 12 19:12:06 UTC 2013



> On April 12, 2013, 4:26 p.m., Aaron J. Seigo wrote:
> > sql/setup.sh, lines 16-17
> > <http://git.reviewboard.kde.org/r/109975/diff/1/?file=138353#file138353line16>
> >
> >     bodega should be replaced here with the $database and $username vars

right


- Giorgos


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


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


More information about the Active mailing list