[Server-devel] XS-0.7 review round 3

[Martin Langhoff]

On Friday, February 10, 2012, Daniel Drake wrote:

> Everything acked so far has been pushed and built, output is at
> http://dev.laptop.org/xs/repos/stable/olpc/xs-0.7/i386/
>
> Pending items are:
>
> 1. Moodle - needs 'pu' branch review, and the possible updates you
> mentioned.
> For now, the repo above includes a RPM from the pu branch (just
> temporarily so that I can build).
>

Your patches look good. I am a bit concerned that mod_admin_extra might
have changed the parameter quoting rules (for the better & saner!). IOWs,
is ejabberdctl_escapeshellarg() still needed? If things work well and your
Online group is named Online (instead if 'Online' with the single quotes as
part of the string...) then it's fine...


> 2. xs-activation - this wasn't working, pushed a new 'pu' branch for
> review.
>

On plane at the moment, will review asap :-)


> 3. Version numbers of the components. I haven't bumped any of them.
> Some had already been bumped for the 0.7 release when it was looking
> like it might be F11/F14-based. Which ones should I bump? ds-backup,
> idmgr, xs-activation, xs-activity-server, xs-config, xs-rsync,
> xs-tools
>

Jerry spun an "Australian" XS build based on my partial work, IIRC. Maybe
he used those numbers. And then again, it's a small number of machines that
have Jerry managing them, so can't be a big issue.

4. I put xs-release as version 6 (matching epel and centos packages),
> with Epoch 1 so that it is seen as an upgrade over the current
> xs-release-9 in XS-0.6. Is that OK?
>

Yep, that's right.

 > I've done an install based on the above repo and done quite a bit of

> testing, things are looking good. We're close!
>

Sounds fantastic!

cheers,




m


-- 
 martin at laptop.org -- Software Architect - OLPC
 - ask interesting questions
 - don't get distracted with shiny stuff  - working code first
 - http://wiki.laptop.org/go/User:Martinlanghoff
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.laptop.org/pipermail/server-devel/attachments/20120211/ec8702c9/attachment.html>