Am Donnerstag, 21. Februar 2019, 14:55:32 CET schrieb Ovidiu Sas: > After the push I realized that I forgot to update the author. My bad. > I wanted to revert the commit, but I didn't know if we have a preferred way > of doing it. > So, is there a guideline for reverting a commit? I will revert it and > re-commit with proper credentials.
Hello Ovidiu, don't worry. Just revert the commit: $ git revert 91b2c5da This will create a new commit that revert this one. Then you can commit the change again with your credentials, e.g. after applying the diff from the first patch set. This is the safest way to do in this situation, as it was already pushed to the public repository. Other ways have more side effects and are usually not necessary. Best regards, Henning > On Thu, Feb 21, 2019, 02:51 Henning Westerholt <[email protected]> wrote: > > Am Mittwoch, 20. Februar 2019, 21:12:07 CET schrieb root: > > > Module: kamailio > > > Branch: master > > > Commit: 91b2c5da34ba7064f858b13bf0a1d27da74336d4 > > > > > URL: > > https://github.com/kamailio/kamailio/commit/91b2c5da34ba7064f858b13bf0a1d2 > > 7 > > > > > da74336d4 > > > > > > Author: root <[email protected]> > > > Committer: root <[email protected]> > > > Date: 2019-02-21T07:11:25+11:00 > > > > > > siputils: fix debug logging > > > > > > --- > > > > > > Modified: src/modules/siputils/contact_ops.c > > > > > > --- > > > > > Diff: > > https://github.com/kamailio/kamailio/commit/91b2c5da34ba7064f858b13bf0a1d2 > > 7 > > > > > da74336d4.diff Patch: > > https://github.com/kamailio/kamailio/commit/91b2c5da34ba7064f858b13bf0a1d2 > > 7 > > > > > da74336d4.patch > > > > Hello, > > > > please add a proper User and e-mail address to your local git > > configuration: > > > > > > https://help.github.com/en/articles/setting-your-commit-email-address-in-g > > it https://help.github.com/en/articles/setting-your-username-in-git > > > > It is necessary to have a proper (human) author for all commits in the > > public > > repository. Please reply to this e-mail that we have it in the archives as > > well. > > > > Otherwise I think we should revert this commit (could be of course applied > > again). -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment _______________________________________________ Kamailio (SER) - Development Mailing List [email protected] https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
