[03:47] <The_LoudSpeaker> wxl: @kc2bez I am unable to edit D32
[03:47] <wxl> why not?
[03:48] <The_LoudSpeaker> arc patch D32 --nobranch is giving me errors
[03:48] <wxl> grab a new repo then do that
[03:48] <The_LoudSpeaker> raman@Aspire-ES1-132:~/Style/Work/work2/libfm-qt$ arc patch D32
[03:48] <The_LoudSpeaker>  Exception 
[03:48] <The_LoudSpeaker> [cURL/35] (https://phab.lubuntu.me/api/differential.querydiffs) <CURLE_SSL_CONNECT_ERROR> There was an error negotiating the SSL connection. This usually indicates that the remote host has a bad SSL certificate, or your local host has some sort of SSL misconfiguration which prevents it from accepting the CA. If you are using a self-signed certificate, see instructions in "libphutil/resources/ssl/README".
[03:48] <The_LoudSpeaker> (Run with `--trace` for a full exception trace.)
[03:48] <The_LoudSpeaker> raman@Aspire-ES1-132:~/Style/Work/work2/libfm-qt$ 
[03:48] <wxl> oh
[03:48] <wxl> well
[03:48] <wxl> that's a network problem
[03:48] <The_LoudSpeaker> Frikking sophos
[04:05] <The_LoudSpeaker> wxl: https://phab.lubuntu.me/P32
[04:08] <wxl> without all that noise, what does it say?
[04:09] <The_LoudSpeaker> same thing
[04:09] <wxl> if you just run `arc diff` it shows ALL of that?
[04:09] <The_LoudSpeaker> aman@Aspire-ES1-132:~/libfm-work/libfm-qt$ arc diff
[04:09] <The_LoudSpeaker>  Select a Default Commit Range 
[04:09] <The_LoudSpeaker> You're running a command which operates on a range of revisions (usually,
[04:09] <The_LoudSpeaker> from some revision to HEAD) but have not specified the revision that should
[04:09] <The_LoudSpeaker> determine the start of the range.
[04:09] <The_LoudSpeaker> Previously, arc assumed you meant 'HEAD^' when you did not specify a start
[04:09] <The_LoudSpeaker> revision, but this behavior does not make much sense in most workflows
[04:09] <The_LoudSpeaker> outside of Facebook's historic git-svn workflow.
[04:09] <The_LoudSpeaker> arc no longer assumes 'HEAD^'. You must specify a relative commit explicitly
[04:09] <The_LoudSpeaker> when you invoke a command (e.g., `arc diff HEAD^`, not just `arc diff`) or
[04:09] <The_LoudSpeaker> select a default for this working copy.
[04:09] <The_LoudSpeaker> In most cases, the best default is 'origin/master'. You can also select
[04:09] <The_LoudSpeaker> 'HEAD^' to preserve the old behavior, or some other remote or branch. But you
[04:09] <The_LoudSpeaker> almost certainly want to select 'origin/master'.
[04:09] <wxl> don't do that jeez
[04:09] <The_LoudSpeaker> (Technically: the merge-base of the selected revision and HEAD is used to
[04:09] <The_LoudSpeaker> determine the start of the commit range.)
[04:09] <The_LoudSpeaker>     What default do you want to use? [origin/master] 
[04:09] <The_LoudSpeaker>  Exception 
[04:09] <The_LoudSpeaker> Command failed with error #128!
[04:09] <The_LoudSpeaker> COMMAND
[04:09] <The_LoudSpeaker> git cat-file -t 'origin/master'
[04:10] <The_LoudSpeaker> STDOUT
[04:10] <The_LoudSpeaker> (empty)
[04:10] <The_LoudSpeaker> STDERR
[04:10] <The_LoudSpeaker> fatal: Not a valid object name origin/master
[04:10] <The_LoudSpeaker> (Run with `--trace` for a full exception trace.)
[04:10] <The_LoudSpeaker> should I do arc diff HEAD^ ?
[04:10] <wxl> no
[04:12] <The_LoudSpeaker> what should I do now?
[04:13] <wxl> what's `git remote -v` give you?
[04:14] <The_LoudSpeaker> https://phab.lubuntu.me/source/libfm-qt.git for both fetch and push
[04:14] <wxl> that's the problem
[04:14] <wxl> you need to use ssh
[04:14] <The_LoudSpeaker> ughh!
[04:14] <The_LoudSpeaker> I was using git coz sophos
[04:14] <The_LoudSpeaker> ssh is blocked on 2222
[04:15] <wxl> sorry
[04:16] <The_LoudSpeaker> lite.
[04:19] <The_LoudSpeaker> tsujan 's email id ploxx
[04:19] <The_LoudSpeaker> also full name?
[04:19] <wxl> it's in the original patch dude
[04:20] <wxl> both are
[04:20] <wxl> you'll slap yourself in the head when you see his name
[04:20] <The_LoudSpeaker> got it!
[04:22] <The_LoudSpeaker> also abot longer description, I like to be verbose. I will remove it tho. If someone asks I am gonna point them at you wxl.
[04:22] <wxl> verbose is good. unnecessary is not.
 Yes, blame wxl
 XD
