/srv/irclogs.ubuntu.com/2008/10/20/#ubuntu-mozillateam.txt

asacsaivann: there also is a menu entry in Tools00:16
ftaasac, http://news.launchpad.net/cool-new-stuff/stacked-branches-holding-post00:31
fta"i.e. Launchpad itself - it now takes less than two minutes to push up a branch. It used to take an hour and a half" ?? 1 & 1/2 hour ?????00:32
ftahttp://ubuntuforums.org/showthread.php?t=95300000:36
asacfta: yeah ... most likely extreme situations like pushing complete ffox tree01:01
ftaasac, in case you missed it, i packaged that: http://www.instantbird.com/01:07
ftaas a xulapp01:07
asacfta: so thats a good example of a xulapp that doesnt shipp everything and fork stuff?01:10
ftamostly, there's no gecko inside, but there's a kind of build system adapted from comm-central, ie needing access to mozilla-central, exactly like tb3 and sm2 now01:12
ftai dropped that build system and replaced it by my xulapp magic, all fine01:13
ftaasac, ^^ https://code.edge.launchpad.net/~mozillateam/instantbird/instantbird.head01:14
asacfta: nice01:18
asacfta: just to check01:18
asacfta: the build-system.tar.gz is a carefully seclected subtree of the xulrunner tree right?01:18
ftaright01:18
asacdo we patch anything before tarring that up?01:18
ftait's done in post-patches::01:21
asacfta: right. but do we patch any of the files that get into that build-system.tar.gz01:21
asacand if so, is that done for the sole purpose of providing that build system?01:21
ftaprobably yes, configure.in is in there01:23
fta?01:24
ftahttp://bazaar.launchpad.net/~mozillateam/xulrunner/xulrunner-1.9.head/annotate/359?file_id=createbuildsystem.sh-20080620161753-mwg01p9oiuq5axbb-101:24
ftaxul provides this tarball in the dev package, callers uses xulapp.mk from m-d, everything else is magic01:27
asachmm01:33
asaci will take a closer look tomorrow maybe. too late now01:34
asacgot bitten by stupid nspluginwrapper01:34
asac;)01:34
asaccu tomorrow01:34
ftacu01:34
NCommanderhey asac05:02
asachi09:01
NCommanderhow goes it asac09:08
* NCommander got his AM report sent to the FD09:08
asacNCommander: cool :)09:13
asacall fine here ;)09:13
asacso given that the frontdesk and DAM work well now, you should be a DD in two weeks09:13
NCommanderasac, so we did one major boo-boo with the icedove upload09:13
NCommanderWe updated control.in09:13
NCommanderBut didn't regenerate control >.<;09:14
* NCommander notes that slipped past both of us, my sponsor, and a release manager09:14
asacNCommander: huh?09:19
asacNCommander: a) why did we update control at all?09:19
asacb) what is the regression you are seeing?09:20
NCommanderasac, Maintainer change09:20
asachmm09:20
NCommanderits not a regression09:20
NCommanderRemember, you wanted me to change Maintainer to Ubuntu Mozilla Team, then add me, yourself, and someone else to uploaders?09:20
NCommanderThe change landed in control.in, but the actual control file didn't get regenerated by accident09:20
asacyeah09:21
asactoo bad09:21
[reed]yeah, too many acronyms... AM, FD, DAM, DD...09:21
[reed]:p09:21
asac\o/ [reed] is back ;)09:21
[reed]except it's 3:22am, and I have a test in a few hours09:22
[reed]:(09:22
asac[reed]: ok. then good luck. would be cool if you could ping me to test something about this EAP issue09:22
asacafter that ;) (or whenever you feel recovered)09:22
[reed]yeah, I'll try to find you later today or so09:23
asaccool09:26
* asac presses thumbs for the test09:26
asacor crosses fingers ;)09:27
asacNCommander: feel free to fix that in bzr. we probably will do an upload sooner or later and than this gets auto-fixed09:27
NCommanderasac, well, I'd like to fix lintian for icedove09:28
XioNoXHi !09:29
asacNCommander: whatever you want ;)09:29
NCommanderasac, its a pity I can't direct commit. When is the next mozillateam meeting so I can be voted in :-)?09:29
asacNCommander: isnt that in topic?09:30
asacoh09:30
asachmm09:30
NCommander:-)09:30
asacThe next Ubuntu Mozilla Team meeting will be held on: Sunday, 2nd November, 18:00 UTC09:30
NCommanderYour last one was awhile ago09:30
asacin #ubuntu-meeting on FreeNode network (irc.freenode.org).09:30
asacwe have every 6 weeks09:31
NCommanderso whens the next meeting that will have quorum :-) (or does mozillateam not suffer from that issue)09:31
=== asac changed the topic of #ubuntu-mozillateam to: Welcome to the Ubuntu Mozilla Team: | Mailing List: ubuntu-mozillateam@lists.ubuntu.com | Please help Mozilla QA tracker: http://tinyurl.com/6yo6g7 | Firefox 3 released to hardy-updates! | Next meeting will be held on Sunday, 2nd November, 18:00 UTC in #ubuntu-meeting (agenda available at: http://tinyurl.com/2ekzoq )
asacNCommander: we have no quorum problems ;)09:32
NCommanderHandy09:32
asacNCommander: just go ahead and push to your branch09:32
* NCommander adds himself to the adenga09:32
NCommander*agenda09:32
asacNCommander: i dont work on the icedove branch at the moment09:32
asacso its just a pull/push for me09:32
asacto publish your changes09:32
asacwhich is ok for 2 weeks  i guess ;)09:32
NCommanderI need your advice on how to fix the engimail serve bug (you said we needed to add .autoreg)09:33
asacNCommander: yes. in icedove we have to ensure that we ship an empty or placeholder file /usr/lib/icedove/.autoreg09:33
NCommanderwhat does that do?09:34
asacNCommander: in enigmail postinst and rm we have to touch that file if it exists09:34
asace.g. if test -e /usr/lib/icedove/.autoreg; then touch /usr/lib/icedove/.autoreg; fi09:34
asacNCommander: it triggers re-registration of system crhome and extensions09:34
NCommanderah09:35
asacotherwise icedove might not see that your extension has been added/changed09:35
NCommanderSo icedove needs that in its post/pre install?09:35
asacNCommander: no. icedove has to ship a placeholder file ... enigmail has to touch it09:35
asace.g. just an empty file in icedove package09:35
NCommanderOh!09:36
NCommanderso that's why engimail broke09:36
asacNCommander: enigmail broke?09:37
NCommanderwell09:37
NCommanderhas an RC bug09:37
asacNCommander: well. thats my story of it. the NMUer did find another story09:37
NCommanderDid you figure out the NMU non-sense?09:38
asaci looked at the diff for a second and it didnt make much sense to me ... at least i didnt understand what it tries to achieve09:38
NCommanderSo you want to NACK it?09:38
asacbut maybe he had a point. unfortunately there is no comment or so about it in bug09:38
NCommanderIsn't that against NMU protocol?09:38
asacNCommander: what is a NACK? isnt that just a new upload without that change?09:38
NCommandernon-acklodge09:38
NCommanderPretty much. As the maintainer, you can reject NMUs09:39
asacNCommander: how?09:39
NCommanderUpload a version removing their changes ;-)09:39
asacNCommander: i mean this most likely made it into the archive already09:39
asacNCommander: right ;)09:39
NCommanderasac, it is already in the archive09:39
asacthats what i know09:39
NCommanderyou can use the dcut command to kill an upload if it was still in the delayed queue09:39
asacyeah remember about dcut ;)09:40
asacanyway. i dont know if his change makes sense. but we should certainly add the .autoreg stuff in postinst09:40
asachave to get some breakfast>/coffee now09:41
NCommandersame here09:41
asacdebian bug 48649109:49
ubottuDebian bug 486491 in enigmail "enigmail: does not upgrade from etch to lenny inside people's" [Normal,Closed] http://bugs.debian.org/48649109:49
asacNCommander: ok. i  looked. that change is ok10:01
asacso we can do -4 on top of 3.210:01
NCommanderSo we want to ack the NMU10:01
NCommanderOk10:01
asacNCommander: how? i only know updload with or without the NMU ;)10:01
NCommanderasac, the next upload must have an entry in the changelog acking the NMU10:02
NCommander(its weird, I know)10:02
asacacking == repeating the Closes: ?10:02
NCommanderasac, no, something like * Ackknlodging the non-maintainer upload by *uploader*10:06
james_whttp://www.debian.org/doc/developers-reference/pkgs.html#ack-nmu10:07
asacNCommander: ok. that procedure is what i had in mind10:11
NCommanderthank you james_w :-)10:11
asac"just keep on going"10:11
asacand remember to get the bugs properply clodsed10:12
armin76we're whalers on the moon10:27
asachow comes?10:37
armin76never seen futurama?10:39
=== asac_ is now known as asac
asacarmin76: didnt really like it ;)11:22
asacbut maybe its just that i never tried ;)11:22
* NCommander works on icedove11:52
NCommanderasac, .autoreg is properly touched by icedove-gnome-support11:55
NCommanderasac, should we simply move those post* scripts to icedove vs icedove-gnome-support?11:55
asacNCommander: no thats right. this simply means that enigmail should do it too12:03
NCommanderasac, what if icedove is installed without gnome support?12:04
asacNCommander: everything that adds/removes chrome/extensions/locales need to touch it in postinst and postrm12:04
NCommanderso both icedove, and icedove-gnome-support need it, right?12:04
asacNCommander: icedove ships the file itself. -gnome-support just touches it12:04
asacNCommander: icedove doesnt need to touch it12:04
asacjust to ship it so its properly registered as a dpkg managed file12:04
asacNCommander: e.g. icedove will touch it implicitly because it ships that file12:05
NCommanderwe do?12:05
NCommandericedove.install says we don't.12:06
asacNCommander: most likely done in rules12:06
NCommanderYup12:06
asacbecause there is no source for that file12:06
NCommanderOk12:06
NCommanderJust making sure12:06
asacsure12:06
NCommanderasac, I'm going to split the icedove package to create an icedove-data (for all the arch all files)12:06
asacthat was one of the points i mentioned: make sure icedove ships that file ;)12:06
asacNCommander: err. why?12:06
asacthat is really unneeded compication12:07
NCommanderKills a lintian warnings. Packages with large amounts of files in /usr/share should split that off12:07
asacyeah i know. but doesnt make much sense here. i had that in the past and eliminated it12:07
NCommanderWhy'd you elimate it?12:07
asacbecause it doesnt make sense ;)12:07
asacits really a lot of work for not much benefit12:08
asacalso _all packages are broken in the archive12:08
asace.g. if anything build depends on it it can cause a bunch of pain on architectures that lack behind12:08
asacnot that a big reason for icedove ... but well.12:08
NCommanderPoint taken-ish12:09
asacthere is one reason to ship _all files in /usr/share12:09
* NCommander is working to clean out the other lintian errors12:09
asacthat is to have a single partition shared across multiple architectures12:09
asacbut i havent heard of anyone actually using that12:09
asacNCommander: there are just a few errors ... the rest are warnings12:09
asaciirc there are warnings about executable icons or something ;)12:10
asacthat should be fixed.12:10
NCommanderYup, working on that12:10
NCommanderThe biggest one that needs to be fixed is that we install a png as icedove's icon12:10
NCommanderWhich works fine in GNOME/KDE, but breaks in say Window Maker12:10
asacNCommander: instead of xpm?12:10
NCommanderyeah12:11
asacNCommander: i think we should ship both and use a reference without .suffix in .desktop12:11
asacand the xpm in icons12:11
NCommanderasac, that's easy to fix12:11
NCommanderall we do is drop the xpm in debian/ and then have it installed on the fly12:11
asacNCommander: if  possible generate the .xpm on the fly during build12:12
NCommanderOh12:12
NCommanderNot difficult12:12
NCommanderBut it means a build-dep on imagemagick12:12
NCommanderand how often do we really change the icon?12:12
asacNCommander: if possible generate. if thats too much work i dont care about .xpm12:12
asacat least it can be in the .diff.gz without uuencode business12:12
asacbut well ... if we can generate it even better12:12
NCommanderconvert *.png *.xpm :-)12:12
asacNCommander: it clutters the diff12:12
NCommanderthats the actual command12:12
asacmost likely there is a lot of clutter, but that doesnt mean we should add more ;)12:13
NCommanderI can modify the repack rule to do it on the fly12:13
asacNCommander: yeah. i think i already do image transformation duing build12:13
asacconvert et al12:13
asacso just hook it in there12:13
NCommanderCan we upload a new orig.tar.gz for Icedove?12:13
asacNCommander: if repack is the rule where i do converts atm then yeah.12:13
NCommanderWithout a new release?12:14
asacNCommander: why?12:14
asacNCommander: unless we also fix the build system to properly ship the .xpm on make install i would prefer to do the .xpn in rules12:14
NCommanderWe can do it in the rules12:14
asacinstead of the ogig12:14
* NCommander figures out how to attach this12:15
NCommander*attack :-)12:15
asacalso mozilla droped support for .xpm so putting that file in the orig doesnt sound right12:15
NCommanderOh, I see how we can do it12:15
NCommanderI'll just do the conversion during make install12:15
asacas gecko itself cannot render it ;)12:15
asacNCommander: urgh12:15
[reed]It's a horrible format.12:15
NCommanderWell, the debian menu system is a hack12:16
asacNCommander: please do it in rules12:16
NCommanderasac, no, I mean during the install rule of rules :-)12:16
asacNCommander: hacks should go to rules ;)12:16
asacah ok thas ok then ... do it whereever i did the other converts12:16
asacunless i dont sdo that anywhere anymore12:16
NCommanderYou don't12:17
asacthen just add it wherever you feel its appropriate ;)12:17
asaclet me get that branch ;)12:17
asacmy memory appears to have lost some details at some point :)12:18
NCommanderasac, a lot of stuff installed into /usr/lib appears platform independent12:20
* NCommander thinks we can thank upstream for that :-P12:20
asacNCommander: thats ok. thats whate i referred to when saying i stopped doing that12:22
asacin the past i had links to /usr/share stuff ... but that was too much work and gave zero benefit12:23
NCommanderSo upstream ships with everything in /usr/lib?12:24
asacNCommander: upstream doesnt ship in /usr at all12:24
asacNCommander: they ship everything in one directory tree and that tree is  not depending on the install location12:24
NCommandereven in the source packages?12:25
NCommanderr12:25
asaceh?12:25
NCommandernm12:26
asacif you are asking if they dont install it in /usr/share on make insteall .. then yes. you can be happy if they install all files on make install at all12:26
NCommanderOuch12:26
NCommanderI knew Mozilla was bad12:26
NCommanderBut I didn't know how bad12:26
NCommanderNext question (yes, I know), what's cdbs-rules for ;-)12:26
[reed]patches welcome.12:28
asacNCommander: its because i have added a feature we shoujld eventually send upstream12:28
NCommanderah12:28
* NCommander is right now trying to figure where to hang a well placed rm -r command to kill the CVS folders in the binaries12:29
asac[reed]: i will re-look at that. but the past told us that it was _too_ hard to maintain proper make install for mozilla devs. benjamin tried to fix that by using the installer code now to pack things up (which appears to be better maintained). not unlikely that we can make this do what linux FHS suggests. and if we have to maintain a separate file we will end up in the situation we had before: make install is always broken and we always run a12:31
asacs/not unlikely/not likely/12:31
[reed];)12:31
asac[reed]: the reason is that nobody uses make install :) ... so whenever a special file is added to dist/... it gets forgotten12:32
asac[reed]: but well. i can live with it as its now ;)12:32
asac[reed]: but i think tbird doesnt use that mechanism ... but fta can tell more about that.12:33
[reed]yeah12:34
asac[reed]: so ... we have the standard situation here. just a few days left and some users say that EAP is working now12:34
[reed]hmm12:34
[reed]so12:34
asaci need a reliable tester ;)12:34
[reed]I'll get with you later today12:34
[reed]it's 6:35am12:34
[reed]I haven't been to bed12:34
[reed]I have to be up at 9am :(12:34
asacotherwise it might be broken if i do nothing ... or if i land some unconfirmed patch it might be broken even though it worked before ;)12:35
asac[reed]: then go to bed you night-owl ;)12:35
asac_just_ this single day :)12:35
[reed]hehe12:35
NCommanderasac, woohoo, I think I got the package almost completely lintian clean on both binaries and source12:37
asacnice ;)12:39
asacdid you do overrides for the all files in /usr/lib?12:39
NCommanderasac, just did a well placed chmod -x12:41
NCommanderasac, added bonus, icedove is now safely binNMUable12:44
asack12:51
asacfixed strict depends?12:51
NCommanderYup12:53
NCommanderThe only lintian warning on the source package now is that a cdbs junk file keeps getting pulled in12:53
asacNCommander: cdbs junk file?12:57
NCommanderasac, its a bug in CDBS, already reported12:58
NCommander(a manifest of files it tars just gets left behind)12:58
asacok ... so its not us12:58
NCommanderYeah12:58
NCommanderWe just didn't see it in a see of lintian warnings (all gone now :-)!)12:58
NCommanderI'm now looking to see what, if any patches we want to include on this upload12:58
asacNCommander: look if debian bug 392603 still applies13:18
ubottuDebian bug 392603 in icedove "thunderbird: Segfault on opening mail folder" [Important,Open] http://bugs.debian.org/39260313:18
NCommanderasac, if the patch applies cleanly I'll add it13:18
asacNCommander: well. lets not just add it ;)13:18
directhexNCommander, you're smart. what's the right version substitution to use for easy binnmuability?13:18
asacNCommander: the kbsd can be added if its applying cleanly13:18
directhex${binary:Version}, yes?13:19
asacNCommander: the manpage is ok. but we should ensure that the examples actually work13:20
asacNCommander: debian bug 378741 is old and if it doesnt have a better icedove icon (i hink its about the old tbird icon), then we should close it13:20
ubottuDebian bug 378741 in thunderbird "taskbar and application icon look very jagged and pixelated" [Minor,Open] http://bugs.debian.org/37874113:21
NCommanderdirecthex, yes13:21
NCommanderdirecthex, if lintian doesn't complain it is usually ok13:21
directhexit's a while since i ran any lintian tests13:21
asacNCommander: xprint using sh instead of bash ... well. do we use xprint at all?13:21
NCommanderNot anymore13:22
asacdebian bug 441038 is fixed upstream ... or if not we have a proper patch for it in firefox 2 packages in ubuntu13:22
ubottuDebian bug 441038 in icedove "icedove: cannot be symlinked" [Minor,Open] http://bugs.debian.org/44103813:22
asaci am quite sure that debian bug 363815 is too outdated to be of much use now13:22
ubottuDebian bug 363815 in mozilla-thunderbird "get-orig-source debian/rules target for Thunderbird" [Wishlist,Open] http://bugs.debian.org/36381513:22
asacwe should close it or remove the patch tag13:23
NCommanderDon't we have a get-orig-tarball rule?13:23
* NCommander will just close it13:23
asacfor debian bug 499309 we should ask him for "evidence from the news" and if thats the case forward upstream13:24
ubottuDebian bug 499309 in icedove "workaround for Cisco PIX text substitution" [Wishlist,Open] http://bugs.debian.org/49930913:24
asacbut not just blindly apply13:24
asacNCommander: we have repack13:24
asaci dont think we have get-orig-source13:24
asacnot sure if thats really important to have though13:24
NCommanderNo, since we still need to repack13:25
directhexurgh. W: moon source: not-binnmuable-all-depends-any libmoon-dev -> libmoon013:25
NCommanderdirecthex, do >= ${source:Version}13:25
asacdebian bug 363858 is somehow unimportant and appears to be fixed upstream in 3.0a313:26
ubottuDebian bug 363858 in mozilla-thunderbird "Thunderbird Won't Read Signature From a Pipe" [Unknown,Open] http://bugs.debian.org/36385813:26
directhexNCommander, is that the right line for a -dev package?13:26
NCommanderdirecthex, your -dev package is arch: all?13:26
NCommanderand your main package is arch any?13:26
directhexNCommander, you know, that's a good question. why IS it arch:all?13:27
asacnot sure what is going on with debian bug 47630213:27
ubottuDebian bug 476302 in iceweasel "iceweasel: ftbfs on armel: temporary object destruction order" [Normal,Open] http://bugs.debian.org/47630213:27
directhexgah, there was literally *nothing* usable in marillat's package, then13:27
NCommanderasac, I know why. Thats just localized insanity. The patch attached fixes it13:27
* NCommander notes armel has some "fun" issues13:27
NCommanderasac, I already put that patch in the included tarball13:28
asacNCommander: we should look what was applied (if any) in iceape13:28
asacor iceweasel13:28
asacor whatever13:28
NCommanderasac, the patch for iceweasel was attached13:28
asacNCommander: ok we can take it. just ensure that autoreconf patchis properly refreshed13:29
asac(if i still use that mechanism at all=)13:29
directhexNCommander, i spy only headers and a pkg-config file. is arch:all wrong? there's nothing arch-speficic that i can see13:30
NCommanderdirecthex, if its a C based library, it should have a static library (.a), and a ****load of symlinks13:30
directhexhm, you're right, there's a missing symlink.13:32
asac-dev usually isnt all ;)13:32
NCommanderdirecthex, what are you packaging?13:32
directhexwait, my mistake. libmoon-dev: /usr/lib/libmoon.so13:33
directhexNCommander, moonlight.13:33
NCommanderasac, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=40684913:33
ubottuDebian bug 406849 in icedove "icedove: Copyright and license info for Debian packaging, including artwork," [Minor,Open]13:33
NCommanderdirecthex, oh, this is a mono based program?13:33
NCommanderThe normal rules *might* not apply13:33
directhexNCommander, at this moment in time, it's just c13:34
NCommanderdirecthex, it needs more than that, it needs the full set of soname symlinks13:34
directhexNCommander, i.e. "1.0 profile" isn't at all monoish, and "2.0 profile" requires mono 2.0 to compile13:34
asacfta: if you want something to sponsor for intrepid, look at Bug 28622513:35
ubottuLaunchpad bug 286225 in vimperator "[intrepid] iceweasel-vimperator: Depends: iceweasel (>= 3.0~) but it is not instalable" [Undecided,Confirmed] https://launchpad.net/bugs/28622513:35
directhexlibmoon0: /usr/lib/libmoon.so.0.0.013:35
directhexlibmoon0: /usr/lib/libmoon.so.013:35
asac:)13:35
NCommanderasac, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=393237 should we slaughter this while we're at it?13:35
ubottuDebian bug 393237 in icedove "icedove: upgrade fail to remove /etc/thunderbird" [Minor,Open]13:35
NCommander(I figure a rm -r in postinst)13:35
asacNCommander: yes. we should remove files that are in dpkg --status of thunderbird, mozilla-thunderbird and icedove and are obsolete13:35
NCommanderok, how do we do that ;-)13:36
asacNCommander: i am not 100% sure. but i think on every upgrade parsing dpkg --status PKGNAME for lines that have "obsolete" ... then checking the md5sum and if the md5sum is still equal rmove that file13:36
asacotherwise keep it as the user has modified it13:37
asacmost likely its too late to migrate them to /etc/icedove anyway13:37
NCommandereh13:37
* NCommander pushes his branch to Launchpad13:37
asacNCommander: have to do a few NM things ... let me know hwen everything is up. will look right next there then13:49
NCommanderasac, I'm still doing a few test builds, this can probably wait for post lenny to upload13:49
asacNCommander: well. lets get the current biuld in. and then get the new build up13:51
asacin == lenny13:51
NCommanderyup13:51
NCommanderIts still not frozen :-/13:51
directhexat this rate jaunty will miss mono 2.0, due to lenny freeze13:53
directhexand maybe killer kilwalla too13:53
james_wNCommander: http://wiki.debian.org/DpkgConffileHandling might be of interest14:21
asacyeah i remember that directly parsing stats will cause lintian to complain14:23
asacsaivann: any nack/ack for the ubufox translations?14:50
asac[reed]: what file .suffix does your cert file for EAP have?14:53
saivannasac : Give me a few minutes and zh-CN locales will be uploaded to my branch. Outside of this locale, all locales works well15:09
asacsaivann: great15:20
asacsaivann: let me know ;)15:20
saivannasac : Ok :)15:26
saivannasac : The're ready, rev 14615:30
asacsaivann: is that against upstream branch?15:39
asacyeah ... seems like (at least the mail suggests it is lp:ubufox)15:39
saivannasac : Yes, your own branch15:39
saivannasac : This one https://code.edge.launchpad.net/~asac/ubufox/main15:40
saivannasac : Let me know if you want me to re-upload the branch, I can do it fast15:40
asacsaivann: what did you do with your email :(15:41
asacsaivann: https://code.edge.launchpad.net/~saivann/ubufox/intrepid_translations15:41
asacsaivann: do you want to recommit that stuff with a good email?15:41
asac(or dont you care)15:41
asac?15:41
saivannasac : Oh.. oups... my ssh key caused this. Well if you merge the branch, will my email appears anywhere?15:42
asacsaivann: yes it would appear in the nested log15:43
asace.g. bzr log  will show the commits as you see them now15:43
asacalso i could just push your branch directly (as i didnt do any additional cahnge)15:43
saivannasac : Ok, I will recommit, thanks for finding this15:44
saivannasac : Sorry for the time it took, I had to re-test all locales. Everything works now : https://code.edge.launchpad.net/~saivann/ubufox/intrepid_translations16:23
asacsaivann: committed. can you test one more round from the lp:ubufox branch directly? (rev 142)16:58
asacso i can bake a package out of it and upload?16:58
saivannasac : Yes, I do it right now16:59
asacthx17:02
saivannasac : I just noticed a problem related to locales. If you start firefox with a locale that does not exist in the ubufox locales, you won't be able to use "alt" functionnalities. The rest of the functionnalities will simply fallback to english language17:08
asacsaivann: hmm. even if the -alt is in en-US?17:09
saivannasac : Yes, see it by yourself : firefox -UILocale gr17:09
saivannasac : it won't happen if you start firefox with a known locale17:10
saivannasac : I tried all locales in your branch, everything works correctly17:15
asacsaivann: have you also tested the "restart" notification?17:15
asacsaivann: shoudl appear when sudo touch /var/lib/update-notifier/user.d/firefox-3.0-restart-required17:16
saivannasac : Oh, let me try it17:16
saivannasac : works well so far, I will test all locales17:16
RainCTHi17:24
saivannHi :)17:25
RainCTIs anyone comming to Mozilla Camp Europe this weekend?17:25
saivannasac : I found a issue with it-IT locale, I'll update my branch in a few minutes17:25
asacsaivann: are you testing the "restart" thing too?17:28
asaccool17:28
asacRainCT: where is that? paris?17:28
RainCTasac: Barcelona17:28
asacoh dear. i should have planned that17:28
asacwhy i nobody telling me17:28
RainCT(https://wiki.mozilla.org/EU_MozCamp_2008)17:28
asacnow its too late unfortunately17:28
saivannasac : Yes, I'm currently testing the "restart" thing, for all locales and it is the only locale that has a problem so far, and I know where is the problem17:30
RainCTToo bad. So no Ubuntu guy is coming? :(17:32
asacRainCT: are you from barcelona?17:39
asacsaivann: cool17:39
asacsaivann: so which local has no -alt?17:39
asac(so i can test the issue you reported above)17:39
saivannasac : it-IT17:39
RainCTasac: Yep :)17:40
* asac installs language-pack-it17:40
saivannasac : Ah wait, do you mean no translations at all?17:40
asacsaivann: you said there are problems when there is no -alt translation with the alt thing17:40
saivannasac : Ok take grc locale (greek) as an example17:42
saivannasac : Currently, all locales that does not have alt translations simply have un-translated english files in my branch, so this bug can't happen with any locales that we install. However, if you remove the files from one of the translations, you'll be able to reproduce the problem17:43
asacok will try with that17:45
saivannasac : You should also be able to reproduce the problem with a locale that we didn't install (like hr, grc, etc)17:45
asacsaivann: well. testing grc now ;)17:45
saivannasac : great17:45
asacsaivann: grc doesnt have a translation at all :/17:46
asacsaivann: and hence alt dialog works for me17:47
saivannasac : Exactly, the normal behavior would be : ubufox fallback in english, but alt is still working, in english17:47
saivannasac : Really?17:47
asacsaivann: i start firefox like: LANG=grc firefox17:47
asacsaivann: do we have an incomplete translation?17:48
asacthat doesnt have alt, but the "normal" thing?17:48
saivannasac : Almost 50% of the translations are incomplete, these locales does have *alt* files un-translated, still in english17:48
asacseems like all we have are complete17:48
asacsaivann: really?17:49
asacsaivann: http://pastebin.com/f5e3fb45017:49
asacthats what i have17:49
saivannasac : Unfortunately, yes, we have a good amount of new locales, but a lot of already existing locales were not updated17:49
asacsaivann: why are those files there for all i have in lp:ubufox then?17:50
asacsaivann: the above are properites ... this is .dtd http://pastebin.com/f3463cae17:50
saivannasac : Because these files are still in english (a copy of the ones in en-US)17:50
asacsaivann: ok. that wouldnt have been needed17:52
asaci splitted the files in such a way that not having one shouldnt break17:52
asacanyway. i cannot reproduce the bug now ;)17:52
saivanncan you try to remove ubufox-restart.properties, ubufox-alt.properties and ubufox-alt.dtd for a locale in your ubufox jar file for a specific locale to see if you can reproduce the problem?17:53
asacsaivann: ah. so you say it was broken without those and now its not broken becaues you copied them?17:54
asacok that makes more sense ... let me check17:54
saivannasac : Yes, exactly17:54
saivannasac : I tried it with my current env. locale here (fr) and alt and restart functionnalies stop to work if translation files does not exist. The rest of ubufox is properly translated.17:58
saivannasac : Revision 143 of my branch https://code.launchpad.net/~saivann/ubufox/intrepid_translations fixes "restart" problem with "it" locale.17:59
asacsaivann: so which "first" tier locales have missing translations? fr? I can surely find someone for that. what other?18:04
saivannasac : No, fr is ok, give me 2 minutes18:06
saivannasac : cs-CZ el-GR lt-LT pl-PL pt-BR ru-RU sl-SL uk18:09
saivannasac : These locales have un-translated ubufox-alt.dtd ubufox-alt.properties and ubufox-restart.properties files18:10
RainCTasac: have you already got the Catalan translation?18:10
=== new3 is now known as wiki
asacRainCT: ask saivann ... he incorporated all the contributions18:35
saivannRainCT : About ubufox locales?18:35
saivannasac : Don't forget  to merge rev 143 of my branch https://code.launchpad.net/~saivann/ubufox/intrepid_translations to fix it locales18:36
asacsaivann: i think i already did that ... didnt i?18:36
saivannasac : you merged rev 142, before you show me how to verify "restart" thing18:37
asacyeah18:37
asacok cool18:37
saivannRainCT : ubufox does have a complete ca translation18:38
saivannasac : Thanks :)18:38
asacsaivann: done18:39
asacrev14418:39
asac;)18:39
saivannasac : rock :)18:39
RainCTsaivann: OK, great. I saw it on the l10n-ca ML but wasn't sure if it was alredy sent to you :)18:40
saivannasac : An entry will still be needed in debian/changelog about these changes, with a LP field to close bug 283517 in your branch, when it will be ready for repositories18:41
ubottuLaunchpad bug 283517 in ubufox "ubufox 0.6pre lacks translations for new strings" [High,In progress] https://launchpad.net/bugs/28351718:41
saivannRainCT : Thanks for asking :)18:41
ftahttp://ubuntuforums.org/showthread.php?t=95064019:09
ftacomment #4+19:09
nxvlasac: hi!19:26
nxvlasac: i just packaged firefox using the new branch, can you please take a look at it19:26
nxvlasac: it's on my ppa: https://edge.launchpad.net/~nvalcarcel/+archive19:26
armin76nxvl: it failed to build :P20:09
nxvlyeah i just saw that20:11
nxvli forgot to bzr add the .desktop20:11
RainCTRe 285321 and bug 286225, the debdiff adds a dependency on firefox. One for abrowser isn't needed, or? (as abrowser provides firefox, and further abrowser depends on firefox-3.0)21:41
ubottuLaunchpad bug 286225 in vimperator "[intrepid] iceweasel-vimperator: Depends: iceweasel (>= 3.0~) but it is not instalable" [Undecided,Confirmed] https://launchpad.net/bugs/28622521:41
NCommanderasac, E: icedove: menu-icon-too-big /usr/share/pixmaps/icedove.xpm: 48x48 > 32x3222:01
NCommanderWe can't win22:01
ftamost of /usr/share/pixmaps/ is far bigger22:03
ftathe whole gnome is 48x4822:04
ftathat what i hate about lintian, it forces you to do unnecessary changes22:04
ftawith no benefit22:04
sebnerfta: therefore lintian overrides exist ^^22:08
fta$ file /usr/share/pixmaps/* | grep -c '48 x 48'22:10
fta9022:10
ftahttp://paste.ubuntu.com/60258/ obviously, noone is respecting that 32x32 rule22:13
RainCTfta: XPM's are 32x3222:20
RainCTfta: PNG's 48x4822:20
RainCTang GNOME is weird and has extra sizes :P22:20
RainCTand it's not unnecesary at all as XPM's are supposed to be for Debian's menu :P22:21
ftayeah, but i still hate it ;)22:27
ftaasac, are all the stuff produced by MacSlow used in some way in ubuntu?22:28
ftahttp://www.techeblog.com/index.php/tech-gadget/iphone-robot22:42
asacfta: no idea ;)22:50
asacwhat does he do?22:50
ftaasac, http://macslow.thepimp.net/22:51
asacfta: not much content on there for me22:52
asac26th april  and then 20th october22:52
ftalook the 1st video22:52
ftawell, it used to be busier, but it stopped22:52
ftaasac, so you pushed another ff3... still no ff-2 dummy package ? and the amd64 branding ?22:54
ftahttp://ubuntuforums.org/showthread.php?t=95332022:58
asaci dont know anything about amd6422:59
asaci uploaded a single bug fix22:59
asacthe dummy package is a glitch23:00
asactoo bad23:00
ftathe abrowser branding used on amd64 ff-branding23:00
ftabug 27908323:00
ubottuLaunchpad bug 279083 in firefox-3.0 "firefox 3.0.3 on intrepid reports 3.0.1 as user agent on amd64" [Medium,Fix released] https://launchpad.net/bugs/27908323:00
ftahm, you said fixed23:01
asacfta: why would a branding issue be arch dependent?23:10
asacfta: well. in that bug report nobody reported it as being "abrowser"23:11
asaci actually assumed you had fixed that in 3.0.3 because i couldnt reproduce it with my firefox23:13
asacso its really just abrowser on amd64. i remember that you showed me that at some point now that you say that23:14
asacbut i still dont get whats the problem there23:14
asacfta: did you rename the firefox.png?23:15
asachmm23:15
asacor was that me23:15
asacthe applet starter was supposed to keep an icon23:15
asacfor the .desktop i mean23:16
asacok its the the rename from firefox.desktop to abrowser.desktop23:22
ftai compared the two remember ? http://paste.ubuntu.com/56044/23:22
asaci remember that. but that doesnt tell me what it is and why it is different23:22
ftai grepped everywhere for the green version and it was the abrowser version23:23
asacfta: http://paste.ubuntu.com/60282/23:35
asacfta: http://paste.ubuntu.com/60284/23:37
asachowever, note that there is still a difference from amd to x8623:37
asacbut the user agent appears to be shipped in a separate file now23:37
asacand that was you who fixed it ;)23:38
asace.g. adding ubuntu-abrowser.js.tmpl23:38
ftai'm i think i did that, right?23:38
fta-i'm23:38
asacyeah23:38
asacbut it fixed that bug23:38
asacyou really confused me now ;)23:38
asacstill a bit of a mystery that the files look so different23:39
ftabut the pref file is still different, while it shouldn't be23:39
asacbut i guess it has something to do with binary-indep and binary-arch23:39
asacmost likely we run someting in -indep which isnt run on amd6423:39
asacfta: right. but in this case it doesnt hurt at least ;=23:40
asacnot saying that it should be ignored23:40
ftathe start page has no locales23:40
asacbut i am quite sure that it has something to do with either a) indep/arch in rules or b) official/unofficial mozilla architecture23:40
ftaand the moz2 codes are used in search engines23:40
asacyeah. moz2 codes are overriden by us (i hope)23:40
asacso tnot that important23:41
asacand homepage is done in ubufox for default install .. of course its not right for amd64 and without ubufox23:41
asacbut most likely its what mozilla build system does on non-official release archs23:41
asacwe should look into that23:41
asacmost likely we have to force that its an OFFICIAL build (or force that its UNOFFICIAL if you want to appraoch this from other side)23:42
ftacomparing buidlogs didn't give me any clue23:42
[reed]why override the mozilla codes? :)23:43
ftanot my change23:43

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