Re: [libvirt] [PATCH v3 2/6] tests: add targets for building libvirt inside docker containers

On Mon, 2019-04-01 at 11:39 +0100, Daniel P. Berrangé wrote: > On Fri, Mar 29, 2019 at 05:39:59PM +0100, Andrea Bolognani wrote: > > On Wed, 2019-03-27 at 17:10 +0000, Daniel P. Berrangé wrote: > > > It is neccessary to disable the gnulib submodule commit check because > > > this fails due to the way we have manually cloned submodule repos as > > > primary git repos with their own .git directory, instead of letting > > > git treat them as submodules in the top level .git directory. > > > > > > make[1]: Entering directory '/src/build' > > > fatal: Not a valid object name origin > > > fatal: run_command returned non-zero status for .gnulib > > > . > > > maint.mk: found non-public submodule commit > > > make: *** [/src/maint.mk:1448: public-submodule-commit] Error 1 > > > > This last part is interesting for people looking at the code but not > > for users, so I'd leave it out of the commit message. > > It is important as it shows the maint.mk rule that is causing > the problem we are fixing. Without that there's not enough context > to undestand the problem. My point is that this is really only interesting to people hacking on Makefile.ci, and it's just noise to all other developers going through the git log. So we should not drop the information, just move it to a comment inside Makefile.ci, possibly with the shell output snipped for brevity.