[04:22] <wxl> go cough on something, @tsimonq2 !
 Hahahahaha
[04:24] <wxl> @tsimonq2: why ARE we using 2222?
 It was a default
[04:25] <wxl> hm
 We should make sure it doesn't interfere with SSHing in though
[04:25] <The_LoudSpeaker> wxl: D32
[04:27] <The_LoudSpeaker> wxl: Should I change the author in debian changelog also?
[04:27] <wxl> no
[04:28] <The_LoudSpeaker> okay
[04:31] -lugito:#lubuntu-devel- [rUPDATENOTIFIERc7e1f1ab58c5: added version and in help] Hans P Möller <hmoller@uc.cl> committed: https://phab.lubuntu.me/rUPDATENOTIFIERc7e1f1ab58c5
[04:31] <The_LoudSpeaker> how do I Get rid of the changes to debian/patches/workaround-missed-file-monitoring.patch ? 
[04:32] <The_LoudSpeaker> copy it before arc patch D32 and then paste it later?
[04:33] <wxl> git checkout <commit> <file>
[04:34] <wxl> or maybe `git reset <commit> -- <file>`
[04:44] <The_LoudSpeaker> check now.
[04:46] <The_LoudSpeaker> @tsimonq2, accept D32 please.
[04:48] -lugito:#lubuntu-devel- [T99: Fix DND with MTP devices] The_LoudSpeaker (Raman Sarda) commented on the task: https://phab.lubuntu.me/T99#2028
 .
 wxl: Please merge and upload.
[04:50] -lugito:#lubuntu-devel- [T99: Fix DND with MTP devices] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T99#2029
[04:51] <wxl> OH
[04:51] <wxl> one other thing
[04:51] <wxl> when you're doing arc diffs.. reference the task
[04:51] <wxl> you can edit the summary and add a T99
 Ah yes, went over my head too
[04:51] <wxl> it will auto close things
[04:52] <wxl> so since you probably don't read your mail: read the task about SRUing, @tsimonq2 
[04:52] <The_LoudSpeaker> okay. Noted.
[04:52] <wxl> so go edit :)
 @wxl [<wxl> so since you probably don't read your mail: read the task about SRUing, @t …], I do read my mail, I saw it but don't have the energy to give you a fully qualified response right now
 tl;dr consult the docs and see if it qualifies
 If it doesn't, the workaround should be documented in the manual
[04:54] <The_LoudSpeaker> wxl: done!
 Or a good exercise for Raman :)
[04:54] <wxl> it's not that it doesn't qualify
 https://wiki.ubuntu.com/StableReleaseUpdate @The_LoudSpeaker
[04:55] <wxl> it's not high impact so that's out
[04:56] <wxl> (1) have an obviously safe patch and (2) affect an application rather than critical infrastructure packages (like X.org or the kernel). 
[04:56] <The_LoudSpeaker> @tsimonq2, I don't I will be able to read that today. or tommorow. Monday possible.
[04:56] <wxl> seems to meet that
[04:56] <wxl> so yeah SRU away @The_LoudSpeaker
[04:56] <wxl> do it after 19.10, though
[04:56] <The_LoudSpeaker> okay but some time ploxxx.
 @wxl [<wxl> do it after 19.10, though], What's the point?
[04:57] <The_LoudSpeaker> How does SRU after 19.10 makes sense?
 ^ :)
[04:57] <The_LoudSpeaker> We will directly include it in 19.10 na?
[04:57] <wxl> what you just did is 19.10
[04:57] <wxl> SRU is for 19.04
[04:57] <wxl> we can worry about that after 19.10 is out
[04:57] <wxl> we need you for all of THAT XD
 Right, but we're encouraging people to install 19.10 after it's released
 That's like two months away
[04:58] <wxl> workaround exists
[04:58] <wxl> not to mention the bug has been there for months already
[04:58] <The_LoudSpeaker> the you mean SRU in 19.04 right?
[04:59] <The_LoudSpeaker> *then
[04:59] <wxl> yep
 Okay, so we document the workaround :)
 Problem solved
