[PATCH] openbsc[master]: rename enum gprs_mm_state to gprs_gmm_state
Review at https://gerrit.osmocom.org/157 rename enum gprs_mm_state to gprs_gmm_state Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 --- M openbsc/include/openbsc/gprs_sgsn.h 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/57/157/1 diff --git a/openbsc/include/openbsc/gprs_sgsn.h b/openbsc/include/openbsc/gprs_sgsn.h index 898b7a5..5fbbf28 100644 --- a/openbsc/include/openbsc/gprs_sgsn.h +++ b/openbsc/include/openbsc/gprs_sgsn.h @@ -23,7 +23,7 @@ enum gsm48_gsm_cause; /* TS 04.08 4.1.3.3 GMM mobility management states on the network side */ -enum gprs_mm_state { +enum gprs_gmm_state { GMM_DEREGISTERED, /* 4.1.3.3.1.1 */ GMM_COMMON_PROC_INIT, /* 4.1.3.3.1.2 */ GMM_REGISTERED_NORMAL, /* 4.1.3.3.2.1 */ @@ -98,7 +98,7 @@ struct llist_head list; char imsi[GSM23003_IMSI_MAX_DIGITS+1]; - enum gprs_mm_state mm_state; + enum gprs_gmm_state mm_state; uint32_t p_tmsi; uint32_t p_tmsi_old; /* old P-TMSI before new is confirmed */ uint32_t p_tmsi_sig; -- To view, visit https://gerrit.osmocom.org/157 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 Gerrit-PatchSet: 1 Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Patch Set 1: Code-Review+2 hwelte had posted +2 on identical commit abandoned for gerrit reasons -- To view, visit https://gerrit.osmocom.org/157 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 Gerrit-PatchSet: 1 Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> Gerrit-HasComments: No
WTF? This has +2 and +1 from Jenkins, and no "merge conflict". Yet when I click on 'submit', Gerrit says: 'Code Review - Error change is new [continue]' Also, the two following commits from the same branch were not merge conflicted yesterday after I submitted them. Today they are marked as conflict. This nonsense is really getting on my case. These commits are really trivial, yet Gerrit can't get its shit together. This is ONE LETTER! <steam> I hate it when simple tasks get overcomplicated by automation (that fails). I'm tempted to merge to master manually, but I should rather calm down and we should resolve the underlying problems, or we'll face this over and over again. I'm going to merge over 100 Iu branch commits like this, so it's worth it. I tried to search online for answers but so far I'm completely stumped. Any ideas? For now I tried to hit the 'rebase' button. Of course now I have to wait for another jenkins +1. Not sure how to handle this -- if I just +1 manually and submit, I may introduce errors. If I don't, this might race indefinitely, while other changes get merged to master I have to repeatedly rebase and wait for jenkins, and my change may never merge, however simple it is. Also, every single patch sitting in the queue has to be rebased all the time, apparently. Really? Again, any ideas? ~Neels On Tue, May 31, 2016 at 11:53:26AM +0000, Neels Hofmeyr wrote:
Review at https://gerrit.osmocom.org/157
rename enum gprs_mm_state to gprs_gmm_state
Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 --- M openbsc/include/openbsc/gprs_sgsn.h 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/57/157/1
diff --git a/openbsc/include/openbsc/gprs_sgsn.h b/openbsc/include/openbsc/gprs_sgsn.h index 898b7a5..5fbbf28 100644 --- a/openbsc/include/openbsc/gprs_sgsn.h +++ b/openbsc/include/openbsc/gprs_sgsn.h @@ -23,7 +23,7 @@ enum gsm48_gsm_cause;
/* TS 04.08 4.1.3.3 GMM mobility management states on the network side */ -enum gprs_mm_state { +enum gprs_gmm_state { GMM_DEREGISTERED, /* 4.1.3.3.1.1 */ GMM_COMMON_PROC_INIT, /* 4.1.3.3.1.2 */ GMM_REGISTERED_NORMAL, /* 4.1.3.3.2.1 */ @@ -98,7 +98,7 @@ struct llist_head list;
char imsi[GSM23003_IMSI_MAX_DIGITS+1]; - enum gprs_mm_state mm_state; + enum gprs_gmm_state mm_state; uint32_t p_tmsi; uint32_t p_tmsi_old; /* old P-TMSI before new is confirmed */ uint32_t p_tmsi_sig;
-- To view, visit https://gerrit.osmocom.org/157 To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 Gerrit-PatchSet: 1 Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
-- - Neels Hofmeyr <nhofmeyr@sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Alt-Moabit 93 * 10559 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschäftsführer / Managing Directors: Holger Freyther, Harald Welte
On 01 Jun 2016, at 13:11, Neels Hofmeyr <nhofmeyr@sysmocom.de> wrote:
Hi!
This nonsense is really getting on my case. These commits are really trivial, yet Gerrit can't get its shit together. This is ONE LETTER! <steam> I hate it when simple tasks get overcomplicated by automation (that fails).
your super negative attitude is super annoying too. I don't have to spend time reviewing, so please go back to patchwork and be happy (but I remember your cursing about setting up a MTA as well). holger
On Wed, Jun 01, 2016 at 02:17:02PM +0200, Holger Freyther wrote:
your super negative attitude is super annoying too. I don't have to spend time reviewing, so please go back to patchwork and be happy (but I remember your cursing about setting up a MTA as well).
I should not have posted that mail. My apologies. ~Neels
Neels Hofmeyr has submitted this change and it was merged. Change subject: rename enum gprs_mm_state to gprs_gmm_state ...................................................................... rename enum gprs_mm_state to gprs_gmm_state Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 --- 0 files changed, 0 insertions(+), 0 deletions(-) Approvals: Neels Hofmeyr: Looks good to me, approved Jenkins Builder: Verified -- To view, visit https://gerrit.osmocom.org/157 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibba054d15c55c7ac570e64ff66ea57964be095e3 Gerrit-PatchSet: 2 Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
participants (3)
-
Holger Freyther -
Neels Hofmeyr -
Neels Hofmeyr