=== jml_ is now known as jml === broder_ is now known as broder [07:39] cjwatson: """+ This is of limited usefulness as LiveFSes can be navigated to via [07:39] + Person:+livefs, but we need something minimal here to allow the [07:39] + "livefses" API root collection to work. [07:39] Why? [07:39] + """ [07:39] A top-level collection can be empty, and even if it's not empty the objects don't have to be subordinate. [07:41] lp.livefses is a thing mostly to avoid having more Soyuz code in Registry. [07:41] Not to actually be a useful collection. [07:47] Hm [07:48] I might just declare inline comments to be in beta for ~launchpad, because 4000 line diffs are unfun to review otherwise. [07:49] Plus it'll give us a week or so to test it out before we open it up more widely once the fixes are done this week. === mpt_ is now known as mpt === ev_ is now known as ev [10:04] wgrant: I don't recall exactly what breakage I was trying to work around there; I'll go back and try removing it again [10:06] I guess it was all a bit messy while you sorted out that versioning crash. [10:06] I know for a fact that it's not required, though. So you should be able to do away with it trivially. [10:07] Quite possible I was dealing with fallout from that and not realising it, indeed. [10:11] I got nearly halfway through with inline comments, so I'll hopefully have another round for you tomorrow morning. [10:17] I'll look forward to it. [10:50] wgrant: OK, this may have been the problem: http://paste.ubuntu.com/7451806/ [10:50] After applying http://paste.ubuntu.com/7451808/ [10:50] Maybe I need some trivial navigation, just without the traversal? [10:51] - for="lp.soyuz.interfaces.livefs.ILiveFSSet" [10:51] - path_expression="string:livefses" [10:51] - parent_utility="lp.services.webapp.interfaces.ILaunchpadRoot" [10:51] - /> [10:51] There's your problem [10:51] Oh duh right [10:55] OK, much better, thanks [10:56] :)