[00:53] <lifeless> can I tempt anyone with a review?
[00:54] <lifeless> mwhudson: I know you have other stuff to do, but I'm hoping you'll be generous :) - its another small one, with -much less- hair than the other.
[00:55] <mwhudson> lifeless: sure, i can take a look
[00:58] <lifeless> mwhudson: thanks; its in topic
[00:59] <mwhudson> oh yeah, that thing
[01:10] <lifeless> mwhudson: so, what did you think?
[01:11] <mwhudson> lifeless: sorry, got a bit distracted, have meld up now :)
[01:14] <mwhudson> can you really not do remove_tracer(self) ?
[01:14]  * mwhudson has no idea
[01:14] <lifeless> dunno, was just moving it so documented the limitation
[01:14] <mwhudson> ah ok
[01:14] <lifeless> it already existed if you passed in a function that itself did record_statements
[01:15] <mwhudson> ah ok
[01:15] <mwhudson> again :)
[01:17] <lifeless> mwhudson: did you alter https://launchpad.net/ubuntu/+assignments in your recent refactorings ?
[01:18] <mwhudson> oh man, lp.testing._webservice.QueryCollector?
[01:18] <mwhudson> what's that doing there
[01:18] <lifeless> mwhudson: existing ?
[01:18] <mwhudson> i just hate the way we accumulate multiple copies of things that do nearly the same thing
[01:18] <lifeless> right
[01:18] <lifeless> so QC was *in a doctest* a month ago
[01:19] <mwhudson> haha
[01:19] <lifeless> its different to record_statements though
[01:19] <mwhudson> so things are moving in the right direction
[01:19] <mwhudson> yeah, but in ways that shouldn't *really* matter, right?
[01:19] <lifeless> oh they should
[01:19] <lifeless> very much
[01:19] <mwhudson> not expecting you to fix it in this branch, just complaining out loud
[01:19] <lifeless> I think it would be a huge mistake to treat them as interchangable
[01:20] <mwhudson> now, more apropos: "Find the persons or teams by which person is subscribed." <- i don't understand this
[01:20] <lifeless> ok
[01:21] <lifeless> so a person can be subscribed to a bug directly or indirectly
[01:21] <lifeless> the bugtask browser code (for me specifically) generates 300 queries whenever I look at a bugtask:+index page.
[01:22] <lifeless> because it was answering the question 'what ways is robert subcribed' by python
[01:22] <lifeless> and its not interested in the BugSubscriptions
[01:22] <lifeless> its interested in the Person or Team records that are subscribed.
[01:23] <lifeless> I find making the one-line summary hard sometimes.
[01:23] <mwhudson> yeah, it can be difficult
[01:24] <lifeless> anyhow, this thing returns a result set
[01:24] <lifeless> which outputs IPerson
[01:24] <lifeless> such that it outputs all IPersons which cause person to be subscribed to the bug.
[01:25] <mwhudson> lifeless: that version makes more sense to me, perhaps
[01:25] <mwhudson> but it's hard to say if that's only because i read it after i already understood what it did
[01:26] <lifeless> 'return all IPersons which cause person to be subscribe to self' ?
[01:27] <mwhudson> lifeless: s/self/this bug/ would be more consistent with other docstrings, but otherwise +1
[01:28] <lifeless> 78 chars.
[01:28] <lifeless> 79 with .
[01:28] <lifeless> :P
[01:28] <mwhudson> hahaha
[01:28]  * lifeless pissess of the narrow-screen crowd.
[01:28] <mwhudson> ok, stick with your version then
[01:29] <lifeless> mwhudson: I think we should use the full 80 columns, but thats an entirely different discussion
[01:29] <mwhudson> yes
[01:30] <thumper> lifeless: want to review my tag cloud change?
[01:30] <mwhudson> lifeless: TestBugTaskView.initialize_view, did you consider using create_initialized_view ?
[01:30] <thumper> lifeless: it'll get rid of the timeouts
[01:30] <mwhudson> lifeless: it's in lp.testing.view iirc
[01:31] <mwhudson> lp.testing.views
[01:33] <mwhudson> lifeless: gosh the browser code you're changing is horrid
[01:34] <lifeless> thumper: sure
[01:34] <lifeless> mwhudson: I wanted to be crystal clear about what I'm exercising
[01:34] <lifeless> mwhudson: can switch it if you prefer
[01:35] <mwhudson> lifeless: hm, that's a fair point too
[01:37] <thumper> lifeless: https://code.edge.launchpad.net/~thumper/launchpad/different-cloud/+merge/34146
[01:37] <lifeless> I'm just doing a task for lynne, brb
[01:39] <mwhudson> lifeless: factory.makeTeam takes a members=[] argument
[01:39] <mwhudson> would reduce some hair in tests
[01:44] <lifeless> mwhudson: thanks, I'll definitely do that in future, can tweak here if you like.
[01:44] <mwhudson> lifeless: i think it would make the tests significantly easier to follow
[01:44] <lifeless> will do
[01:44] <mwhudson> ta
[01:46] <mwhudson> lifeless: in the browser doctest, "# count with 2 teams" should say 4 shouldn't it?
[01:46] <lifeless> oh bugger, yes.
[01:47] <lifeless> s/2/many/
[01:47] <mwhudson> :)
[01:47] <mwhudson> future proofing!
[01:48] <mwhudson> lifeless: in the same test class, how do you feel about s/initialize_view/record_view_initialization/ ?
[01:49] <lifeless> sure
[01:49] <mwhudson> otherwise i think i'm fine :)
[01:49] <lifeless> changes are done
[01:49] <mwhudson> can you push the branch again and i'll mark it so in the web
[01:49] <mwhudson> ah ok
[01:50] <lifeless> pushing in a second
[01:53]  * lifeless ignores the 79 limit issue
[01:54] <lifeless> ok push running now
[01:56] <lifeless> mwhudson: you can clicky click now
[01:56] <lifeless> mwhudson: (though I don't know why you wait)
[01:57] <mwhudson> lifeless: it's been a somewhat distracted review from my end (my fault), i want to look over the diff again, may as well look over the one with the changes
[01:58] <lifeless> mwhudson: well its there
[01:59] <lifeless> thumper: nuke this:
[01:59] <lifeless> 653	+
[01:59] <lifeless> 654	+def test_suite():
[01:59] <lifeless> 655	+ return unittest.TestLoader().loadTestsFromName(__name__)
[01:59] <mwhudson> lifeless: click clicked
[01:59] <mwhudson> +y
[02:02] <lifeless> thanks
[02:06] <thumper> lifeless: OK, should also remove it from the standard_test_template file then
[02:06] <thumper> because that is where it came from
[02:07] <lifeless> please do
[02:07] <lifeless> if we had gettext
[02:07] <lifeless> the pluralisation would be easier for you
[02:08] <lifeless> thumper: I know you didn't touch this
[02:08] <thumper> lifeless: tweaked and pushed
[02:08] <lifeless> but
[02:08] <lifeless> sorted(
[02:08] <lifeless> 76	list(getUtility(IBranchCloud).getProductsWithInfo(num_products)))
[02:08] <lifeless> ==
[02:08] <lifeless> sorted(getUtility(IBranchCloud).getProductsWithInfo(num_products))
[02:08] <lifeless> (sorted takes an iterable)
[02:08] <thumper> yep
[02:09] <lifeless> is there some reason to have a list( there ?
[02:09] <thumper> nope
[02:09] <thumper> removed
[02:10] <lifeless> the slave store by default.
[02:10] <lifeless> is a lie
[02:10] <lifeless> 'the slave store.'
[02:10] <thumper> ok
[02:10] <lifeless> would be true
[02:11] <thumper> it used to be defaulted :)
[02:11] <lifeless> thats ok
[02:11] <lifeless> have you tried the resulting queries on staging ?
[02:11] <thumper> tweaked
[02:11] <thumper> yes
[02:12] <thumper> I have
[02:12] <thumper> and it's fast
[02:12] <lifeless> 269+
[02:12] <lifeless> that new VWS should be removed; the auto formatter can do it for you I think
[02:13] <thumper> what is VWS?
[02:13] <lifeless> vertical white space
[02:13] <thumper> 269+ what?
[02:13] <thumper> I see it
[02:14] <thumper> are we running the autoformatter?
[02:14] <lifeless> not automatically
[02:14] <thumper> I thought we weren't
[02:14] <lifeless> but you can run it by hand AFAIK
[02:14] <thumper> but it makes my nice import lines messy
[02:15] <lifeless> its more stubborn than you
[02:15] <thumper> my making something like "from storm.locals import And, Or" take 4 lines
[02:15] <thumper> which I've been told is OK
[02:15] <lifeless> See, I think that should be fine on one line.
[02:15] <thumper> me too
[02:15] <thumper> so...
[02:15] <thumper> fix the autoformatter
[02:15] <lifeless> and the policy
[02:16] <thumper> agreed
[02:16] <thumper> so I can leave the VWS?
[02:16] <lifeless> oh the VWS is wrong regardless :)
[02:16] <thumper> why?
[02:16] <thumper> are we not separating canonical and lp?
[02:16] <lifeless> no, because they are both 'local'
[02:16] <lifeless> pep8 defines three classes of imports
[02:16] <lifeless> stdlib
[02:16] <lifeless> nonlocal
[02:16] <lifeless> local
[02:16] <thumper> ok
[02:16] <lifeless> in that order
[02:17] <thumper> fixed
[02:17] <lifeless> so an external canonical.landscape import, say, would be nonlocal.
[02:17] <lifeless> I doubt the autoformatter is sophisticated enough for that yet, but that would be the right thing to do
[02:18] <thumper> I've moved on
[02:19] <lifeless> reviewed +1
[02:19] <lifeless> clicky clicky set your commit message etc
[02:19] <thumper> ta
[02:24] <lifeless> how many ms do you think it will take to render now ?
[02:39] <thumper> no freaking idea
[02:39] <thumper> although wallyworld_ says 42
[02:49] <lifeless> lies
[02:49] <lifeless> Would be nice
[02:49] <lifeless> but lies.
[07:11] <lifeless> if anyone wants a break from coding and would like to review whats in the topic, that would be grand.
[08:04] <noodles775> Can someone rs the following testfix, otherwise I'll just rs it myself :) https://code.edge.launchpad.net/~michael.nelson/launchpad/test-fix-db-buildbase/+merge/34166
[08:23] <lifeless> done
[08:23] <noodles775> Thanks lifeless it's on pqm already.
[08:30] <lifeless> speaking of buffer 65
[08:30] <lifeless> do you have time to do a review perhaps ?
[08:35] <noodles775> lifeless: I can, although gmb might be starting his OCR round now too?
[08:35] <lifeless> gmb: are you?
[08:46] <noodles775> lifeless: looking now.
[08:48] <lifeless> thanks!
[09:16] <noodles775> lifeless: have you seen the pre-iterator-hook option for DecoratedResultSet? It might allow you to prefetch the data for all results at once, rather than when each result item is accessed?
[09:17] <lifeless> noodles775: yes, it's not sufficient
[09:17] <noodles775> OK
[09:17] <lifeless> because you need to discard the uninteresting columns so that you return Specification
[09:17] <lifeless> not (Specification, .., ..,..)
[09:17] <lifeless> in all other regards it would be equal
[09:18] <lifeless> (this patch prefetches all at once; its just patches them up row by row)
[09:18] <noodles775> Yes, you would still do that with your decorator function, the pre-iter-hook just allows you to do your caching all at once.
[09:18] <noodles775> Oh, does it?
[09:18]  * noodles775 looks again.
[09:18] <lifeless> it does one query
[09:19] <lifeless> on Specification, Person(aliased assignee), Person(aliased approver), Person(aliased drafter), EmailAddress(aliased for assignee), Account(aliased for assignee), EmailAddress(aliased for approver ...
[09:19] <lifeless> the decorator does two things:
[09:20] <lifeless>  - pulls out the Specification, so that it looks like we only queried Specifications to clients of the api
[09:20] <lifeless>  - preloads the Person objects for assignee, approver and drafter for that row - no new queries are issued.
[09:20] <lifeless> [I has the tests to prove it]
[09:22] <noodles775> Yes, I don't doubt that that is what its doing, what I do doubt is that it's doing the preloading once for all people in your result set, rather than for each person individually.
[09:23] <lifeless> noodles775: performance wise that doesn't matter though
[09:23] <noodles775> but that may not be possible here.
[09:23] <noodles775> Ah, ok.
[09:23] <lifeless> noodles775: I don't see why it would be of advantage (or disadvantage)
[09:24] <lifeless> because:
[09:24] <lifeless>  - we have to have a row decorator to return just Specification
[09:24] <noodles775> The advantage of the pre-iter-hook is that you can do one single query to pre-load all of your result set (or slice of your result set), rather than one query to pre-load each item in your result set. But as you say, it might not apply here.
[09:24] <lifeless>  - doing it in pre_iter_hook would then save no function calls, but add one function call to use the pre_iter_hook
[09:25] <noodles775> Yes, its about saving queries, not function calls.
[09:25] <lifeless> noodles775: if you're preloading via separate queries, I suspect you're doing it wrong :)
[09:25] <lifeless> noodles775: though I can imagine a use case when the query optimiser goes beserk
[09:25] <noodles775> lifeless: Sorry, s/preloading/caching
[09:26] <noodles775> ie. currently cache_validity is called when each item in your result set is accessed, which may be the best way, I just wasn't sure if you'd considered whether it was possible to do this once for the result set instead.
[09:27] <lifeless> sure, yes.
[09:27] <lifeless> note that cache_validity does no queries
[09:27] <noodles775> Ah, right. I see... because its ownly using the preloaded data. Gotit.
[09:40] <lifeless> noodles775: thanks for doing this
[09:41] <noodles775> no probs, almost done. Looks great, so far the only question I've got is whether you think the call-sites might be easier to read if _validity_queries returned a dict instead of a tuple?
[09:42] <lifeless> hmm, happy to change that.
[09:42] <lifeless> namedtuple would be ideal
[09:42] <lifeless> is that in 2.5 ?
[09:42] <noodles775> Not sure, but yeah, that would be ideal.
[09:45] <lifeless> I'll do a dict or namedtuple
[09:49] <noodles775> Great, r=me (I had one or two other questions). Thanks lifeless.
[09:54] <lifeless> cool
[09:54] <lifeless> _ is a convention in python for 'ignored'
[09:54] <lifeless> or unused
[09:56] <lifeless> namedtuple is 2.6
[09:56] <gmb> lifeless, Is the branch in the queue the one that noodles775 just reviewed?
[09:56] <noodles775> Yeah, among other things (translatable strings, last result on console etc.).
[09:57] <noodles775> gmb: yep.
[09:57] <gmb> Righty.
[09:58] <lifeless> noodles775: I like having it - its much more pithy than ignored_because_we_have_to_have_a_name_expr_here in range(...):
[09:59] <lifeless> noodles775: Its the only single codepoint variable name I use ;)
[09:59] <noodles775> lifeless: fine by me, might be worth adding to the style guide.
[10:02] <lifeless> noodles775: http://pastebin.com/Q2dAz2ts
[10:02] <lifeless> noodles775: is my proposed diff
[10:02] <lifeless> please let me know if you think it is an improvement
[10:03] <lifeless> lastly, _webservice for QueryCollector - its not exported from testing/__init__.py; it seems a bit inconsistent to me to reexport there and not elsewhere; we probably want to keep iterating and shuffling that stuff around anyhow...
[10:03] <noodles775> Yep.
[10:05] <noodles775> lifeless: and yes, I think the dict references are a bit more obvious than tuple indexes. Thanks!
[10:05] <lifeless> my pleasure
[10:05] <lifeless> thanks heaps for the review
[10:05] <noodles775> np.
[10:05] <lifeless> I shall push that change and send to ec2
[10:05] <noodles775> lifeless: erm, don't you need to change the callsites too?
[10:06] <noodles775> ah, that is the only one?
[10:06] <lifeless> noodles775: two
[10:06] <lifeless> one in specification, one in person
[10:06] <noodles775> yep, never mind me.
[10:06]  * noodles775 goes for a cuppa.
[10:29] <wgrant> gmb: I have a massive (~30 lines) branch for you, if you've time.
[10:31] <gmb> wgrant, Unless you're out by two orders of magnitude, in your estimation, I think I can handle that :)
[10:31] <lifeless> +1
[10:33] <wgrant> gmb: https://code.edge.launchpad.net/~wgrant/launchpad/bug-510180/+merge/34175
[10:33] <gmb> wgrant, Nice and subtle. I like it. r=me
[10:33] <wgrant> gmb: Heh. Thanks. Can you throw it into ec2?
[10:34] <gmb> wgrant, Will do.
[10:34] <wgrant> Thanks.
[14:24] <bac> hi gmb can i get a review?  https://code.edge.launchpad.net/~bac/launchpad/bug-39212/+merge/34196
[14:24] <gmb> bac, Sure.
[14:25] <bac> thanks
[14:31] <gmb> bac, What's with the wacky indenting around line 93-96 of the diff?
[14:31] <bac> gmb: dunno.  let me look
[14:33] <bac> gmb: dewhacked
[14:33] <gmb> bac, Eyefankyoo
[14:34] <gmb> bac, ALso, line 109 needs to be indented by one more stop to be consistent.
[14:34] <bac> gmb: i am going to open a but about URIWidget not respecting displayWidth, too.
[14:34] <bac> gmb, yep, i caught that one too
[14:34] <gmb> bac, Thanks x2
[14:35] <bac> i think we have css that is harmful and should be removed, but i'm not certain of all of the areas affected.
[14:35] <gmb> Fair neough.
[14:35]  * gmb switches lcoations; brb
[14:41] <gmb> bac, I thought that ReST headers just had to have the underline rather than the overline, too.
[14:41] <gmb> e.g.
[14:41] <gmb> Person's wikinames
[14:41] <gmb> [14:41] <gmb> (etc.)
[14:41] <bac> gmb: really?  i've often seen over/under
[14:41] <gmb> bac, In that case, we have a problem with consistency
[14:41] <bac> gmb: if you know better that's one less line...
[14:42] <bac> gmb: duh
[14:42] <gmb> (Not least because I do underline only :))
[14:42]  * gmb checks TestsStyleGuide
[14:43] <gmb> bac, The example in TestsStyleGuide uses underline only, so let's standardise on that.
[14:43] <bac> gmb: will do
[14:43] <gmb> Thanks.
[14:51] <gmb> bac, r=me with the above changes.
[15:39] <EdwinGrubbs> gmb: can you review lp:~edwin-grubbs/launchpad/bug-490659-series-timeout-part2
[15:40] <EdwinGrubbs> oops, I meant to paste https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-490659-series-timeout-part2/+merge/34159
[15:40] <gmb> EdwinGrubbs, Sure
[15:47] <gmb> EdwinGrubbs, Those two tests are *awfully* similar. Any chance they could be unified in their own TestCase (e.g. TestFooBatchNavigators)
[15:48] <EdwinGrubbs> gmb: sure
[15:48] <gmb> EdwinGrubbs, Cool.
[15:48] <gmb> EdwinGrubbs, Other than that, r=me.
[16:00] <EdwinGrubbs> thanks
[16:19] <noodles775> Hi abentley ! Will you have a chance view the changes I did in response to your review yesterday, or should I ask Leonard to take a look?
[16:20] <noodles775> https://code.edge.launchpad.net/~michael.nelson/launchpad/distro-series-difference-model/+merge/34086
[16:20] <abentley> noodles775, I can have a look.
[16:20] <noodles775> Great, thanks.
[16:40] <abentley> noodles775, re: getComments, I don't understand how implementing it directly on DistroSeriesDifference would be programming to interfaces, not implementations.
[16:41] <noodles775> abentley: It wouldn't. It would be programming to implementations, rather than programming to interfaces wouldn't it?
[16:42] <noodles775> ie. it would be using the comment model object etc., where as at the moment it only deals with interfaces. But I don't mind changing it.
[16:42] <abentley> noodles775, sorry, I still don't understand why moving it there would be the wrong thing.
[16:43] <noodles775> abentley: It wouldn't be wrong, I just thought it was *better* to separate out so that the DSDifference model doesn't need to know anything about the comment model.
[16:43] <abentley> You just wind up calling a method on DistroSeriesDifferenceCommentSource that makes very implementation-specific assumption about DistrosSeriesDifference
[16:45] <abentley> The implementation of getComments *must* deal with the implementation of how those comments are stored.  The best encapsulation is to make it an instance method on DistroSeriesDifference.
[16:45] <noodles775> Calling a method on a utility that implements an interface, yes. And the comment model needs to know about the implementation of DSDifference, as its implementation references it. But as I said, I don't feel strongly about it.
[16:48] <abentley> noodles775, my own rule is to be suspicious of staticmethods, because they are often cleaner as instance methods.
[16:48] <abentley> noodles775, I don't think your rule applies to objects that implement an interface.
[16:48] <noodles775> oh, I only used a static method because that's what most of the classes in lp.code do.
[16:49] <noodles775> See lp.code.models.sourcepackagerecipe.SourcePackageRecipe.new, for example.
[16:49] <abentley> noodles775, I'm glad you did.  Lots of people go around making "instance" methods that are really staticmethods, and I'm even more suspicious of those.
[16:51] <noodles775> OK, so you'd like me to remove DSDifferenceComment.getForDifference() and put that functionality directly in the existing DSDifference.getComments() right?
[16:51] <abentley> noodles775, right.
[16:52] <abentley> noodles775, An example of an "instance" method that is really a staticmethod is BranchSet.getRecentlyChangedBranches()
[16:55] <noodles775> abentley: pushing http://pastebin.ubuntu.com/486363/
[16:56] <noodles775> which passes with: bin/test -vvm test_distroseriesdifference -m test_distroseriesdifferencecomment
[16:57] <abentley> noodles775, r=me.  Thanks for your changes.
[16:57] <noodles775> np, thanks for the review :)
[18:57] <bac> hi leonardr ... has the API infrastructure changed with respect to batching lately?  We're seeing OOPSes on getting all of the members of a very large team.
[18:58] <leonardr> bac, yes, see bug 627442, caused by bug 620508, recently triggered and incompletely worked around
[18:58] <_mup_> Bug #627442: lazr.restful and batching cause a FeatureError <oops> <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/627442>
[18:58] <_mup_> Bug #620508: Slicing a ResultSet breaks subsequent len() calls. <Storm:New> <https://launchpad.net/bugs/620508>
[18:58] <leonardr> gary and benji are working on it
[18:59] <bac> leonardr: yep, that's the one!  thanks.
[19:20] <jcsackett> leonardr: https://code.edge.launchpad.net/~jcsackett/launchpad/deprecate-official_rosetta/+merge/34229
[19:21] <jcsackett> linting on this one is a bit odd, b/c it hit about a zillion files and the review would be unreadable with the fixes; i'll hit everything from make lint, but didn't push it up for now, so the MP and diff is legible.
[19:40] <leonardr> jcsackett, ok
[19:42] <leonardr> salgado, can i ask you to take a look at https://code.edge.launchpad.net/~leonardr/launchpad/publish-tokens-2 ?
[19:42] <leonardr> just the last two revisions
[19:43] <leonardr> abently isn't around, but it's all oauth stuff so you should be able to do it
[19:43] <salgado> leonardr, r11454 and r11455?
[19:47] <leonardr> salgado: actually, it looks like it's everything from 11452..11455
[19:47] <salgado> leonardr, it'd be nice if you could generate an incremental diff for me to look, then
[19:48] <leonardr> salgado: sure
[19:51] <leonardr> salgado, http://pastebin.ubuntu.com/486435/
[19:52] <leonardr> i converted a pagetest to a unit test, fixed a bunch of indentation, and did the interface switch i showed you earlier
[20:01] <leonardr> jcsackett, basic question: what is the difference between official_rosetta and translations_usage?
[20:04] <jcsackett_> leonardr: official_rosetta is a bool, translations_usage is an enum letting us know more about how a service is used.
[20:04] <jcsackett_> We're trying to phase out the official_ bools because they don't really provide enough data.
[20:04] <leonardr> so 'official' means that launchpad is the official handler of a project's translations
[20:06] <jcsackett> Right, which isn't aleays enough data. For example in project configs we can't tell that translations aren't even being done, just that no translations are on LP.
[20:17] <leonardr> jcsackett, you should file a bug to track the work done to fix the xxx on line 422, and mention that bug in the xxx
[20:18] <jcsackett>  leonardr ok.
[20:18] <leonardr> you should do the same for the xxx on line 453
[20:18] <jcsackett> Got it.
[20:21] <jcsackett> There's a repeated xxx, I'll update them all.
[20:37] <leonardr> jcsackett, r=me with the bugs filed
[20:39] <lifeless> leonardr: hi
[20:40] <lifeless> leonardr: I have a branch which was oked., but ec2 showed me the error of my code
[20:40] <lifeless> leonardr: I'd like you to comment on the last commit (only) on it, if you could
[20:40] <leonardr> lifeless, sure
[20:40] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/bug-618367/revision/11477
[20:40] <lifeless> is the commit
[20:40] <lifeless> https://code.launchpad.net/~lifeless/launchpad/bug-618367/+merge/34162
[20:40] <lifeless> is the whole MP, if you want context
[20:40] <lifeless> basically, all I do in the commit is move checks for 'valid person' from WHERE to the JOIN
[20:41] <jcsackett> leonardr: Thanks.
[20:46] <leonardr> lifeless, what comes out of Person._validity_queries?
[20:46] <lifeless> leonardr: a dict
[20:46] <lifeless> leonardr: used in building queries
[20:47] <lifeless> Person._all_members uses it, I refactored the code from Person._all_members to make this bit of it usable by specifications
[20:47] <lifeless> leonardr: I've shrunk the dict
[20:47] <lifeless> because it now did not need to return WHERE clauses at all.
[20:50] <leonardr> lifeless: ok, i see where you're moving stuff from the WHERE to the JOIN, and i see how someone with only non-preferred emails would be excluded
[20:50] <leonardr> you're also changing the account_table join so that it only picks up active accounts. is that right?
[20:51] <lifeless> same bug
[20:51] <lifeless> if you had an account that wasn't active, you'd be excluded rather than being included but marked invalid
[20:51] <lifeless> sorry, the *row* would be excluded
[20:51] <lifeless> which drops the specification out
[20:52] <lifeless> in exactly the same way; the account row would match the old join, making account.status non-null, but then the where clause would exclude the whole row.
[20:54] <leonardr> ok, i understand. and oyu took the code out of specificatoion.py because _validity_queries no longer returns any constraints
[20:54] <leonardr> r=me
[20:56] <lifeless> right
[20:56] <lifeless> thanks


