I created a second pull request, since the original one did not shot the transcript properly on the terminal (there was a problem with my use of 'tail'). The failure summary still has a problem with newlines, but it is generally more informative.
Simon On Mon, Oct 16, 2017 at 8:46 PM, Simon Matthews <[email protected]> wrote: > II think that I have created a PR. Please let me know if you can see it. > > I forgot to change the order of the builds to fix the issue with api and > identity not being in the local repository. This change request addresses > the logging of failures. > > Simon > > On Sun, Oct 15, 2017 at 11:41 PM, Markus Geiss <[email protected]> > wrote: > >> Thanks for finding that. >> >> Can you please implement your suggested enhancements and send a PR? >> >> Cheers >> >> Markus >> >> .::Yagni likes a DRY KISS::. >> >> On Mon, Oct 16, 2017 at 1:07 AM Simon Matthews <[email protected]> >> wrote: >> >>> One more comment: it would be nice if the script exited at the first >>> error, instead of continuing, and potentially hiding an early build >>> failure. >>> >>> Or, it could save the text from the build failure and present it to the >>> user after finishing. >>> >>> Simon >>> >>> On Sun, Oct 15, 2017 at 3:45 PM, Simon Matthews < >>> [email protected]> wrote: >>> >>>> Getting a bit further, it looks like there are other dependency issues: >>>> >>>> ... >>>> Download https://jcenter.bintray.com/org/eclipse/jetty/jetty-server/ >>>> 9.3.11.v20160721/jetty-server-9.3.11.v20160721.jar >>>> [composite-build] Configuring build: /home/simon/FineractCN/permitt >>>> ed-feign-client/component-test >>>> :api FAILED >>>> >>>> FAILURE: Build failed with an exception. >>>> >>>> * What went wrong: >>>> Could not resolve all dependencies for configuration >>>> ':api:detachedConfiguration5'. >>>> > Could not find io.mifos.identity:api:0.1.0-BUILD-SNAPSHOT. >>>> Searched in the following locations: >>>> ... >>>> >>>> Simon >>>> >>>> >>>> On Sun, Oct 15, 2017 at 3:43 PM, Simon Matthews < >>>> [email protected]> wrote: >>>> >>>>> In the current version of the initial-setup.sh script, async is built >>>>> before api, but async appears to require libraries that are created in the >>>>> api build. >>>>> >>>>> Hence initial-setup.sh fails if the user has an empty local >>>>> repository. >>>>> >>>>> Simon >>>>> >>>> >>>> >>> >
