[00:59] rbelem: Good evening. Do you want to look at liquid stuff tonight? [01:01] hi persia :-) [01:01] persia, yep :-) [01:02] What time works best for you? [01:02] persia, i'm finishing plasma-mobile copyright [01:02] persia, this time if fine for me [01:03] OK. Let me know whenever you're at the next step, and we'll finish plasma-mobile, and start looking at k*-mobile [01:03] cool! :-) [01:05] persia, i have two script files in plasma-mobile without license, but with two lines code. should i ignore it? [01:06] pastebin them? Two lines of code are usually able to be considered insufficiently significant for copyright, unless they do something really nifty. [01:07] (like the 1-line perl implementation of a cryptography algorithm) [01:08] persia, it is just a gettext command [01:08] http://paste.ubuntu.com/359324/ [01:08] rbelem: Yeah. I don't think that's significant enough to carry copyright: we may as well submit it. [01:08] The archive admins may disagree though, in which case we need to pester upstream. [01:09] cool! [01:29] persia, is packagecheck available in ubuntu? [01:32] Not to my knowledge. What would it do? [01:45] there is licensecheck and lintian ... [01:49] The surprising bit is that even search engines aren't helping identify "packagecheck". [02:06] persia, http://pkg-perl.alioth.debian.org/howto/quilt.html [02:07] Odd. Should be there. Anyway, are you patching the sources? I didn't think we'd gotten there yet. [02:08] persia, plasma-mobile needs some changes to make it buildable [02:09] Ah. You already added "--with quilt" to your dh call? [02:09] yep [02:10] Cool. I usually just copy /usr/share/doc/quilt/README.source into my packages, but I'll hunt for packagecheck [02:12] rbelem: Whilst you're changing stuff, you might want to build-dep on pkg-kde-tools and include /usr/share/pkg-kde-tools/makefiles/1/debhelper/kde.mk in your debian/rules (this handles the cmake stuff) [02:13] persia, nice! :-D [02:14] i did not know that :-) [02:15] I had thought you were still chasing copyright, or I would have given you that and quilt sooner :) [02:15] eheheh [02:15] persia, i think i finished [02:16] i will paste it to you [02:17] persia, http://paste.ubuntu.com/359346/ [02:19] rbelem: "Maintainer" in debian/copyright should be the upstream maintainer. As you don't appear elsewhere in the file, I doubt that's you. [02:19] persia, i got this warning http://paste.ubuntu.com/359349/ [02:19] Obviously my rarely-used KDE packaging skills are out of date :) [02:20] i thought it was related to package maintainer [02:20] ehehehe [02:20] :-) [02:20] Nope, that would be in debian/control [02:21] You can omit the first Copyright: paragraph, or if you don't omit it, you need to procede it with a Files: specifier. [02:22] You must include a License: entry in each Files: paragraph [02:23] rbelem: See #kubuntu-devel : we need to change debian/rules to use --with kde and drop the include. [02:24] :-D [02:27] persia, i think i'm missing something [02:27] persia, it is not building now [02:27] Using --with kde and dropping the include? [02:27] What fails? [02:28] persia, i will paste [02:34] rbelem: About that error? [02:35] oops... [02:35] forgot :-) [02:35] eheheh [02:35] Thought you might. Simultaneous multi-channel conversations can be tricky :) [02:36] eheheh [02:36] persia, http://paste.ubuntu.com/359353/ [02:40] rbelem: I think you need to build-depend on kdelibs5-dev to get KDE4Macros.cmake to define kde4_add_plugin [02:40] persia, it is already [02:41] Strange. Can you put your .diff.gz somewhere? I'll take a deeper look. [02:41] yep [02:48] persia, http://paste.ubuntu.com/359360/ [03:13] persia, any ideas? [03:13] rbelem: Still build-testing [03:22] persia, did you get the build working? [03:22] rbelem: No, but I've managed to replicate the error :) [03:22] persia, :-) [03:23] persia, it seems that it is not applying the patch [03:23] My build log agrees with that: "No patches in series" [03:25] persia, is the patch there in .pc? [03:25] .pc oughtn't matter, but .pc isn't in your patch [03:26] Strange thing being that if I run `quilt push -a` manually, it reports that it applies. [03:26] I'm still digging: I'll let you know as soon as I have something. === asac_ is now known as asac [03:32] persia, http://paste.ubuntu.com/359372/ [03:33] persia, it is how CMakeList.txt should look like [03:34] I have http://paste.ubuntu.com/359374/ [03:35] Maybe refresh the patch? [03:35] persia, already refresh [03:35] (`quilt push -a; $editor CMakeList.txt; quilt refresh; quilt pop -a`) [03:36] And that makes it build? [03:36] nope [03:36] heh. OK. I'll update my copy of the patch and keep investigating. [03:36] only builds when i run cmake by hand [03:37] Right, which doesn't help us as much as we'd like. [03:48] persia, i think i'm going to sleep in some minutes [03:48] Fair enough. It's getting late. I'll troubleshoot, and see if I can get the build to complete. [03:49] I'll email you my diff.gz later, and we can touch base tomorrow evening. [03:50] persia, cool! [03:50] Have a good night. [03:50] persia, have a good day :-D [03:50] bye === njpatel_ is now known as njpatel === Zic is now known as Guest90451 === dmart_ is now known as dmart === dmart_ is now known as dmart === kojack is now known as kolbyjack === robbiew is now known as robbiew-afk === robbiew-afk is now known as robbiew