[00:25] <TheMuso> calc: Do you know where I may be able to find a PPS file with audio, so I can test bug 298494?
[00:33] <TheMuso> Or anyone else? ^^
[00:33] <[Relic]> is anyone else alive, or are they all bots like pitti :)
[00:34] <TheMuso> [Relic]: I'm alive.
[00:38] <calc> TheMuso: the one in the bug report doesn't work?
[00:40] <TheMuso> calc: Oh there is one in the bug? Haven't looked at the bug in that much detail yet. Thanks.
[00:42] <calc> ok np
[02:28] <TheMuso> calc: Am I correct in deducing that openoffice now uses gstreamer for its audio output?
[02:34] <calc> TheMuso: yes
[02:36] <TheMuso> calc: Right, well that audio stutering with pps bug is very weird, because openoffice doesn't seem to be respecting the settings from the sound preferences applet.
[02:36] <TheMuso> I get audio stutter whether I use OSS, alsa or pulseaudi in those settings. Examining open devices from a terminal, when I have oss selected, no oss device node is being used what soever.
[02:36] <TheMuso> Only alsa nodes get used.
[02:37] <TheMuso> So I am not convinced its an audio infrastructure bug.
[02:37]  * TheMuso will update the bug in a bit when I've done a bit more testing.
[02:38] <calc> ok well it might not be after all, i just thought it might be since he claimed it worked previously
[02:39] <calc> feel free to bounce it back to openoffice.org with that info added to the report :)
[02:40] <TheMuso> calc: Ok will do.
[02:42] <wasabi> hmm. at some point UDF vols stopped auto mounting.
[02:42] <wasabi> it's trying iso9660 first and succeeding.
[05:58] <dholbach> good morning
[06:29] <vhaarr> Hey, what does "SRU" and "FFE" mean? I see these abbrevations in some bugs on launchpad, for example.
[06:29] <Hobbsee> !sru
[06:29] <Hobbsee> !ffe
[06:29] <Hobbsee> ^
[06:30] <vhaarr> ffe == uvf?
[06:30] <Treenaks> vhaarr: no, FFE is exception to the freeze
[06:30] <vhaarr> ah
[06:30] <vhaarr> thank you
[06:31] <vhaarr> is jaunty in feature freeze at the moment?
[06:33] <StevenK> vhaarr: No
[06:34] <vhaarr> why is it that nvidia-graphics-drivers-180 is stuck in https://launchpad.net/ubuntu/jaunty/+queue then, does it still need some sort of "initial approval" since it's a new package?
[06:37] <vhaarr> ah, I should probably be in #ubuntu-motu with these questions
[06:37] <vhaarr> apologies for spamming the devel channel
[06:43] <Hobbsee> vhaarr: yes
[06:44] <Hobbsee> @ initial approval
[06:44] <vhaarr> righty, thanks
[08:09] <fabbione> soren: ping?
[08:19] <soren> fabbione: wazzup?
[08:20] <fabbione> soren: hey.. do you still maintain kvm/libvirt stuff?
[08:20] <soren> fabbione: Yes.
[08:21] <fabbione> soren: when you have time, can you give me a crash course on how to setup libvirt networking?
[08:21] <fabbione> soren: I am fighting with bridges and stuff that I am not very familiar with
[08:21] <soren> Sure.
[08:21] <soren> fabbione: With bridging, it's really quite simple. Do you have a bridge setup already and just want libvirt to use it?
[08:22] <fabbione> soren: i only have the default installation (hardy)
[08:22] <soren> Alright.
[08:22] <fabbione> soren: basically i want the guests to access the real network directly
[08:22] <soren> Right.
[08:22] <fabbione> soren: not via all that NAT stuff
[08:22] <NCommander> hey apachelogger
[08:22] <fabbione> soren: and make that permanent
[08:23] <soren> fabbione: Could you put your /etc/network/interfaces on a pastebin? I can make the changes there and you can see what I did. That's lots easier than explaining it.
[08:23] <NCommander> Well, you could just set it up to use IPv6 ...
[08:23] <fabbione> soren: sure thing.. sec
[08:24] <fabbione> soren: http://www.fabbione.net/interfaces
[08:25] <soren> fabbione: I presume it's the bond0 interface you want to bridge to?
[08:25] <fabbione> soren: well that's connected to the real net.. so yeah
[08:28] <soren> fabbione: Any particular reason you're not using the "slaves" directive in your interfaces file?
[08:28] <soren> (for setting up the bonding)
[08:29] <fabbione> soren: not really
[08:29] <soren> fabbione: Ok. We can fix that afterwards :)
[08:29] <fabbione> soren: i didn't even know ifup did grow bonding support :)
[08:29] <fabbione> soren: <- very very old school
[08:29] <fabbione> soren: it doesn't really matter as long as it works
[08:30] <soren> fabbione: Ok, for the really simple, general case: http://people.ubuntu.com/~soren/interfaces-example.txt
[08:31] <fabbione> soren: looking
[08:32] <fabbione> soren: well i doubt there is network manager installed on the server
[08:32] <fabbione> it's not a desktop + kvm thingy
[08:32] <soren> So, for your setup it, this should probably work: http://people.ubuntu.com/~soren/fabbione.txt
[08:32] <fabbione> it's a decent _real_ server :)
[08:32] <soren> Just leave that bit out, then :)
[08:33] <soren> fabbione: This one should deal with the bonding as well: http://people.ubuntu.com/~soren/fabbione-even-better.txt
[08:34] <fabbione> eheheh
[08:34] <fabbione> soren: one thing I am not sure I understand
[08:34] <fabbione> soren: the bond will not get the ip address
[08:34] <soren> Hm?
[08:34] <fabbione> (sec)
[08:35] <soren> Oh, I think I know what you mean..
[08:35] <soren> The bond0 interface no longer gets an ip address. Only the bridge interface does.
[08:35] <fabbione> ah ok
[08:35] <soren> That's the way it works.
[08:35] <fabbione> so i also need to make sure the bridge will get the same mac address
[08:36] <soren> IIRC, the bridge assumes the numerically lowest MAC address of the interfaces attached to it.
[08:37] <soren> ...so it doesn't change over time, if that's what you mean.
[08:37] <fabbione> ok perfect :)
[08:37] <fabbione> yeah
[08:37] <fabbione> that's what I was looking for
[08:37] <soren> Ok, so now you've got a bridge.
[08:38] <fabbione> ok just a sec.. customer on the phone
[08:38] <soren> To hook a virtual machine into that, you put <interface type='bridge'><source bridge='ifbr0'/></interface>  in your domain XML definition.
[08:38] <soren> ...and that's it.
[08:39] <soren> fabbione: sure, take your time.
[08:39] <fabbione> thanks :)
[08:48] <RAOF> wgrant: I'm sure you'll be overjoyed to hear that syndaemon in Jaunty fails to work with a BadDevice error.
[08:51] <fabbione> soren: re
[08:51] <fabbione> ok... testing fancy-interface-file-from-soren
[09:00] <soren> fabbione: First: Does the host's network still work?
[09:00] <fabbione> soren: you need to add auto bond0 otherwise the bridge can't bind
[09:00] <fabbione> soren: i am fixing bits and pieces :)
[09:00] <fabbione> soren: bond0 needs to be UP
[09:00] <fabbione> now.. i am testing again.. one sec
[09:00] <fabbione> soren: this is that kind of testing that I hate the most.. "if the server reboots.. will it work?"
[09:01] <soren> fabbione: Ah, yes, good point on the bond0.
[09:01] <fabbione> soren: and this thing has something like 10 pages of POST before you get to grub
[09:01] <soren> fabbione: Yikes.  Just shoot.
[09:09] <fabbione> soren: ok.. host networking works fine now
[09:37] <asac> doko_: armel seems to have built fine. Thanks a bunch!
[09:39] <NCommander> doko_, do you know why there seems to be no implicate cast from float to double? (or via versus I suppose, since that works on i386)
[09:39] <NCommander> (on arm)
[09:52] <glatzor> morning james_w, do you arrive at sfo?
[09:53] <glatzor> james_w, if so we could share a taxi, since I would arrive three minutes after you
[10:01] <tseliot> Riddell: you're the archive admin today, aren't you?
[10:02] <Riddell> tseliot: I would be if I wasn't on holiday
[10:02] <Riddell> tseliot: what do you need?
[10:02] <tseliot> Riddell: can you see nvidia-glx-180 in NEW?
[10:03] <tseliot> (Jaunty)
[10:03] <Riddell> tseliot: yes
[10:04] <tseliot> Riddell: good. Can you accept it, please?
[10:04] <Riddell> into restricted presumably?
[10:04] <tseliot> yes
[10:05] <tseliot> BTW it makes KDE4 usable with nvidia cards
[10:05] <tseliot> :-)
[10:06] <Riddell> yay
[10:06] <Riddell> accepted
[10:06] <tseliot> Riddell: thanks a lot
[10:11] <NCommander> hey Riddell
[10:12] <Riddell> morning NCommander
[10:12] <NCommander> how goes it Riddell?
[10:13] <Riddell> NCommander: lovely thanks, currently enjoying the hospitality of the Kubuntu Stuttgart Office
[10:14] <NCommander> o_O?
[10:15] <NCommander> Riddell, I'm working on running down the all evil pyKDE4 arm bindings FTBFS
[10:15] <NCommander> bah
[10:15] <Riddell> brave man
[10:17] <NCommander> I can't figure out where its choking
[10:17] <NCommander> If its the Python-C API, QT, bindings, or sip
[10:17] <NCommander> :-/
[10:17] <NCommander> Are we the only distro with KDE4 and an ARM port?
[10:17] <Riddell> NCommander: there's Debian experimental
[10:18] <Riddell> NCommander: have you tried asking Sime?
[10:18] <NCommander> who?
[10:18] <Riddell> pykde maintainer
[10:18] <NCommander> armel doesn't appear to have an experimental buildd ...
[10:18] <Riddell> NCommander: on IRC as Sime_
[10:19] <NCommander> hrm
[10:20] <james_w> glatzor: I do, that would be great
[10:21]  * directhex wonders if ScottK tried to catch up with him again after ~11pm yesterday
[10:22] <lool> NCommander: Hmm the error in the build log seemed pretty clear, did you get past that one?
[10:22] <NCommander> lool, it's not clear
[10:22] <NCommander> lool, its FTBFS on a generated file
[10:22] <NCommander> (that file is generated by the KDE python binder lexer AFAIK)
[10:22] <NCommander> I'm hoping once I get a chance to fully look through the lexers output I can see whats going wrong
[10:23] <NCommander> As far as I can tell from the build log, it LOOKS some stupidity w.r.t. to float/double on ARM
[10:23] <NCommander> (man, ARM is picky about its floating points)
[10:24] <NCommander> Since this issue with float/doubles seems to only happen in KDE and Qt, I'm thinking its Qt specific vs. some bug in GCC
[10:25] <glatzor> james_w, yeah, great. Do you have got a credit card? have already reserved a taxi?
[10:25] <NCommander> I'm hoping/praying its just something in the input sip file I can change to clear the build failure right up
[10:25] <NCommander> (although knowing my luck)
[10:26] <james_w> glatzor: nijaba is arriving at the same time as well, I haven't, but I don't know if he has
[10:27] <nijaba> glatzor: no, I haven't booked a cab, but there generally is no need to do so, plenty are waiting there.  I do have a cc
[10:28] <NCommander> man, my external is making death noises
[10:28] <NCommander> But its working fine
[10:28] <NCommander> ...
[10:28]  * NCommander wonders how long until it fails
[10:28] <nijaba> glatzor: we are you flying in from? International?
[10:29] <glatzor> nijaba, I will fry from London to SF with UA995
[10:29] <glatzor> fly
[10:29]  * NCommander wants to share a cab if possible
[10:29] <nijaba> glatzor: I certainly hope you won't fry ;)
[10:30] <glatzor> :)
[10:30] <glatzor> james_w, nijaba: have you ever been to sf? where is good place to meet?
[10:30] <nijaba> glatzor: ok, so we'll join at custom exit in the international terminal
[10:31] <nijaba> glatzor: about a 100 times.  If you do not find me, I might be outside getting stoned from a first cigarette in about 12h
[10:43] <NCommander> lool, so I found the FTBFS
[10:43] <NCommander> and its ugly
[10:44] <NCommander> Real ugly
[10:45] <NCommander> But it does explain why Qt is a miserable pile of exceptions on ARM
[10:46] <lool> NCommander: I think I'd file it under sip
[10:47] <NCommander> lool, its a bug in kdelibs
[10:47] <lool> NCommander: What did you find?
[10:47] <NCommander> Under normal cirmstances, qreal is a typedef on double
[10:47] <NCommander> But on Windows CE, and ARM its a float
[10:47] <NCommander> kdelibs has a few cases where it explicately assumes double and ...
[10:48] <NCommander> The hardcoded doubles need to be changed to qreal in kdelibs, AND in kdebindings
[10:48] <NCommander> (unless we want to change Qt on ARM and then transition Qt's rdepends)
[10:48] <lool> NCommander: Well are you sure they intended qreal and not double?
[10:48] <NCommander> lool, http://pastebin.ca/1273380
[10:48] <NCommander> lool, QList<double> in the source :-)
[10:48] <NCommander> If we change it to QList<qreal>, it would make the assignments work properly
[10:49] <ogra> are you sure it gets the QT_ARCH_ARM properly handed over ?
[10:50] <NCommander> ogra, yeah, qreal becomes a float
[10:50] <NCommander> I think that check exists because ARM's FPU performance kinda sucks
[10:50] <NCommander> But there is no comment explaining it :-/
[10:50] <lool> NCommander: My question is: are you sure they want to handle qreals and not doubles; it's the same question for QList<double> versus QList<qreal>
[10:51] <NCommander> I'm not sure I'm following
[10:51] <NCommander> QList is simply a container
[10:52] <NCommander> lool, http://paste.ubuntu.com/79246/ - this is the generated code its choking on
[10:52] <lool> NCommander: level 1) qreal is defined to foo or bar depending on various conditions; we could look into this, perhaps armel's qt is using doubles or other projects, and that would make our life simpler; level 2) kdelibs might be using qreal and doubles for various things; perhaps it calls into other APIs or simply wants double in some places
[10:52] <NCommander> Its two function calls that use double vs. qreal, I think its an isolated fluke that just didn't get catched. Since the behavior is the same on all architectures expect ARM ...
[10:53] <lool> level 3) sip generates code from kdelibs (AIUI) but doesn't handle some cases of type mistmatches
[10:53] <NCommander> Qt's docs recommend you use their own types over C's to prevent fun mistakes like this
[10:54] <NCommander> As a general rule of thumb, I don't like changing library APIs like QTs, and we'd break binary compatbility with Qt on other ARM systems if we changed qreal to double
[10:54] <NCommander> (not that we really care, but ...)
[10:54] <lool> NCommander: Sure, but you might have to use other types for various reasons; perhaps it's truly wrong to use double, perhaps it's not?
[10:55] <lool> NCommander: TBH I don't know how much binary compatibility there is for running qt bits on arm platforms
[10:55] <ogra> heh
[10:55] <NCommander> Chanigng qreal's type would break it
[10:56] <NCommander> Personally, I'm against changing a core library against Qt unless there is a serious reason to do so
[10:56] <lool> NCommander: Well by changing double to qreal in kdelibs, you'd break kdelibs binary compatibility as well anyway
[10:56] <NCommander> lool, no it won't
[10:57] <NCommander> qreal == double on !ARM
[10:57] <NCommander> ABI doesn't change
[10:57] <lool> It doesn't change on !ARM
[10:57] <lool> and !WINCE
[10:57] <NCommander> Considering KDE doesn't compile on ARM ... :-)
[10:57] <lool> You want to avoid an ABI change on ARM with an ABI change on ARM
[10:57] <NCommander> O_o?
[10:57] <NCommander> oh
[10:58] <NCommander> I perfer smaller ABI changes
[10:58] <lool> Anyway, I think we're being distracted
[10:58] <NCommander> yeah
[10:58] <NCommander> The question is do we file a bug with the Qt devs, or a bug with the KDE guys :-)
[10:58] <NCommander> (there a compile time arguements to force qreal to double on ARM it seems)
[10:58] <lool> Best would be to check what others are doing for qreal (1), and whether upstream kdelibs thinks of this (double -> qreal)
[10:59] <lool> These are in fact orthogonal and we can solve both
[10:59] <NCommander> Well, I found the note on why this was done in Qt
[10:59] <NCommander> Doubles eat performance for lunch on ARM
[10:59] <lool> Which ARM?  with vfp?
[10:59] <ogra> which arm ? :) i heard v7 behave differently
[11:00] <NCommander> We technically target v5, with v7 optimized libraries
[11:00] <NCommander> and we're softfloated, unless we have an optimized library, VFP isn't used on ARM
[11:01] <ogra> using the VFP of the v7 will definately be in our focus
[11:01] <NCommander> Well, to get the VFP used, optimize specific binaries and libraries which would benefit
[11:01] <NCommander> We'll have to figure out a way to give dpkg a brain for this case
[11:01] <NCommander> (I think we need subarchitecture support in dpkg/apt TBH, but for another time)
[11:02]  * ogra didnt say it would be easy ... or happen right now :)
[11:02] <lool> NCommander: I think I care most for being compatible with Debian on this particular point of ABI anyway
[11:02] <lool> So let's check whether Debian's qt is patched for armel and live with the same for Ubuntu's armel qt
[11:02] <NCommander> Good idea
[11:02]  * NCommander ventures to Debian's source packages
[11:03] <lool> NCommander: In all cases, why wouldn't we want to fix sip to handle this gracefully?
[11:04] <NCommander> lool, its not a bug in sip.
[11:04] <lool> NCommander: Are you subscribed to the upstream kdelibs lists and all?  Perhaps you want to bring this up there
[11:04] <NCommander> lool, anything trying to use these APIs would have caused a similar issue I think
[11:05] <lool> NCommander: You're saying sip can't handle API's mixing doubles and qreal?
[11:06] <NCommander> lool, I don't think its a bug in sip. the API and qreal should always match if you want compability with Qt types
[11:06] <NCommander> sip can't be blamed if the author ignored the user friendly typedefs
[11:06] <lool> NCommander: What if I need to pass doubles and qreals to two sub APIs?
[11:06] <NCommander> We're dealing with the return type here of a template
[11:07] <ScottK> directhex: No.  I didn't gry to get ahold of you again.  I ran into a related problem that's nothing to do with the mono transition and got stuck. I suspect, but cannont yet prove, that adjusting the build-dep as you suggested is sufficient for the transition.
[11:08] <NCommander> lool, Debian doesn't change Qt's behavior
[11:16] <lool> NCommander: yup
[11:16] <lool> I checked the headers and the patches and didn't find any such change either
[11:16] <lool> (and it does set QT_ARCH_ARM)
[11:16] <NCommander> lool, Debian hasn't run into the issue on the virtue they still use KDE3
[11:17] <NCommander> lool, so now what :-)?
[11:18] <asac> doko_: do you maintain all java packaging in a svn repo? or are there packages that i just need to upload if i want to extend those Xpp- headers?
[11:18] <lool> NCommander: Well we eliminated level 1 from the equation
[11:18] <lool> NCommander: you filed level 2
[11:18] <NCommander> o_o?
[11:18] <NCommander> level 1/2? huh?
[11:21] <NCommander> lool, well, beside KDE, I think the ARM port is fairly good in main
[11:21] <NCommander> (universe is another story)
[11:22] <lool> NCommander: levels: see discussion we had earlier here?
[11:24] <NCommander> lool, nope
[11:24]  * NCommander has been MIA over a week
[11:24] <NCommander> lool, well, I looked deep within the sourcecode of kplotaxis.cpp, and its just a return of an internal variable
[11:25] <NCommander> which boils down to a         QList<double> m_MajorTickMarks, m_MinorTickMarks;
[11:25] <lool> NCommander: ?
[11:25] <NCommander> No, I'm finding the code in kplotpoint that we're considering changing to qreal after upstream guidance
[11:25] <NCommander> I'm seeing what it does specifically :-)
[11:25] <lool> NCommander: By "earlier" I meant a couple of hours earlier, not days
[11:26] <lool> NCommander: /lastlog level
[11:26] <NCommander> ooooh
[11:26] <NCommander> handy IRC command!
[11:27]  * NCommander looks for KDE's SVN repo
[11:27] <NCommander> Time to create a proposed patch now that I know what needs editing
[12:14] <Silicium> hi there
[12:15] <Silicium> i want to create a custom installer CD, this works fine with getting the packages from a external repository
[12:15] <Silicium> now i want to add the custom packages to the local CD Repository
[12:16] <Silicium> should i create a own new repository or should i merge my packages into the exist hardy repo?
[12:18] <asac> slangasek: can you take a look at network-manager-pptp SRU in -proposed. has been pending for a while and all bugs fix committed have verification-done. thanks
[13:44] <doko_> asac: "all java packaging" ?
[14:00] <asac> doko_: all java plugins ;)
[14:01] <hwilde> anybody an ssh port forwarding expert
[14:02] <hwilde> i've got all these leftover sshd's waiting for an EOF that netcat will never send...
[14:04] <doko_> asac: sun-java* in svn, openjdk in bzr
[14:05] <asac> doko_: what other java plugins do we have?
[14:05] <asac> or is that it for jaunty?
[14:08] <doko_> gcjwebplugin will go away
[14:48] <lamont> wow.  d-i actually picked the first(?) link-present interface for network config.  kewl.
[14:48] <lamont> (eth2)
[15:46] <lamont> dear open&*%)&$%&%vpn.  stop dropping the damn local network route. kthx
[15:47] <bddebian> Fix it! :)
[15:51] <lamont> so... at what point does the intrepid openvpn go ripping through the routing table and remove all routes to subnets that have supernets routed over the vpn?
[15:51] <lamont> thereby leaving me with a default route pointing to, um, me.
[15:57] <lamont> bddebian: well, I thought I had fixed it.. by explicitly doing the routing table addition myself, rather than telling openvpn to do it via the config.  however... it appears to do it after it runs the up script
[15:58] <lamont> and it's only a problem when I plug the machine into the lan cable at home...
[15:59] <bddebian> Ah :)
[16:00] <psusi> TheMuso: you remember that dmraid bug with isw breakage caused by the raid10 patch I was working on?  I basically gave up as fixing it correctly is far more complicated than I first thought.  What are the odds on just backporting rc15? ;)
[16:15] <Silicium> hi there
[16:15] <Silicium> i have created a custom install disk
[16:16] <Silicium> so how i can regenerate te main Packages file with all hashes?
[16:43] <asac> doko_: gcjwebplugin.so  or IcedTeaPlugin.so what is used?
[16:45] <doko_> hardy: gcjwebplugin.so, later: IcedTeaPlugin.so
[16:46] <asac> doko_: and the  .so filename in the IcedTeaPlugin.so case is "IcedTeaPlugin.so" ... right?
[16:46] <asac> (seems obvious, but better ask ;))
[16:50] <doko_> asac: yes
[16:50] <doko_> asac: but you should only see libjava-plugin.so
[16:50] <asac> doko_: because of the alternative?
[16:51] <asac> i need the filename of the link target
[16:51] <asac> thats what mozilla sees internally
[16:54] <doko_> asac: this is ugly
[18:00] <slangasek> tseliot: it appears to be in binary new
[18:34] <slangasek> asac: published, cheers
[18:40] <slangasek> kees, Mithrandir: ah, pam_env user-environment support merged upstream
[18:42] <kees> slangasek: oooh nice
[19:05] <asac> doko_: you dont even know what this is about ;)
[19:17] <slangasek> ScottK: thanks for filing the lintian MIRs ahead of me :)
[19:23] <ScottK> slangasek: You're welcome.  I figured I asked for the sync without checking for that, so I earned it.
[19:24] <sebner> ScottK: well, no I'm here :)
[19:24] <ScottK> slangasek: Now we just need someone from the MIR team to actually look at them ...
[19:43] <slangasek> kirkland: so why is per-user encrypted homedirs preferable to either use of ~/Private, or system-level encryption of /home?
[19:45] <ScottK> Per user would be better if the user didn't trust the sysadmin?
[19:48] <slangasek> ScottK: how does ~/Private not already solve that, with less of a performance hit?
[19:48] <ScottK> slangasek: Dunno.
[19:48]  * ScottK was taking a stab in the dark.
[19:52] <lifeless> if you don't trust the sysadmin you're already wedged
[19:52] <lifeless> once your CPU can read your data, so can they
[19:53] <slangasek> maybe you're running SELinux at the same time ;)
[19:53] <lifeless> even so :)
[19:53] <lifeless> you might think you're running SELinux, but are they :)
[19:54] <slangasek> my attempts to compromise the machine say yes
[19:54] <lool> slangasek: Re: p7zip MIR: I wonder whether we could replace zip/unzip with p7zip
[19:55] <slangasek> lool: possibly!  I have no preference, I just don't want us to ship both zip/unzip /and/ p7zip on the CDs :)
[19:55] <lool> That would be more functionality and space efficient
[19:55] <lifeless> slangasek: humour aside, a VM with selinux in it, and you are talking to the VM, sysadmin has raw hardware
[19:55] <lool> slangasek: full ack
[19:55] <pwnguin> slangasek: ive been looking at p7zip as part of squashfs
[19:56] <pwnguin> slangasek: unfortunately, debian packages aren't optimized for compression
[19:56] <pwnguin> but i hear the livecd doesn't have traditional .debs?
[19:56] <slangasek> er?  what does p7zip (the userspace package) have to do with squashfs (a kernel fs implementation)?
[19:56] <pwnguin> p7zip is the lzms implementation ;)
[19:57] <pwnguin> lzma i mean
[19:58] <slangasek> AIUI, p7zip is a container format.  We have an "lzma" package that implements lzma compression.
[19:59] <pwnguin> huh
[20:00] <pwnguin> look at the lzma package description
[20:03] <slangasek> what about it?
[20:04] <pwnguin> it's got 7zip all over it ;)
[20:04] <pwnguin> i dare say 7zip and lzma are interchangeable terms ;)
[20:04] <slangasek> they are not
[20:04] <pwnguin> anyways, back to business as usual
[20:07] <pwnguin> oh that's right, i remember now; 7z lets you use a lot of compression algorithms but mainly promoted lzma as it's 'new hotness'. clearly i got up too early today
[20:14] <psusi> 7zip and lzma are as interchangable as gzip and lz
[20:15] <TheMuso> psusi: There would really have to be a strong case for it. We could try backporting just the intel code.
[20:15] <pwnguin> its more like avi and divx ;)
[20:16] <TheMuso> psusi: But even then backporting rc15 is probably not a good idea.
[20:16] <psusi> TheMuso: even to get it into -updates?
[20:16] <psusi> err, -backports?  which was it?
[20:16] <psusi> the one you have to choose to enable to get the latest and greatest
[20:17] <TheMuso> psusi: -backports if an option of course, but thats not always useful to everyone.
[20:17] <TheMuso> psusi: I thought you meant intrepid-updates.
[20:17] <psusi> right.... but it would be better than having them install a broken version from my ppa to get their system working
[20:18] <TheMuso> Agreed.
[20:18] <psusi> and don't forget, it is a regression
[20:19] <TheMuso> Yes I know.
[20:19] <TheMuso> psusi: But just putting a newer version in backports is not ideal either.
[20:20] <psusi> all I know is that raid10 patch from mandriva is a lot different than the raid10 implementation in rc15, so it seems a lot easier to just go with that rather than try to fix the very broken mandriva patch
[20:20] <TheMuso> Yep agreed.
[20:21]  * TheMuso will have a look at it during UDS, since rc15 allows the creation of dmraid metadata now, so I'll try creating a raid10 vm and get rc14 to try looking at it and see if I can reproduce locally.
[20:21] <psusi> now if only I could make heads or tails out of this code to figure out why it doesn't like volumes with long names with spaces in them....
[20:22] <psusi> ohh yea, that's true... good idea
[20:22] <psusi> I've been using the metadata dumps people have attached to the bug reports to simulate the setup for testing
[20:23] <TheMuso> that hasn't worked for me I've found, as you need same drives with same models, serial numbers, etc.
[20:23] <psusi> you have to build dmraid with a testing option so it considers /dev/dm-* as underlying disks, place the drive serial numbers in /dev/dm-*.serial, and use dmsetup to create a device mapper that returns zeros for everywhere but where the metadata is linearly mapped to a loopback device
[20:25] <TheMuso> meh I think I'll use rc15 and a vm.
[20:25] <TheMuso> Anyway, I must continue to prepare for my flight today.
[20:25] <psusi> hehe
[20:26] <tseliot> slangasek: ok, thanks for the information
[21:23] <hwilde> last chance before reinstall... anybody know why this is failing reference two different kernels ? http://pastie.org/329186
[23:00] <Lancao> hi
[23:02] <ScottK> Is it's a regression that qualifies for SRU, then it's really inappropriate for -backports.
[23:08] <Mez> can someone confirm me this? if you install a minimal system (mine's done through debootstrap) - and then try to install openssh-server - it won't work
[23:08] <Mez> I've tested it and it seems to fail with log_daemon_msg
[23:09] <superm1> Mez, generally you need to divert invoke-rc.d if you are working directly in a chroot doing these kinds of things unless you bind mount /proc and possibly /sys
[23:09] <ScottK> Mez: Does it depend on lsb-base?
[23:10] <superm1> Mez, you can look at livecd-rootfs for an example of what's done while live disks get build
[23:11] <Mez> ScottK: lsb-base is installed, and it has the call for /lib/lsb/init-functions too...
[23:11] <Mez> other scripts in init seem to work, which is a bit weird.
[23:14] <Mez> superm1: I am working in a chroot, yes (trying to setup an EC2 image)
[23:16] <Mez> superm1: seems it was proc


[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base amd64 [Zesty Final] has been updated (20170411)
[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base armhf [Zesty Final] has been updated (20170411)
[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base ppc64el [Zesty Final] has been updated (20170411)
[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base arm64 [Zesty Final] has been updated (20170411)
[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base i386 [Zesty Final] has been updated (20170411)
[04:21] -queuebot:#ubuntu-quality- Builds: Ubuntu Base s390x [Zesty Final] has been updated (20170411)
[04:32] -queuebot:#ubuntu-quality- Builds: Lubuntu Alternate i386 [Zesty Final] has been updated (20170411)
[04:32] -queuebot:#ubuntu-quality- Builds: Lubuntu Alternate amd64 [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Server amd64 [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Server i386 [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Server s390x [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Server arm64 [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Server ppc64el [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Lubuntu Desktop i386 [Zesty Final] has been updated (20170411)
[04:51] -queuebot:#ubuntu-quality- Builds: Lubuntu Desktop amd64 [Zesty Final] has been updated (20170411)
[04:53] -queuebot:#ubuntu-quality- Builds: Ubuntu Server armhf+raspi2 [Zesty Final] has been updated (20170411)
[05:05] -queuebot:#ubuntu-quality- Builds: Xubuntu Desktop amd64 [Zesty Final] has been updated (20170411)
[05:05] -queuebot:#ubuntu-quality- Builds: Xubuntu Desktop i386 [Zesty Final] has been updated (20170411)
[05:10] -queuebot:#ubuntu-quality- Builds: Ubuntu Desktop amd64 [Zesty Final] has been updated (20170411)
[05:10] -queuebot:#ubuntu-quality- Builds: Ubuntu Desktop i386 [Zesty Final] has been updated (20170411)
[05:19] -queuebot:#ubuntu-quality- Builds: Ubuntu Kylin Desktop amd64 [Zesty Final] has been updated (20170411)
[05:19] -queuebot:#ubuntu-quality- Builds: Ubuntu Kylin Desktop i386 [Zesty Final] has been updated (20170411)
[05:40] -queuebot:#ubuntu-quality- Builds: Ubuntu GNOME Desktop amd64 [Zesty Final] has been updated (20170411)
[05:40] -queuebot:#ubuntu-quality- Builds: Ubuntu GNOME Desktop i386 [Zesty Final] has been updated (20170411)
[05:43] -queuebot:#ubuntu-quality- Builds: Ubuntu Budgie Desktop amd64 [Zesty Final] has been updated (20170411)
[05:43] -queuebot:#ubuntu-quality- Builds: Ubuntu MATE Desktop amd64 [Zesty Final] has been updated (20170411)
[05:43] -queuebot:#ubuntu-quality- Builds: Ubuntu Budgie Desktop i386 [Zesty Final] has been updated (20170411)
[05:43] -queuebot:#ubuntu-quality- Builds: Ubuntu MATE Desktop i386 [Zesty Final] has been updated (20170411)
[05:44] -queuebot:#ubuntu-quality- Builds: Kubuntu Desktop i386 [Zesty Final] has been updated (20170411)
[05:44] -queuebot:#ubuntu-quality- Builds: Kubuntu Desktop amd64 [Zesty Final] has been updated (20170411)
[05:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Studio DVD amd64 [Zesty Final] has been updated (20170411)
[05:51] -queuebot:#ubuntu-quality- Builds: Ubuntu Studio DVD i386 [Zesty Final] has been updated (20170411)
[12:02] -queuebot:#ubuntu-quality- Builds: Ubuntu MATE Desktop amd64 [Zesty Final] has been updated (20170411.1)
[12:02] -queuebot:#ubuntu-quality- Builds: Ubuntu MATE Desktop i386 [Zesty Final] has been updated (20170411.1)
[13:24] -queuebot:#ubuntu-quality- Builds: Ubuntu Studio DVD amd64 [Zesty Final] has been updated (20170411.1)
[13:25] -queuebot:#ubuntu-quality- Builds: Ubuntu Studio DVD i386 [Zesty Final] has been updated (20170411.1)


[14:13] <nkn> folositi vreunul dc++ ?
[17:33] <ppine> buna buna
[18:11] <ppine> mda mda buna asa
[20:03] <amorphous1> I'll login right after you leave...I wanna have the time to put all my cases in order by the time you leave
[20:03] <amorphous1> this is my last day..I'll be back in 2 weeks
[20:04] <amorphous1> scuze...mesajede de mai sus ^^ erau destinate altcuiva


[00:02] <dewman> mrand, ok...I had some mixed results, it seems to find all the channels with the default timeout, but..when tuning to live tv it doesnt actually tune the channel and then pops back to the menu. So what I did was add a longer timeout and then I checked the decryption status on each channel and I got better results. Are you using the normal release firmware on your tuner or the beta version?
[03:26] <m4xx|work> anyone care to help me get the linuxtv cx18 drivers to compile?
[03:27] <m4xx|work> in an attempt to get both of my cards working i've updated to 10.10
[03:27] <m4xx|work> the latest media_build.git doesn't seem to have the cx18 drivers and i'm not sure how to get the older version that did
[03:42] <manmountain> I'm a complete newbie with linux. I want to change my primary OS from Windows to a linux system. But I need to be able to stream media to other machines in the house (Laptops, xbox 360, ps3 and smartphones). Will mythtv to the job well?
[04:03] <mrand> dewman: released.  And honestly, I've actually only used it once or twice.  I've spent more time playing with the scanner now than tuning real channels for viewing or capture.
[15:32] <htpc> Hello,
[15:32] <htpc> I've installed mythbuntu and changed the mysql password
[15:33] <htpc> now mythTV can startup
[15:33] <htpc> So I've found a instruction to reinstall the mysql database, so I deleted it and try to reinstall but with no success.
[15:34] <htpc> How can a reinstall MythTV or should I reinstall mythbuntu completely?
[16:01] <htpc> have solved the problem by reinstalling the complete mythtv packages, thanks
[16:01] <htpc> and bye ;)