[PATCH] openbsc[master]: rename enum gprs_mm_state to gprs_gmm_state

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Neels Hofmeyr nhofmeyr at sysmocom.de
Wed Jun 1 11:11:52 UTC 2016


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 at sysmocom.de>
> Gerrit-Reviewer: daniel <dwillmann at sysmocom.de>

-- 
- Neels Hofmeyr <nhofmeyr at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20160601/66fe4caf/attachment.bin>


More information about the OpenBSC mailing list