=== Rcart_ is now known as Rcart === chilicuil_away is now known as chilicuil === chilicuil is now known as chilicuil_away [07:51] good morning === didrocks1 is now known as didrocks === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away === lan3y is now known as Laney === Quintasan_ is now known as Quintasan === didrocks1 is now known as didrocks === Tonio_ is now known as Tonio_aw === Tonio_aw is now known as Tonio_ === Tonio_ is now known as Tonio_aw === Tonio_aw is now known as Tonio_ [16:42] Hunk #1 succeeded at 17 with fuzz 1. what does it mean? [16:44] It means that a part of a patch applied correctly, but it was one line off of what is specified in the patch. [16:45] Logan_: Isn't that offset 1? [16:47] Fuzz is the number of lines of context that had to be ignored to make the patch apply. [16:47] It's if the context is a bit different [16:47] patch(1) explains it [16:47] Oh, right, my bad. [16:47] obounaim: It merits manual investigation to make sure the patch applied where you wanted and still does what you want. [16:49] Ok, thanks all of you. [16:50] jtaylor: sorry I didn't get to look at your backport this weekend, I'll try to squeeze it in at some point [16:50] Logan_: Are you Logen Rosen? [16:50] That's me. [16:51] Logan_: Forwarding new version should be packaged bugs to Debian while they are in pre-release freeze isn't particularly likely to be of benefit. [16:52] (I noticed the skanlite one because I'm subscribed to the relevant Debian mailing list) [16:52] True, but it encourages the maintainer to fix it after the freeze. And we can always submit a patch if the change is applied in Ubuntu first. [16:55] It also causes maintainer annoyance at Ubuntu. === [ESphynx] is now known as ESphynx === LordOfTime is now known as TheLordOfTime === yofel_ is now known as yofel === Logan__ is now known as Logan_ === chris|| is now known as chris| === LordOfTime is now known as TheLordOfTime === lifeless_ is now known as lifeless [22:30] jtaylor: poke [22:30] Logan_: ? [22:31] jtaylor: I'm doing a merge of pyg from Debian, and I noticed that you "include required string.h in wlp/C/commands.{l,y}" - this was done in Debian, but only for the .y file - should the .l delta remain? [22:32] hm is the l file autogenerated? [22:32] i.e. include 'string.h' was only done in commands.y in Debian, and not in .l, as it was in Ubuntu [22:32] I'm not very familiar with this lexing stuff [22:32] actually, it probably is [22:33] I just patched both and the warning disappeared, possibly the l patch was not needed [22:33] if the implicit declaration warning is gone you can drop it [22:33] alright, cool, thanks === jtechidna is now known as JontheEchidna