/srv/irclogs.ubuntu.com/2008/03/19/#ubuntu-mozillateam.txt

[reed]jemalloc00:00
[reed]jason evans00:00
[reed]:p00:00
asacfta: can you please check if epiphany crashes as well00:00
asacthats important to know. if it does there is no way to put this into libxul00:01
asac[reed]: commented (with some uncertainty) - well bugzilla is still processing my submit :)00:12
asacsomething going on?00:12
[reed]hmm, go back and submit again?00:14
asaci cannot even open https://bugzilla.mozilla.org/show_bug.cgi?id=42333400:14
asacanymore00:14
asac[reed]: ?00:14
[reed]oh, looks like server problems00:15
asacyeah :)00:15
asac[reed]: http://paste.ubuntu.com/5849/00:16
asacmy comment00:16
asaci go to bed in a minute or tw00:16
asaco00:16
asachave meeting in about 6h00:16
[reed]I'll post your comment if bugzilla doesn't come back in a few00:17
asacthanks00:17
ftawell, my build failed:00:28
ftahttp://paste.ubuntu.com/5850/00:28
ftait's not related00:28
[reed]cvs up00:29
[reed]I think that got fixed00:29
ftayes, saw a few bustages earlier00:29
ftabut it's 1:30am for me00:29
acesfull9does anyone know the proper way to create a icalendar feed so that it can be imported as a remote calendar in sunbird03:17
acesfull9I am using php to generate it, and I can download it and import it just fine, but I want to be able to subscribe to it remotely03:18
acesfull9I set content-type: text/calendar but got no luck, I get an error03:18
acesfull9the wierd thing is if I download it locally and import it, it works just fine03:19
asacfta: there?07:00
asacfta: so without the static patch things work well?07:00
asacfta: i have to know that, so i know if its ok for us to have jemalloc at all07:01
asac(e.g. is backing out the static patch good enough)07:01
asacplease test epiphany as well07:01
asacthanks a lot07:02
asacfta_: if it works, can you please try to rename the xulrunner directory (and fix the system.conf obviously) ... to see if firefox would still be able to load jemalloc library07:55
asacfta_: i doubt that works and thats why i would vote to not use jemalloc at all for the time being07:56
asacbut i need verification for that ... if you have a package with the backed out patch i can test that as well07:56
=== fta_ is now known as fta
ftahi09:03
ftaasac, as i said, trunk works when i revert that patch09:03
ftasee the 2 .head branches i've just pushed09:04
ftaepy works too09:04
asacfta: well i know that it works without that09:46
asacquestion is if it works as well if you rename the xul dir09:46
asac(without respinning)09:46
asacfta: so did you push latest trunk to PPA? or do i need to do a rebuild here?09:47
asacfta: mozclient is broken here ... get-orig-source DEBIAN_DATE=... doesn't work09:53
asacthe date is properly transformed for checkout of client.mk09:54
asacbut not for make -f client.mk checkout09:54
asacthere is a bug09:55
asack found it09:56
asacnow it fails to get the modules09:59
asachttp://paste.ubuntu.com/5861/10:00
asacyeah ... mozilla-devscripts is completely borked right now10:05
asaceven without date i get: http://paste.ubuntu.com/5862/10:08
asacthats with 0.0510:08
asac0.06 is worse ... latest branch is the same as 0.0510:08
asacstrange ... is moz cvs broken?10:10
asac[reed]: wake up ... big big issue with CVS :) ... http://paste.ubuntu.com/5863/10:11
=== Greenery_ is now known as Greenery
asac[reed]: ok. i think there are plenty of people on it for now10:12
asac:)10:12
asaci guess its time to get my cvs account10:13
* asac doing breakfast10:17
asaccvs is still broken12:42
asacwth12:42
asacmozilla bug 42383512:52
asacsigh12:52
asac*sigh*12:52
ubotuMozilla bug 423835 in Server Operations ""cannot find module" errors at checkout from cvs-mirror" [Critical,New] http://bugzilla.mozilla.org/show_bug.cgi?id=42383512:52
[reed]the sysadmins are trying to fix major problems from last night's switch failure13:45
asac[reed]: thx. switch failure?13:47
asacwhat does that mean?13:47
asacis it still a network/routing issue?13:47
jetsaredimasac: uploaded useragentswitcher extension last night13:48
asacjetsaredim: great. i think i will do the upload batch if you have finished webdeveloper :)13:48
asacor is that ready as well?13:49
jetsaredimcan't figure it out13:49
jetsaredimwas going to ask you to take a look at my branch13:49
jetsaredimfor some reason it's not calling the build command (ant) during debuild13:49
asacshow :)13:49
jetsaredimI'm sure I've done something wrong, but not sure what13:49
asacurl?13:49
jetsaredimhttp://bazaar.launchpad.net/~jetsaredim/firefox-extensions/firefox-webdeveloper.ubuntu/annotate/jgreenwa%40d620-20080319110706-akcupz1vlllaxpff?file_id=rules-20080318131529-7rzcyz4htw13mt6r-3213:50
jetsaredimpretty much the identical rules file to what is in useragentswitcher and that one works fine13:50
jetsaredimand if I just run "ant" from the top of that tree - it works like a champ13:53
asacdoes userswitcher use ant as well?13:55
jetsaredimyea13:55
jetsaredimhttp://bazaar.launchpad.net/~jetsaredim/firefox-extensions/useragentswitcher.ubtuntu/annotate/jgreenwa%40d620-20080319044826-00d0v66vnmtjaazq?file_id=rules-20080319034613-dcsqz3umdthkvygg-613:56
asacjetsaredim: ok i see13:56
asacthe EXTENSION_PKG must be the _binary_ package name that will ship the extension. not the source package13:57
asacthat should fix it13:57
asace.g. firefox-webdeveloper13:57
jetsaredimin control?13:57
asace.g. in xpi.mk its hooked in like:13:58
asacbuild/$(MOZ_EXTENSION_PKG)::13:58
asacifneq (,$(MOZ_XPI_BUILD_COMMAND)) $(MOZ_XPI_BUILD_COMMAND)13:58
asacendif13:58
asacbuild/webdeveloper ... will never be run by cdbs13:58
jetsaredimo in rules13:58
jetsaredimhmm13:58
asaconly build/firefox-webdeveloper13:58
asacyoull figure13:58
jetsaredimok13:58
jetsaredimi'll play around with it13:58
asacthere is no EXTENSION_PKG in control13:58
asaci will try to bail out in xpi.mk if a package not matching any binary package is used13:59
asacjetsaredim: in xpi.mk the documentation is accurate:14:00
asac#        MOZ_EXTENSION_PKG (MANDATORY):14:00
asac#               define the binary package name used to ship this xpi14:00
asacgood14:00
jetsaredimok - that seems better14:01
asac:)14:02
asacdoes it work now?14:02
asacjetsaredim: you imported the complete package into the .upstream branch14:03
asacthats wrong14:03
asacjust the orig.tar.gz14:03
jetsaredimdoes it really matter?14:03
asacyes14:03
jetsaredimsince I ditched it and replaced it with the new tree14:03
asacotherwise the upstream doesn't make sense14:03
asacit matters in any case14:03
asacthe current .upstream branch has commits for debian/14:03
asacimporting the current package helps you to learn the procedure14:04
asacthough not mandatory14:04
asacbut still the .upstream branch must not contain any debian/ directory :)14:04
asacjetsaredim: i don't see that you replaced the .upstream code with new files yet14:05
asacat least thats not in the log14:05
asachttps://code.edge.launchpad.net/~jetsaredim/firefox-extensions/firefox-webdeveloper.upstream14:05
asac1. By  Jared Greenwald <jgreenwa@d620>   on 2008-03-1814:05
asac* import of existing package14:05
jetsaredimi made a .ubuntu branch14:05
asacthats good14:05
asacyes the ubuntu branch should never receive new upstream files14:05
asaconly through merge from .upstream branch14:05
jetsaredimawesome14:05
asacjetsaredim: i don't see any new files in log of .ubuntu branch as well14:06
asacjetsaredim: i think you accidentially pushed the .ubuntu branch to .upstream14:06
asacyou can uncommit till you are at revision 1 again and push --overwrite to .upstream14:06
asacbut still i am not sure where you upgraded the upstream sources ... if you did that at all14:07
jetsaredimum14:07
asacso ... un .upstream branch you would have 2 commits14:07
asac1. import existing upstream (v. XXXX) <- please name the version here14:07
asac2. upgrade to new upstream (v. XXXX) <-14:07
asacthe ubuntu branch would get14:08
asacbzr branch -r 1 /path/to/upstream xxx.ubuntu14:08
asacrevision 2. import debian/ from packaging (version XXX)14:08
asaceither you fix build system then or do it after upstream merge14:08
asaclets assume you fix it now14:08
asac3. fix build system to make use of xpi.mk and add build-depends on mozilla-devscripts accordingly14:09
asac4. merge new upstream sources from .upstream branch (v. XXX)14:09
asac(revision 4 you are doing like:)14:09
asacbzr merge /path/to/upstream/branch14:09
asacwhile inside the .ubuntu branch14:09
asacand then just bzr commit14:09
jetsaredimwhy would a merge be necessary since I would have branched after importing the new sources14:10
asacjetsaredim: you branched after revision 1 (current package)14:10
asacso you need to merge revision 2 (new upstream)14:10
jetsaredimok - so when you say existing upstream - you mean the current package minus the debian dir?14:11
asacjetsaredim: the current orig.tar.gz14:11
asacnothing more14:11
jetsaredimexpanded?14:11
asacthere might be differences outside the debian/ dir in the diff.gz14:11
asacso minus debian/ dir is not accurate14:11
asacyes of course14:12
asacmakes sense?14:12
asac(i mean in general)14:12
jetsaredimin general14:12
jetsaredimi think14:12
jetsaredimi think I'll just blow away the bzr branches and start over14:13
asacas you wish ... save the current debian/ dir so you can use that once you have arrived at that stage :)14:13
jetsaredimright14:13
jetsaredimand I modified the build.xml file too14:13
asacyes. that only goes to .ubuntu branch14:14
jetsaredimright14:14
asacthats why importing debian package minus debian/ dir is not .upstream14:14
asaconly .orig.tar.gz should be upstream14:14
asaci don't want to be picky about the exact procedure. i only care the in the end .upstream has the pure upstream sources. and .ubuntu is based on that branch14:15
asaci just think that after doing this once you will be able to use that easily14:16
jetsaredimsure you do ;)14:16
asacjetsaredim: i see that you have a wierd dupe branch :) ... ~jetsaredim/firefox-extensions/firefox-webdeveloper.ubtuntu14:18
asacread "ubtuntu" :)14:18
jetsaredimheh14:19
asac~jetsaredim/firefox-extensions/firefox-webdeveloper.ubuntu exists as well ... so i think its a glitch :)14:19
jetsaredimyea14:19
jetsaredimi do that all the time when typing ubuntu14:19
asacjetsaredim: yeah. bzr will remember where you initially pushed to14:19
asacso you just need to do it once14:19
asacafter that just bzr push should be enough14:19
asacjetsaredim: you can see where it would pull/merge and push to when running bzr info14:20
jetsaredimcrap - i think i just nuked the useragentswitcher.ubuntu branch14:20
asacjetsaredim: yes ... if you still have it on your system you don't need to bother14:21
asacyou can just push it again14:21
jetsaredimthat would be convenient wouldn't ot14:22
jetsaredim*it14:22
jetsaredimwell - at least I built the new package and can recover it from there14:22
asacdo you still have it on your disc?14:22
asacyes ... you could just branch from .upstream and apply the .diff.gz14:23
jetsaredimyea14:23
jetsaredimasac: better...? https://code.edge.launchpad.net/~jetsaredim/firefox-extensions/firefox-webdeveloper.upstream14:37
asacthe comment looks good14:37
asachow did you do it? like i said: rm -rf * .. and then extract new upstream source and run bzr add .14:38
asac?14:38
jetsaredimyep14:38
asacyes great14:38
asacnow branch the initial revision to an .ubuntu branch14:38
asacapply the current .diff.gz14:38
asac(as current package)14:38
jetsaredimwait - what?14:38
jetsaredimwhat do you mean by "branch the initial version"?14:39
asacthe idea is to create a .ubuntu branch that has the initial packaging14:39
asacthat should be done on revision 1 of course14:39
jetsaredimok14:39
asacso bzr branch -r 1 /path/to/upstream ...ubuntu14:39
asacthen apply diff.gz14:39
asaccommit that as "* import packaging as of XXXX-0ubuntu1"!14:39
asacor something14:40
asaconce that is done you can merge the other revision over by just running bzr merge /path/to/upstream14:40
asacbump the changelog14:40
asacfix the packaging ... and so on14:40
asac:)14:40
jetsaredimok - so now I've got the upstream and ubuntu14:45
jetsaredimupstream is current - ubuntu is old but has packaging14:45
jetsaredimnow I merge?14:45
jetsaredimshoot14:46
asacyeah14:46
jetsaredimsomething got munged14:46
asacyou go to .ubuntu14:46
asacbzr merge /path/to/upstream14:46
asacwhat happened?14:47
asacyou can return to last committed state by running bzr revert14:47
jetsaredimwhen I applied the diff it just put the files that should go into the debian dir at the top of the tree14:47
asacyes14:47
asacrevision 2 most likely?14:47
jetsaredimi suppose I can just create the dir and move them in14:47
jetsaredimyea14:47
asache?14:47
asacjetsaredim: maybe retry to apply the diff14:48
jetsaredimi have things like control and rules at top of tree14:48
asacif you are in the branch you run14:48
jetsaredimpatch < patchfile14:48
asacgunzip -c /path/to/diff.gz  | patch -p114:48
jetsaredimah p114:48
asacyes14:48
jetsaredimthat's prolly the problem14:48
asacthen you most likely (if there are no conflicts) you run bzr add .14:48
jetsaredimyea14:49
asacand commit that saying "packaging for 1.0.5-0ubuntuX"14:49
asacor something14:49
jetsaredimi had done that, but like i said the thing got fuxed14:49
jetsaredimhow do I roolback a revision?14:49
jetsaredimactually - i can just branch r114:49
asacjetsaredim: what got committed?14:50
asacyou can bzr uncommit14:50
asacbzr revert14:50
asacthat should bring you to the revision below14:50
asacbzr ensure with bzr status that you don't have any unknown files14:50
asacotherwise you might add them accidentially when you commit next time14:51
asacjetsaredim: but remember that bzr uncommit is evil and should usually _never_ be done for branches already pushed to a remote place14:51
asacyou can use it for local reshuffeling of changes though14:51
jetsaredimright - i already pushed it14:51
asacjetsaredim: well ... in this case you can redo14:51
jetsaredimwas thinking that I could branch r1 re-apply the patch and then push14:52
asacits like an excersize14:52
asacand nobody is currently depending on it ... just "evil as a general rule"14:52
asacjetsaredim: well rebranching revision 1 is similar to uncommit from the evilness ... do what you like more14:52
asacit doesn't matter in this case14:52
jetsaredimis there something magic that happens during the merge?14:59
jetsaredimbecause the dev changed some of the directory structure and it seems completely different14:59
asache?14:59
asacwhats different?14:59
asacplease do a bzr diff15:00
asacand show me the output15:00
asacand bzr status15:00
asacjetsaredim: is the new directory structure wrong?15:01
jetsaredimno15:01
jetsaredimthe old dir structure is15:01
jetsaredimthey work for the package they are with15:01
jetsaredimthe old works with the old extension - new with the new extension15:02
asacso why do you see a problem?15:02
asacjetsaredim: bzr could track moves, but it doesn't know that things got moved15:03
asaci don't think that there should be a problem15:03
jetsaredimis the merge going to get rid of all of the old files?15:04
asacjetsaredim: yes. if they are removed from the upstream branch it will (unless you have modified them ... then you would get conflicts)15:04
asacjetsaredim: bzr status should yield something like this:15:04
asachttp://paste.ubuntu.com/5873/15:04
asacyou see which files where removed15:04
asacwhich added15:04
asacand which modified15:04
asac(you would also see meta info at the bottom about the merge)15:05
jetsaredimis there a way to resolve conflicts?15:08
asacwhat kind of conflicts do you get?15:09
jetsaredimnot sure15:09
asacthere shouldn't be any15:09
jetsaredimwhich files are which?15:09
asache?15:09
jetsaredimthere is BASE, OTHER and THIS?15:09
asacshow me bzr st15:09
asacwhere do you get the conflict?15:09
asacBASE is probably the current file on your branch ... OTHER the onmodified file from the other branch and THIS the result of the merge15:10
jetsaredimhttp://paste.ubuntu.com/5877/15:10
asacjetsaredim: ok. so the original debian package indeed hat the build.xml touched15:11
asacjetsaredim: you can open build.xml and search for the conflict markers15:11
asac"<<<<<<<<"15:11
asac======15:11
asac>>>>>>15:11
asacresolve them :)15:11
jetsaredimwhy not just bring in the OTHER?15:11
jetsaredimsince this is just a temporary branch15:12
asacbecause then you loose the changes in build.xml. its better to review manually15:12
asacfurther there might be other parts sucessfully merged in already15:12
asacin this case it might turn out that OTHER would have been right, but manually looking is required as a general rule to not drop things by mistake15:13
asacjetsaredim: if the changes conflicting tried to do something similar you have to do for the new packaging, you can also do it in this step15:13
jetsaredimlooks like he just re-wrote the build.xml file for the most part - there's like 85 changes15:20
jetsaredimhe did do similar stuff to the build file that I had to do15:21
jetsaredimfyi - i asked the bzr folks and they said i can just plaster whatever over and it won't screw anything up from vcs point of view15:23
jetsaredimso I'm going to plaster on my patched version of the new build.xml15:25
asacjetsaredim: yes15:36
asacthats ok15:36
asacjetsaredim: but if you do so please document that in commit log15:36
asacjetsaredim: and add an initial changelog bump in the same commit using15:36
asacdch -v NEWPACKAGE-0ubuntuversion -D UNRELEASED15:36
asacyou can do that in a second commit as well15:37
asacjetsaredim: all going well?15:42
asac:)15:42
jetsaredimI think so15:44
jetsaredimI'm just pushing a new version15:44
jetsaredimwith the new build.xml and new debian/* files15:45
asacgood15:47
asacdoes it build?15:47
jetsaredimok - so now I have a tree that should be the 1.1.5 source + my changes to the debian/* files and build.xml15:48
jetsaredimlemmie check15:48
* jetsaredim screams obscenities at the screen15:49
jetsaredimdh_install -pfirefox-webdeveloper temp-xpi-NQFL5237/chrome temp-xpi-NQFL5237/chrome.manifest temp-xpi-NQFL5237/install.js temp-xpi-NQFL5237/install.rdf temp-xpi-NQFL5237/license.txt /usr/share/firefox-webdeveloper15:49
jetsaredimcp: cannot stat `./chrome': No such file or directory15:49
jetsaredimsry for the paste15:49
asacjetsaredim: there is something wierd15:50
jetsaredimasac: you're telling me?15:50
jetsaredimthing is that I had it all working15:51
asacjetsaredim: no .. i mean .. what happened to the original debian/rules15:51
jetsaredimstill have the dir from that around - lemmie see what is different15:51
asacaeh sorry ... the changelog15:51
asaci look at the commit revision 415:51
asacthere is a complete new changelog in15:51
asacisn't that supposed to be in revision 2 already?15:51
jetsaredimuh - i suppose15:52
* jetsaredim goes back to square one...15:52
asacjetsaredim: ah ... you forgot to bzr add15:54
asacafter diff.gz patch :)15:54
jetsaredimi coulda swore I did15:55
asac;)15:55
jetsaredimthis is a pain15:58
asacwell :)15:59
asaclearning by doing15:59
asacone doesn't do such mistakes often15:59
asacthen things get efficient15:59
asacthen they suddenly appear to be done by instinct :-D16:00
jetsaredimyea16:00
jetsaredimtrue16:00
jetsaredimnot like i've never used vcs systems before16:00
jetsaredimmaking me feel like an amateur16:01
jetsaredim:)16:01
jetsaredimgotta run for lunch16:01
jetsaredimbbl16:01
asacwell, but you must admit that bzr is quite comprehensible :)16:02
asacits just the procedures for debian packaging that are different ;)16:02
fta2asac, i have rendering issues on http://en.wikipedia.org/wiki/Biang16:34
fta2the [edit] on the right below each pictures are misplaced16:34
fta2and if you zoom several times, the background becomes black16:35
* asac looking16:43
asacfta2: whats the problem? the edit links are all three next to each other slightly to the right16:48
asachow is it supposed to be (i can't remember)16:49
asaci don't see any edit for other pictures16:49
asacare those edit things related to the pictures at all?16:49
fta2asac, http://www.sofaraway.org/ubuntu/tmp/biang.png17:21
asacok i had to fix the ip for cvs-mirror to get a cvs checkout going again17:22
asacfta2: yes, but from which elements are they?17:22
asacthe one next to mnemonics looks ok17:22
asacok its most likely for the paragraphs17:23
asacabove17:23
asacsummary + about the noodle and phonetic sub.17:23
fta2maybe yes17:23
fta2but this looks weird anyway17:23
fta2at least the 3 on the same line overwriting the text17:24
fta2leaving.. it's raining here, lots of fun with my e-bike on perspective.17:27
asachow are those implemented in html? floats?17:28
fta2donno, select the text and use view selection source17:31
* fta2 gone17:31
asacjetsaredim: thats about right, yes.17:32
asacjetsaredim:  aber brnach you don't need to commit17:33
asacyou don't need to push until you are finished either17:33
asacbut thats all that you should fix besides the merge conflicts17:35
asacjetsaredim: ^^^17:35
* asac jetsaredim well, nobody is perfect. nothing to bother about17:44
asacjetsaredim: just go ahead17:45
* jetsaredim proceeds with caution17:45
jetsaredimI don't really need these firefox-webdeveloper.{dirs,links,install} do I?17:46
asacjetsaredim: no you don't need them17:49
* asac off traveling17:59
=== dholbert is now known as dholbert_lunch
* jetsaredim uses pidgin20:12
jetsaredimasac: ok - so something I'm doing is not right20:16
jetsaredimcause I keep getting a fatal error during dh_install on debuild -b20:17
asacwhats the problem?20:17
jetsaredimlemmie paste the output20:17
asacy20:17
jetsaredimhttp://paste.ubuntu.com/5900/20:17
jetsaredimthat's just the end of the debuild -b, but the relevant part20:17
asacjetsaredim you still have any debhelper file in debian/ ?20:18
asaclike *install ?20:18
jetsaredimo duh20:18
asacyou need to drop all of them20:18
asacits all automatically handled now20:18
asace.g. *install, *links *dirs20:18
asacbzr rm FILENAME :)20:19
jetsaredimahh better20:20
jetsaredimnow to update the changelog20:21
asacyes20:21
jetsaredimwhat was the example of the changelog you wanted me to follow again?20:22
asacjetsaredim: look firefox-3.0.head for instance20:23
asacor xulrunner-1.9.head20:23
asac(those are branch names)20:23
asacor look at the changelog of apt-get source xulrunner-1.920:23
asacbut looking at branch you can also see how we document during commit20:23
asacwhich is basically the same as in changelog20:24
ftadamn, i have a corrupted oo file :(20:31
jetsaredimthat sounds like a personal problem ;)20:32
ftait was fine up to yesterday20:33
jetsaredimasac: ok - this is looking much much better20:37
jetsaredimthough this file list is looking odd... http://paste.ubuntu.com/5902/20:38
asacjetsaredim: yes indeed20:38
asacdoes it work20:38
jetsaredimseems to20:38
jetsaredimI'm no webdeveloper expert user20:38
jetsaredimbut I suddenly have the webdeveloper toolbar20:39
jetsaredimso i'm guessing that's a good sign20:39
jetsaredimit could be part of the xpi20:40
jetsaredimcause useragentswitcher looks similar20:40
asacyeah ... thats good enough i guess20:42
jetsaredimgoing to push it and then upload to my ppa20:43
asacyep20:44
[reed]asac / fta: check #developers on moznet20:45
[reed]talking about the --with-libxul-sdk jemalloc crash bug20:45
fta[reed], what did i miss ?20:49
[reed]fta: they don't want to consider it a b5 blocker20:49
[reed]and I'm trying to change their mind20:50
ftaoh20:51
fta[reed], should I let you handle this or do you need me to say something ?20:53
[reed]fta: please speak up20:53
jetsaredimasac: to reconstitute that useragentswitcher tree I should be able to just use the tree from an apt-get source, right?21:05
asacjetsaredim: no idea what you are planning to do21:05
jetsaredimi accidentally nuked the useragentswitcher.ubuntu tree21:06
jetsaredimso I was trying to re-constitute it21:06
jetsaredimsince I already built the package and its in my ppa21:07
fta[reed], why a block on 418016 ? 418016 has been committed/fixed21:07
[reed]because that's how we mark regressions21:07
[reed]they block the bug that caused them21:07
ftaand btw, i don't see a discussion there. it's no, b5 is not concerned by -with-libxul-sdk against we need it21:08
[reed]er, what?21:08
[reed]asac / fta: can one of you post a mozconfig?21:12
[reed]for firefox21:12
[reed]that includes --with-libxul-sdk21:12
ftawe don't really use a mozconfig but i can post either our configure lines or about:buildconfig21:12
[reed]do you --enable-libxul ?21:14
jetsaredimasac: I'll have to figure this out later21:22
fta[reed], posted21:30
[reed]fta: where?21:30
ftabug21:30
[reed]er21:30
[reed]I meant let me see21:30
[reed]lo21:30
[reed]lol21:30
ftait's public anyway21:31
[reed]k21:31
ftaanyone could dl our source package or read our bzr branches21:31
[reed]patch in bug21:32
[reed]try it?21:32
[reed]you'll need to reapply the jemalloc in libxul patch ;)21:33
asachey hey ... calm down :)21:33
asaci am sure its not working with-libxul-sdk21:33
asacand bsdsmedberg things so too21:33
asacif it works for caillon then its his --enable-libxul flag (whcih conflicts with libxul-sdk)21:34
asac... but that means that he is not using system xul ... another option would be that he uses some wierd linker tweaks21:34
asaclike LD_PRELOAD ... and so on.21:34
asacbut all that is not an option for me :)21:34
asacok let me look in bug again21:35
asacno that doesn't make sense with-libxul-sdk21:35
asacoops21:35
asacbackscrolled answer ;)21:35
[reed]ugh, fta21:36
armin76asac: fta: you guys are still using myspell? :P21:36
[reed]stop overwriting bug options!21:36
ftaI did nothing at all except add a comment21:36
[reed]did you not get a "Mid-air" page ?21:36
ftagot caught in mid air, yes21:36
[reed]yes, you don't go through the mid-airs21:37
[reed]heh21:37
[reed]you killed the blocking flag21:37
[reed]and two other things21:37
armin76lolz21:37
ftasorry21:37
armin76thats what happens when you aren't used to bugzilla :P21:37
ftabad bugzilla, it should ask me if i want to do that21:38
asacwhy would it21:38
asacif you change the form fields it changes it21:38
ftai didn't21:38
armin76and asac fails as well :P21:38
asacyou did by accident21:38
fta?21:38
armin76he didn't21:38
armin76he just had an old version of the bug page21:39
ftayes21:39
asacok, whatever :)21:39
armin76fta: when you get a mid air collision, just go back with your browser, reload the page, and resubmit21:39
* fta got shot in mid-air21:39
asachehe21:40
asacso what was the bugid?21:41
ftahttps://bugzilla.mozilla.org/show_bug.cgi?id=42333421:42
ubotuMozilla bug 423334 in XPCOM "crash at startup in [@ NS_CompareVersions] when using --with-libxul-sdk" [Critical,New]21:42
asacthx21:42
asacfta: ok will you take the patch instead of the drop?21:43
asacthat looks good21:43
ftai can try it for sure21:45
asacyes, please do _with_ static jemalloc21:46
armin76what happened with mozilla bug 38690421:49
armin76its fixed or?21:49
ubotuMozilla bug 386904 in Build Config "DIST_FILES and DIST_CHROME_FILES not implemented for install:: target in config/rules.mk" [Normal,Resolved: invalid] http://bugzilla.mozilla.org/show_bug.cgi?id=38690421:49
ftapulseaudio crashed, again21:58
ftaasac, [reed], it crashes again22:51
ftasame place22:51
ftaasac, epiphany crashes too22:53
asacwhen does it crash?22:57
asacwhat did you do?22:57
ftaasac, http://paste.ubuntu.com/5904/22:57
asacwhat did you do?22:58
ftaas we just discussed, _with_ static jemalloc + the leak patch22:58
asaci don't know how the initial ephy crash looked like. get backtrace from firefox22:58
asacis it still in jemalloc22:59
ftahm hold on22:59
ftaff3, yes, same place23:00
ftahttp://paste.ubuntu.com/5905/23:00
ftacrappy trace23:01
asacbut wierd that it happens there at all23:09
ftanot really, if i understand what bs said, the malloc is from system (because libxul has not been loaded yet) while the free is from jemalloc23:11
asacfta: yes thats clear. wierd that it happens there ;)23:14
ftawhy ?23:14
asaclook at the function ... it allocates the bits it frees.23:15
asaclibxul is not loaded in between allocation and freeing23:15
asaconly thing that might be  is that the prototypes for version strings strduped are allocated differently. but can't see how that would break the free of the fresh strdups23:17
asacNS_CompareVersions(appData.minVersion, gToolkitVersion23:18
asacappData.minVersion is most likely libc malloced23:18
asacwhile gToolkitVersion might be jemalloc23:18
asachowver those are strduped23:18
asacso both should be allocated through the same mechanism thereafter23:19
ftamaybe, my brain is too tired to tell23:20
ftai don't know if it's a greader bug or a xul regression but reading articles with the space key is no longer possible23:20
ftait used to jump to the next one, now it does more than one so it often skips the next23:22
ftain both prism and firefox23:22
asaci would bet on xul as the source23:23
ftaI need a non gecko browser23:24
ftamaybe I could give webkit a try23:25
fta[reed], what is litmus ?23:25
[reed]fta: litmus.mozilla.org23:33
ftathx23:34

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