[bt-devel] Backend Proposal

Tue Jun 21 14:55:32 MST 2011

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 20.06.2011 21:24, Patrick Sebastian Zimmermann wrote:
> I have a pretty much finished rewrite of the config system in a branch. The 
> changes include a unit testing framework (based on QTest) and are meant to 
> unify config option and session handling. The code probably still has some 
> bugs, but I have tested the code quite a lot and I think I got most of the 
> bugs out already.

I've tried to review your changes several times, but never finished.
These are big changes and take a lot of time to review. Last time I
tried was months ago, so I don't remember any more and need to start
from scratch again. It would help a tiny bit if you could again merge
master into your branch and also my merge request.

> I am not sure whether you think this config system refactor is a good idea. But 
> if you think it should go into master it really makes sense to merge before 
> starting the frontend/backend refactor.

I believe you've done a serious amount of work. We should definitely
work this out before any other major refactoring efforts start. I
suspect that this won't be an easy merge, but I'm willing to study your
changes and try... when I have time...

> I am currently very short in time, but if it's not much that is needed to get 
> this into the tree I can at least give it a try.

Same problem with most of us. :|

> ps. There are three smaller bugfix merge requests still hanging in the queue...

I merged one of them, and commented on the other two because they need a
bit of clarification. I don't yet fully understand the problems they fix
nor am I sure whether these fixes are indeed correct.

> 
> On Monday, 20. June 2011 00:46:13 Greg Hellings wrote:
>> On Sun, Jun 19, 2011 at 4:54 PM, Gary Holmlund <gary.holmlund at gmail.com> 
> wrote:
>>> I would be willing to do the structural work of moving files into new
>>> directories and changing the backend and rendering into libraries. We
>>> would want to get agreement on the details before any work starts. The
>>> work should be done so that there is never a commit that does not
>>> compile and run. It can be broken down into a series of small steps that
>>> accomplish this.
>>
>> It's probably good to identify one component at a time to pull out of
>> the code base and do it that way.  And it probably should be the task
>> of whoever is going to take the reins for that component to move the
>> files and modify them as necessary (in connection with people working
>> on any components that rely on it).  Since each component probably
>> requires more than just copying the files to the new folder structure,
>> it's probably best for each component to be done in a branch of its
>> own, that way every commit on main works even if there is some
>> breakage in the branch. It also allows for multiple people, such as
>> the person creating the component and people working on components
>> that rely on it, to collaborate on factoring out each piece. Due to
>> the interconnectedness, commits on the branch may not work but we'll
>> be sure the project works before merging each component back onto
>> main.
>>
>>> I wonder if 3 components (backend, render, and BibleTime) are the right
>>> breakdown. There may be other things we should separate such as the
>>> config system.
>>
>> I'm not sure what our config system does that isn't part of the normal
>> Qt cross-platform config system.  I would have guessed that any config
>> systems would be part of the GUI component, but you probably know
>> better than I.
>>
>> --Greg
>>
>> _______________________________________________
>> bt-devel mailing list
>> bt-devel at crosswire.org
>> http://www.crosswire.org/mailman/listinfo/bt-devel
> 
> _______________________________________________
> bt-devel mailing list
> bt-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/bt-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)

