Dear all,
it is a pity to see that since many weeks there has been no progress on merging any of Andreas' work. I think everyone involved is to be blamed to some extent. Andreas because the provided branches are not kept clean, Ivan probably for lack of time, and me for not paying more attention and helping this process along.
The general life-cycle in any project, including osmo-pcu, should be as follows:
* contributor starts on some improvements, creates private branch * master advances meanwhile * contributor rebases (not merges!) his changes on top of more recent master * contributor indicates that his branch is ready for merge * maintainer merges (some of?) the commits of the contributor branch * contributor re-bases remaining commits on new master * contributor requests merge of remaing patches ...
If I open 'gitk' on any of the branches (master / jolly / jolly_dsp) I get grey hair (to say the least). One would normally expect:
1) 'jolly' to be based on some (maybe not current) master
2) 'jolly_dsp' to be based on 'jolly'
Neither of the two is the case. I've tried to * rebase jolly on top of master as well as * rebase jolly_dsp on top of jolly
and both are impossible due to the series of merges and the unclear ancestry / relationship of the branches. Or, just for fun, try 'git diff origin/jolly..origin/jolly_dsp. It shows you anything else but what one would expect
Andreas: Please take some time to clean up the mess. Your 'jolly' branch should contain a series of clean per-feature commit's on top of current master, and 'jolly_dsp' should be a set of few patches on top of 'jolly'.
This way there is always a clean queue of changesets that Ivan can merge (or cherry-pick). I can understand that if I was in Ivan's place, I would not really know where to even start merging some of those contributions.
Thanks for everyone's attention. Please take some time to get this resolved. Let me know if you have some questions. There are plenty of people with lots of git experience around (Holger, Pablo, myself) who can help if something is unclear.
Regards, Harald
Hi, all!
I absolutely agree with Harald and I am ready to use described life-cycle model for osmo-pcu. Before that time I merged branch 'jolly', when the differences between master and jolly become critical. I didn't coordinate merges with Andreas and jolly branch wasn't rebased before merges. I think, it was my mistake. Now I am ready to merge jolly branch at any time, when Andreas indicates that his branch is ready for merge.
2012/11/25 Harald Welte laforge@gnumonks.org
Dear all,
it is a pity to see that since many weeks there has been no progress on merging any of Andreas' work. I think everyone involved is to be blamed to some extent. Andreas because the provided branches are not kept clean, Ivan probably for lack of time, and me for not paying more attention and helping this process along.
The general life-cycle in any project, including osmo-pcu, should be as follows:
- contributor starts on some improvements, creates private branch
- master advances meanwhile
- contributor rebases (not merges!) his changes on top of more recent master
- contributor indicates that his branch is ready for merge
- maintainer merges (some of?) the commits of the contributor branch
- contributor re-bases remaining commits on new master
- contributor requests merge of remaing patches
...
If I open 'gitk' on any of the branches (master / jolly / jolly_dsp) I get grey hair (to say the least). One would normally expect:
'jolly' to be based on some (maybe not current) master
'jolly_dsp' to be based on 'jolly'
Neither of the two is the case. I've tried to
- rebase jolly on top of master
as well as
- rebase jolly_dsp on top of jolly
and both are impossible due to the series of merges and the unclear ancestry / relationship of the branches. Or, just for fun, try 'git diff origin/jolly..origin/jolly_dsp. It shows you anything else but what one would expect
Andreas: Please take some time to clean up the mess. Your 'jolly' branch should contain a series of clean per-feature commit's on top of current master, and 'jolly_dsp' should be a set of few patches on top of 'jolly'.
This way there is always a clean queue of changesets that Ivan can merge (or cherry-pick). I can understand that if I was in Ivan's place, I would not really know where to even start merging some of those contributions.
Thanks for everyone's attention. Please take some time to get this resolved. Let me know if you have some questions. There are plenty of people with lots of git experience around (Holger, Pablo, myself) who can help if something is unclear.
Regards, Harald
--
- Harald Welte laforge@gnumonks.org
============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)
Hi Alexander,
On Mon, Nov 26, 2012 at 06:22:57PM +0400, Alexander Chemeris wrote:
From what I see, all (or almost all) recent commits from Andreas are bug fixes. Ideally I'd love to see Andreas developing them on top of the current 'master'
agreed.
and submitting them to the mailing list as soon as he push them. This way Ivan (and everyone else) would be able to review them and merge as soon as possible.
We can configure a post-commit hook to automatically send any commit (or only commits to certain branches) to the osmo-pcu mailing list. I think right now they only end up at the generic osmocom.org commitlog list, whcih probably except Holger and me nobody is really following...
I hate the situation when fixes are stored in a private branch and are not merged into mainline.
agreed.
hi,
because i am not a git expert, i did not start cleanup yet. i will hopefully talk to harald on the phone this week. he might give me some hints about cleaning up my two branches (jolly and jolly_pcu). as soon as it is done, i will inform you on this list.
regards,
andreas
Hi Harald,
On Mon, Nov 26, 2012 at 7:23 PM, Harald Welte laforge@gnumonks.org wrote:
On Mon, Nov 26, 2012 at 06:22:57PM +0400, Alexander Chemeris wrote:
and submitting them to the mailing list as soon as he push them. This way Ivan (and everyone else) would be able to review them and merge as soon as possible.
We can configure a post-commit hook to automatically send any commit (or only commits to certain branches) to the osmo-pcu mailing list. I think right now they only end up at the generic osmocom.org commitlog list, whcih probably except Holger and me nobody is really following...
Well, I'm also following it with filter set to "pcu". :)
I don't think it's useful to have another post-commit hook. My point is that Andreas (and everyone else) should write to the mailing list when they think their patches are ready, which may be well after it's actually committed. I.e. it should be human decision when to post a patch.
-- Regards, Alexander Chemeris. CEO, Fairwaves LLC / ООО УмРадио http://fairwaves.ru
hi all,
i just moved the commits of jolly branch and jolly_dsp branch and created jolly_merge branch. it is now based on current master's head and includes all additional work i have done so far. therefore the other jolly-branches have been deleted. once the jolly_merge branch is merged (rebased) with master branch, it can be removed. harald suggested not to do any cleanups on master, because it is allot of work, and it could break functionality, if a mistake would be done while cleaning.
best regards,
andreas
Hi all,
jolly has gone through the effort to rebase all his changes on top of current master. How do we proceed from here?
My understanding was so far that Ivan is maintaining the PCU and repsonsible for reviewing/merging any suggested changes.
The current 'jolly_merge' branch seems to work quite fine, especially now with the fixes of the last two days. I would appreciate if we could get that merged, _before_ any other changes happen in master and again cause fragmentation.
Alternatively, we could of course see if Jolly should be permitted to generally push directly to master, or if all patches should be sent via 'git send-email' to this list, or if Jolly (or even I, but I know the code not well and have too many other projects) should become a co-maintainer of the project.
Let's work together to find a solution for the responsibilities and workflow. Thanks!
Hi, Harald!
I merged jolly_merge branch to master today, sorry for so long delay. I have done some tests and I think that now we have first stable version of osmo-pcu. So at this stage I guess that Andreas or you can commit or merge bug fixes to master without any approvals, I think that bug fixes shouldn't be stored in a private branches. But all features and fundamental changes in the code should be developed in a private branches and should be discussed in the mailing list before merging to master. I plan to resume the development of osmo-pcu soon, we should implement PCCCH/PBCCH support, so I will create new branch for this feature and at this stage I suggest to merge and commit only bug fixes to master branch.
2013/1/17 Harald Welte laforge@gnumonks.org:
Hi all,
jolly has gone through the effort to rebase all his changes on top of current master. How do we proceed from here?
My understanding was so far that Ivan is maintaining the PCU and repsonsible for reviewing/merging any suggested changes.
The current 'jolly_merge' branch seems to work quite fine, especially now with the fixes of the last two days. I would appreciate if we could get that merged, _before_ any other changes happen in master and again cause fragmentation.
Alternatively, we could of course see if Jolly should be permitted to generally push directly to master, or if all patches should be sent via 'git send-email' to this list, or if Jolly (or even I, but I know the code not well and have too many other projects) should become a co-maintainer of the project.
Let's work together to find a solution for the responsibilities and workflow. Thanks!
--
- Harald Welte laforge@gnumonks.org http://laforge.gnumonks.org/
============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)
Hi Ivan,
On Thu, Jan 17, 2013 at 01:21:27PM +0300, Ivan Kluchnikov wrote:
I merged jolly_merge branch to master today, sorry for so long delay. I have done some tests and I think that now we have first stable version of osmo-pcu.
FYI: I've applied some very minor fixes (copyright statement, printing of version information) and tagged the current master as 0.1.
osmocom-net-gprs@lists.osmocom.org