/srv/irclogs.ubuntu.com/2018/01/17/#ubuntu-devel.txt

sarnoldbdmurray: polite poke re https://bugs.launchpad.net/ubuntu/+source/apport/+bug/1738581  :)01:31
ubottuLaunchpad bug 1738581 in apport (Ubuntu) "apport is leaking environment variables (including passwords!) to public bug reports" [Undecided,New]01:31
slangaseknacc: enabled> ugh.  I'd say there are no good interfaces here, and it'll be hard to implement this in a way that won't break once things start to move to systemd (not necessarily in tandem)05:46
=== cpaelzer_ is now known as cpaelzer
* xnox wonders if i am back on irc10:14
Odd_Blokexnox: You appear to be.10:15
xnoxwhoop whoop10:16
Unit193Nah, you aren't, xnox.10:17
juliankI don't entirely understand MoM's days old. For example, wpa's Oct 2017 upload was merged, but it says 326 days old11:06
cjwatsonI believe it's supposed to be the date on which it was superseded - i.e. the creation date of the first Debian source publication with a higher version11:09
juliankcjwatson: hmm, there was a newer version in experimental before that, amybe it picked thatu p11:13
juliankjbicha: Seems we can sync libsecret now? gjs is back on s390x, and you pushed the disable vala & python there patch to Debian11:13
rbasakI think MoM only sees unstable, not experimental.11:19
rbasakEven if Ubuntu is synced to experimental.11:19
* rbasak is guessing11:19
jbichajuliank: yes11:20
juliankjbicha: ok, done.11:20
cjwatsonrbasak,juliank: mm, except there is a bug here.11:27
cjwatsonat least arguably11:27
cjwatsonjuliank: fixed for the next run, fingers crossed.  http://bazaar.launchpad.net/~ubuntu-core-dev/merge-o-matic/trunk/revision/30311:32
juliankcjwatson: awesome!11:32
rbalintlool: do you mind if i merge flash-kernel from unstable?11:35
juliankoh, it's about time for that one :)11:36
juliank3.0~rc.4ubuntu6611:36
rbalintjuliank: i figured no one wanted touch it :-)11:37
juliankI wonder if I should have a go at wpa11:38
juliank^ mostly mdeslaur 's playground :)11:39
rbalintjuliank: it is a desktop package, so i'm trying to fix foundations ones first11:39
juliankThere aren't many _good_ ones left :D11:40
juliankOh coreutilsmight be nice11:41
juliankstrace is blocked on 3 broken tests, I should figure out why they're broken. ugh.11:42
juliankWe should add some classes and javascript to the MoM output so we can filter out the green and gray stuff, and perhaps a search thingy, that would be nice.11:49
rbalintjuliank: i'd prefer a way different strategy, driving the outstanding merges so low that we should not care about the filtering :-)11:52
juliankthat works too11:53
juliankI already took out 4 of them today so far11:53
julianktwo on monday, two more queued in a ppa11:53
juliankand what, 7 or 8 last week?11:53
rbalintjuliank: great! :-)11:53
juliankMy bionic is a bit unstable. From time to time, it does not register clicks11:54
juliankonly for like 5 seconds, then it works again11:54
juliankusing the X session :D11:55
mdeslaurjuliank: I'm not a wpa expert, go ahead if you feel like it!12:05
willcooke juliank hey!  Re: mouse clicks, could it be this?  https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/118166612:25
ubottuLaunchpad bug 1181666 in GNOME Shell "gnome-shell randomly blocks mouse clicks from working in app windows" [Critical,Confirmed]12:25
juliankwillcooke: sounds like it12:42
juliankwillcooke: though for me it's only temporary and solves itself12:43
willcookejuliank, I think soving itself is in keeping with what duflu is saying.12:59
willcookeI'll follow up with him and see whats going on there12:59
rbalintpitti: i'm looking at merging procps from unstable if you don't mind13:04
pittirbalint: sure, go ahead; I don't have taps on anything right now13:04
loolrbalint: absolutely not, would love if you would13:10
rbalintpitti, lool: thanks, taking them then13:25
juliankapw: if you don't mind I'll have a go at merging cryptsetup14:09
juliankapw: Though I don't quite understand why you did all these changes like dropping c99, they seem pointless to me.14:11
juliankoh, that might just be wrongly indented changelog14:11
juliankxnox: seems you made the _BSD_SOURCE => _DEFAULT_SOURCE, removal of -std=c99 changes in cryptsetup. May I ask why? seems like a useless diff14:12
juliank* last year in August14:13
xnoxjuliank, because the new world order is c11; and it FTBFS; _BSD_SOURCE no more; it's _DEFAULT_SOURCE now14:14
juliankIt FTBFS due to std=c99?14:14
xnoxno, it's just std=c99 used be higher than the default standard; now it's lower than the default standard; and hence I upgraded to c11.14:14
xnoxand _BSD_SOURCE was tripping up the preprocessor i believe.14:15
juliank#define _GNU_SOURCE14:15
juliank#define _BSD_SOURCE14:15
juliank#define _POSIX_C_SOURCE 114:15
xnoxyes14:15
juliankthat's what Debian uses now....14:15
xnoxwell, that's nuts =)14:15
juliankseems more complicated than yours :D14:15
juliank"Oh, let's see what we can add to make it compile"14:15
xnoxjuliank, i'm all for dropping delta's and going with whatever debian has, on the grand scheme of things, this is marginally irrelevant.14:16
xnoxjuliank, i hope they have dropped c99, or have they not?14:16
juliankxnox: No they have not.14:16
* xnox ponders if _DEFAULT_SOURCE works on kfreebsd and hurd14:16
xnoxmaybe i should forward them a patch, but not sure if i care about these thigns.14:16
juliankxnox: But the thing is, it could be a mere statement that this is supposed to be C9914:16
juliankNot neccessarily for technical reasons14:17
xnoxi think it would fail to compile with 8914:17
xnoxcause it actually relied on c9914:17
* xnox that's from memory14:17
juliankxnox: They also pass -pedantic though, so they kind of have to pass -std=c99 or something, as gnu99 or gnu11 is not pedantic enough :D14:18
juliankI don't think you even need _DEFAULT_SOURCE in gnu11 mode14:20
juliankBut since this is debian/-only, I wonder anyway why they want to build in ISO C mode14:20
rbasaksil2100: any idea what's going on with bug 1718824 and pulseaudio in Xenial genreally?14:47
ubottubug 1718824 in HWE Next "The analogue audio does not work on the Dell USB Dock" [Critical,In progress] https://launchpad.net/bugs/171882414:47
rbasakFor example publishing history https://launchpad.net/ubuntu/+source/pulseaudio/+publishinghistory for 1:8.0-0ubuntu3.514:47
rbasakI see no SRU process comments in the bug14:48
rbasakBut you've been involved, I think14:48
rbasakAnd there is stuff in the queue.14:48
sil2100rbasak: hm, I don't know anything about this sadly14:52
sil2100rbasak: I don't remember being involved in this bug, but maybe my memory is just wrong?14:52
rbasakI don't see your involvement in the bug specifically.14:52
rbasakBut I saw you in publishing history for  1:8.0-0ubuntu3.714:52
sil2100I did some publishing of it in the past indeed, let me take a look at what's in this SRU14:57
naccslangasek: ok, for context, LP: #174089215:42
ubottuLaunchpad bug 1740892 in corosync (Ubuntu Bionic) "corosync upgrade on 2018-01-02 caused pacemaker to fail" [Medium,In progress] https://launchpad.net/bugs/174089215:42
juliankcjwatson: The MoM change seems to have worked well :)20:28
juliankBetter merge prioritisation possible :D20:29
tsimonq2Speaking of MoM, now that I know where the source code is and I'm remembering to do it: https://code.launchpad.net/~tsimonq2/merge-o-matic/increase-graph-size/+merge/33626521:51
tsimonq2:D21:51
juliank+121:52
tsimonq2juliank: Could you merge it then? You're a Core Developer ;)21:54
julianktsimonq2: merged21:58
julianktsimonq2: not sure if it needs somebody to update some infra or not21:59
tsimonq2Good point. cjwatson?22:01
juliankI should add search filter javascript stuff22:04
tsimonq2+122:05
juliankand of course make MoM useful for non-native packages by using unapplied patches states22:05
juliank* more useful22:05
tsimonq2YES22:07
tsimonq2That annoys me SO MUCH22:07
* tsimonq2 slides juliank a preemptive cold one22:07
tsimonq2(er, maybe not the right word, you get it :) )22:08
julianktsimonq2: it's not that easy it seems22:10
juliankor it should be22:10
tsimonq2juliank: Right, it might involve some tweaking irt patches.ubuntu.com22:11
tsimonq2afair it grabs the patches from there and then applies them22:11
julianktsimonq2: https://code.launchpad.net/~juliank/merge-o-matic/skip-patches/+merge/33626722:17
julianktsimonq2: it generates patches.ubuntu.com22:18
juliankAFAICT22:18
tsimonq2oh22:19
tsimonq2juliank: lgtm (although I'm not a ~core-dev so you'll have to merge it yourself :) )22:20
juliankI'll leave that up to someone who knows what they're doing22:21
tsimonq2Ok22:22
naccslangasek: infinity: when there is a buggy prerm of a package, is there any pattern established for how to handle that in a fix? In this case, it's corosync, where the old package's prerm stops corosync, which ends up stopping pacemaker, and nothing currently starts pacemaker back up. I don't know of a way to determine in the postinst (when using --restart-after-upgrade to dh_installinit) that22:22
naccpacemaker was running when the old prerm ran.22:22
naccmy initial stab tried to use /etc/init.d/pacemaker status in the init script, but that's run from the postinst, which means pacemaker is in the stoppe state22:24
cjwatsonjuliank,tsimonq2: deployed MoM r30522:35
tsimonq2cjwatson: ack, many thanks22:36
tsimonq2cjwatson: How often does cron run there?22:37
cjwatsontsimonq2: 05,35 * * * *   /srv/patches.ubuntu.com/code/cron.daily22:37
cjwatsonthough that's subject to locking22:37
juliankcjwatson: Opinions on https://code.launchpad.net/~juliank/merge-o-matic/skip-patches/+merge/336267 welcome too, if you have time :D22:38
cjwatsonnot tonight I'm afraid22:38
juliankcjwatson: no problem :)22:39
tsimonq2cjwatson: thanks22:40
julianktsimonq2: big graphs look better, yeah23:09
juliank:D23:09
julianklatest merge page has them now23:09
tsimonq2I saw :D23:10

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