wgrant | nigelb: Are you sure that branch fixes it completely? | 00:11 |
---|---|---|
wgrant | nigelb: If you're not pretty sure that's enough to totally fix it, we need to revert your revision. | 00:11 |
nigelb | wgrant: as embarassing as it sounds, yes. | 00:11 |
nigelb | I added tests | 00:11 |
nigelb | *more* tests | 00:11 |
nigelb | wgrant: Is there more I could do to actually prove it works? | 00:13 |
nigelb | Is there a test server I could push to? | 00:13 |
nigelb | without getting it committed that is. | 00:13 |
wgrant | I'll try merging it on DF. | 00:14 |
nigelb | DF? | 00:15 |
wgrant | dogfood.launchpad.net | 00:15 |
wgrant | Normally used for Soyuz testing. | 00:15 |
nigelb | Can I visit it? | 00:15 |
nigelb | i.e., can people not in ~launchpad visit it. | 00:16 |
wgrant | yes. | 00:16 |
nigelb | Ok, if you can get it running there, that'd be great. | 00:16 |
wgrant | Although it's very slow, as everything including the database runs on a machine from 2004. | 00:16 |
nigelb | I just need to load a bug. | 00:17 |
nigelb | wow, OOOPs all over the place. | 00:18 |
wgrant | Ah, yeah, loooks like we might have issues with testing this on DF. | 00:21 |
wgrant | Because its database is old. | 00:21 |
wgrant | And doesn't have BugMessage.owner set. | 00:21 |
nigelb | ah | 00:22 |
nigelb | argh | 00:22 |
nigelb | let me try something | 00:22 |
wgrant | So, I think I need to roll this back, and perhaps you can ask a LOSA to merge it temporarily on (qa)staging next week. | 00:23 |
wgrant | So you can test it out. | 00:23 |
nigelb | Is it possible to do a merge on DF? | 00:24 |
nigelb | i.e. code merge | 00:24 |
wgrant | Yes, but its DF is crap so that won't be much use. | 00:24 |
wgrant | Er. | 00:24 |
nigelb | \o/ | 00:24 |
wgrant | s/DF/DB/ | 00:24 |
nigelb | Okay, so I can see a merge | 00:24 |
nigelb | If you can push the code, I can test it in an MP as well. | 00:25 |
wgrant | You'll have to create a new MP. | 00:25 |
nigelb | I created one | 00:25 |
nigelb | https://code.dogfood.launchpad.net/~summit-hackers/summit/i18n/+merge/60007 | 00:25 |
wgrant | JS should be updated | 00:27 |
nigelb | Code's still old. | 00:29 |
wgrant | Which code? | 00:29 |
nigelb | linkchecker.py needs to be updated as well | 00:29 |
wgrant | Hmm, but it should already be running the latest devel revision... | 00:30 |
nigelb | Its r13954 | 00:30 |
nigelb | While qastaging is r13978 | 00:30 |
wgrant | Well, it says that, but it often lies. | 00:30 |
wgrant | Let's see. | 00:30 |
nigelb | I know that linkchecker.py doesn't contain my changes, because I'm getting what used to happen earlier. | 00:31 |
nigelb | I'm guessing you just brought it down | 00:32 |
wgrant | It's restarting. | 00:33 |
wgrant | Slowly. | 00:33 |
wgrant | And it's back. | 00:33 |
wgrant | Hopefully with the new Python code this time. | 00:33 |
* nigelb does hard refresh | 00:33 | |
nigelb | wgrant: \o/ | 00:35 |
nigelb | Seems to work! | 00:35 |
wgrant | Do you have a bug that would be interesting to test? I can fix particular bugs to render on DF> | 00:36 |
nigelb | one sec | 00:36 |
nigelb | let me see the one we were testing on qastaging | 00:37 |
nigelb | bug 755937 | 00:38 |
_mup_ | Bug #755937: phaseshift version 0.40-13.2 failed to build on i386 <ftbfs> <natty> <oneiric> <universe> <phaseshift (Ubuntu):Confirmed> <phaseshift (Ubuntu Oneiric):Confirmed> < https://launchpad.net/bugs/755937 > | 00:38 |
nigelb | could you fix that on DF? | 00:38 |
wgrant | It renders now. | 00:38 |
nigelb | Trying to duplicate | 00:40 |
nigelb | Gah. | 00:41 |
nigelb | I'm awesome | 00:41 |
wgrant | Oh? | 00:41 |
nigelb | I made the qastaging bug render :p | 00:41 |
wgrant | Heh | 00:41 |
nigelb | Now I can't use that to compare | 00:41 |
nigelb | Works as expected in dogfood. | 00:42 |
nigelb | let me violate another qastaging bug :P | 00:42 |
nigelb | ZOMG. | 00:43 |
nigelb | QAstating did *not* timeout on me. | 00:43 |
nigelb | This needs celebration. | 00:43 |
nigelb | (spoke to soon. Damn) | 00:43 |
nigelb | wgrant: Seems to work like it should | 00:49 |
* wgrant lands. | 00:52 | |
nigelb | wgrant: Sorry about the mess :) | 00:52 |
nigelb | I think I've made a whole lot of revisions undeployable | 00:52 |
wgrant | Indeed, but it's by no means the worst we've had recently :) | 00:53 |
nigelb | HA | 00:53 |
nigelb | I'm really glad I invested in the time to write javascript tests | 00:53 |
nigelb | I would never have fixed this on time without those | 00:53 |
wgrant | Yes, tests are handy :) | 00:54 |
wgrant | nigelb: Could you set a commit message? | 00:54 |
nigelb | yeah, sec | 00:54 |
nigelb | wgrant: done | 00:55 |
nigelb | wgrant: Oh. No test run? | 01:00 |
wgrant | nigelb: No point. It's a weekend, and the deployment pipeline is blocked until this is fixed anyway. | 01:00 |
nigelb | \o/ | 01:01 |
wgrant | Plus it's a safe change. | 01:01 |
nigelb | Right. You just ran it in dogfood and I didn't bring it down :-) | 01:01 |
LPCIBot | Project devel build #1,083: STILL FAILING in 36 sec: https://lpci.wedontsleep.org/job/devel/1083/ | 01:04 |
nigelb | ^ Didn't even start | 01:06 |
LPCIBot | Project devel build #1,084: STILL FAILING in 1.7 sec: https://lpci.wedontsleep.org/job/devel/1084/ | 01:16 |
=== almaisan-away is now known as al-maisan | ||
=== al-maisan is now known as almaisan-away | ||
LPCIBot | Project devel build #1,085: STILL FAILING in 1.8 sec: https://lpci.wedontsleep.org/job/devel/1085/ | 01:31 |
LPCIBot | Project devel build #1,086: STILL FAILING in 1.7 sec: https://lpci.wedontsleep.org/job/devel/1086/ | 01:46 |
LPCIBot | Project devel build #1,087: STILL FAILING in 1.7 sec: https://lpci.wedontsleep.org/job/devel/1087/ | 02:01 |
LPCIBot | Project devel build #1,088: STILL FAILING in 1.8 sec: https://lpci.wedontsleep.org/job/devel/1088/ | 02:16 |
LPCIBot | Project devel build #1,089: STILL FAILING in 1.7 sec: https://lpci.wedontsleep.org/job/devel/1089/ | 02:31 |
LPCIBot | Project devel build #1,090: STILL FAILING in 1.7 sec: https://lpci.wedontsleep.org/job/devel/1090/ | 02:46 |
wgrant | cjwatson: Is remove-package.py used at all these days? It was superseded by lp-remove-package.py years ago, but never deleted... | 04:54 |
StevenK | I can not recall remove-package being used while I've been an AA | 05:12 |
wgrant | I'm waiting for mawson to tell me if anyone has. | 05:13 |
StevenK | It imports dak_utils for crying out loud | 05:13 |
wgrant | The last source it was used on was in 2007... binaries I'm still waiting for. | 05:13 |
wgrant | Yep. | 05:13 |
wgrant | But then again so does sync-source... | 05:13 |
StevenK | steven@liquified:~/launchpad/lp-branches/devel% grep -c '^#' scripts/ftpmaster-tools/remove-package.py | 05:13 |
StevenK | 145 | 05:13 |
wgrant | But that can hopefully die soon. | 05:13 |
wgrant | Yes. | 05:13 |
wgrant | There's a revdep check all commented out in there :/ | 05:13 |
StevenK | Yes. | 05:14 |
StevenK | Because removing code is hard, or something. | 05:14 |
StevenK | An XXX from elmo. Neat. | 05:14 |
StevenK | That is disgusting, kill it. | 05:15 |
wgrant | It's already gone. | 05:15 |
wgrant | Just wanting confirmation. | 05:16 |
* StevenK stares at scripts/_ginalog.py | 05:19 | |
wgrant | Yes, pretty much. | 05:20 |
wgrant | But it doesn't use initZopeless, so this branch won't delete it. | 05:20 |
StevenK | Oh, nice, you're killing it already? | 05:20 |
wgrant | Well, porting most scripts that use it directly to use LaunchpadScript instead. | 05:20 |
wgrant | And deleting lots of scripts that use it but don't work any more. | 05:21 |
StevenK | Does that mean canonical.lp dies? | 05:21 |
wgrant | Hopefully in the next couple of days. | 05:21 |
wgrant | Still a little bit to go. | 05:21 |
wgrant | Hmm, untested script written in 2006 and unchanged since then except for compatibility fixes. | 05:24 |
wgrant | I think it can die. | 05:24 |
StevenK | Which one? | 05:33 |
wgrant | scripts/rosetta/check-distroseries-translations-diffs.py | 05:34 |
StevenK | Nothing seems to reference scripts/_ginalog.py, I'm tempted to just delete it. | 05:40 |
wgrant | I believe that would be the correct course of action. | 05:42 |
wgrant | 540KB of accidental addition, I suspect. | 05:42 |
wgrant | It hasn't changed since it was added around r1149 | 05:44 |
StevenK | 1 file changed, 15640 deletions(-) | 05:48 |
wgrant | Heh | 05:48 |
nigelb | Woah | 05:49 |
nigelb | With the amount of "die" and "kill" you both use, a casual observer would think you both are professional hitmen. | 05:49 |
StevenK | I used 'rm' this time! | 05:50 |
nigelb | ha | 05:50 |
StevenK | wgrant: Tossed at PQM | 05:50 |
wgrant | The code with the lowest maintenance cost is code that doesn't exist :) | 05:50 |
nigelb | +1 to that | 05:50 |
StevenK | Neat. r13980 | 05:54 |
nigelb | 20 more revs to r14000. Neat. | 05:55 |
wgrant | StevenK: Have you read sync-source.py lately? | 06:13 |
wgrant | def init(): global Blacklisted, Library, Lock, Log, Options | 06:13 |
wgrant | That's a pretty good summary of its style. | 06:13 |
StevenK | I don't think I want to. | 07:23 |
wgrant | 31 files changed, 558 insertions(+), 2198 deletions(-) | 07:28 |
wgrant | And canonical.lp is dead. | 07:28 |
StevenK | \o/ | 07:29 |
StevenK | Have you pushed it? | 07:29 |
wgrant | The three branches are in ec2. | 07:36 |
wgrant | Will see how much is broken. | 07:36 |
wgrant | Ah, this is handy. | 08:49 |
wgrant | I think our manual escaping stuff might do the wrong thing with postgres 9.1's escaping changes. | 08:51 |
wgrant | :( | 08:51 |
cjwatson | wgrant: yes, feel free to kill remove-package.py | 08:54 |
wgrant | cjwatson: Thanks. | 08:54 |
cjwatson | we still need sync-source.py for a while until a few more bugs in the new-style thing are worked out | 08:56 |
wgrant | Yep. | 08:56 |
cjwatson | particularly sponsorship | 08:56 |
cjwatson | *sigh* must find time to finish writing that autosync API script too | 08:56 |
wgrant | cjwatson: Are archive-{integrity,override,cruft}-check used? | 08:58 |
cjwatson | cruft is very heavily used. | 08:58 |
wgrant | Ah, so we still use it for NBS? | 08:58 |
cjwatson | I can't remember about the other two. | 08:58 |
cjwatson | Yes. | 08:58 |
cjwatson | I kind of feel like we ought to use integrity but I haven't done so for ages. | 08:59 |
wgrant | It also does ASBA, which is a bit odd. | 08:59 |
wgrant | Yeah, I want to run something like integrity, that basically compares the pool with the DB. | 08:59 |
wgrant | Because there is heaps of cruft there, and some stuff is probably missing. | 09:00 |
cjwatson | I think this is the first time I've heard of archive-override-check. Exactly what "inconsistences" [sic] does it report on? | 09:00 |
wgrant | It's the one I don't know about, too. | 09:00 |
wgrant | Let me read it. | 09:00 |
wgrant | I think it might check consistency between architectures. | 09:02 |
cjwatson | So like http://people.canonical.com/~ubuntu-archive/architecture-mismatches.txt except unused? :-) | 09:02 |
cjwatson | That report is just done based on Packages and Sources files. | 09:02 |
wgrant | Indeed, probably. I assumed NBS there was similar -- didn't realise you actually used archive-cruft-check. | 09:03 |
cjwatson | So, OK, it'll miss things with inconsistent overrides that aren't built, but ... | 09:03 |
cjwatson | Yeah, we use it as the first stage of the input | 09:03 |
cjwatson | I wouldn't object to rewriting it at some point | 09:03 |
cjwatson | http://paste.ubuntu.com/692165/ - don't vomit all at once | 09:04 |
wgrant | I plan to write a new integrity checker in the short term, but have no plans for cruft, apart from the port to LaunchpadScript that I did this afternoon. | 09:04 |
wgrant | Not bad, not bad. | 09:04 |
wgrant | That explains why the output was unrecognisable. | 09:04 |
cjwatson | I particularly like the grep '^ *o ' | 09:04 |
wgrant | Yeah. | 09:06 |
wgrant | AFAICT archive-cruft-check's ASBA support is unused and doesn't even make sense. | 09:06 |
wgrant | Perhaps it originated with dak. | 09:06 |
cjwatson | I don't believe I've used it for some time | 09:21 |
cjwatson | If you want to delete that part, that's fine by me; or you could just ignore it until I rewrite cron.NBS and propose a branch that removes the script entirely | 09:23 |
cjwatson | OK, WTF was _ginalog.py about? | 09:23 |
cjwatson | Oh. Wow. ArchiveCruftChecker doesn't even talk to the database except to get the current distroseries and such (and the removal code which we don't use). I could just pull that out wholesale. | 09:26 |
cjwatson | (Perhaps not on a Sunday morning though.) | 09:27 |
wgrant | cjwatson: Yeah, most of archive-*-checker don't really use the DB for anything useful. | 09:43 |
wgrant | There's a *lot* of cruft in LP :) | 09:44 |
wgrant | nigelb: Thanks. | 09:53 |
nigelb | wgrant: \o/ | 10:16 |
nigelb | Not my happiest moment. Really. | 10:17 |
wgrant | Heh :) | 10:19 |
StevenK | Hehe, even cjwatson comments on _gina log | 11:32 |
=== Pendulum_ is now known as Pendulum | ||
m4n1sh | where do all the translation strings like here come from? Do they come from po/messages.pot file present in branch associated with series which is focus of development? | 18:11 |
nigelb | Not the best of times. Oceania still hasn't woken up. | 18:22 |
dobey | m4n1sh: "like here" ? | 21:27 |
dobey | m4n1sh: also, Oceania is at war with East Asia, so that whole thing might get in the way. ;) | 21:28 |
m4n1sh | errrr? had too much of whisky? | 21:28 |
m4n1sh | :) | 21:29 |
dobey | no | 21:29 |
dobey | did you miss the 2 minutes hate, brother? | 21:29 |
mwhudson | morning | 21:30 |
dobey | m4n1sh: http://en.wikipedia.org/wiki/Nineteen_Eighty-Four#The_War | 21:30 |
m4n1sh | ahhh | 21:31 |
m4n1sh | got it | 21:31 |
m4n1sh | :) | 21:31 |
dobey | m4n1sh: anyway, i'm not sure what you mean by 'like here' given you provided no link :) | 21:32 |
m4n1sh | yes | 21:32 |
m4n1sh | I know | 21:32 |
m4n1sh | that was the mistake | 21:32 |
m4n1sh | I realized it just now | 21:32 |
m4n1sh | it was the http://translations.launchpad.net/pinta | 21:33 |
m4n1sh | page | 21:33 |
m4n1sh | was trying my hands on gettext and all those, not able to understand many things | 21:33 |
dobey | ah; i think translations on a project are imported from the .pot file in the series, if there is one | 21:35 |
dobey | ugh | 21:36 |
dobey | what the heck is pinta doing | 21:36 |
dobey | eww | 21:36 |
m4n1sh | dobey: where? | 21:38 |
dobey | with translations | 21:39 |
m4n1sh | it got a rebirth | 21:39 |
m4n1sh | I dont think the translations work properly | 21:39 |
m4n1sh | the translations should be named like es.pot, ro.pot etc | 21:39 |
dobey | it's doing weird stuff manually with gettext, i presume to try and support windows also | 21:39 |
dobey | no | 21:39 |
m4n1sh | like locale.pot | 21:39 |
dobey | no | 21:39 |
m4n1sh | right now it is messages-<locale>.pot | 21:40 |
dobey | there is only one pot file | 21:40 |
m4n1sh | sorry | 21:40 |
dobey | no, it's .po | 21:40 |
m4n1sh | I mean | 21:40 |
m4n1sh | right now it is messages-<locale>.po | 21:40 |
m4n1sh | it should be just locale.po | 21:40 |
dobey | well, a lot of things should be different | 21:40 |
m4n1sh | but all the files in pinta repo is named messages-<locale>.po | 21:40 |
m4n1sh | differnet like? | 21:40 |
dobey | it's not clear how to me pinta expects to support windows/osx though | 21:41 |
m4n1sh | it does | 21:41 |
dobey | well, "messages" is not a proper gettext package name | 21:41 |
dobey | m4n1sh: i know it does. i mean in the technical sense | 21:41 |
dobey | ie, a single .exe, or does it require cygwin to work, or what | 21:42 |
m4n1sh | dobey: it works on windows and osx too | 21:42 |
m4n1sh | over mono | 21:42 |
m4n1sh | runs over mono for windows | 21:42 |
m4n1sh | or probably even .NET | 21:42 |
dobey | yes but that doesn't tell me anything | 21:42 |
m4n1sh | https://github.com/PintaProject/Pinta/tree/master/po | 21:42 |
m4n1sh | check this | 21:42 |
dobey | i looked at it | 21:42 |
dobey | but it doesn't tell me anything about it, other than it's being done wrong | 21:42 |
m4n1sh | lol | 21:43 |
dobey | anyway, not an issue with launchpad itself | 21:43 |
m4n1sh | I dont think so | 21:43 |
m4n1sh | launchpad exports it as <locale>.po | 21:43 |
m4n1sh | AFAIK | 21:43 |
m4n1sh | this messages-foo thing breaks pure gettext based files | 21:43 |
m4n1sh | like pinta.desktop.in | 21:43 |
dobey | yes, like i said. the way pinta is doing translations is totally broken :) | 21:44 |
m4n1sh | I get entries in pinta.desktop as | 21:44 |
m4n1sh | Name[messages-ro] = blah blah | 21:44 |
m4n1sh | I expect to get | 21:45 |
m4n1sh | Name[ro] = blab blah | 21:45 |
dobey | eh? | 21:45 |
m4n1sh | yes | 21:45 |
m4n1sh | after running make I get "Name[messages-ro] = blah blah" in pinta.desktop file | 21:45 |
m4n1sh | not what I expect | 21:45 |
dobey | i don't see how | 21:46 |
dobey | are you building/installing on windows? | 21:46 |
m4n1sh | no | 21:46 |
m4n1sh | Ubuntu | 21:46 |
dobey | is bzr way behind what's in git? or do you have some patch to make it build? | 21:47 |
m4n1sh | it looks for all the files in po/ dir and checks for the string I Used in _Name | 21:47 |
m4n1sh | so the translation for that string is in messages-ro.po | 21:47 |
m4n1sh | so it substitues Name[messages-ro] = fo fo | 21:47 |
m4n1sh | dobey: this way is not related to git or bzr | 21:48 |
dobey | is there a pinta irc channel? | 21:48 |
m4n1sh | yes | 21:48 |
m4n1sh | dead | 21:48 |
m4n1sh | only me and Laney are there :) | 21:48 |
m4n1sh | this gettext thing is done by intltool and autotools | 21:48 |
m4n1sh | two highly confusing things | 21:49 |
m4n1sh | nothing git or bzr specific | 21:49 |
dobey | let's move discussion there | 21:49 |
m4n1sh | yes | 21:49 |
lifeless | :P | 21:59 |
nigelb | morning lifeless | 22:05 |
lifeless | hi nigelb | 22:59 |
nigelb | Sleepless nights are so not fun. | 23:00 |
nigelb | wallyworld_: \o/ I landed that bug title via XHR that I was working on! After qa-bad twice though :( (wgrant may stab me anytime :P) | 23:01 |
=== _thumper_ is now known as thumper | ||
wallyworld_ | nigelb: excellent. well done. i've been fighting with unity this morning :-( | 23:06 |
nigelb | Ouch, that sounds like a bad way to start a Monday | 23:06 |
wallyworld_ | yep. been bad for a few days. hopefully beta2 will be better | 23:07 |
wgrant | lifeless: Are we still forbidden from fastdowntime during beta week? | 23:09 |
lifeless | wgrant: on some days yes | 23:11 |
wgrant | lifeless: Wed/Thu I could understand, but it seems like two minutes early in the week shouldn't hurt. | 23:12 |
wgrant | I guess it depends when they aim to have images, too. | 23:12 |
* wgrant stabs Optus a bit. | 23:13 | |
cjwatson | I wouldn't expect a Monday fastdowntime to be a problem. After that I think we'd prefer you held off | 23:27 |
cjwatson | and Friday should be OK, given that the main bit that isn't fast yet is the publisher downtime | 23:28 |
wgrant | cjwatson: Thanks. | 23:29 |
lifeless | cjwatson: wgrant: I think kate asked for wed/thu | 23:56 |
lifeless | skaet: ^ was it those two days you wanted blacklisted, or tuesday as well ? | 23:56 |
wgrant | Does she realise it means missing a publisher run? | 23:58 |
wgrant | Although it may be on manual at that point anyway. | 23:58 |
lifeless | https://code.launchpad.net/~lifeless/launchpad/use-oops-timeline/+merge/75935 | 23:58 |
lifeless | wgrant: does it ? | 23:58 |
wgrant | lifeless: It finishes at :26-33 or so, so we disable it an hour before. | 23:59 |
wgrant | Meaning that the 0802 run doesn't happen. | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!