Hi Imesh/Pubudu, Please find my comments inline.
On Tue, Aug 25, 2015 at 12:31 AM, Pubudu Gunatilaka <pubu...@wso2.com> wrote: > Hi, > > build.sh file fails when there are no plugins in the {product/plugins} > folder. We need to change the scripts as well. > pushd ${wso2_product_plugin_path} cp * ${script_path}/plugins popd The above 'cp *" command will not fail if there are no files inside ' wso2_product_plugin_path'. But however it will fail if there's no ' wso2_product_plugin_path'or'script_path}/plugins'which I guess is not possible. > I could see the same syntax errors in the other docker images as well. > Will do the needful. > > Thank you! > > On Mon, Aug 24, 2015 at 11:47 PM, Imesh Gunaratne <im...@wso2.com> wrote: > >> One other problem I noticed with this build.sh file is that it does not >> copy the plugin files to plugins folder when executed with clean. >> >> On Mon, Aug 24, 2015 at 11:41 PM, Imesh Gunaratne <im...@wso2.com> wrote: >> >>> Hi, >>> >>> I found the following issue in ESB 4.8.1 build.sh file and fixed: >>> >>> echo "-----------------------------------" >>> - echo "Building" ${wso2_product_type^^} - ${wso2_product_version} >>> "template module" >>> + echo "Building" ${wso2_product_type} - ${wso2_product_version} >>> "template module" >>> echo "-----------------------------------" >>> >>> >>> https://github.com/wso2/product-private-paas/commit/f2f763e7612eeae1a1e211b1fd97a970b5c369f1 >>> >>> This in-fact will change the 'wso2_product_type' to uppercase. However it will not work with bash versions below v4.0. So it's ok to remove them for now. > -- >>> *Imesh Gunaratne* >>> Senior Technical Lead >>> WSO2 Inc: http://wso2.com >>> T: +94 11 214 5345 M: +94 77 374 2057 >>> W: http://imesh.gunaratne.org >>> Lean . Enterprise . Middleware >>> >>> >> >> >> -- >> *Imesh Gunaratne* >> Senior Technical Lead >> WSO2 Inc: http://wso2.com >> T: +94 11 214 5345 M: +94 77 374 2057 >> W: http://imesh.gunaratne.org >> Lean . Enterprise . Middleware >> >> >> _______________________________________________ >> Dev mailing list >> Dev@wso2.org >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > > *Pubudu Gunatilaka* > Software Engineer > WSO2, Inc.: http://wso2.com > lean.enterprise.middleware > mobile: +94 77 4078049 > Thanks, -- *Lasindu Charith* Software Engineer, WSO2 Inc. Committer & PMC Member, Apache Stratos Mobile: +94714427192 | Web: blog.lasindu.com
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev