/srv/irclogs.ubuntu.com/2009/12/02/#ubuntu-mozillateam.txt

ftadtchen, seems it was recommended by the pulse author00:04
=== asac_ is now known as asac
=== micahg1 is now known as micahg
=== ripps_ is now known as ripps
=== debfx_ is now known as debfx
asac[reed]: one issue i just noted looking at ffox 3.6 is this: http://people.canonical.com/~asac/tmp/Namoroka_Choose_User_Profile.png10:50
asacthat utf issue ... its the same for all "..." ... also in the menus10:50
asacis that known?10:50
asac[reed]: same for all ... in UI: http://people.canonical.com/~asac/tmp/Namoroka_HomeDots.png10:52
asacfta: still building: https://edge.launchpad.net/~ubuntu-mozilla-security/+archive/nss3.12.3/+packages ;)12:48
asac15h ++12:48
ftaasac, heheh13:22
asacfta: those 10 minutes for each test ... is 10 minutes really needed? how often is that run?13:27
asaclike if its run 10 times, we might be able to reduce build time everywhere considerably by moving it to 3 minutes ;)13:27
asachow did you guess that time?13:27
ftait's to avoid the FTBFS after 4h of inactivity when a given test is blocked on a network access.13:30
ftabut it's just a timeout, not a running time, like if the test completes in 3 sec, that's 3 sec, not 10 min13:32
ftamaybe it's not enough for arm13:32
ftaif you don't have a restricted env like in the builders, you could remove the timeout and see how long those tests really need13:33
ftaasac, ^^13:34
asachmm13:46
asacok13:46
asacfta: but how likely is it that it will take 10 minutes?13:46
ftasome take several minutes13:46
asack13:48
ftaasac, http://paste.ubuntu.com/333128/13:50
ftaasac, "Started 17 hours ago" ??? wtf, didn't it take 9h the 1st time?14:56
asacno14:57
asacit took 9h after fixing the first build failure14:57
asace.g incremental from there on14:57
ftaoh15:06
ftai see it failed on karmic in opts_check_SSE2.cpp15:06
ftaskia15:06
ftawhat did you do to make that work?15:07
ftaoh, armv7=115:07
ftaso why set that on lucid only??15:08
ftaasac, ^^15:08
ftaposted my evo patch15:14
ftauh? https://edge.launchpad.net/~fta/+oauth-tokens15:25
asacfta: because only on lucid we support armv7 and higher only16:07
asacon karmic we still support older chips16:07
asacso we cannot use armv7 there16:07
asacotherwise stuff would die when starting it on older target platforms16:08
ftahm, i have no solution to offer then, upstream clearly supports only v716:13
ftasomeone has to port it :P16:13
asacfta: if they would only support v7 ... then they should add that as a default ;)16:14
asacanyway16:14
asacyes.16:14
asaci think the problem you see above is what armin76 complained about16:14
asacalready16:14
ftaafaics, it's coming from android16:15
asaclucid is main target. just wanted to see if karmic works out of box16:15
asaconce i have hardware i will try to port it (or maybe cross compiler setup)16:15
asacandroid definitly targets older arm versions16:15
asacarmv7 is quite new and there is not much hardware out there yet16:15
asacmost stuff is armv5 afaik and v616:15
asaclike iphone etc.16:15
ftaompiling /build/buildd/chromium-browser-4.0.261.0~svn20091201r33442/build-tree/src/sconsbuild/Release/obj/skia/skia/__/third_party/skia/src/opts/SkBlitRow_opts_arm.o16:16
fta/tmp/ccRUIN1V.s: Assembler messages:16:16
fta/tmp/ccRUIN1V.s:131: Error: thumb conditional instruction should be in IT block -- `moveq ip,#8'16:16
ftascons: *** [/build/buildd/chromium-browser-4.0.261.0~svn20091201r33442/build-tree/src/sconsbuild/Release/obj/skia/skia/__/third_party/skia/src/opts/SkBlitRow_opts_arm.o] Error 116:16
ftait failed on lucid too16:16
asacok16:16
asacthats adding CFLAGS=...16:17
asacone second16:17
asacsupposed to be in toolchain by default16:17
asacfor now we have to add it16:17
asachttps://wiki.ubuntu.com/ARM/Thumb216:17
asacso adding -Wa,-mimplicit-it=thumb to CFLAGS and CXXFLAGS is needed16:17
asacjust for lucid16:17
asacor someone should port this properly to arm ;)16:18
asacthumb216:18
asacbut implicit-it is fine for now16:18
ftawill also report that upstream..16:18
asacthx16:19
asacfta: the wiki page above has details ... you can hand them16:19
asachttps://wiki.ubuntu.com/ARM/Thumb216:19
asacthat one16:19
asacand maybe16:19
asachttps://wiki.ubuntu.com/Mobile/ARMv7AndThumb16:20
asacwhich has a few typed slides with more details16:20
asacthx16:22
asacfta: will you add the implicit-it=thumb  to CFLAGS/CXXFLAGS for lucid+arm or should i commit it?16:23
ftajust let me get the 1st answer from upstream (it's earlier for them)16:23
asacok. as long as its in tomorrows daily i am fine16:24
asacbut implicit-it is almost certainly the only thing we can do for now ;)16:24
asacwill be changed in toolchain in a week or two so this will just go away16:24
ftayep, it's too late for today, it's past 2516:24
asacits 24 ;)16:24
asachehe16:24
fta24:5116:24
asacok ping me if you want me to add those flags ;) (though i am not sure how to best inject that for scons)16:25
ftawill do, don't worry16:26
ftahm, metacity crashed twice today because of chromium16:30
=== mac_v_ is now known as mac_v
ccheneyasac: do we want binary package names or the source for the runtime rdepends?16:36
ccheneyeg something like this: zcat Packages.gz | grep-dctrl -n -F Depends xulrunner-1.9 -s Source | sort -u16:38
asacccheney: i think sources is what we need to know16:38
asacbut if we have binaries first we can later compress them16:38
ccheneyok, yea its easy to find out either of them16:39
ccheneyjust change -s Source or -s Package16:39
asacright. i think having list of binaries is good still16:40
asacso we know if its a plugin etc. without looking up16:40
ccheneyok16:40
asacso both ;)16:41
ccheneyok16:41
ftahu, grep-dctrl, nice :)16:41
* ccheney running mirror dists now :)16:42
asacinnovative commands can ease the task ;)16:42
ccheneyyea16:43
ftaasac, apparently, your theme work-around for evo is not enough.. or has been reverted (karmic), i jsut double-clicked on an att, and evo looped, 100% cpu, 100% mem :(16:43
asacfta: backtrace :/16:43
asacwe can then see if it was reverted or never committed to trunk or whatever16:43
asacor maybe a new engine that was forked _before_ applying it16:44
asacor maybe something unrelated :-P16:44
ftalol, it was completely frozing the box, i had to go to a console and kill it16:44
asacfta: lucid?16:44
ftano, karmic16:44
asachmm16:44
asacif its a regression, check what packages came recently16:44
asacwe should nail that down (if it is)16:44
ftastrangely, i no longer have anything in /var/crash16:44
asacfta: we disable apport by default for release16:45
asacotherwise we would get covered in bugs ;)16:45
ftaoh, how can i re-enable that?16:46
fta[1473971.334865] totem-plugin-vi[11482]: segfault at 6863732f ip 007ce645 sp 07398ec0 error 4 in libxcb.so.1.1.0[7c6000+1c000]16:46
fta[2163811.388737] totem-plugin-vi[15733]: segfault at 0 ip 009bf645 sp 046edec0 error 4 in libxcb.so.1.1.0[9b7000+1c000]16:46
fta[2173254.714962] totem-plugin-vi[14326]: segfault at 21 ip 03828d36 sp 030d3ec0 error 4 in libX11.so.6.2.0[37e8000+12a000]16:46
ftahm, no evo, no metacity, no ff16:46
ftastrange16:46
asacthere is a gconf setting afaik16:46
asaclet me check16:46
asachmm16:47
asacno manpage either ;)16:47
asacccheney: remember how to enable/disable apport?16:48
fta /etc/default/apport16:48
asacoh ;)16:48
asacright. enabled= .-P16:48
=== yoasif__ is now known as yoasif_
ftaasac, http://code.google.com/p/chromium/wiki/LinuxChromiumArm16:57
ccheneyasac: we need to track for updates/backports/etc also right?17:01
asacfta: were most of the dev stuff in the ia32-libs package for chromium?17:03
asacccheney: to be safe we need to check updates and security ... backports just a brief check i guess17:04
ccheneyasac: ok17:04
ftaasac, eh?17:04
ftaia32-libs is dead17:04
ftai mean, no longer needed17:05
=== yoasif_ is now known as yoasif
asacfta: right. just wondered how many extra packages you needed17:18
asacin your chromium ia32 package17:18
asacfolks complained that they cannot build mozilla stuff that way17:18
asacand hoped i could say: check what fta did ... maybe extend it a bit and there you go ...17:18
ftaasac, what do they want to build? my chromium ia32 package was just a kind of addon to ia32-libs, to add the missing bits for each distro17:21
ftabut it was empty for karmic17:21
ftaasac, https://bugs.launchpad.net/ubuntu/+source/nspluginwrapper/+bug/48714117:31
ubottuUbuntu bug 487141 in nspluginwrapper "[Patch included] NPN_GetStringIdentifiers implementation is broken" [Undecided,New]17:31
ccheneyasac: should it only be checking for xulrunner-1.9* or any xulrunner?17:38
ccheneyasac: for eg dapper17:38
* ccheney got a hit on sun-java5 but needs to look at the actual record still17:40
ftahttp://paste.ubuntu.com/333333/ :)19:25
ccheneyasac: i got the list together for binary packages, should i stick in the ubuntu wiki, or somewhere else?19:30
* ccheney put it in https://wiki.ubuntu.com/xulrunner-list19:31
bdrungasac: were there a reason why xpi.mk is maintained in mozilla-devscripts and not in a separate package? the main purpose of mozilla-devscripts is for packaging mozilla apps and not mozilla extensions. why not splitting them up?22:02
ftabdrung, just because it was easier that way, m-d was already there, and xpi was just one file22:55
bdrungfta: now we have more files ;) src/xpi*22:56
bdrungfta: would it be useful to split them?22:56
ftai don't really think it's necessary, and it would mean re-applying for the new package to be accepted, and changing all the build-deps22:58
ftai don't see any benefit22:59
bdrungfta: linux philosophy: one tool (here: package) for one purpose22:59
ftawould be like splitting devscripts, why? it's a collection of scripts23:00
ftaall are mozilla related, so it makes sense to keep them grouped23:00
bdrungxpi* is extension oriented. so we have the mozilla app group and the xul extension group23:01
ftait's small, you want to make it even smaller23:03
ftaasac, i've committed the thumb2 arm cflag, but i didn't test it, the syntax may be a problem GYP_DEFINES="foo bar release_extra_cflags=-Wa,-mimplicit-it=thumb" .. i'm not sure gyp will parse the a=b=c correctly23:05
ftawith a comma in between23:05
bdrungfta: k, you convinced me23:10
ftagreat :) imho, it's less work to keep it like it is23:11
asacfta: hmm. maybe "foo bar release..:='....'"23:30
asacbdrung: its went to be a collection of devscripts23:30
asacno reason to split it up23:30
ftawe'll see, that would be the next try23:30
asacmozilla-devscripts basically should have all stuff from xulapp.mk, over xpi.mk to package-up.mk and so on ...23:31
ftaasac, i'm respinning ucd right now (today's failed) so feel free to grab it in a few minutes so it builds during the night23:32
asacwith '' ?23:32
asacthx23:32
bdrungasac: can you have a look at my changes to m-d?23:33
asacfor me more compressed has always to be balanced with readability ;)23:34
ftaasac, nope, if it works like i think it works, '' would be passed as they are and wrong, but we'll see23:34
asaclike unifying xpath expressions to one23:35
asacmight be a good excersize, but doesnt gain anything23:35
bdrungasac: speed!23:37
asacneglectable for this i would think ;)23:37
bdrungasac: it's faster. we use lazy evaluation for all those install.rdf magic is evaluated multiple times.23:38
bdrungit sums up to some seconds.23:38
asacpeople feel more important if their build takes a few minutes than just a few second ;)23:39
bdrungasac: maybe it would be useful to put all the xpi-recommends stuff into an separate script (maybe dh_xul-ext?)23:39
bdrungasac: i have a testsuite here with many extensions and i am impatient :P23:40
asacte permission fixup looks ok.23:40
asaca bit too much explicitly imo. but i think thats ok unless i get a great other idea ;)23:40
bdrungasac: the maintainer can disable it, if it is too strict.23:41
asaci know ;)23:42
bdrungbut the normal ext contains no file, which needs the exec bit.23:42
asacright. thats why i even wonder why we would to look for !#23:43
asacother approach woudl be to make all stuff not executable ...23:44
bdrungbecause i found one example for it23:44
asacyes. but thats an exception23:44
asacwhich is fine and for which we have those vars23:44
bdrungasac: that's not enough. there are files with 777 (stupid *** user)23:44
bdrung*** = otherOS23:45
asacbdrung: my suggestion is: make everything 644. period. and add explicit whitelist feature23:45
asacbut i guess you want a full on/off knob ;)23:45
bdrungyes. i want to have a working default for all packages, that i know. (and some knobs for unknown freeking packages)23:46
asacsure. i think its better to start with knobs for the cases we know and extend if needed. to some degree one has to be vain :) ... like: freeking unknown packages should adapt a bit or wait to be supported ... rather than having big "force/unforce everything flags"23:48
bdrungasac: do you have an idea where to put the config files (debian bug 558490)? my idea: symlink defaults/preferences/*.js to /etc/xul-ext/$package.js23:48
ubottuDebian bug 558490 in mozilla-devscripts "adblock-plus / xul-ext-adblock-plus: move configuration to /etc" [Normal,Open] http://bugs.debian.org/55849023:48
asacbdrung: at best we would ship exactly one .js file that is empty by default that the admin can modify23:49
asacand not link the whole preferences dir to /etc/.. etc.23:50
bdrunghow should this work?23:50
asacits plain wrong to allow users to change the "base" defaults23:50
asacso basically three layers is what is wanted for packages like firefox etc:23:50
asac 1. base defaults -> shipped in package, not modifiable23:50
asac 2. admin defaults -> shipped in /etc/ ... one config file is enough23:51
asac 3. user defaults -> in user profile23:51
asacso mozilla doesnt support those three steps yet23:51
bdrungso we are talking about 2.23:51
asacyes.23:51
asacat least not everywhere23:51
asacso xulrunner has a special directory called "syspref"23:51
asacthat is loaded after the defaults/preferences/ ...23:51
asacbut that doesnt exist for extensions23:52
asacso for extensions we can trick by choosing a filename that comes first in alpabetic order23:52
asaclike:23:52
asac/usr/share/ubufox/defaults/preferences/000system.js23:52
asacthat file can be modified by admins ... and they can go back to factory defaults still by simply cleaning the config file23:52
asacits linked to23:53
asac/etc/firefox-3.0/pref/ubufox.js23:53
asacls -l /usr/lib/firefox-3.6b5pre/defaults/syspref/23:54
asacthats the admin dir for firefox for instance23:54
asaccheckout the defaults/pref directory ... there is so many config23:54
bdrungwhy have we firefox in the path? what is with an package, that works in firefox and thunderbird?23:54
asacand if users do a single merge error on package update they might screw full firefox23:54
asacthats why it is bad to put all those configs up to editing23:54
asacbdrung: its probably a bug23:55
asacatm ubufox only supports firefox23:55
asacand i didnt want to add /etc/ubufox23:55
asacand we didnt have /etc/xul-ext ... yet23:55
asacanyway. the idea should be not to link all the .js, but carefully add one .js link to the file the admin can later on tweak23:56
asacnot sure how to best do it ;)23:56
asacmaybe ls the dir23:56
bdrungbut for the future? where should be the config file in /etc?23:56
asacbdrung: whatever the debian extension team is happy with23:57
asaccould be /etc/ubufox23:57
asacor /etc/xul-extensions/ubufox/...23:57
asacor /ext/xul-extensions/ubufox.js23:57
bdrungcan you write a mail to the mailing list?23:57
asac(if we have a strict one config-file policy)23:57
asacis there an thread where this should go?23:58
bdrungi have to go to sleep now.23:58
asacme too ;)23:58
asacso wont do it today anyway23:58
bdrungi checked some extensions. they had only one js file for configuration.23:58
asacyes. but that shouldnt be linked there23:58
asacthat one should always stay in the original place23:58
asacso the user can always easily go back to factory defaults23:59
asacby wiping his /etc/ file23:59
asacits really messy to fight with the config file system for users. and users find instructions to change them and then fail to merge them and then get huge bustage and then cause support headache ;)23:59

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