| ajmitch | directhex: lucky you, I had to compile a kernel only 2 days ago | 00:01 |
|---|---|---|
| ajmitch | though it was just grabbing the ubuntu kernel, modifying 1 file to add a USB device ID, and push it through pbuilder | 00:01 |
| freinhard | hi! | 00:19 |
| freinhard | is it possible to write a debian/watch file for google code? doesn't provide directory listing | 00:19 |
| jpds | freinhard: It is. | 00:19 |
| jpds | See: pidgin-facebookchat | 00:19 |
| freinhard | jpds: great, works! | 00:26 |
| freinhard | would be nice if someone could comment (and maybe advocate) http://revu.ubuntuwire.com/p/ctemplate | 00:31 |
| persia | freinhard: From a quick glance, you have some unanswered lintian reports (you don't need Section: in the source stanza) you're forcing debhelper 7, but using CDBS: what feature from debhelper 7 do you require? You don't need .dirs files for directories identified to dh_install, you haven't included a symbols file to check for ABI shifts. | 00:35 |
| persia | There's likely more, but that's a batch to dig at for now :) | 00:35 |
| freinhard | persia: just reuploaded witout the unneeded second section line | 00:36 |
| persia | OK. How about the rest? | 00:37 |
| freinhard | just removed debhelper and i'll check the .dir files | 00:37 |
| persia | removed debhelper? Check the debhelper changelog: you may want something newer than the 5.0.30 that cdbs pulls. Also, be sure to adjust debian/compat if you're supporting older debhelper. | 00:38 |
| persia | Also, there's a lintian bug that some people work around by adding an unversioned debhelper build-dependency. If you end up with the complaint that there's no build-depends on debhelper, and you do have build-depends on CDBS, you can ignore it because CDBS depends on debhelper. | 00:39 |
| freinhard | in fact i've no idea which debhelper and compat i need. build on karmic, that's all i know. upstream requires 4.0.0 but no idea if this is still valid | 00:40 |
| freinhard | s/build/builds/ | 00:40 |
| persia | upstream shouldn't require any version of debhelper: it's only used in packaging. | 00:40 |
| persia | Check the debhelper changelog (in reverse order). The first time you see a feature you think you're using, set that as a minimum version of debhelper. | 00:41 |
| === SuBmUnDo_ is now known as SUBMuNdO__ | ||
| persia | If you get back to the version that CDBS requires (5.0.30 for karmic), stop, and don't bother with a versioned depends. | 00:41 |
| freinhard | i know, they provided some two years old debian/ folder which, except for the controls and .install files, i dumped | 00:41 |
| persia | Similarly, it's likely worth checking the CDBS changelog to see if you need a specific version. | 00:42 |
| === SUBMuNdO__ is now known as SuBmUNDo`OuTzZ | ||
| === SuBmUNDo`OuTzZ is now known as SUBMuNdO__ | ||
| freinhard | the only thing i can do is to require karmics current cdbs version. i know that it builds on karmic and i can't test anything else. | 00:42 |
| persia | Well, don't bother with a versioned build-dep on cdbs then. | 00:43 |
| persia | If someone tries to backport it, they may file a bug. | 00:43 |
| freinhard | that's what i wanted to hear :D | 00:44 |
| freinhard | any package where i can have a look how the debian/symbols file works? | 00:45 |
| persia | https://wiki.ubuntu.com/stefanlsd/dpkg-gensymbols is an excellent write-up (which stefanlsd really ought push into the PackagingGuide at some point) | 00:47 |
| persia | Dunno about sample packages. | 00:47 |
| freinhard | if i don't depend on debhelper anymore, can i drop the debian/compat file? | 00:53 |
| persia | No. | 00:56 |
| persia | You need to tell debhelper which version of the files you're using. | 00:56 |
| freinhard | ok so i just ignore the lintian warning about debhelper and keep the compat file | 00:57 |
| persia | But if you're using CDBS, you can drop it to compatibility level 5, unless you're depending on some special feature of debhelper that requires a newer compatibility level (and you oughtn't be if you checked the changelog and aren't adding a versioned dependency) | 00:57 |
| persia | Yes, that's the lintian bug I mentioned earlier, that it fails to detect when debhelper will be installed by CDBS. | 00:57 |
| persia | As I said before, some people work around this with an unversioned debhelper build-dependency, but I usually just leave it there, because I know lintian is wrong in this case. | 00:58 |
| persia | Fixing it is tricky because the real fix would involve lintian building a recursive build-depends tree based on a current apt-cache for the target listed in debian/copyright and checking that, which would be slow (and quite possibly completely wrong in cases where special targets (e.g. UNRELEASED) are being used). | 00:59 |
| freinhard | so this will probably never happen. | 01:00 |
| persia | If you're super picky, you can set up a lintian override file that explains why lintian is brain-dead in this case and overrides the warning, but since it's not critical, it doesn't really require an explanantion in an override file. | 01:00 |
| freinhard | no,i'm not picky at all. if it works, i'm fine with it ;) | 01:00 |
| persia | Well, it's a tricky problem. Needs someone who cares enough about CDBS and lintian and hiding the warning to spend a lot of time on it. | 01:01 |
| persia | Most such people would rather fix other stuff in lintian or make CDBS more flexible instead. | 01:01 |
| freinhard | lintian complains about leading spaces in the description when i run it on one of the resulting .deb files | 01:02 |
| freinhard | is the description indented with one or two spaces? | 01:04 |
| freinhard | i guess one... | 01:05 |
| persia | http://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Description describes the syntax for Description fields. | 01:06 |
| persia | Sometimes you want one space, sometimes two. It depends on what you're doing. | 01:06 |
| === SUBMuNdO__ is now known as SuBmUNDo`OuTzZ | ||
| === ScottK changed the topic of #ubuntu-motu to: Ubuntu 9.10 released! | Lucid Alpha 2 Released! | Want to get involved with the MOTU? https://wiki.ubuntu.com/MOTU/Contributing | Sponsor queue: http://is.gd/2y76G | http://qa.ubuntuwire.com/ftbfs | http://people.canonical.com/~ubuntu-archive/NBS/ | http://qa.ubuntuwire.com/debcheck | latest rebuild failures: http://udd.debian.org/cgi-bin/ubuntu_ftbfs.cgi | Outstanding merges: http://people.ubuntuwire.com/~lucas/merges.html | ||
| SoftwareExplorer | I'm tying to make a debdiff for bug 461104. I get the messageNOTICE: 'ubiquity' packaging is maintained in the 'Bzr' version control system at:http://bazaar.launchpad.net/~ubuntu-installer/ubiquity/trunk Please use:bzr get http://bazaar.launchpad.net/~ubuntu-installer/ubiquity/trunk to retrieve the latest (possibly unreleased) updates to the package. What one should I use? | 01:31 |
| ubottu | Launchpad bug 461104 in ubiquity "Timezone combo doesn't always change (doesn't retranslate) if I change language" [Undecided,Confirmed] https://launchpad.net/bugs/461104 | 01:31 |
| ScottK | SoftwareExplorer: For that package you're better off to work from bzr and ask to have your branch reviewed. | 01:32 |
| persia | SoftwareExplorer: ubiquity development is a bit special, because the ubiquity branch doesn't quite match the ubiquity package (because the ubiquity package includes chunks of d-i). | 01:32 |
| persia | SoftwareExplorer: https://wiki.ubuntu.com/Ubiquity/Contributing might be a good place to start to untangle things | 01:33 |
| persia | SoftwareExplorer: https://wiki.ubuntu.com/Installer/Development has more pointers. | 01:34 |
| SoftwareExplorer | The main reason I was going to try to do this bug is that it seems to already have a patch for the python. Is this not a good bug to try to fix for my first time? | 01:35 |
| persia | Probably not. That's an odd source package and runs in a tricky-to-test environment. | 01:36 |
| persia | Speaking of my experience, I ran into lots of difficulties when I first tried to change ubiquity, even after several years of experience with packaging and bugfixing in Ubuntu. | 01:37 |
| SoftwareExplorer | Ok. Well, at least I was able to confirm the bug. | 01:37 |
| persia | Alternately, if you have a real interest in how the installer works, more help in that area would likely be appreciated :) | 01:37 |
| SoftwareExplorer | I'm just trying to start learning to package. I don't really have a specific interest yet, mostly I just want to help. | 01:38 |
| ScottK | SoftwareExplorer: Were glad to have you. | 01:40 |
| ScottK | kamalmostafa: If you're around, now's not bad. | 01:41 |
| persia | ScottK: By the way, did you ever get an answer to your question about plymouth and art? | 01:41 |
| persia | (or am I misremembering the channel in which you asked?) | 01:42 |
| ScottK | persia: I did not, but davmor2 said he'd file the bug. | 01:42 |
| ScottK | persia: I think it was -devel, but thanks for remembering. | 01:42 |
| persia | In another context, I was told that the filename was hardcoded in the ./configure call in the plymouth source package. | 01:42 |
| kamalmostafa | ScottK: just a moment | 01:42 |
| ScottK | OK. | 01:42 |
| ScottK | persia: That's "special" | 01:42 |
| persia | I wondered if it might make sense to have that be some well-known-location and use alternatives for the different flavours, but I'm not likely to dig into that. | 01:43 |
| persia | Indeed :) It broke several flavours. | 01:43 |
| ScottK | It also FTBFS on ports. Dies in configure | 01:43 |
| persia | no artwork? :) | 01:43 |
| ScottK | The error message isn't helpful | 01:43 |
| ScottK | I didn't get a chance to investigate | 01:44 |
| persia | It's the --with-logo=/lib/plymouth... that handles the artwork there. | 01:45 |
| persia | ScottK: Seems the issue is: configure:11953: error: Package requirements (libdrm libdrm_intel libdrm_radeon libdrm_nouveau) were not met: | 01:46 |
| persia | I'm guessing that string needs adjustment for any ports DRM that support it (but I'm not at all familiar with how DRM works for ports) | 01:46 |
| ScottK | Ah. No, they wouldn't be, would they. | 01:46 |
| * ScottK neither. | 01:47 | |
| ScottK | Maybe tseliot knows. | 01:47 |
| persia | Maybe we should have been having this discussion in -devel :) | 01:47 |
| persia | (but tseliot isn't there either) | 01:47 |
| ScottK | kamalmostafa: What have you got? | 01:48 |
| kamalmostafa | ScottK: ok, first off, I'm not at all sure that I actually did what you asked for, but here goes... | 01:50 |
| ScottK | kamalmostafa: That's fine. Learning is a big part of the point of this. | 01:50 |
| kamalmostafa | Lets start with libtifiles: https://code.launchpad.net/~kamalmostafa/ubuntu/lucid/libtifiles/libti-backmerge | 01:50 |
| ScottK | ok | 01:50 |
| kamalmostafa | - Merged libtifiles2 --> libtifiles | 01:51 |
| kamalmostafa | - I did not re-autoconf (ran into problems) so kept ubuntu versions' conf (autoconf-1.10.1) | 01:51 |
| kamalmostafa | - Significant changes to debian/copyright files. Is it okay? | 01:51 |
| kamalmostafa | - I may have missed some instances of "2-removal" (po stuff? doc stuff?) | 01:51 |
| kamalmostafa | First question: Is the debian/changelog version number correct? | 01:52 |
| kamalmostafa | I bumped up the "libtifiles2" version number by one "ubuntuN" unit to construct 1.1.2-0ubuntu2 but it looks weird as the next revision above libtifiles (1.1.1-1). .. | 01:53 |
| ScottK | Bumping the version number was right. | 01:56 |
| ScottK | As first glance it looks pretty good. | 01:56 |
| ScottK | I don't think there's a way to do the changelog that wouldn't end up looking a bit odd. | 01:57 |
| kamalmostafa | Yeah, I fought it out with the version numbering for quite awhile -- this was the only way to convince Lucid to use *this* libtifiles instead of the one that appears newer from libtifiles2. | 01:57 |
| ScottK | kamalmostafa: It does look like there are some things you can do to minimize the diff from debian. | 01:57 |
| ScottK | kamalmostafa: Exactly. | 01:58 |
| ScottK | As an example, you have ordered the build-depends differently. | 01:58 |
| ScottK | That's a pointless diff and you should change to follow Debian. | 01:58 |
| ScottK | OTOH, the more precise debhelper version requirement in the Ubuntu package is good to retain | 01:58 |
| ScottK | You should make standards version and home page the same as Debian | 01:59 |
| kamalmostafa | But here's the thing... All those settings are what I lifted from "libtifiles2". | 01:59 |
| kamalmostafa | I mean to say -- that's the debian/control from "libtifiles2", except modified for the new name "libtifiles". | 02:00 |
| ScottK | Yes. | 02:01 |
| ScottK | So I think there's a little more work to do to make our diff from Debian only the meaningful ones | 02:01 |
| ScottK | For example I'd also use the Debian descriptions unless you think the Ubuntu ones are enough better to be worth sending a patch to Debian. | 02:01 |
| ScottK | It looks like debian/rules could be better aligned similarly. | 02:02 |
| ScottK | kamalmostafa: This is a very good start. | 02:02 |
| kamalmostafa | Okay that seems like a reasonable endeavor for e.g. debian/control, but what about the generated files in the root directory -- those are sure going to differ greatly from Debian. Is that going to be a problem? | 02:02 |
| ScottK | I just looked in the debian dir so far. | 02:03 |
| ScottK | For that we'll keep one or the other, I don't know yet which. | 02:04 |
| kamalmostafa | :-) Oh, you don't want to look up at Makefile.in ;-) | 02:04 |
| ScottK | I'm banking on keeping "the one that works". | 02:04 |
| ScottK | Those should disappear on the next upstream release. | 02:05 |
| kamalmostafa | Well, as it stands, I think it actually "works" -- the two packages together do build in PPA and install on my Karmic machine. But yes, I'll make another pass over the debian/* files to reduce the diff. I'm for leaving the generated stuff alone though. | 02:05 |
| ScottK | OK. Let's see. You're off to a good start. | 02:06 |
| kamalmostafa | ScottK: you still looking at tifiles? or ready to move on to a ticalcs? | 02:07 |
| ScottK | kamalmostafa: I suspect if I did, I'd just tell you the same thing. Want to take another pass at them both first? | 02:11 |
| kamalmostafa | Sure that's just fine, one side note: What about libticables2? (libticalcs-dev depends on libticables2-dev). Is it going to need the same "2-removal" treatment? | 02:12 |
| ScottK | Yes. | 02:12 |
| ScottK | We'll want to switch to the Debian package names and if there are any reverse-build-depends they'll have to be changed | 02:13 |
| ScottK | Looks like tilp2 is affected as well. | 02:13 |
| === asac_ is now known as asac | ||
| kamalmostafa | What a mess. Well, since those ones aren't breaking (yet), how about we just deal with tifiles & ticalcs as "stage 1", and then deal with the rest as "stage 2". Will that work? | 02:14 |
| ScottK | Yes. | 02:18 |
| ScottK | libticables2-dev will be NBS (Not Built from Source) once we upload the new libticables, but won't actually be removed until it had no reverse-build-depends | 02:19 |
| ScottK | (or depends) | 02:19 |
| kamalmostafa | Okay, good plan. Thanks for the review Scott -- I'm off to dinner, then will make another pass at these two. Will let you know when ready for the next review. All good? | 02:19 |
| ScottK | Ye[ | 02:23 |
| ScottK | Yep | 02:23 |
| dhillon-v10 | hi all, I am working on this merge and bzr says that there are conflicts in control and changelog, I understand the changelog part but my question is shouldn't there be conflicts in more than that or is that okay | 03:20 |
| crimsun | dhillon-v10: generally that means the merge is clean | 03:21 |
| dhillon-v10 | crimsun, this is a merge from main so I was hoping something super complicated that I can't do but it seems plausible :) | 03:22 |
| RAOF2 | There will only be conflicts where the both Ubuntu and Debian have made changes in the same place; unless we're both fixing the same bug independently, that should be fairly rare. | 03:24 |
| dhillon-v10 | RAOF, crimsun thanks :) | 03:24 |
| dhillon-v10 | RAOF2, oh btw what does >> mean in version numbers, is it greater than | 03:25 |
| RAOF2 | Yup, greater than. | 03:27 |
| RAOF2 | Strictly greater than, in fact. | 03:27 |
| dhillon-v10 | RAOF2, thanks again | 03:28 |
| micahg | does debcommit automatically use --fixes lp:xxxxxx? | 04:26 |
| crimsun | yes | 04:28 |
| micahg | yay! | 04:28 |
| micahg | indeed | 04:29 |
| crimsun | example: http://pastebin.com/d66dc32c3 | 04:30 |
| crimsun | O:-) | 04:30 |
| dhillon-v10 | crimsun, I did the merge here: https://bugs.launchpad.net/ubuntu/+source/acpid/+bug/507778 does it look right | 04:35 |
| ubottu | Ubuntu bug 507778 in acpid "Please merge acpid (1:2.0.0-1) from Debian testing" [Undecided,New] | 04:35 |
| JontheEchidna | jdong: ping on an SRU | 04:40 |
| JontheEchidna | wondering if I can get approval for bug 350902 | 04:41 |
| ubottu | Launchpad bug 350902 in kdepimlibs "[ubuntu 8.10] kio_imap4 hangs" [Medium,In progress] https://launchpad.net/bugs/350902 | 04:41 |
| jdong | sure, gimme 5 minutes | 04:42 |
| JontheEchidna | Thanks | 04:42 |
| slytherin | Why do we have a 'unapproved' queue for -proposed uploads. We never upload packages to -proposed without a SRU approval? So what is point of another approval? | 04:47 |
| micahg | slytherin: according to the wiki process, upload is supposed to occur before approval | 04:47 |
| slytherin | micahg: AFAIK, approval form SRU team is required before upload. | 04:48 |
| jdong | well... | 04:49 |
| micahg | both https://wiki.ubuntu.com/StableReleaseUpdates#Procedure and https://wiki.ubuntu.com/ArchiveAdministration#Stable%20release%20updates seem to imply you don't | 04:50 |
| jdong | in the old days, ubuntu-sru tended to be a subset of ubuntu-archive | 04:50 |
| jdong | and archive admins found it much more natural to use queuediff to process SRU's as they're clearing the upload queues | 04:50 |
| jdong | hence upload and reference LP #, wait for approval | 04:50 |
| jdong | but MOTU-SRU (at first didn't have, later didn't know of) the tools, and preferred using LP team subscriptions | 04:50 |
| jdong | hence for MOTU, it tended to be ACK before upload | 04:51 |
| jdong | now, I guess upload-and-wait is the correct procedure? :) | 04:51 |
| micahg | well, if it's different the docs should be updated... | 04:51 |
| jdong | well I think for now, the right thing to do is to update the docs and slap the people who don't follow it | 04:52 |
| jdong | (innocently looks away) | 04:52 |
| micahg | jdong: approval first? | 04:52 |
| jdong | upload first | 04:52 |
| micahg | jdong: that's what it says :) | 04:53 |
| slytherin | jdong: And what happens when when Martin Pitt gives approval, package is uploaded and is waiting in queue? Whom do I need to bug about it. | 04:53 |
| jdong | slytherin: archive admins, in that case | 04:53 |
| ScottK | slytherin: What package? | 04:53 |
| jdong | wow, what a big debdiff :) | 04:54 |
| slytherin | ScottK: evolution-mapi. | 04:54 |
| ScottK | slytherin: What bug? | 04:54 |
| jdong | oh that's not a debdiff | 04:54 |
| * jdong obviously needs caffeine | 04:54 | |
| slytherin | ScottK: bug 472552 | 04:54 |
| JontheEchidna | jdong: would you prefer a debdiff? | 04:54 |
| ScottK | I can accept it if pitti approved it. | 04:54 |
| ubottu | Launchpad bug 472552 in evolution-mapi "Please upgrade evolution-mapi to 0.28.1" [Medium,In progress] https://launchpad.net/bugs/472552 | 04:54 |
| ScottK | Looking | 04:54 |
| jdong | JontheEchidna: haha no no, it's fine, I got it now :) | 04:54 |
| JontheEchidna | :) | 04:55 |
| jdong | silly midnight blind clicking | 04:55 |
| JontheEchidna | I was a bit unclear on the process too. Generally pitti just uploads and accepts at the same time :P | 04:55 |
| jdong | JontheEchidna: haha yeah, since the unification of the teams I guess there's been a bit of a shakeup :) | 04:56 |
| jdong | the dust will settle soon | 04:56 |
| jdong | JontheEchidna: ok you've got an ACK, carry on! | 04:56 |
| JontheEchidna | jdong: thanks | 04:57 |
| ScottK | slytherin: Done. | 04:57 |
| slytherin | ScottK: thanks. Do I need to add that response in bugs 'please test package from -proposed'? | 04:57 |
| ScottK | slytherin: No. Did that as part of the process. | 04:58 |
| slytherin | great | 04:58 |
| slytherin | ScottK: I am wondering if I should request for standing exception for evolution-mapi. | 04:59 |
| ScottK | No opinion. I'm not on the SRU team. | 04:59 |
| ScottK | You'd have to ask the tech board for the exception. | 05:00 |
| slytherin | I meant for micro releases actually | 05:00 |
| slytherin | Ok. I will mail them. | 05:00 |
| jdong | is there a standing-exception... yeah what ScottK said :) | 05:00 |
| === ripps is now known as ripps|sleep | ||
| slytherin | ScottK: You forgot to add comments on other bugs fixed in evolution-mapi upload. Should I do that? | 06:09 |
| ScottK | slytherin: Add a comment that test results should go in the bug I marked. | 06:09 |
| slytherin | Ok | 06:10 |
| slytherin | Or I could add comment and set the bugs is verification-needed. | 06:10 |
| slytherin | s/is/as/ | 06:10 |
| jariq | Could someone please review new package ipwatchd ? http://revu.ubuntuwire.com/p/ipwatchd | 07:01 |
| persia | jariq: Regarding point 4 of my previous feedback: why do changes in debian/rules need to wait for an upstream release? | 07:03 |
| jariq | persia: hm.. i thought you were talking about "rm -f" in Makefile in upstream package | 07:04 |
| persia | Well, it's a good idea to fix that too :), but your debian/rules is also a makefile. | 07:04 |
| persia | Mind you, without all the other stuff, it becomes minor enough not to warrant another upload, I just wondered. | 07:05 |
| persia | My personal preference is to have the logs show if anything unexpected happens, just so that when something does fail, I can figure it out. | 07:05 |
| persia | On the other hand, the way you're doing it might save kilobytes of storage space over the package lifetime. | 07:05 |
| persia | (and it's exceedingly unlikely that failure to remove some stamp files would interrupt a build) | 07:06 |
| persia | Bah, your package is in good enough shape that it needs a real review. | 07:06 |
| persia | I'll hit it if I get time while I'm doing reviews tonight, but there's nothing I can tell you in a couple minutes that would improve it. | 07:07 |
| jariq | persia: ok thanx a lot.. I'll take a look at that 'rm -f ' stuff meanwhile | 07:08 |
| persia | Really, it's so minor it's not worth updating the package to make the change. | 07:08 |
| persia | It's just a stylistic thing. | 07:08 |
| persia | Probably more important if the target was an interesting file that might not be there or might have been created at the wrong time. | 07:09 |
| persia | But for stamp files, it doesn't matter at all. | 07:09 |
| jariq | ok | 07:10 |
| fabrice_sp | anyone has an idea why I'm getting this error: Function `gtk_tree_view_column_get_cell_renderers' implicitly converted to pointer at ignoregui.c:192 | 07:11 |
| * persia idly wonders if a dh(1) package using rules.tiny could just ship debian/rules as a symlink to /usr/bin/dh | 07:11 | |
| persia | fabrice_sp: In what context? | 07:11 |
| fabrice_sp | line is GList *list = gtk_tree_view_column_get_cell_renderers (col); | 07:11 |
| fabrice_sp | and function is declared as GList * gtk_tree_view_column_get_cell_renderers (GtkTreeViewColumn *tree_column); | 07:12 |
| fabrice_sp | (my upload of xchat is FTBFS in amd64) | 07:12 |
| persia | OK. What's the type of the col variable at the time of the function call? | 07:12 |
| fabrice_sp | GtkTreeViewColumn *col; | 07:13 |
| * persia has no idea given that code | 07:13 | |
| fabrice_sp | the same for me :-/ | 07:13 |
| fabrice_sp | it's not a FTBFS as is, because the build is fine, but some sort of check is run on code after the building | 07:14 |
| * fabrice_sp never saw that before | 07:14 | |
| persia | So it's a make tests failure? | 07:14 |
| fabrice_sp | it seems to be an "automated build log filter" | 07:15 |
| fabrice_sp | http://launchpadlibrarian.net/37913661/buildlog_ubuntu-lucid-amd64.xchat_2.8.6-4ubuntu3_FAILEDTOBUILD.txt.gz | 07:15 |
| persia | Check the rules file. Some rules files will run a test suite. It may be that someone patched something and forgot to update the test suite. | 07:15 |
| fabrice_sp | hmm, actually, the message gives some links | 07:16 |
| * fabrice_sp should have looked at that before :-/ | 07:16 | |
| fabrice_sp | point to http://wiki.debian.org/ImplicitPointerConversions | 07:17 |
| persia | Also, look at line 1732 of the build log. | 07:17 |
| persia | The error is reported there, it's just not enough to kill the make. | 07:17 |
| fabrice_sp | ok | 07:18 |
| persia | It looks like some header is missing, because the compilation claims "ignoregui.c:192: warning: implicit declaration of function 'gtk_tree_view_column_get_cell_renderers'" | 07:18 |
| persia | So it may never be seeing " GList * gtk_tree_view_column_get_cell_renderers (GtkTreeViewColumn *tree_column);" for some reason. | 07:18 |
| persia | Maybe a scoping error also. | 07:19 |
| fabrice_sp | make sense. I'll have a look later. Thanks! | 07:19 |
| persia | Good luck. | 07:19 |
| fabrice_sp | thanks ;-) | 07:20 |
| persia | Note that this really only affects ia64, but it's worth checking for everything :) | 07:20 |
| persia | (because amd64 will *try* to use 32-bit-safe pointers, if it can) | 07:20 |
| fabrice_sp | I know: the problem is that it really blocks amd64 also :-/ | 07:20 |
| fabrice_sp | FTBFS | 07:21 |
| persia | Doesn't it block all architectures on the buildds? | 07:21 |
| fabrice_sp | no: only ia64 and amd64 | 07:21 |
| persia | If not, it's worth asking a buildd admin why not | 07:21 |
| persia | That seems silly to me, but OK. | 07:21 |
| fabrice_sp | have to go. Bye ;-) | 07:21 |
| persia | Bye :) | 07:21 |
| dholbach | good morning | 07:30 |
| cemc | morning | 07:36 |
| === SuBmUNDo`OuTzZ is now known as GotchA | ||
| === slytherin1 is now known as slytherin | ||
| hakaishi | Hi! Could someone please review qt-shutdown-p (I'm waiting since days)? http://revu.ubuntuwire.com/p/qt-shutdown-p | 10:10 |
| bdrung_ | dholbach: please unsubscribe u-u-s (and subscribe yourself) when sponsoring a merge | 10:14 |
| dholbach | bdrung_: why? | 10:14 |
| dholbach | bdrung_: once the bug is closed it shouldn't matter any more who's subscribed, or does it? | 10:15 |
| bdrung_ | dholbach: when searching lp directly or when someone writes a comment | 10:17 |
| dholbach | hum, generally the bug (that's been closed after upload) shouldn't turn up in bug lists any more... only if you search for all bugs ever | 10:19 |
| bdrung_ | dholbach: yes, but sometimes i search for all | 10:20 |
| dholbach | what is your use-case? | 10:21 |
| dholbach | I never bothered to do it and am not sure who else does :) | 10:21 |
| dholbach | but yeah I could do it if it's really necessary | 10:22 |
| dholbach | anybody else who has an opinion? | 10:22 |
| persia | dholbach: The sponsoring procedure documents unsubscribing, and lots of people do it. | 10:30 |
| dholbach | I see | 10:30 |
| dholbach | maybe I'm the only lazy guy then ;-) | 10:30 |
| persia | I wrote that bit, to cover two cases 1) often bugs will have other tasks not currently needing sponsorship, which clogs the LP view | 10:30 |
| persia | and 2) sometimes bugs need later attention because something went wrong, so it's good for the original sponsor to be paying attention, but nobody else needs to care (unless there's an issue, in which case the submitter might again subscribe the queue) | 10:31 |
| persia | dholbach: You're not the only lazy guy, just one of them :) | 10:32 |
| hakaishi | Could someone please review qt-shutdown-p? ';.;' *pleeeease* http://revu.ubuntuwire.com/p/qt-shutdown-p | 10:33 |
| persia | hakaishi: Some reviewers have an informal policy that any package that gets reviews requested with less than 24 hours passing between requests get ignored. | 10:34 |
| persia | Not that your package doesn't deserve review, but just a warning that asking too much may have the opposite effect. | 10:34 |
| hakaishi | hmm... the last view days I asked just two times a day... I think this isn't too much to bear | 10:35 |
| hakaishi | but, thanks | 10:36 |
| persia | You may be right. Depends on the reviewer. I don't believe we have an active REVU coordinator this cycle, so I'm not sure there's any firm policy. | 10:36 |
| hakaishi | okay, then I'll just try to ask every 24 hours and 5 minutes ^^' | 10:38 |
| persia | That's what I used to advise people when I was REVU Coordinator. | 10:38 |
| persia | Well, actually, I used to advise about every 30 hours, to have the best chance of catching different people at different times of day. | 10:39 |
| hakaishi | okay, thank you! See you then ^^' (you seem to be online almost all the time) | 10:40 |
| persia | I'm almost always online, but sometimes I have significant lag (rarely more than 100Ksec) | 10:41 |
| === shriekout is now known as rororororo | ||
| === rororororo is now known as shriekout | ||
| freinhard | 0hi! | 11:42 |
| freinhard | fixed all lintian warnings on http://revu.ubuntuwire.com/p/ctemplate and the last thing remaining is, what is that .symbols file for and where do i get usefull documentation or a package using it? | 11:44 |
| slytherin | freinhard: http://wiki.debian.org/UsingSymbolsFiles | 11:54 |
| freinhard | slytherin: no example there | 11:58 |
| slytherin | freinhard: for example you can check gtk+2.0 package | 11:59 |
| freinhard | slytherin: ok, undestood what that file does and what it is for. | 12:03 |
| freinhard | slytherin: these files aren't manually created, are they? | 12:03 |
| slytherin | freinhard: I haven't used it so don't know the details. | 12:03 |
| persia | freinhard: https://wiki.ubuntu.com/stefanlsd/dpkg-gensymbols is fairly good documentation. | 12:04 |
| persia | I tend to create the file at packaging time (using tools), so that the package FTBFS if the symbols don't match. | 12:05 |
| freinhard | persia: not the shortest docs for a single file i've ever seen ;) | 12:05 |
| persia | Well, no, but that's not a bad thing. | 12:05 |
| persia | Consider all the documentation that exists on how to deal with debian/rules, when one can just use /usr/share/doc/examples/rules.tiny | 12:05 |
| slytherin | persia: is that path correct? | 12:10 |
| freinhard | slytherin: no it isn't, there's a debhelper missing | 12:11 |
| slytherin | :-) | 12:11 |
| persia | Good catch, both of you :) | 12:11 |
| freinhard | ok, from the docs i see that exporting a single environment variable should trigger dpkg-gensymbols and i can gather the output for the symbols file after building the dsc with pbuilder? | 12:12 |
| persia | freinhard: I've added some comments to ctemplate | 12:18 |
| persia | freinhard: Right. Once done, copy that file back into your source package, so it will compare the generated results against the defined results at build-time. | 12:19 |
| persia | When upstream introduces new symbols without breaking ABI, add the right version hints. | 12:19 |
| persia | When upstream breaks the ABI, create a new symbols file. | 12:19 |
| freinhard | hmm didn't work with cdbs, looks like this gets called for debug packages only | 12:23 |
| freinhard | persia: if the environment variable is set, dh_makeshlibs should have called dpkg-gensymbols? | 12:40 |
| persia | freinhard: I don't remember having set an environment variable, but perhaps. You might also try `touch debian/libctemplates.symbols` to generate a broken one and get the conflict output. | 12:44 |
| freinhard | just tried that, still running | 12:45 |
| === Pici` is now known as Pici | ||
| freinhard | persia: where did you find a .la file? | 13:01 |
| freinhard | persia: just checked the .deb file with gdebi and couldn't see one. | 13:02 |
| persia | freinhard: I saw it listed in your .install file. | 13:03 |
| persia | http://revu.ubuntuwire.com/revu1-incoming/ctemplate-1001150403/ctemplate-0.96/debian/libctemplate-dev.install | 13:03 |
| persia | So if you're not seeing it in the package, then something else is wrong. If you are seeing it in the package, you should change it so you don't (referenced to the squeeze release goal) | 13:04 |
| === jtechidna is now known as JontheEchidna | ||
| === azeem_ is now known as azeem | ||
| bddebian | Heya gang | 14:29 |
| persia | bddebian: !!! | 14:30 |
| slytherin | Need help with oython. I am trying to install reviewboard which defines dependency on paramiko. I have already installed package form repository but reviewboard installer is not detecting it. | 14:31 |
| bddebian | Heya persia! | 14:32 |
| sebner | huhu bddebian :) | 14:34 |
| bddebian | Heya sebner | 14:35 |
| iulian | Hey. | 14:37 |
| * sebner waves at iulian too | 14:38 | |
| ^arky^ | hi, who's the right person to answer a question on upstart event handling | 14:46 |
| persia | ^arky^: I'm not sure there's any right person. | 14:47 |
| persia | But it also depends on context. If you're looking to put an upstart job in a package, we might be able to help. If you're writing an upstart job from scratch, you might want to talk to the upstart team. | 14:48 |
| ^arky^ | Then I will ask the question anyway, https://answers.edge.launchpad.net/upstart/+question/97511 | 14:48 |
| ^arky^ | need to write upstart job for gnome-orca package, upstart team looks like a good bet | 14:48 |
| persia | Yeah, for that sort of thing, you probably need upstream support. | 14:50 |
| ^arky^ | Yeah, I have bug 507953 created for this, how do I bring it to the notice of upstart team | 14:59 |
| ubottu | Launchpad bug 507953 in gnome-orca "Upstart keybinding event to restart orca" [Undecided,New] https://launchpad.net/bugs/507953 | 14:59 |
| ^arky^ | think I'll subscribe upstart developers team and keep my fingers crossed | 15:00 |
| persia | Um, surely there's a better way. | 15:00 |
| persia | ^arky^: Try asking in #upstart (based on http://upstart.ubuntu.com/development.html) | 15:01 |
| ^arky^ | persia: I already did | 15:07 |
| ^arky^ | no response | 15:07 |
| persia | Have you tried the mailing list? | 15:08 |
| freinhard | persia: do i need to pass some magic commandline option to dh_makeshlibs to get the symbols output? | 15:26 |
| persia | freinhard: I'm afraid I don't remember offhand | 15:26 |
| persia | You can always run it manually if you do a local binary build (debuild -b) in a chroot (schroot -c or pbuilder --login) | 15:27 |
| === ripps|sleep is now known as ripps | ||
| === MTeck-engaged is now known as MTecknology | ||
| === yofel_ is now known as yofel | ||
| freinhard | what's the benefit of having /usr/lib/libfoo.so in the -dev package? | 16:52 |
| geser | you can link against -lfoo | 16:52 |
| freinhard | ok, my question should have been: why put libfoo.so in the -dev and not in the main package? | 16:53 |
| geser | because you only need it during building other packages (or software in general) | 16:54 |
| freinhard | thank you for clearing that up! | 16:55 |
| sebner | huhu geser :D | 16:56 |
| freinhard | still trying to figure out how to get that symbols file | 16:56 |
| geser | Hi sebner | 16:56 |
| freinhard | tried debian/(dev-packagename|package-name|parent-directory-name).symbols | 16:56 |
| freinhard | none of them cause dh_makeshlibs to call dpkg-gensymbols | 16:57 |
| freinhard | what does "if (-e" in perl check? exists? | 16:58 |
| geser | have you also tried debian/libfoo1.symbols (or how exactly your library package is named) | 16:59 |
| freinhard | yes, do they need to contain something? just put a empty file there | 17:00 |
| geser | don't know, haven't need to use this yet | 17:00 |
| freinhard | if a package contains a optional .vim syntax file that's usually goes to /usr/share/docs, should i install it to /usr/share/vim/vimcurrent/syntax/ ? | 17:04 |
| freinhard | hmmi guess not, would imply having vim installed... | 17:06 |
| === cyphermo1 is now known as cyphermox | ||
| freinhard | persia: about the copyright file: copied that from upstream svn, so i guess this should be fine? | 17:12 |
| randomaction | What is the correct procedure for sponsored SRUs? Suppose we have a good debdiff and bug description. What happens next? ubuntu-sru ack or upload to -proposed? | 17:39 |
| freinhard | http://revu.ubuntuwire.com/p/ctemplate could need some more review :) | 17:51 |
| ScottK | randomaction: It's a bit confused, but I think the consensus is upload. | 17:57 |
| randomaction | Also, what does opening a task for a stable release imply? It means you think that the problem is worth a SRU, right? | 17:59 |
| ScottK | yes. | 18:01 |
| randomaction | thank you | 18:02 |
| ScottK | Although sometimes I'll open one against all affected releases and wontfix the ones I think we shouldn't do for clarity | 18:02 |
| === micahg1 is now known as micahg | ||
| freinhard | anyone willing to sponsor my (almost ;) ) perfect ctemplates package? http://revu.ubuntuwire.com/p/ctemplate | 18:12 |
| fabrice_sp | anyone knowing how to fix xchat in amd64? The log is there http://launchpadlibrarian.net/37913661/buildlog_ubuntu-lucid-amd64.xchat_2.8.6-4ubuntu3_FAILEDTOBUILD.txt.gz | 18:31 |
| fabrice_sp | The line that fails is this one: GList *list = gtk_tree_view_column_get_cell_renderers (col); and I tried to include <gtk/gtktreeviewcolumn.h> | 18:33 |
| fabrice_sp | bbl | 18:35 |
| kamalmostafa | ScottK: Hi Scott. I've re-done the libtifiles2 merge -- this time keeping the older Debian autoconf products and most of the debian/ file contents. After folding in the one actual code change that Ubuntu picked up from the maintainer, the merge diff is really only a dozen lines or so: https://code.launchpad.net/~kamalmostafa/ubuntu/lucid/libtifiles/libti-backmerge | 18:37 |
| kamalmostafa | fabrice_sp: I see that in that header file, the prototype you need is wrapped by #ifndef GTK_DISABLE_DEPRECATED -- is that the problem perhaps? | 18:38 |
| ScottK | kamalmostafa: Great. I'm busy with $WORK right now. I'll try and look at it tonight. | 18:41 |
| kamalmostafa | ScottK: That's fine. | 18:41 |
| fabrice_sp | kamalmostafa, will have a look. Thnaks for the idea! :-) | 18:49 |
| kamalmostafa | fabrice_sp: glad to help! | 18:50 |
| === and`_ is now known as and` | ||
| kamalmostafa | Problem: After updating a .gmo file, my bzr builddeb -S fails: dpkg-source: error: cannot represent change to .../po/fr.gmo: binary file contents changed | 19:58 |
| kamalmostafa | Advice? | 19:58 |
| fabrice_sp | kamalmostafa, update the source (.po file?), and modify the debian/rules to generate it again, if it's not the case | 20:08 |
| kamalmostafa | fabrice_sp: I'm not at all familiar with the po/ methodology. There is a fr.po file (and a fr.gmo) already checked into this bzr tree. I'm trying to replace them with the fr.po and fr.gmo from a later version of the same package. So I don't think the .gmo needs to be generated again -- just needs to be rolled into this change. I can "bzr commit" it no problem -- just can't build a test source package with "bzr builddeb - | 20:13 |
| fabrice_sp | AFAIK, you can't commit a binary change | 20:14 |
| kamalmostafa | bzr commit accepts it with no argument. :-) | 20:15 |
| kamalmostafa | Is it abnormal that a .gmo file would be checked in at all then? | 20:16 |
| maxb | kamalmostafa: Yes, quite abnormal | 20:17 |
| maxb | Unless it's by way of it being shipped in an upstream tarball, and thus imported. | 20:17 |
| fabrice_sp | sorry: building classpath is killing my computer :-/ | 20:17 |
| maxb | In which case, what needs to happen is that the .gmo is unchanged whilst building the debian source package | 20:18 |
| * fabrice_sp was going to say the same thing :-) | 20:18 | |
| kamalmostafa | maxb: okay, I think that does fit this situation -- both the "original" and my "target" packages were imported from upstream. | 20:19 |
| fabrice_sp | generally, in that cases, I modify the clean target to delete the generated binary files, and build them | 20:20 |
| fabrice_sp | even if the binary file comes from upstream | 20:20 |
| kamalmostafa | So now when exactly will the new .gmo be regenerated? (just part of 'make'?) | 20:21 |
| maxb | kamalmostafa: OK. Make sure that _you_ don't commit any changes to those binary files - they should be left identical to those coming from the upstream import. | 20:21 |
| maxb | Well, that's dependent on the specific package's buildsystem, but hopefully yes | 20:21 |
| kamalmostafa | Got it (and I'll test it to make sure that happens). So let me get it straight... Its fine to change the .po, but leave the .gmo alone when building the source package; the .gmo should get regenerated as part of the build anyway. Yes? | 20:23 |
| maxb | Correct | 20:23 |
| kamalmostafa | maxb and fabrice_sp: Thanks very much! | 20:23 |
| qnix | hi, is there a page about the Prevu tool ? Who said exactly it does in its automated backporting ? | 20:46 |
| sistpoty | hi folks | 21:01 |
| sebner | huhu sistpoty :D | 21:01 |
| sistpoty | hi sebner | 21:01 |
| freinhard | hi | 21:01 |
| sistpoty | hi freinhard | 21:01 |
| maxb | qnix: prevu is mainly just a wrapper around pbuilder. If you want to use it for anything complex, I recommend learning about pbuilder directly. | 21:02 |
| freinhard | looks like people with sparetime! go for it: http://revu.ubuntuwire.com/p/ctemplate :D | 21:02 |
| maxb | qnix: See also cowbuilder / cowdancer, which will save you so much time if you do lots of things with pbuilder | 21:02 |
| qnix | ah, I already use cowbuilder. | 21:02 |
| maxb | Then you don't really care much about prevu | 21:03 |
| qnix | I thought prevu was something that fixes incompatibilities between different version of debhelper etc.. | 21:03 |
| qnix | Ok | 21:03 |
| freinhard | selling stuff here is really really hard ;) | 21:03 |
| maxb | qnix: Well, I admit I only looked at it last a couple of distributions ago, but it didn't do anything of that sort back then | 21:03 |
| qnix | maxb: all right | 21:04 |
| sistpoty | freinhard: I'm already looking at it, just not telling you :P | 21:07 |
| freinhard | sistpoty: yay :D | 21:08 |
| === jtechidna is now known as JontheEchidna | ||
| sistpoty | freinhard: commented on it, some tiny tidbits, but nothing really serious. Nice package! | 21:26 |
| sistpoty | nixternal: from the try right now to fix gdcm it looks like I don't need ppc access after all... it ftbfs on amd64 now with the same reason (which I had slightly suspected) | 21:30 |
| sistpoty | nixternal: so feel free to disable/remove my account | 21:30 |
| freinhard | sistpoty: i was told that there should be a .symbols file for lib packages, but nobody seems to be able to tell me how to do it right. | 21:31 |
| siretart | please don't add .symbol files if you don't understand them. a broken symbol file can be very annoying... | 21:32 |
| nixternal | sistpoty: I will leave it, just in case you ever need it again :) | 21:32 |
| siretart | hi folks, btw! | 21:32 |
| nixternal | howdy sistpoty | 21:32 |
| sistpoty | hi siretart and nixternal | 21:32 |
| sistpoty | nixternal: ok :) | 21:32 |
| freinhard | siretart: i undestood what it's good for but i didn't even get that far to produce any content for the .symbols file | 21:33 |
| freinhard | sistpoty: thx for commenting | 21:33 |
| siretart | freinhard: you can use dpkg-gensymbols for creating a first version | 21:34 |
| sistpoty | freinhard: man dpkg-gensymbols might give some hints... however (and that's how I make it) unless libctemplate has many reverse dependencies, a symbol file is not too much needed in the first place | 21:34 |
| freinhard | sistpoty, siretart: i thaught dh_makeshlibs calls dpkg-gensymbols if there is a libctemplate0.symbols file in the debian folder, but that didn't happen, or there was no output on commandline | 21:35 |
| siretart | freinhard: it calls dpkg-gensymbols to check against the referencs .symbols file. you still have to generate and maintain the reference file | 21:36 |
| === gnomefreak76 is now known as gnomefreak | ||
| === mezgani is now known as p3rror | ||
| sistpoty | ... grumble... gdcm... grumble... it looks like it should work but doesn't :( | 22:09 |
| sistpoty | meh, I'm stupid, I really should read the context, now it's suddenly all clear | 22:12 |
| * Laney spanks RainCT | 22:26 | |
| Laney | you broke pbuilder-dist in 560 | 22:26 |
| RainCT | Laney: wha? | 22:26 |
| Laney | /usr/bin/pbuilder | 22:26 |
| Laney | should be sbin! | 22:26 |
| * Laney fixes | 22:27 | |
| RainCT | uhm yeah sorry | 22:27 |
| sebner | lol | 22:27 |
| sebner | hi Laney | 22:27 |
| RainCT | I wanted to get ride of the hardcoded path entirely but apparently forgot to do so :P | 22:27 |
| Laney | hiya sebner | 22:28 |
| sebner | Laney: side note: Eagerness rather than immaturity. I care for my stuff and tend to give 110% for it. Might seem odd for some though. Call it immaturity if you think (evil hard) life will change me | 22:41 |
| sistpoty | sebner: huh? | 22:41 |
| Laney | I was trying to defend you | 22:48 |
| === micahg1 is now known as micahg | ||
| sistpoty | ScottK: I think I've got a fix for gdcm now, which should make it build on ppc (not tested, it ftbfs'd as I thought in the same way on amd64 now). problem is that it's a hack to fix debian 562775 and I'm not 100% sure if it will also cure rdepends | 22:55 |
| ubottu | Debian bug 562775 in libvtk-java "FTBFS: Could not find vtk.jar file, VTK_JAVA_JAR is wrong, please set proper GDCM_VTK_JAVA_JAR replacement var" [Serious,Open] http://bugs.debian.org/562775 | 22:55 |
| sistpoty | (due to lack of knowledge about both cmake and java) | 22:55 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!