| === asac_ is now known as asac | ||
| === davida is now known as davidascher | ||
| 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 | 07:56 |
| === _Tsk_ is now known as Usul | ||
| === Usul is now known as _Tsk_ | ||
| 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:57 |
| _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:58 |
| _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 | 07:59 |
| * micahg needs to upload 3.1.7 build 3 to natty and go to sleep | 08:01 | |
| === yofel_ is now known as yofel | ||
| fta | cyphermox_, hi, rahhhh, n-m once again messed up with my sendmail servers (because of its /etc/hosts crazy changes) | 15:10 |
| cyphermox_ | fta, that's what I'm working on today | 15:19 |
| fta | oh, great | 15:20 |
| 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:44 |
| chrisccoulson | jdstrand, ok, uploaded now | 15:54 |
| jdstrand | thanks | 15:54 |
| micahg | jdstrand: do you have a minute for a PM? | 16:48 |
| jdstrand | micahg: a PM? | 17:08 |
| * jdstrand racks brain for meaning of 'PM' | 17:08 | |
| micahg | jdstrand: private messsage | 17:09 |
| jdstrand | micahg: go ahead | 17:10 |
| 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:36 |
| micahg | janimo: ./debian/rules patch | 18:39 |
| 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 | 19:33 |
| BUGabundo | yo | 21:27 |
| 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:28 |
| micahg | janimo: http://paste.ubuntu.com/541162/ | 21:29 |
| micahg | that's from firefox, but should basically be the same | 21:29 |
| 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:30 |
| 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:32 |
| janimo | micahg: great to hear that. Anything planned yet, or stoping the switch? | 21:33 |
| 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:34 |
| 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:35 |
| 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:46 |
| 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:47 |
| * janimo tries again with renewed confidence | 21:48 | |
| 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:27 |
| 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:28 |
| janimo | do I need a quilt pop -a after refresh? | 22:29 |
| micahg | janimo: in theory, debuild -S should clean that | 22:30 |
| micahg | janimo: could you use this to refresh the patch? quilt refresh --diffstat -U8 --no-timestamps | 22:35 |
| janimo | ok | 22:35 |
| micahg | that'll just make it look like the other patches | 22:36 |
| 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:42 |
| 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:43 |
| janimo | 3 of those are creatd from .in files | 22:44 |
| micahg | janimo: yep, please file a bug if you like, patches welcome, I can fix it later tonight otherwise | 22:45 |
| 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:46 |
| 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:47 |
| chrisccoulson | i was going to do that soon ;) | 22:48 |
| 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:49 |
| 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:50 |
| janimo | micahg: I fixed those uncleaned files, will add to the debdiff | 22:59 |
| micahg | janimo: great, thanks, i'll massage that into the bzr commits | 23:02 |
| janimo | so bzr merge request is preferred over debdiff? | 23:03 |
| micahg | janimo: up to you, it actually makes it easier to give attribution, but I'll take it in any form :) | 23:04 |
| micahg | janimo: if you do a bzr merge, please make it one commit per change | 23:05 |
| 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:09 |
| fta | cyphermox, excellent! thank you | 23:10 |
| janimo | I'll attach the debdiff for review for now, in the meantime I am testing the package building on armel | 23:11 |
| 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:30 |
| asac | what is the statement? | 23:31 |
| asac | damn shitty bamf.index shit | 23:31 |
| 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:33 | |
| 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:34 |
| 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:35 |
| 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:36 |
| 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:37 |
| janimo | micahg: ok, arm build got past where it used to error so that is good | 23:39 |
| 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:41 |
| 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:42 |
| 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:43 |
| asac | fta: we can rewrite this ... its ugly anyway ;) | 23:44 |
| 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:45 |
| 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:47 |
| 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:48 |
| asac | well ... maybe Exec=([^ ]+) | 23:49 |
| asac | to cut of that early in the process ;) | 23:49 |
| asac | but will tweak it | 23:49 |
| asac | or add a special token like WM_CLASS:$2 | 23:50 |
| 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:58 |
| fta | oh, you don't want the full Exec | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!