On Sun, Feb 09, 2014 at 01:28:05PM +0400, Alexander Chemeris wrote:
Hi,
I'm glad that you share my sadness that jolly
branches are not yet
merged. I hope you'll find some time to continue reviewing the
patchset for L1 parts, which jolly submitted quite a while ago.
but as you pointed out in the other SMS email, he needs to add tests
and things like (errno == 111) is an obvious no go.
As much as I love programming and contributing, at
this moment I have
to focus on enabling other people to contribute. I hope that one day I
could give up my management tasks and dive deep into writing lots of
code again. That said, I'm happy that Fairwaves is funding substantial
amount of work on Osmocom, including OsmoPCU, OsmoTRX, UmTRX and a bit
of OsmoBTS and OsmoNITB. And we're only increasing amounts of this
funding, as well as attracting more community members.
Please stop the marketing here. But if you really want to help, maybe
mentor Andreas and review his patches before it goes to the mailinglist.
I have spent way too much time on this already, it is not economic for
me to do his QA. I have cats to feed!
Please try to
contribute and finish this patch or at least just make
your modification available to the community.
I realized, that this is actually the final working version of the
patch. I took you so long to get back to it, so I was thinking it's
the original patch which was broken. Feel free to merge this into
master.
It lacks a test case and the doubt if it works or not doesn't help
either. But sure it is easy to demand things from others. ;)
holger