On Mon, Apr 01, 2019 at 01:03:21PM +0200, Andrea Bolognani wrote: > 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. I really disagree. I want the commit messages to be more verbose so that details of what's being fixed are clearly visible. If you don't care about this detail fine, just ignore it, but that's not a reason to purge cut useful info out of the commit messages. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list