[04:59] <The_LoudSpeaker> okay. I will add it to todo
[04:59] <wxl> i mean we just add the bug to the release notes and we're done
[04:59] <wxl> cuz the bug has documented the workaround for a long time
[04:59] <The_LoudSpeaker> but @tsimonq2 's idea seems more interesting.
 Mmkay
[05:00] <wxl> huh?
[05:00] <wxl> what ideA?
 @The_LoudSpeaker you should read the SRU docs anyway :)
[05:00] <The_LoudSpeaker> I mean like if we document the workaround then we don't need to sru right?
 Yes
[05:00] <wxl> we don't need to
[05:00] <wxl> we could though
[05:00] <wxl> wouldn't be a bad idea
[05:00] <The_LoudSpeaker> @tsimonq2, Yup! I will read it.
[05:01] -lugito:#lubuntu-devel- [rLIBFMQTPACKAGING3663fb0ed1c2: Fix DND MTP Bug] wxl (Walter Lapchynski) committed: https://phab.lubuntu.me/rLIBFMQTPACKAGING3663fb0ed1c2
 The SRU docs are in the Lubuntu Developers page, right?
 And @HMollerCl @aptghetto are going to apply like yesterday, right? :P
[05:02] <wxl> ruh roh
[05:03] <wxl> patch unexpectedly ends in middle of line
 Oh that's fun
