 @tsimonq2 [@HMollerCl https://phab.lubuntu.me/T97], you mean 79 chars line? Or other?
[00:41] -lugito:#lubuntu-devel- Error: T97], is an invalid task reference.
 @HMollerCl [you mean 79 chars line? Or other?], Want me to go through it?
 i have a pep8 compliance checker
 can dump you the list of chaos
 that would be nice
 `black .`
 Problem solved
 thanks!
 @teward001 [i have a pep8 compliance checker], There's a Python module for this already
 *shot*
 yes there is :P
 use flake8
 flake8 is the module?
 @HMollerCl [you mean 79 chars line? Or other?], unrelated, the 79chars line isn't a forced PEP8 compliance
 with the dev team in agreement it can be extended to 120
 Just run black on it, really. :P
 88 chars, but well who cares
 https://github.com/psf/black
 sorry, 100 is the number I meant
 To quote PEP8:
 > Some teams strongly prefer a longer line length. For code maintained exclusively or primarily by a team that can reach agreement on this issue, it is okay to increase the nominal line length from 80 to 100 characters (effectively increasing the maximum length to 99 characters), provided that comments and docstrings are still wrapp
 @tsimonq2 just to be sure, you ran lintian agains the code in the PPA or the one in the repo https://phab.lubuntu.me/source/lubuntu-update-notifier/
 wow i can't see that repo heh
 makes sense.
 @HMollerCl [@tsimonq2 just to be sure, you ran lintian agains the code in the PPA or the one …], The one in the repo
 @teward001 [wow i can't see that repo heh], You can't??
 hmm
 > Users with the "Can View" capability: … Members of the project "Development Team" can take this action.
 means you didn't give me Phab rights
 @tsimonq2 [You can't??], is private
 Fixed
 now i can see it :)
 thanks Simon
 is no longer private
 @tsimonq2 I don't see a .git dir in the repo.
 W: lubuntu-update-notifier source: diff-contains-git-control-dir .git
 You take diffs, @tsimonq2?
 i believe that's handled during cleaning to clean it Hans
 during the build process (don't quote me tho)
 ewwwwww
 EWWWWWWWWW
 you want to see the pep8 violations Hans?
 @HMollerCl [@tsimonq2 I don't see a .git dir in the repo.], That's because it's part of the repo once you clone it :P
 @UniversalSuperBox [You take diffs, @tsimonq2?], Hmm?
 @teward001 [i believe that's handled during cleaning to clean it Hans], Yes
 @teward001 [you want to see the pep8 violations Hans?], hahahahahaha
 Apply this and try again @teward001 :P
 so you fixed the 100+ PEP8 violations regarding indentation using tabs and not spaces?
 messes with diff
 Maybe?
 Yes, it appears to have
 ah, yes, you did
 there's other PEP8 violations though
 Depending on your flake8 config
 not using flake8 on this one
 using PyCharm inbuild
 oooo, I thought that linesbreak where much clearer with tan than spaces......
 Well, that was my problem, I thought... should have search for it.
 notifier.py, line 51: `if self.upg_path == None:` should be `if self.upg_path is None:`
 which is a pep8 rule
 line wrap at 100 is the max allowed by PEP8 as well
 function names should be lowercase and variables should be lowercase whne function local
 but those're the lower ones
 Four spaces as indentation please ;)
 yep
 unrelated also
 here's the flake8 dump with defaults in it: https://paste.ubuntu.com/p/3snGYS3VJF/
 @teward001 [notifier.py, line 51: if self.upg_path == None: should be if self.upg_path is No …], "== None" should be always is "None ??"
 lots of E501s
 @HMollerCl ["== None" should be always is "None ??"], Yes, since equality to None is not necessarily defined
 @HMollerCl ["== None" should be always is "None ??"], None should not be equality-operatored.  `if var is None` is the proper way to check NoneType being the type
 because None isn't always defined in equality checks
 Python recognizes it and can handle it but it's a coding violation
 thanks, good to know.
 there's some unused imports too
 a bunch
 unused local vars too
 but most of your problem are strings that need linewrapped down
 if the Lubuntu team agrees
 then you can amend it to 100 as the length for wrap
 but you should not go beyond 100 (PEP8)
 tahnks, with what did you the checks?
 flake8
 it has his pakcahes
 packages
 `cd /temp/lubuntu-update-notifier && sudo pip3 install flake8 && python3 -m flake8 .`
 hmoller@xpsM1210:~/lubuntu-phab/lubuntu-update-notifier$ apt list *flake8* … Listing... Done … flake8/eoan,eoan 3.6.0-1 all … python-flake8/eoan,eoan 3.6.0-1 all … python3-flake8-docstrings/eoan,eoan 1.1.0-1 all … python3-flake8-polyfill/eoan,eoan 1.0.2-1 all … python3-flake8/eoan,eoan 3.6.0-1 all
 my system here is Bionic
 and my Python is 3.7.4 (local pyenv deployment) so having the newest flake from PyPI solved that headache
 but the PEP8 violations still are a thing so :)
 there's also a bug in flake8-docstrings it seems
 @HMollerCl as for Lintian checks...
 it cleared source build checks, running Eoan sbuild now with Lintian
 @teward001 [it cleared source build checks, running Eoan sbuild now with Lintian], I already gave him a full `-EvIL +pedantic` Lintian check.
 Fully annotated, in the task
 *shot*
 i'm being thorough lol
 oh what's this, a lintian failure!
 How much more thorough? :P
 W: lubuntu-update-notifier source: diff-contains-git-control-dir .git … P: lubuntu-update-notifier source: file-contains-trailing-whitespace debian/control (line 20) … P: lubuntu-update-notifier source: source-contains-empty-directory .git/branches/ … P: lubuntu-update-notifier source: source-contains-empty-directory .git/objects/in
 well
 the E items should be fixed :P
 I don't get this "W: lubuntu-update-notifier: script-with-language-extension usr/bin/upgrader.py"
 @teward001 [the E items should be fixed :P], All of them should
 :P
 @HMollerCl [I don't get this "W: lubuntu-update-notifier: script-with-language-extension usr …], > When scripts are installed into a directory in the system PATH, the script name should not include an extension such as .sh or .pl that denotes the scripting language currently used to implement it. The implementation language may 
 @tsimonq2 [All of them should], yes, you're not wrong.
 but the copyright file issues are the larger ones 😜
 @HMollerCl if you don't get what the tag means, https://lintian.debian.org/tags-all.html - usually self explanatory for most of the detections
 ^^
 thanks @teward001
 and apparently debhelper's clean in the rules doesn't purge `.git` currently 😐
 @teward001 [> When scripts are installed into a directory in the system PATH, the script nam …], what to they mean exactly with "system PATH"
 ?
 yes.  what the default $PATH variable is
 of which /usr/bin is default in in Debian and Ubuntu systems
 see `/etc/environment/ and its `PATH` definition.  This is mine on Bionic: `PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games`
 @kc2bez I don't suppose you've made any more progress tonight?
[04:15] <kc2bez> @tsimonq2 Been reading the docs and what you sent. Was on BDLL too. Here is where I am at: `arc land --onto ubuntu/eoan` then `debuild -S` then `dput ubuntu ../lubuntu-default-settings_19.10.2_source.changes`
[04:15] <kc2bez> Little bugaboo the version in the archive doesn't match phab.
 AHA.
 wxl's fault XD
[04:20] <tsimonq2> kc2bez: I'll merge in wxl's changes, merge your diff, and then you can push from there.
[04:21]  * tsimonq2 slaps wxl[m] with a wet fish
[04:23] <tsimonq2> Wait a minute
[04:23] <tsimonq2> O_o
 @tsimonq2 copyright thing is awfull
[04:23] <tsimonq2> It's the most awful thing in packaging, I swear.
[04:23] <tsimonq2> Copyright is just stupid.
[04:24] <tsimonq2> Okay, I'm confused. What did wxl mess up... :P
[04:27] <kc2bez> ¯\_(ツ)_/¯
[04:27] -lugito:#lubuntu-devel- [rDEFAULTSETTINGSd7e494fb108c: Adding back translations.] tsimonq2 (Simon Quigley) committed: https://phab.lubuntu.me/rDEFAULTSETTINGSd7e494fb108c
[04:28] <tsimonq2> Okay, so what he had was a messed up local copy.
[04:28] <tsimonq2> According to the VCS, it's just adding a changelog entry.
[04:28] <tsimonq2> Well job, wxl.
[04:28] <tsimonq2> Anyway, enough making fun of wxl. XD
[04:30] <tsimonq2> Let's see if this hacky hack will work.
[04:32] -lugito:#lubuntu-devel- [rDEFAULTSETTINGS744503987915: Add more mimetypes.] tsimonq2 (Simon Quigley) committed: https://phab.lubuntu.me/rDEFAULTSETTINGS744503987915
[04:32] <tsimonq2> Bleeeh, it has to put me as the author.
[04:32] <tsimonq2> Whatevs.
[04:33] <tsimonq2> @kc2bez: Alright, go for it. Tip of ubuntu/eoan should be all set.
[04:33] <tsimonq2> debuild -S -d
[04:33] <tsimonq2> Then dput ubuntu ../lubuntu-default-settings_19.10.3_source.changes
[04:34] <kc2bez> Alright, I will clone that in a fresh working directory.
[04:34] <tsimonq2> Sounds goos.
[04:34] <tsimonq2> *good
[04:42] <tsimonq2> kc2bez: How's that coming along for you?
[04:42] <kc2bez> good
[04:43] <kc2bez> about to dput
[04:43] <tsimonq2> I'm waiting in anticipation :D
[04:45] <kc2bez> I couldn't tell XD
[04:45] <kc2bez> I got an email
[04:47] <tsimonq2> So did I!
[04:48] <tsimonq2> Look at that: https://launchpad.net/ubuntu/+source/lubuntu-default-settings/19.10.3
[04:48] <tsimonq2> Nice work!
[04:48] <tsimonq2> Congrats on your first upload :)
[04:48] <kc2bez> \o/
[04:48] <kc2bez> Couldn't do it without you.
[04:48] <tsimonq2> :D
[04:49] <tsimonq2> Next time should be slightly easier ;)
[04:49] <kc2bez> next I need to git tag or no?
[04:49] <tsimonq2> Yes
[04:49] <tsimonq2> So git tag ubuntu/19.04.3 or something similar
[04:49] <tsimonq2> And then git push --tags
[04:52] <kc2bez> Ok, should be done too. 
[04:53] <tsimonq2> Can confirm :)
[04:53] <kc2bez> Sweet!
[04:54] <tsimonq2> :D
[04:55] <tsimonq2> kc2bez: Do you want to try merging Calamares from Debian when you get a chance?
[04:56] <kc2bez> What is in Debian? We should be on the latest version.
[04:57] <tsimonq2> It's the latest release, but it's more of a packaging exercise.
[04:58] <kc2bez> Sure. Maybe not tonight though :)
[04:58] <tsimonq2> If you get merges from Debian down to a science, I can point you at plenty of LXQt packages. ;)
[04:58] <tsimonq2> Sure, that's no problem :)
[04:58] <kc2bez> Yeah I would like to. 
[04:59] <tsimonq2> It'll certainly get you more comfortable uploading to the archive. :D
[04:59] <kc2bez> More experience will help for sure.
[05:00] <tsimonq2> And if anyone wants it, it's certainly not exclusive to Dan (none of this is, in fact, it'd be cool to get you to sponsor something eventually), libfm-qt is a really fun symbols puzzle.
[05:01] <tsimonq2> - _ZNSt8_Rb_treeIjjSt9_IdentityIjESt4lessIjESaIjEE16_M_insert_uniqueIjEESt4pairISt17_Rb_tree_iteratorIjEbEOT_@Base 0.14.1
[05:01] <tsimonq2> + _ZNSt6vectorIN2Fm8FilePathESaIS1_EED1Ev@Base 0.14.1+git201907270128~disco-0ubuntu1~ppa1
[05:01] <tsimonq2> + _ZNSt6vectorIN2Fm8FilePathESaIS1_EED2Ev@Base 0.14.1+git201907270128~disco-0ubuntu1~ppa1
[05:01] <tsimonq2> + _ZNSt6vectorISt4pairISt10shared_ptrIKN2Fm8FileInfoEES5_ESaIS6_EE17_M_realloc_insertIJS6_EEEvN9__gnu_cxx17__normal_iteratorIPS6_S8_EEDpOT_@Base 0.14.1+git201907270128~disco-0ubuntu1~ppa1
[05:01] <tsimonq2> + _ZNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEE12_M_constructIPKcEEvT_S8_St20forward_iterator_tag@Base 0.14.1+git201907270128~disco-0ubuntu1~ppa1
[05:01] <tsimonq2> +#MISSING: 0.14.1+git201907270128~disco-0ubuntu1~ppa1# _ZNSt8_Rb_treeIjjSt9_IdentityIjESt4lessIjESaIjEE16_M_insert_uniqueIjEESt4pairISt17_Rb_tree_iteratorIjEbEOT_@Base 0.14.1
[05:01] <tsimonq2> The error is only present on Disco.
[05:01] <kc2bez> ewww
[05:02] <tsimonq2> >:D
[05:02] <tsimonq2> Hey RikMills, want a break from PIM? :P
[05:02]  * tsimonq2 runs
[05:03] <kc2bez> Thanks for your help tsimonq2
[05:03] <tsimonq2> No problem at all!
[05:03] <tsimonq2> Have a great night
[05:04] <kc2bez> Thanks, morning now though ;)
[05:04] <tsimonq2> True ;)
[08:56] -lugito:#lubuntu-devel- [T90: Bionic locker should be light-locker] apt-ghetto (apt-ghetto) commented on the task: https://phab.lubuntu.me/T90#1570
 "The error is only present on Disco." … That smells like a compiler version issue
 @tsimonq2 [<tsimonq2> Hey RikMills, want a break from PIM? :P], To be honest, I have given up caring about PIM symbols. Just ignoring them and rebuilding the 5 or 6 extenal packages that use the PIM ABI each time seems less work
[14:02] <The_LoudSpeaker> wxl: did you receive my email ? 
[19:02] -lugito:#lubuntu-devel- [T21: Our xscreensaver theme is too old] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T21#1571
[20:06] -lugito:#lubuntu-devel- [T91: Update mimeapps.list] kc2bez (Dan Simmons) commented on the task: https://phab.lubuntu.me/T91#1573
[20:51] <wxl[m]> Is phab just spinning away for anyone else?
[20:52] <kc2bez> I'm in.
[20:52] <kc2bez> Which part in particular?
[20:53] <wxl[m]> Nevermind I guess. Wherever I'm at is in some weird internet black hole
[20:53] <kc2bez> I hate that
[20:53] <wxl[m]> Or maybe Firefox issue
[20:55] <kc2bez> I use firefox but mobile networks can be a challenge.
[21:02] -lugito:#lubuntu-devel- [rUPDATENOTIFIER930acffb78ae: fixed PEP8] hmollercl (Hans P. Möller) committed: https://phab.lubuntu.me/rUPDATENOTIFIER930acffb78ae
[22:33] -lugito:#lubuntu-devel- [rMANUALc1e8dba98759: Fix styling] lynorian (Lyn Perrine) committed: https://phab.lubuntu.me/rMANUALc1e8dba98759
[22:36] -lugito:#lubuntu-devel- [rMANUALbdc3aa2417cf: Add destination need to accept] lynorian (Lyn Perrine) committed: https://phab.lubuntu.me/rMANUALbdc3aa2417cf
[22:41] -lugito:#lubuntu-devel- [T21: Our xscreensaver theme is too old] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T21#1574
[23:10] -lugito:#lubuntu-devel- [T21: Our xscreensaver theme is too old] wxl (Walter Lapchynski) commented on the task: https://phab.lubuntu.me/T21#1575
[23:47] <guiverc> kc2bez, ack on t91  (mimetypes)
[23:47] <kc2bez> Thanks guiverc 
[23:53] <wxl> i got to finish some things but it's on my TODO, too
[23:53] <wxl> i think i've exhausted myself on screensavers/locks for one day
[23:54] <kc2bez> Thanks wxl I appreciate it.