iQgcBAEBAgAGBQJOARNMAAoJEHeFv6jk2xWmwalAAKfxu4lYdKzfPeJCYydshzBQ
oc6oI69nMKUedwmXp5X7cPaINTZ8cMwov0ZPNs11fXy5M6cI0GsyKgsFAGnVzi+6
UzVTmkj0poLo3S2e5pLiSfFS7ys3ozzBWuCfWlP1QKjQfZ+kb6xnbquO2zYeSgWG
UMVvX8zafm/30gWBsshmh12Ts0AaUEPtglvmFnyHOE4PNtzScF0iwc+5DJ99izyh
aNGByeZvylIS51TabDWJLXfpqucTbozTCotooN6+CCJR7ldIyjya/N8GJRhDzH89
CNHQ6Rqt6d6LJtgrf2hYUM9mVu2TwQCRVP5ycLg9vqgzJ2r/vJM9ll2lC/ptnFkB
jg0Tf2LMQwvatUAHbKvmQfrSiDbT/iRnsPlPeYPt9ob4vZs2enyB+c5Fu8BYrIVg
pl+ZonLEWH7aeL9POJ66T43zGlIp+ywsqLcQX8MYr0nLLq71CYM1GFYLy4B8aoFL
DPSwtZFg9OfoH72iUzJdzON/Q1w0QQIAasSaVVSvpe6AvRw0j8hmrjoYOfAwcX6b
eAMv6mnpVt8hMzNeAo3dVCWX2cG0UA2KPqvUoMBYhB4l0RUyRb2xjOvmaYdEmBnq
MHI8uYz/eKSU4kbhxFgpzhgCl2Ycd+Ku71vyFzatOd82FUorpj9MRSqsMbYcprpA
cpLjPt9cKQ44KYRMq4YvKImCVvI+1tV4ZO9UI+Pj+22f7axGnb8tVbnBh+n9HZX5
dJ0J8fWuZHe5lpasUMyt+Hq/AcibsjpkAUkyTjWRuV0D5mG7/BYvBXS2C73tQT/C
N8HeGQ0j1MiFwk+bSK7pdGQ+rdUJslVs1/f4QC7ipmAdP0hDedqtWcZZ/6cMnULy
VZfaaPZafO/F3DHsXAVJFi9WWMC6fhUdc3Ok3zkNP6XxZwBfxu3hmakkUPN5gQje
SjsJbll9Hzm2sTiKFsexPBcI38jdfwElcH7o95Y6FXHHufyXrbMZ2li61JQgaVbt
W+5xyuc2e0pUjT+6GGOP+sW4S3/IVYl/aqpcvmxT4Ctg9d1EUeG0b74dFsy2Gv2b
Z2hTwpkiVBHzPcmwxF8+tDhLDsiZQFo1jc84WInVpSQK0xoxv7ZuMfjWQZgZyvuw
wCvmSuuip5T7cPOgG1zDBND7PufrPBEOeEq0kpCaiH4PzA/bKET1DlaQbtV9FMOA
cPfcEDf7PNNLbERk9g82UziFpOuxHBINfIBXaNcpbMcgdY2VnNAXquyGLhMopXZI
WghIkiyf1X4BcEGSzqnAUVGYNSJaApkf6b8Po37K5u4VloiUA7jAeetoAyefPJ22
aOo3V6VE09CGIK+yrPTi0yyIMLgtspWxPPmqVVP8trUPnVROk3fHsAREeq8spxhD
+66YI9jCeLhZS+uvj/TyO0NEvw3dp1ziWGCnHIfpDqudRLacEI0374BRVMAPQBE1
4rkTOnSfNaH3u4ZLxNS/gQnESC72OzjHAjAzzAJpRFh1VwogogcUd26GKNcPeLT8
0xIsLkph4vCDSuOI3/GCk5U6Q10bSQ22ZYjqUmjAVpDC35InY3hHPTRDN0gAK8k6
FID3d4Ia6rdNW5zs7iuNjg/595Dm8FsC02mDfD56j3fJ7MtSE9E+Y0XSezyCnTyg
exseq6f3t0iB7BXkQOBSVi9oGKUPXZqKEMhbmLKjOVE1QCHWtTsT8Y3hUNnqBmaU
mjDzM1CXQvTDS+I8kN+pOUgcTCDDkAlQWCs0IQvIIyN5858I349pxVzjIjiWOb0/
DWD/o2vyrK7T4K/IgeM5mysOVkFkdGVnW9OWBwsAQ/m0q7Oke6nza5IvMVAox29w
W662LDq80JU+7kshSSW7jHdV3MUsIbPfA0VTZetAU1+3Cva71DjwoyhCaDCKNj9y
8A6hMcPgPdY4KDsfqOkI9Zc7SELGURpxsaSFN6a+e4wzcBc/QgmTPnIayxVNR8DZ
hR7mWOEwTNXr9r+p+RR8H+zmYyAKaAzVxSlOlHl1eprfqBCXE/llfrfk0reFrATe
qxSqc+fcLF8cuE4Gf3QAsijOmtTfbJv3yYLyucdSs/HtzN6dsyFn5QkKDhxGGAHY
ivsTtwarhIMue2n649/xqENKEJtH2crnIXE05cwunQWufJutUwmTee/cDE5QW03i
spaC8Dfyke9+NYHP+62QFpg7L2shBM4AmfNL574yIpWnRPVot4BVAffdsWEJzGiH
gQY5r1jZtBL4cgp0MEubLu+74253QUWc40cV8QcwlovsuMhgSv235pBBfKlT/6Nd
3nP9L83b05vS0hDqgZYZe+FsOGLO0EkkqOoxIQV06803QdaU1eDeZOThSziPPCO8
hvUr9AcrY8gbuCcCo0S/FMYQUV2l2PTg3jjVPo3cdm3EJ4Bd9zvHPO5Gt4S50zhd
r2e0NOLIs28xqRbJqLE+YSlkX73L5En3eb0FHTzLNzWO1p0aZdq7gSymvjQPnyRZ
V8Rz2aBnEjMKTl2XJ4i+8TQMos07E+2E3H5PdwGaVkb0m83fORG5wHL0rRmIYhBI
gREYU4f32fQONSxIh3lVvxV/puMrTksArG5h+9fI2pSHPvxQg3GMUXEIXJDH7ftF
RbUYHVY2rp8UbEhG7g6ykv6zn6ZW+t6Q/XUxLA7nuFOhincnvWoA6ld2tlS5Mlta
R7TAq/df4CKDthLBXoVRxfEm3lyer7VB7uMv+PaQA/1mOtvZNcyq/iwn5crsuic/
VoKAZkvc6ggK7SDF3px7
=lrxX
-----END PGP SIGNATURE-----