[05:03] <wxl> hunk #3 failed at 445
[05:03] <The_LoudSpeaker> which patch?
 @wxl [<wxl> hunk #3 failed at 445], Where are you seeing this ooc?
[05:03] <The_LoudSpeaker> fix-dnd-mtp onr?
[05:03] <wxl> debuild -S
[05:03] <wxl> yep
[05:04] <wxl> it applied cleanly before. strange. i even tried it and it worked fine.
[05:04] <The_LoudSpeaker> It didn't fail when I used earlier. might be failed due to changes in P31
[05:04] <The_LoudSpeaker> it applies but faills see lines 607-610 of P30
[05:05] <wxl> the issue that seems different is the extra line at the end
 Whaaaaaaatttt?
 You gotta be kidding me.
[05:05] <wxl> nope
[05:05] <wxl> that's fuzz, my friend
 F
[05:06] <wxl> talk to @tsimonq2 about trailing whitespace
 I will have to it again then?
 @wxl [<wxl> talk to @tsimonq2 about trailing whitespace], BAD
 Where's the trailing whitespace?
 >_>
[05:06] <wxl> wait
[05:06] <wxl> huh
[05:06] <wxl> i'm wrong
[05:06] <wxl> hold on.
[05:06] <wxl> i'm confused.
 What happened?
[05:07] <wxl> ok that is really weird
 Drop the bomb already!
[05:07] <wxl> the patch i got was missing an empty line
[05:07] <wxl> but on phab it's there
[05:07] <wxl> they should be identical
[05:08] <wxl> P: libfm-qt source: file-contains-trailing-whitespace debian/changelog (line 24)  
 Weird.
 I had just done dch -r and changed email id
[05:09] <wxl> well that's the thing
[05:09] <wxl> on phab it looks right
[05:17] -lugito:#lubuntu-devel- [rLIBFMQTPACKAGING4cfa12445f16: Add missing empty line to end of patch.] wxl (Walter Lapchynski) committed: https://phab.lubuntu.me/rLIBFMQTPACKAGING4cfa12445f16
[05:18] <wxl> DUDE WTF
[05:20] <wxl> sheesh
[05:21] <wxl> ohhhhhhhhhhhhh
[05:21] <wxl> if you do the side-by-side view you can see "no newline at end of file"
[05:25] -lugito:#lubuntu-devel- [T99: Fix DND with MTP devices] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T99#2035
[13:30] <kc2bez> wxl: to answer your question why T99 didn't autoclose: you need to put `resolves T99` for it to it automatically else it just becomes a mention (also good if you have more to do). 
[14:43] -lugito:#lubuntu-devel- [T103: Properly deal with translations on the blog] Noumeno (Roberto) commented on the task: https://phab.lubuntu.me/T103#2038
[17:16] -lugito:#lubuntu-devel- [rMANUAL1c43d6a6eac1: Capitilize IRC] lynorian (Lyn Perrine) committed: https://phab.lubuntu.me/rMANUAL1c43d6a6eac1
[17:21] -lugito:#lubuntu-devel- [rMANUALfb23781c8b22: Add Quit quassel by right click Quit] lynorian (Lyn Perrine) committed: https://phab.lubuntu.me/rMANUALfb23781c8b22
[17:30] -lugito:#lubuntu-devel- [rMANUALd80aeb54d289: Remove unnesecary whitespace] lynorian (Lyn Perrine) committed: https://phab.lubuntu.me/rMANUALd80aeb54d289
 I was installing lubuntu 18.04 in a friend's laptop. In live usb rn.
 Pressed windows key. Application menu didn't show up.
 Last I used, it was set to open on pressing windows key.
 Did I miss something?
 ^ @kc2bez @tsimonq2 wxl:
[18:01] <wxl[m]> 18.10 and beyond only
[18:01] <wxl[m]> Tl;Dr no one wants LXDE
 That guy needs a stable system. He doesn't want to update every 6 momths.
 Also, We are supporting 18.04 till next year na?
 That's enough for him.
 Also, Idk why brightness keys functions are inverted? The slider also works invertedly. Moving to the Left side increases brightness and right side decreases it.🤔
 @The_LoudSpeaker [Pressed windows key. Application menu didn't show up.], In 18.04 only openbox manage shortcuts
[19:05] <wxl[m]> To quote agaida, in biology stable has another meaning: dead 
 @The_LoudSpeaker [Also, Idk why brightness keys functions are inverted? The slider also works inve …], Any idea about this @HMollerCl ?
[19:11] <kc2bez> wxl[m]: That might be one of agaida's best quotes.
[19:14] <wxl[m]> You'll find the SRU process painfully long. Needless to say, there are not lots changes in LTS. Which means you get to enjoy old bugs for a long time
 @wxl[m] [<wxl[m]> You'll find the SRU process painfully long. Needless to say, there are …], > painfully long
 nuh uh
 @The_LoudSpeaker [Any idea about this @HMollerCl ?], Nope
[20:45] <wxl> @Guephren i got a really easy job for you. i need a transparent 32x32 PNG to match this guy https://phab.lubuntu.me/source/lubuntu-artwork/browse/ubuntu%252Feoan/src/usr/share/sddm/themes/lubuntu/shutdown.png that is to indicate suspending rather than shutting down. i think something like a pause symbol would suffice (⏸ )
[21:17] -lugito:#lubuntu-devel- [T110: add suspend button to sddm theme] wxl (Walter Lapchynski) just created this task: https://phab.lubuntu.me/T110
[21:19] -lugito:#lubuntu-devel- [T111: create suspend button for sddm theme] wxl (Walter Lapchynski) just created this task: https://phab.lubuntu.me/T111
[21:27] <wxl> @tsimonq2: can we get sddm or at least sddm-theme-lubuntu in phab?
[21:27] -lugito:#lubuntu-devel- [T112: add code to support suspend button] wxl (Walter Lapchynski) just created this task: https://phab.lubuntu.me/T112
 @wxl [<wxl> @tsimonq2: can we get sddm or at least sddm-theme-lubuntu in phab?], sddm-theme-lubuntu is a binary package of lubuntu-artwork
[21:28] <wxl> oh?
 We only import source packages XD
[21:28] <wxl> oh derp
[21:29] <wxl> i don't know how i ignored that
[21:42] -lugito:#lubuntu-devel- [rART590321e0c6e3: add support for suspend button. resolves T112] wxl (Walter Lapchynski) committed: https://phab.lubuntu.me/rART590321e0c6e3
[21:57] <wxl> @tsimonq2: i'm thinking of adding fcitx-{hangul,pinyin,mozc,unikey} to the seed to ensure support of korean, chinese, japanese, and vietnamese. is that silly?
 @wxl [<wxl> @tsimonq2: i'm thinking of adding fcitx-{hangul,pinyin,mozc,unikey} to the …], Only if Cala can thwack it off of the installed system if the user doesn't need it
[21:58] <wxl> how do we determine if they do or don't?
 Packages module language variable something or other?
[21:58] <wxl> harumph.
[21:59] <wxl> but there are folks that will use multiple input methods
[21:59] <wxl> and some use input methods that their locales, keyboards, and languages, would not suggest
[22:00] <wxl> maybe i should give this to the global team to figure out XD
[22:12] <kc2bez> Yeah, not the cala team XD
[22:25] -lugito:#lubuntu-devel- [T113: figure out fcitx language support] wxl (Walter Lapchynski) just created this task: https://phab.lubuntu.me/T113
[22:33] <wxl> @tsimonq2: please respond to this https://phab.lubuntu.me/D30#664
[22:33] -lugito:#lubuntu-devel- [Needs Revision] added screengrab: https://phab.lubuntu.me/D30#664
[22:54] -lugito:#lubuntu-devel- [rSEEDe0d400c0171d: remove fcitx-ui-qimpanel] wxl (Walter Lapchynski) committed: https://phab.lubuntu.me/rSEEDe0d400c0171d
[23:12] -lugito:#lubuntu-devel- [T59: fix keyboard shortcuts] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T59#2088