=== asac_ is now known as asac
=== davida is now known as davidascher
[07:56] <micahg> _Tsk_: good morning, does the support crew ML discuss bugs affecting users A?TM
[07:56] <_Tsk_> no
[07:56] <_Tsk_> but we could
=== _Tsk_ is now known as Usul
=== Usul is now known as _Tsk_
[07:57] <micahg> Usul: k, then I guess it's not really for me then, I guess I technically don't "support" users, but rather insure as close to bug free as we can get
[07:58] <_Tsk_> yes for that we use mdat and tag the post with [tb-qa] in the subject
[07:58] <micahg> what is mdat, I keep seeing that
[07:59] <_Tsk_> mozilla.dev.apps.thunderbird - sorry I thougt you knew
[07:59] <micahg> oh, yes, I'm familiar with the gruop
[07:59] <micahg> just diidn't connect the two :-/
[07:59] <_Tsk_> I mean I thought youhad made the connection
[08:01]  * micahg needs to upload 3.1.7 build 3 to natty and go to sleep
=== yofel_ is now known as yofel
[15:10] <fta> cyphermox_, hi, rahhhh, n-m once again messed up with my sendmail servers (because of its /etc/hosts crazy changes)
[15:19] <cyphermox_> fta, that's what I'm working on today
[15:20] <fta> oh, great
[15:44] <jdstrand> chrisccoulson: hi! what is the status of tbird build3?
[15:44] <chrisccoulson> jdstrand, oh, i just need to upload it now
[15:44] <chrisccoulson> i'll do that now
[15:44] <jdstrand> thanks
[15:54] <chrisccoulson> jdstrand, ok, uploaded now
[15:54] <jdstrand> thanks
[16:48] <micahg> jdstrand: do you have a minute for a PM?
[17:08] <jdstrand> micahg: a PM?
[17:08]  * jdstrand racks brain for meaning of 'PM'
[17:09] <micahg> jdstrand: private messsage
[17:10] <jdstrand> micahg: go ahead
[18:36] <janimo> after unpacking the tb source package how do I add a new patch? quilt and edit-patch both complain. How do I unpack the tarball first?
[18:36] <janimo> I did not find it described in the wiki
[18:36] <janimo> micahg: ^
[18:39] <micahg> janimo: ./debian/rules patch
[19:33] <fta> chrisccoulson, do you still see the unexpected load on your desktop? if so, please add a comment in bug 683959, i feel alone there
[19:33] <ubot2> Launchpad bug 683959 in linux (Ubuntu) "unexpected load with 0% CPU and no activity (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/683959
[21:27] <BUGabundo> yo
[21:28] <janimo> micahg: any special QUILT_PATCHES settings in ~/.quiltrc ? Patches in debian/patches are relative to build-tree.
[21:28] <micahg> janimo: yes, there should be a note in the README.Source
[21:28] <micahg> :( no readme
[21:29] <micahg> janimo: http://paste.ubuntu.com/541162/
[21:29] <micahg> that's from firefox, but should basically be the same
[21:30] <janimo> ok, I saw that rune in patchsys-quilt.mk as well
[21:30] <janimo> but it seems strange as qult in cdbs runs with --quiltrx /dev/null
[21:32] <janimo> ok, quite a custom workflow - the flexibility of quilt
[21:32] <micahg> janimo: yes, we might be able to fix some of that when we switch to source format 3
[21:33] <janimo> micahg: great to hear that. Anything planned yet, or stoping the switch?
[21:34] <micahg> janimo: time, it's not critical for this cycle, but we'd like to do it, also the tools are currently set up to use CDBS, so those have to be ported as well
[21:34] <janimo> which tools do you mean?
[21:34] <janimo> somehitng outside the package?
[21:34] <micahg> janimo: mozilla-devscripts
[21:35] <janimo> is this in debian too or ubuntu specific?
[21:35] <micahg> janimo: it's shared between Ubuntu and Debian, but the packaging for Firefox and Thunderbird are Ubuntu specific
[21:46] <janimo> micahg: I still get .pc dirs is that expected?
[21:46] <micahg> janimo: where?  you should get the .pc dir in build-tree/mozilla
[21:46] <janimo> there indeed
[21:46] <micahg> yep
[21:47] <janimo> but is not the debian/patches the only patch dir?
[21:47] <janimo> why the QUILT_PATCHES settings then?
[21:47] <micahg> janimo: not for quilt, debian/patches is where the actual patch is, .pc is where stuff gets applied so it can be reverted
[21:47] <janimo> ok
[21:47] <micahg> .pc is like a working copy
[21:48]  * janimo tries again with renewed confidence
[22:27] <janimo> micahg: debian/rules pre-build + debian/rules apply-patches is the same as debian/rules patch ?
[22:27] <janimo> they seem to have the same result
[22:28] <janimo> I finally got a new patch under debian/patches
[22:28] <micahg> janimo: might be,  idk offhand
[22:28] <janimo> before debuild -S do I need to do anythng else?
[22:28] <janimo> the debdiff adds some files under debian/ as well like the manpage
[22:29] <janimo> do I need a quilt pop -a after refresh?
[22:30] <micahg> janimo: in theory, debuild -S should clean that
[22:35] <micahg> janimo: could you use this to refresh the patch? quilt refresh --diffstat -U8 --no-timestamps
[22:35] <janimo> ok
[22:36] <micahg> that'll just make it look like the other patches
[22:42] <janimo> micahg: so there are 4 generated files in debian/ that do not go away with debian/rules clean
[22:42] <janimo> and they show up in the debdiff
[22:43] <janimo> I can delete them by hand but there must be a better way
[22:43] <janimo> manpage, tbird.sh postinst and prerm
[22:43] <micahg> janimo: that would be a bug then :(
[22:44] <janimo> 3 of those are creatd from .in files
[22:45] <micahg> janimo: yep, please file a bug if you like, patches welcome, I can fix it later tonight otherwise
[22:46] <janimo> I'll first have to bult test my patch
[22:46] <janimo> before sending it
[22:46] <micahg> janimo: ok, don't worry about those files then
[22:47] <micahg> janimo: I actually need to gut the Thunderbird packaging to bring it in line with the changes chrisccoulson made to the Firefox packaging
[22:47] <janimo> it sure needs a shekup
[22:47] <janimo> shakeup
[22:47] <micahg> that'll probably wait until the end of the month when I have some time off
[22:48] <chrisccoulson> i was going to do that soon ;)
[22:49] <micahg> chrisccoulson: if you have time, otherwise, I was going to have it ready before the next 3.3 alpha which is scheduled for mid January
[22:50] <micahg> I don't think we need to do too much for the 3.1 branch, but for thunderbird-trunk I think it should be cleaned up
[22:59] <janimo> micahg: I fixed those uncleaned files, will add to the debdiff
[23:02] <micahg> janimo: great, thanks, i'll massage that into the bzr commits
[23:03] <janimo> so bzr merge request is preferred over debdiff?
[23:04] <micahg> janimo: up to you, it actually makes it easier to give attribution, but I'll take it in any form :)
[23:05] <micahg> janimo: if you do a bzr merge, please make it one commit per change
[23:09] <cyphermox> fta, good news, I just finished writing a patch to disable NM touching /etc/hosts. I'll do some more cleanup and testing tonight and should be ready to upload soonish
[23:10] <fta> cyphermox, excellent! thank you
[23:11] <janimo> I'll attach the debdiff for review for now, in the meantime I am testing the package  building on armel
[23:30] <asac> fta: there?
[23:30] <asac> fta: perl -ne 'if (/^Exec=(.+)$/) { $f = (split "/", $ARGV)[-1]; print "$f\t$1\n"; }' /usr/share/applications/*.desktop > $I.new || rm -f $I.new
[23:30] <asac> if i want to include StartupWMClass= from the .desktop file as well in a third column ;)
[23:31] <asac> what is the statement?
[23:31] <asac> damn shitty bamf.index shit
[23:33] <fta> asac, hi
[23:33] <asac> hey ;)
[23:33] <fta> asac, what are you trying to achieve?
[23:33]  * asac is trying to fix unity to allow keep firefox dailies in launcher ;)
[23:34] <asac> they dont use StartupWMClass but try to guess whether a window belongs to a .desktop file by matching WINDOW_CLASS to the .desktop file name
[23:34] <asac> and they create the bamf.index file so it doesnt need to parse all desktop files
[23:34] <asac> so what i want is to also include StartupWMClass=$VALUE
[23:34] <asac> in $I.new
[23:34] <asac> as a third column if that is available in .desktop ;)
[23:35] <asac> and i found this perl statement which is easy for this case, but i dont know how to make an optional multiline match to include StartupWMClass=(.+)
[23:35] <asac> if its .desktop ... if not just omit it
[23:35] <fta> oh, ok. let's see.
[23:35] <asac> and make print "$f\t$1\t$2\n"
[23:36] <asac> but $2 is optional and only if StartupWMClass is in .desktop ;)
[23:36] <asac> so atm:
[23:36] <asac> perl -ne 'if (/^Exec=(.+)$/) { $f = (split "/", $ARGV)[-1]; print "$f\t$1\n"; }' /usr/share/applications/*.desktop  | pastebinit
[23:36] <asac> http://pastebin.com/VDRLCjgx
[23:37] <asac> but for firefox-4.0.desktop i want another column that has "Minefield" in it basically
[23:37] <asac> strange that there is no StartupWMClass for openoffice btw in its .desktop
[23:37] <asac> guess it just takes too long to start anyway ;)
[23:39] <janimo> micahg: ok, arm build got past where it used to error so that is good
[23:41] <micahg> janimo: cool, do you have time to do a full build?
[23:41] <fta> asac, the way it's written, it couldn't be multiline, it nees to be rewritten
[23:41] <janimo> the build is progressing now
[23:41] <janimo> but takes a log of time on arm
[23:41] <fta> asac, let me try another way
[23:42] <micahg> janimo: should take around 5.5 hours
[23:42] <janimo> it's 2 AM here so I'll only know for sure tomorrow
[23:43] <micahg> janimo: can you add DEP-3 tags for the patch? http://dep.debian.net/deps/dep3/
[23:43] <janimo> but I am almost cretain it works since I have make -k before and this was the only issue
[23:43] <janimo> never used DEP 3 before
[23:43] <micahg> janimo: ok, well, do you want to ping me in the morning to let me know, I can at least commit to the dailies tonight, but I'd rather be sure it completes before uploading to teh archive
[23:44] <asac> fta: we can rewrite this ... its ugly anyway ;)
[23:45] <micahg> janimo: I don't want to scare you away ;)
[23:45] <asac> i was about to write a C file to do this ... but then i thought of mr. magic perl fta ;)
[23:45] <janimo> micahg: well, it is a bit tedious indeed
[23:45] <janimo> :)
[23:45] <micahg> janimo: we're just trying to make patches in teh archive have proper references
[23:45] <janimo> ah, ok
[23:45] <micahg> janimo: I can take care of that if it's too much
[23:47] <micahg> janimo: I appreciate the patch, my guess is it would've taken me at least half a day to figure it out
[23:47] <janimo> I added a reference to a patch
[23:47] <janimo> I figured it out then saw it was done for xulrunner in some other platform
[23:48] <janimo> I looked at other simliar fixes for thumb2 failures and a guideline in the wiki
[23:48] <janimo> about thumb2 porting
[23:48] <asac> bummer i also need to add a token because Exec can be multi token itself
[23:49] <asac> well ... maybe Exec=([^ ]+)
[23:49] <asac> to cut of that early in the process ;)
[23:49] <asac> but will tweak it
[23:50] <asac> or add a special token like WM_CLASS:$2
[23:58] <fta> asac, perl -ne '/^(.*?)=(.*)/; $$d{$ARGV}{$1} = $2; END { for $f (keys %$d) { printf "%s\t%s%s\n", $f =~ m{.*/([^/]+)$}, $$d{$f}{'Exec'}, $$d{$f}{'StartupWMClass'} ? "\t$$d{$f}{'StartupWMClass'}" : "" } }' /usr/share/applications/*.desktop  ?
[23:59] <fta> oh, you don't want the full Exec