After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 766026 - Cerbero doesn't use my name/email when writing commits
Cerbero doesn't use my name/email when writing commits
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: cerbero
unspecified
Other Linux
: Normal normal
: 1.8.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-05-05 13:33 UTC by Xavier Claessens
Modified: 2016-05-08 17:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
cerbero/git: Only set user if none is already configured (2.37 KB, patch)
2016-05-05 13:53 UTC, Xavier Claessens
none Details | Review
cerbero/git: Only set user if none is already configured (2.46 KB, patch)
2016-05-05 14:03 UTC, Xavier Claessens
committed Details | Review

Description Xavier Claessens 2016-05-05 13:33:10 UTC
When I'm hacking on GStreamer modules, I write my code and commits into cerbero's checkout, so I can easily test my code. But it won't put my name/email on those commits, and use "Cerbero Build System <cerbero@gstreamer.freedesktop.org>" instead.

That's a regression introduced by:
https://cgit.freedesktop.org/gstreamer/cerbero/commit/?id=17d4c4d64ca6947da6837fb119c94253d9e32b58
Comment 1 Tim-Philipp Müller 2016-05-05 13:50:50 UTC
As I see it, cerbero is not really a system for 'hacking'. I'm not a fan of how it uses git here, but that's how it is. The commit was needed to make stuff work at all, but I agree it should be made optional based on whether a global name is set or not. Care to make a patch?
Comment 2 Xavier Claessens 2016-05-05 13:53:27 UTC
Created attachment 327349 [details] [review]
cerbero/git: Only set user if none is already configured
Comment 3 Xavier Claessens 2016-05-05 13:59:16 UTC
(In reply to Tim-Philipp Müller from comment #1)
> As I see it, cerbero is not really a system for 'hacking'.
Not sure what it is then... ;-)

> Care to make a patch?
Sure!
Comment 4 Xavier Claessens 2016-05-05 14:03:35 UTC
Created attachment 327350 [details] [review]
cerbero/git: Only set user if none is already configured
Comment 5 Nicolas Dufresne (ndufresne) 2016-05-05 15:00:44 UTC
I think we should admit that there is some basic hacking ability that we use all the time when dealing with cerbero in real life projects. Specially when we need to test a small patch, going through recipe update with file bases patches all the time is not efficient.
Comment 6 Nicolas Dufresne (ndufresne) 2016-05-05 15:01:58 UTC
Review of attachment 327350 [details] [review]:

.
Comment 7 Nicolas Dufresne (ndufresne) 2016-05-05 15:23:10 UTC
commit f465d7ba52b4633044d27eb21d2146f2431c76d1
Author: Xavier Claessens <xavier.claessens@collabora.com>
Date:   Thu May 5 09:52:02 2016 -0400

    cerbero/git: Only set user if none is already configured
    
    https://bugzilla.gnome.org/show_bug.cgi?id=766026
Comment 8 Tim-Philipp Müller 2016-05-08 17:15:49 UTC
Thanks Xavier!