/srv/irclogs.ubuntu.com/2015/07/28/#launchpad-dev.txt

mwhudsonblr, wgrant: woo the (somewhat) despecial casing of launchpad in the go tool just landed03:31
wgrantYay03:37
blrmwhudson: nice, progress of sorts!03:37
mwhudsonyeah03:37
mwhudsonthe last two days' worth of fun is finding out what breaks in the archive with go 1.5...03:38
mwhudsonvs what's broken already03:38
wgrantEverything and most things?03:38
wgrantOr, on x86, more than everything.03:38
wgrants/x86/!x86/03:38
mwhudsonheh well, poking as few bears as possible03:39
mwhudsonso not building 1.5 for arm64 or ppc64le yet03:39
mwhudsonor using shared libs03:39
mwhudson20 packages have failures, about half of which are not my fault03:39
mwhudson(already broken with gccgo in archive, or ftbfs on any arch for some reason)03:40
wgrantEw03:40
mwhudsonhttps://docs.google.com/spreadsheets/d/1hsw1qAy0yLAfqUnZ9fVbXKwe7HxwUzpx5kJ72bF6nUE/edit#gid=003:40
mwhudsonwgrant: btw, chdist is super handy, thanks for reminding me about that03:41
wgrant:)03:43
wgrantcjwatson: Is --check-distribution useful given the --check-archive option?11:37
wgranteg. ubuntu-archive-tools always infers distribution from archive.11:37
cjwatsonu-a-t provides distribution options too in case you didn't want to specify an archive ...11:38
wgrantOh, so it does.11:38
wgrantThough in this case it's --check-distribution=ubuntu in place of --archive=ubuntu, which doesn't seem like much of a win.11:38
cjwatsonI dunno, I was mostly being regular11:38
cjwatsonCan remove them if you prefer11:39
cjwatsons/them/it/11:39
wgrantI'm not fussed, but it doesn't seem particularly useful.11:40
cjwatsonInferring distribution from archive makes sense at least; I'd forgotten that u-a-t did that.11:40
wgrantThe non-archive methods only exist on the API for compatibility reasons and for partner.11:40
wgrantIn u-a-t it's probably just compatibility.11:40
wgrant-A ubuntu vs -d ubuntu is hardly an ease of use problem.11:41
cjwatsonDepends on the tool; in e.g. manage-chroot it makes more sense to talk about the distribution.11:41
wgrantIndeed.11:41
wgrantBut it's not operating on an archive at all in that case (potentially a misfeature, as we've run into a few times...)11:42
cjwatsonwgrant: updated MP, how about now?11:48
wgrantcjwatson: Looks good, thanks.11:50
cjwatsonwgrant: as for https://code.launchpad.net/~cjwatson/launchpad/db-livefs-build-ordering/+merge/265660, the existing index becomes inoperative anyway once the new code is deployed.  Since it's a smallish table I decided not to worry too much11:52
wgrantcjwatson: Ah, fair point.11:52
wgrantcjwatson: My main concern was the long-running transaction thing, anyway.11:54
wgrantDon't want to be stuck for a 12 hour backup.11:54
cjwatsonRemind me of the backup schedule?11:54
cjwatsonMaybe we should just FDT it shortly after the code update.11:55
wgrantNah, live is fine.11:55
cjwatsonI was planning to request the whole thing once sourcherry gets round to applying this one.11:56
cjwatsonDid you get a chance to look at any of my snap branch stack?11:56
wgrantWas unwell and off yesterday, and not 100% today, so I've merely pondered them with no obvious objections.11:58
cjwatsonAh, sorry to hear that.11:58
cjwatsonI'm still slightly recovering from camping in a field for three days, but my inbox has more than enough to keep me occupied?11:58
cjwatsons/\?/!/11:58
wgrantOh dear, a field!11:58
cjwatsonDistressingly outdoors.11:58
wgrantThe weather looked nice, though.11:59
cjwatsonUh11:59
wgrantWell, the temperature, at least.11:59
wgrantI didn't check the rest :)11:59
cjwatsonWe got pretty wet on Sunday.  Traditional festival weather.11:59
cjwatsonGood fun though12:00
wgrantExcellent.12:00
cjwatsonwgrant: Hmm, I wonder what to do about stuff like https://launchpad.net/ubuntu/+source/gem2deb/0.20.1/+build/7722524.  It's beginning to look like we might be best off copying the entirety of Dpkg::Deps into our backported sbuild package as Sbuild::Deps14:02
cjwatson(an approach previously taken in Debian buildd branches ...)14:03
wgrantcjwatson: Hum, what happened there? It just excluded anything with a profile restriction, even when it was a negative?14:05
cjwatson  DB<2> x deps_parse("ruby-rspec <!nocheck>", reduce_arch => 1, host_arch => "i386", build_arch => "i386", build_dep => 1, reduce_profiles => 1, build_profiles => [])14:06
cjwatson0  Dpkg::Deps::AND=HASH(0x9286abc)14:06
cjwatson   'list' => ARRAY(0x90fb1e4)14:06
cjwatson        empty array14:06
cjwatson^- that happened14:06
cjwatsonI guess that's a long way of saying "yes"14:06
wgrantQuite.14:06
wgrantHow unfortunate.14:07
cjwatsonI don't know why there were no warnings or anything14:07
cjwatsonDpkg::Deps is probably fairly self-contained on top of an otherwise modernish libdpkg-perl.14:07
cjwatsonI think it just needs Dpkg::BuildProfiles (possibly only on precise).14:08
cjwatsonIn fact on both precise and trusty, since trusty's doesn't have parse_build_profiles or evaluate_restriction_formula.14:09
cjwatsonNeeds Dpkg::BuildEnv on precise, but I think that's the lot.14:11
cjwatsonwgrant: Proposed sbuild patches: http://paste.ubuntu.com/11954598/ (precise) http://paste.ubuntu.com/11954600/ (trusty)17:06
rpadovanicjwatson, o/ how are you? :-)20:35
=== blr_ is now known as blr
cjwatsonwgrant: hmm, https://launchpad.net/ubuntu/+archive/test-rebuild-20150722/+build/7682014 looks weird - it's not a non-UTF-8 issue, because the byte reported is the first byte of รถ in UTF-823:19
cjwatsonbut it seems astonishing that we would still have bugs with UTF-8 .changes23:20
wgrantcjwatson: Oh, interesting.23:20
wgrantcjwatson: I saw a couple of instances of that in the OOPS reports a week or so ago.23:20
wgrantBut it wasn't widespread, so I assumed it was legitimate badness.23:20
wgrantIt's presumably the maintainer email fixing changes failing to decode it somewhere.23:21
cjwatsonRegression from r17611, perhaps23:22
wgrantYeah, must be.23:22
wgrantI'll try to repro.23:22
cjwatsonThanks23:22
cjwatsonI remember there being functions in there that had decoding as kind of a side-effect, and thought I'd managed to disentangle all of that but maybe not totally correctly.23:24
wgrantAh, it wasn't in OOPS reports, it was in process-upload.py cronspam.23:25
wgranthttps://pastebin.canonical.com/136298/23:25
wgrantRight, reproducible easily.23:29
wgrantWill fix.23:29
wgrantYay SQLObject compat layer.23:29
cjwatsonWell23:29
cjwatsonIt's not really sqlobject's fault23:30
wgrant(test suite would have caught this if it were native Storm)23:30
cjwatsonAh, OK23:30
wgrantBecause Storm whines if you try to put a str in a unicode column.23:30
wgrantIt's much more Python 3 about things.23:30
cjwatsonThat value would previously have been ascii_smashed, so still a str but without any non-ASCII23:30
cjwatson(And a lot less useful)23:30
wgrantVery annoying for whipping up code quickly, very handy for not writing dodgy code.23:30
cjwatsonI made it be str for compatibility, but maybe it would have been better to fix all the callers to accept unicode23:31
wgrantALEFLAMMEIFORGETTHEREST23:31
cjwatsonExactly23:31
cjwatsonSemantics of that value are "the RFC822-encoded version of this field", so you can probably make a case for either octet-stream or text23:31
cjwatsonOh in fact it's just the name isn't it23:32
cjwatson    name = force_to_utf8(name)23:32
cjwatsonSo I guess just .decode23:33
wgrantreally?23:33
cjwatsonAll very silly code.23:33
wgrantwhat on earth23:33
wgranthave you *read* force_to_utf8?23:33
cjwatsonI have.23:34
wgrantThat is certainly the stupidest thing I've seen today.23:34
cjwatsonIt probably sort of made sense with the prevailing common case of pre-UTF-8 control files.23:34
cjwatsonKind of.23:34
wgrantNo23:34
wgrantRead it23:34
wgrantOh23:34
wgrantI misread.23:35
cjwatsonYes, I know :)23:35
wgrantOops23:35
cjwatsonISO-8859-1 was probably the most common non-UTF-8 encoding for those though23:35
wgrantI misread it as doing something with the decoded UTF-8 value23:35
wgrantSo it returned a unicode in one case and not the other.23:35
wgrantYeah23:35
wgrantHmmm23:35
cjwatsonIt should arguably be turned into force_to_unicode23:36
cjwatsonIf that doesn't make all the twisty callers explode too badly, and we already know our test coverage is poor ...23:37
cjwatsonAnyway, I must to bed23:37
wgrantNight.23:37

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!