[PATCH 3/5] sms: Add a function to update DB scheme v3 to v4.

[Holger Hans Peter Freyther]

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