[03:47] wxl: @kc2bez I am unable to edit D32 [03:47] why not? [03:48] arc patch D32 --nobranch is giving me errors [03:48] grab a new repo then do that [03:48] raman@Aspire-ES1-132:~/Style/Work/work2/libfm-qt$ arc patch D32 [03:48] Exception [03:48] [cURL/35] (https://phab.lubuntu.me/api/differential.querydiffs) 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] (Run with `--trace` for a full exception trace.) [03:48] raman@Aspire-ES1-132:~/Style/Work/work2/libfm-qt$ [03:48] oh [03:48] well [03:48] that's a network problem [03:48] Frikking sophos [04:05] wxl: https://phab.lubuntu.me/P32 [04:08] without all that noise, what does it say? [04:09] same thing [04:09] if you just run `arc diff` it shows ALL of that? [04:09] aman@Aspire-ES1-132:~/libfm-work/libfm-qt$ arc diff [04:09] Select a Default Commit Range [04:09] You're running a command which operates on a range of revisions (usually, [04:09] from some revision to HEAD) but have not specified the revision that should [04:09] determine the start of the range. [04:09] Previously, arc assumed you meant 'HEAD^' when you did not specify a start [04:09] revision, but this behavior does not make much sense in most workflows [04:09] outside of Facebook's historic git-svn workflow. [04:09] arc no longer assumes 'HEAD^'. You must specify a relative commit explicitly [04:09] when you invoke a command (e.g., `arc diff HEAD^`, not just `arc diff`) or [04:09] select a default for this working copy. [04:09] In most cases, the best default is 'origin/master'. You can also select [04:09] 'HEAD^' to preserve the old behavior, or some other remote or branch. But you [04:09] almost certainly want to select 'origin/master'. [04:09] don't do that jeez [04:09] (Technically: the merge-base of the selected revision and HEAD is used to [04:09] determine the start of the commit range.) [04:09] What default do you want to use? [origin/master] [04:09] Exception [04:09] Command failed with error #128! [04:09] COMMAND [04:09] git cat-file -t 'origin/master' [04:10] STDOUT [04:10] (empty) [04:10] STDERR [04:10] fatal: Not a valid object name origin/master [04:10] (Run with `--trace` for a full exception trace.) [04:10] should I do arc diff HEAD^ ? [04:10] no [04:12] what should I do now? [04:13] what's `git remote -v` give you? [04:14] https://phab.lubuntu.me/source/libfm-qt.git for both fetch and push [04:14] that's the problem [04:14] you need to use ssh [04:14] ughh! [04:14] I was using git coz sophos [04:14] ssh is blocked on 2222 [04:15] sorry [04:16] lite. [04:19] tsujan 's email id ploxx [04:19] also full name? [04:19] it's in the original patch dude [04:20] both are [04:20] you'll slap yourself in the head when you see his name [04:20] got it! [04:22] 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] verbose is good. unnecessary is not. [04:22] Yes, blame wxl [04:22] XD [04:22] go cough on something, @tsimonq2 ! [04:22] Hahahahaha [04:24] @tsimonq2: why ARE we using 2222? [04:25] It was a default [04:25] hm [04:25] We should make sure it doesn't interfere with SSHing in though [04:25] wxl: D32 [04:27] wxl: Should I change the author in debian changelog also? [04:27] no [04:28] okay [04:31] -lugito:#lubuntu-devel- [rUPDATENOTIFIERc7e1f1ab58c5: added version and in help] Hans P Möller committed: https://phab.lubuntu.me/rUPDATENOTIFIERc7e1f1ab58c5 [04:31] how do I Get rid of the changes to debian/patches/workaround-missed-file-monitoring.patch ? [04:32] copy it before arc patch D32 and then paste it later? [04:33] git checkout [04:34] or maybe `git reset -- ` [04:44] check now. [04:46] @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 [04:49] . [04:49] 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] OH [04:51] one other thing [04:51] when you're doing arc diffs.. reference the task [04:51] you can edit the summary and add a T99 [04:51] Ah yes, went over my head too [04:51] it will auto close things [04:52] so since you probably don't read your mail: read the task about SRUing, @tsimonq2 [04:52] okay. Noted. [04:52] so go edit :) [04:53] @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 [04:53] tl;dr consult the docs and see if it qualifies [04:54] If it doesn't, the workaround should be documented in the manual [04:54] wxl: done! [04:54] Or a good exercise for Raman :) [04:54] it's not that it doesn't qualify [04:55] https://wiki.ubuntu.com/StableReleaseUpdate @The_LoudSpeaker [04:55] it's not high impact so that's out [04:56] (1) have an obviously safe patch and (2) affect an application rather than critical infrastructure packages (like X.org or the kernel). [04:56] @tsimonq2, I don't I will be able to read that today. or tommorow. Monday possible. [04:56] seems to meet that [04:56] so yeah SRU away @The_LoudSpeaker [04:56] do it after 19.10, though [04:56] okay but some time ploxxx. [04:57] @wxl [ do it after 19.10, though], What's the point? [04:57] How does SRU after 19.10 makes sense? [04:57] ^ :) [04:57] We will directly include it in 19.10 na? [04:57] what you just did is 19.10 [04:57] SRU is for 19.04 [04:57] we can worry about that after 19.10 is out [04:57] we need you for all of THAT XD [04:57] Right, but we're encouraging people to install 19.10 after it's released [04:58] That's like two months away [04:58] workaround exists [04:58] not to mention the bug has been there for months already [04:58] the you mean SRU in 19.04 right? [04:59] *then [04:59] yep [04:59] Okay, so we document the workaround :) [04:59] Problem solved [04:59] okay. I will add it to todo [04:59] i mean we just add the bug to the release notes and we're done [04:59] cuz the bug has documented the workaround for a long time [04:59] but @tsimonq2 's idea seems more interesting. [04:59] Mmkay [05:00] huh? [05:00] what ideA? [05:00] @The_LoudSpeaker you should read the SRU docs anyway :) [05:00] I mean like if we document the workaround then we don't need to sru right? [05:00] Yes [05:00] we don't need to [05:00] we could though [05:00] wouldn't be a bad idea [05:00] @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 [05:02] The SRU docs are in the Lubuntu Developers page, right? [05:02] And @HMollerCl @aptghetto are going to apply like yesterday, right? :P [05:02] ruh roh [05:03] patch unexpectedly ends in middle of line [05:03] Oh that's fun [05:03] hunk #3 failed at 445 [05:03] which patch? [05:03] @wxl [ hunk #3 failed at 445], Where are you seeing this ooc? [05:03] fix-dnd-mtp onr? [05:03] debuild -S [05:03] yep [05:04] it applied cleanly before. strange. i even tried it and it worked fine. [05:04] It didn't fail when I used earlier. might be failed due to changes in P31 [05:04] it applies but faills see lines 607-610 of P30 [05:05] the issue that seems different is the extra line at the end [05:05] Whaaaaaaatttt? [05:05] You gotta be kidding me. [05:05] nope [05:05] that's fuzz, my friend [05:05] F [05:06] talk to @tsimonq2 about trailing whitespace [05:06] I will have to it again then? [05:06] @wxl [ talk to @tsimonq2 about trailing whitespace], BAD [05:06] Where's the trailing whitespace? [05:06] >_> [05:06] wait [05:06] huh [05:06] i'm wrong [05:06] hold on. [05:06] i'm confused. [05:07] What happened? [05:07] ok that is really weird [05:07] Drop the bomb already! [05:07] the patch i got was missing an empty line [05:07] but on phab it's there [05:07] they should be identical [05:08] P: libfm-qt source: file-contains-trailing-whitespace debian/changelog (line 24) [05:08] Weird. [05:08] I had just done dch -r and changed email id [05:09] well that's the thing [05:09] 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] DUDE WTF [05:20] sheesh [05:21] ohhhhhhhhhhhhh [05:21] 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] 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:03] Kalvin was added by: Kalvin [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 [17:59] I was installing lubuntu 18.04 in a friend's laptop. In live usb rn. [17:59] Pressed windows key. Application menu didn't show up. [17:59] Last I used, it was set to open on pressing windows key. [18:00] Did I miss something? [18:01] ^ @kc2bez @tsimonq2 wxl: [18:01] 18.10 and beyond only [18:01] Tl;Dr no one wants LXDE [18:09] That guy needs a stable system. He doesn't want to update every 6 momths. [18:09] Also, We are supporting 18.04 till next year na? [18:10] That's enough for him. [18:14] 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.🤔 [18:45] @The_LoudSpeaker [Pressed windows key. Application menu didn't show up.], In 18.04 only openbox manage shortcuts [19:05] To quote agaida, in biology stable has another meaning: dead [19:08] @The_LoudSpeaker [Also, Idk why brightness keys functions are inverted? The slider also works inve …], Any idea about this @HMollerCl ? [19:11] wxl[m]: That might be one of agaida's best quotes. [19:14] 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 [19:14] @wxl[m] [ You'll find the SRU process painfully long. Needless to say, there are …], > painfully long [19:14] nuh uh [20:39] @The_LoudSpeaker [Any idea about this @HMollerCl ?], Nope [20:45] @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] @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 [21:28] @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] oh? [21:28] We only import source packages XD [21:28] oh derp [21:29] 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] @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? [21:58] @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] how do we determine if they do or don't? [21:58] Packages module language variable something or other? [21:58] harumph. [21:59] but there are folks that will use multiple input methods [21:59] and some use input methods that their locales, keyboards, and languages, would not suggest [22:00] maybe i should give this to the global team to figure out XD [22:12] 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] @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