#nemomobile log for Wednesday, 2020-03-25

T42<neochapay> (Photo, 800x451) https://irc.thaodan.de/.imgstore/Vx3Y0CaKFA.png08:15
PureTryOut[m]Well fixed the libiodata problem I was having, a missing include09:32
PureTryOut[m]I got multiple fixes for multiple packages, but I have no way to create an account on https://git.sailfishos.org. How would I submit them?09:32
PureTryOut[m]Or at least I don't know where to register for a SailfishOS LDAP account09:33
DylanVanAsscheAFAIK, you have to ask someone on IRC to create it for you? `lbt` manages the Mer repos09:33
lbtyeah - let me know your username and email09:35
PureTryOut[m]Ah thanks. Username would be "PureTryOut" and email "bribbers@disroot.org"09:36
PureTryOut[m]And I see no place to create issues, but I see some references to an external issue tracker, is that right?09:36
DylanVanAsschebugs.sailfishos.org is a Bugzilla instance, same credentials as git.sailfishos.org09:37
PureTryOut[m]Cool thanks09:37
DylanVanAsschelbt: However, Bugzilla seems down atm09:37
lbtNo - the only bug reporting is at TJC09:38
PureTryOut[m]Btw who controls the Matrix bridge here? locusf maybe? Could somebody add +freenode:matrix.org to the room flairs?09:40
PureTryOut[m]Cool thanks πŸ‘οΈ09:42
locusfnp09:43
PureTryOut[m]Now just replace the IRC <-> Telegram bridge for a Matrix <-> Telegram bridge and we have a nice looking room on all platforms πŸ˜ƒ09:43
PureTryOut[m]So how exactly should `timed` be compiled? According to the spec file making a symlink of `qmacro.h` to `src/h/timed-qt5` should do it, but that doesn't seem to be the case for me, it just keeps complaining that `<timed-qt5/qtmacro.h>` is `No such file or directory`09:45
PureTryOut[m]Then again, the spec file links it from outside the source directory it seems, no clue where it gets it from09:45
T42<eekkelund> @neochapay [I still wait @eekkelund patches], Finally on it! :)09:51
T42<neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/PUbfmykGS1.webp09:51
PureTryOut[m]Actually, seems I just made the symlink wrong. Works now, but now it complains about failing to find `tzdata.type.h` and `queue.type.h`. Where am I supposed to get those files from? πŸ€” This is the timed package still09:52
T42<aa13q> PureTryOut: you also could look here: https://github.com/nemo-packaging/timed-git (since APKBUILDs are similar)09:52
T42<aa13q> and here https://build.merproject.org/package/show/home:neochapay:mer:mw/timed-qt5 β€” since Sergey recently ported it for rpm-based mer with qt5.1209:52
T42<aa13q> hope that helps :)09:52
PureTryOut[m]Oh PKGBUILDs! Nice thanks that will help. Not necessarily for finding the required deps (Arch leaves a lot out somehow), but it will for build commands and such09:54
PureTryOut[m]In this case I think I already found it, seems there is a tzdata-timed09:54
PureTryOut[m](which has some interesting versioning scheme)09:54
PureTryOut[m]aa13q: note that the license listed in the PKGBUILD of that package is wrong. It's listed now as GPL while it is actually Public Domain. Then again, I don't know what SPDX identifier to use for it09:57
T42<eekkelund> PureTryOut: This is how Asteroid does it on OpenEmbedded side, tzdata-timed is not required for them09:57
T42<eekkelund> https://github.com/AsteroidOS/meta-asteroid/blob/03ad5c25ec1a7eee13498e4b5916c22a6b9166bc/recipes-nemomobile/timed/timed/0001-Fixes-build.patch09:57
PureTryOut[m]eekkelund: ooh that's better, thanks!09:58
PureTryOut[m]eekkelund: do you know if Asteroid has any intention to upstream their patches back to Mer?10:10
T42<neochapay> We still waiting :)10:16
T42<eekkelund> PureTryOut: No, I don't know10:22
T42<abranson> is asteroid still going. it's been very quiet since mid-201910:32
T42<abranson> I know that when the qt5.9 work started on the sailfish lipstick, the asteroid patches were used as a base. so I guess they already are 'upstreamed'10:33
T42<abranson> well, let's say used as 'inspiration'10:33
PureTryOut[m]aa13q: I'm looking at your packaging for lipstick, but you don't seem to have nemo-qml-plugin-systemsettings even though those are required by lipstick. Do you still have lipstick building somehow or are you getting that dep from somewhere else?10:38
PureTryOut[m]Does anyone have any luck compiling libshadowutils? Besides it feeling like a big hack in general, it fails to compile those 3 .c files from shadow because of "No such file or directory" on `<config.h`>10:54
PureTryOut[m]Ok that libshadowutils thing is insane tbh, an out-of-tree attempt to make it into a library πŸ˜• Why has no one tried upstreaming it into regular shadow?11:07
PureTryOut[m]I just can't get it compiling because I have no clue where `config.h` is supposed to come from11:09
T42<aa13q> PureTryOut: still work-in-progress at spare time, didn't managed to build whole lipstick with upstream Qt via alpm packages yet.11:10
T42<aa13q> Talking about libshadowutils, last week I though about the same thing probably: https://git.sailfishos.org/mer-core/nemo-qml-plugin-systemsettings/merge_requests/3#note_4284211:10
PureTryOut[m]Dropping that dep would be great11:12
T42<aa13q> we could provide a patch/MR probably and save it for future upstream tasks I guess πŸ€”11:16
T42<aa13q> I'm not a pro for dropping libshadowutils for the whole system though, should be checked twice for other lipstick deps, etc11:16
PureTryOut[m]Is it used in more places then?11:19
T42<aa13q> not sure πŸ€”11:24
T42<eekkelund> @PureTryOut[m] [Is it used in more places then?], According to meta-nemomobile OpenEmbedded dependency list, it is only used in nemo-qml-plugin-systemsettings11:26
PureTryOut[m]Ah, in that case dropping it shouldn't be that much effort11:26
T42Toby %lastname% was added by: Toby %lastname%11:39
T42<Sam %lastname%> 611:39
T42<neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/JGWti9ioPb.webp11:40
PureTryOut[m]Is a tarball release considered "dirty" for libresource? According to https://git.sailfishos.org/mer-core/libresource/blob/master/build-aux/git-version-gen#L135 it is12:02
PureTryOut[m]But I'm not sure what "dirty" means here.  <version>-dirty is an invalid version for pkconf anyhow12:03
PureTryOut[m]Ah I see the problem. This build environment works in a git repo, and the script just assumes it's dirty because of that, even though the git repo isn't libresource12:17
PureTryOut[m]Will fix locally12:17
T42<aa13q> PureTryOut: not sure is it related, but there's also script for the obs to calculate rpm package version https://github.com/MeeGoIntegration/boss-launcher-webhook/blob/master/src/service/tar_git#L99312:22
T42<aa13q> afaik spec version is replaced according to the script here for example: https://build.merproject.org/package/view_file/home:neochapay:mer:mw/libresourceqt-qt5/_service:tar_git:libresourceqt-qt5.spec?expand=112:22
PureTryOut[m]Neh I found the problem already, the script assumes that if it's working in a git repo, it's the libresource git repo while in this build environment, that is false12:23
PureTryOut[m](`git-version-gen` that is)12:23
T42<aa13q> πŸ‘12:24
PureTryOut[m]A pain in the ass to fix though. I hoped it would be as simple as adding this check https://git.sailfishos.org/mer-core/libresource/blob/master/build-aux/git-version-gen#L82 again on line 131 to line 144, but it seems that doesn't work...12:38
PureTryOut[m]Actually nvm that's just me being dumb, fixed it12:47
T42<aa13q> PureTryOut: talking about timed package licence, thanks for the notice, nemo-packaging contributors should update it to more specific SPDX id, it looks like it's `LGPL-2.1-or-later`:12:47
T42<aa13q> https://git.sailfishos.org/mer-core/libresource/blob/master/rpm/libresource.spec12:47
T42<aa13q> https://git.sailfishos.org/mer-core/libresource/blob/master/COPYING12:47
PureTryOut[m]You sure it's -or-later? Seems -only to me12:48
T42<aa13q> hm, my bad!12:53
PureTryOut[m]But yeah it'd be nice if they used more specific SPDX id's12:54
T42郝秋棉 %lastname% was added by: 郝秋棉 %lastname%13:18
PureTryOut[m]Hmm seems https://git.sailfishos.org/mer-core/libmlocale/blob/master/configure#L339 doesn13:23
PureTryOut[m]*doesn't work, warnings are still being treated as errors13:23
PureTryOut[m]Instead I just fixed the deprecated function warnings lol. Will upstream13:45
PureTryOut[m]So... Much... Stuff... To package....14:01
T42<aa13q> true :) only once though, should be much easier with an updates :)14:02
PureTryOut[m]Definitely14:03
PureTryOut[m]Although then I have to start keeping track of package updates14:03
PureTryOut[m]Which'll be interesting...14:03
PureTryOut[m]How would you handle that actually? I know there are services like repology which help, but that requires other distros to have packaged a newer version than you already14:04
PureTryOut[m]Oh fun, nemo-qml-plugin-systemsettings hard depends on PackageKit, but ofc that doesn't support the Alpine package manager and is a deprecated tool as well14:10
T42<aa13q> also keep in mind that packagekit-qt wrapper have some unfortunate project stucture and have problems with qmake builing at least :)14:15
T42<aa13q>  https://github.com/hughsie/PackageKit-Qt/issues/3214:15
T42<aa13q> with cmake it is possible to build apper app from kde, for example, but includes are "broken" (there's no `<PackageKit/something...> just some file names that is luckily building)14:15
T42<aa13q> alpine support for the packagekit could be probably ported from alpm backend: https://github.com/hughsie/PackageKit/tree/master/backends/alpm14:17
PureTryOut[m]We don't really want to add Alpine support to packagekit tbh14:17
T42<aa13q> anyway, that dependency should be optional not hardcoded imho14:17
PureTryOut[m]PackageKit is outdated and doesn't get new features/backends14:17
PureTryOut[m]Instead the maintainer now recommends to add direct support to the individual packaging systems in applications that need it14:17
PureTryOut[m]Yeah agreed14:17
PureTryOut[m]For now I'm just seeing if I can remove it entirely, will try to make it optional later I guess14:18
PureTryOut[m]Got my first Mer MR merged though πŸŽ‰ https://git.sailfishos.org/mer-core/libmlocale/merge_requests/1014:19
T42<aa13q> cool! :)14:19
PureTryOut[m]> tmw it's easier to fix the code rather than changing the build configuration14:20
T42* aa13q received some dejavu flashback since he definetly patched qGreater and qSort somewhere else too14:21
PureTryOut[m]Any clue what sailfishusermanager is?14:22
T42<aa13q> sailfish-mdm packages probably14:24
T42<aa13q> https://sailfishos.org/develop/docs/sailfish-mdm/14:24
PureTryOut[m]>  PKGCONFIG += sailfishpolicy sailfishmdm14:24
PureTryOut[m]That doesn't seem to be it14:24
PureTryOut[m]In this case it's PKGCONFIG += sailfishusermanager14:25
T42<aa13q> sailfishusermanager14:27
ol<PureTryOut[m] "PackageKit is outdated and doesn"> Then it needs a new maintainer. Dropping this useful piece of software just becase current maintainer lost interest is counterprosuctive, IMHO.14:28
PureTryOut[m]Yeah there is no such package14:28
PureTryOut[m]Maybe, but so far no one is stepping up to do it14:29
PureTryOut[m]Then again, as far as nemo-qml-plugin-systemsettings goes, the bit that needs PackageKit is really SailfishOS specific14:30
PureTryOut[m]For enabling/installing the developer mode14:30
T42<aa13q> about packagekit and similar projects in general: sound like the chicken or the egg dilemma for me: nobody iterested because nobody started to contribute :) In best-case it could be solved by start contributing step-by-step small iterations (for example, porting from alpm backend :)14:34
T42<aa13q> anyway, yes, it could be dropped in the mentioned case with devmode and patched as optional dependency.14:34
T42<aa13q> PureTryOut: is https://github.com/sailfishos/user-managerd the thing that you searched for the sailfishusermanager, right?14:35
PureTryOut[m]I couldn't find it but yes I guess that's it14:36
PureTryOut[m]Why is that not on https://git.sailfishos.org/mer-core? πŸ€”14:36
DylanVanAsscheI guess that's not part of `mer`, but `sailfish os` specific?14:37
PureTryOut[m]But a part of Mer is depending on it lol14:37
T42<aa13q> PureTryOut: not sure, but probably some historical reasons, guys from jolla could share some details (if they didn't done it already, maybe it's somewhere at the wiki / irc logs)14:39
DylanVanAsscheI don't understand the organisation either πŸ˜›14:39
T42<aa13q> @PureTryOut[m] [Got my first Mer MR merged though πŸŽ‰ https://g …], found the reason of my dejavu πŸ˜… https://git.sailfishos.org/aa13q/libmlocale/commit/44df7696a3388359b359e8ef29c83b7db0249a1d β€” I've tried the same thing last week, but didn't managed to upstream yet! Great that it's already in master! β˜ΊοΈπŸ‘14:46
PureTryOut[m]Haha nice14:47
PureTryOut[m]Hmm, what version of OpenSSL is used in SailfishOS/Nemo Mobile? `nemo-qml-plugin-systemsettings` fails to compile with 1.1.1d at least15:04
PureTryOut[m]`error: invalid use of incomplete type 'X509' {aka 'struct x509_st'}`15:05
T42<eekkelund> PureTryOut: https://github.com/AsteroidOS/meta-asteroid/commit/cbdf72747e5362b33c74aef3059f1ef024f7edcc ?15:08
PureTryOut[m]I like how that fix was mostly removing stuff. Thanks, I'll try it out!15:09
T42<eekkelund> Fix for not packaging ssusysinfo15:12
T42<eekkelund> https://github.com/AsteroidOS/meta-asteroid/blob/master/recipes-nemomobile/nemo-qml-plugins/nemo-qml-plugin-systemsettings/0001-Disable-SSU-dependency.patch15:12
T42<eekkelund> πŸ˜ƒ15:12
T42<eekkelund> But mer-version of usb-moded needs ssusysinfo, upstream version doesn't. For OpenEmbedded Nemo those(nemo-qml-plugin-systemsettings and usb-moded) are two packages that rrequires ssu-sysinfo15:12
T42<neochapay> @eekkelund [Fix for not packaging ssusysinfo15:14
T42<neochapay> https://githu …], not good way...maybe add read this parametrs from ini like /etc/hw.ini ?15:14
PureTryOut[m]What's wrong with ssu-sysinfo now? Why would I remove that?15:15
T42<neochapay> yea...what ?15:16
T42<eekkelund> Nothing wrong, just another example of 'fixing' by removing :)15:22
T42<neochapay> @eekkelund [Nothing wrong, just another example of 'fixing …], where MR15:23
T42<neochapay> hehe15:25
T42Queena %lastname% was added by: Queena %lastname%15:28
PureTryOut[m]I don't get why they haven't upstreamed their changes15:47
T42<aa13q> you've meant AsteroidOS guys? Probably just didn't have enough resourses for that. Sailfish is still at qt5.6 for now due to open lgpl2->3 transition (afaik), so at that moment upstreaming everything wasn't a high priority task for the both sides (just my guess)15:50
PureTryOut[m]Well I've finally packaged everything up to lipstick. Now just waiting on a version of lipstick that works on Qt 5.14 πŸ˜ƒ15:50
T42<aa13q> awesome!15:50
T42<aa13q> runtime fun is the other part thought :)15:51
PureTryOut[m]Well yeah, let's worry about that later lol15:51
PureTryOut[m]For now I just completely patched out developer mode from systemsettings. It's very SailfishOS specific anyway15:51
T42<aa13q> Got it! Do you have a public repo link so I could translate missing part to pkgbuilds too? hehe ☺️15:53
T42<aa13q> would be great to check it on arch/manjaro systemd side :)15:54
T42<AJSlye> @aa13q [Got it! Do you have a public repo link so I co …], Keep me updated to your progress. I've tried compiling what we have so far and ran into issues as well. Also if we have any PKGBUILD that are no longer needed we need to mark them depriciated in the README.MD.15:56
T42<aa13q> @AJSlye [Keep me updated to your progress. I've tried c …], β˜ΊοΈπŸ‘Œ15:56
PureTryOut[m]aa13q: yes of course, the branch is here https://gitlab.com/postmarketOS/pmaports/-/tree/feature/mer15:57
T42<neochapay> can you write article about progres for nemomobile-ux.github.io/15:57
T42<aa13q> thank you!15:58
PureTryOut[m]neochapay: shall I write about packaging Glacier for postmarketOS? I guess I can do, but so far there isn't much useful stuff yet. Haven't got Lipstick compiling yet due to the Qt thing 😜15:59
T42<neochapay> prorgess it's good news hehe ^_^16:00
PureTryOut[m]Still I rather wait till I got at least Lipstick compiling, and preferably the majority of Glacier. There is not much to write atm16:04
PureTryOut[m]Although I can make a WIP post for now maybe16:04
T42<eekkelund> What is wrong with lipstick?16:05
T42<neochapay> just write about progress - it's goot - see how somethinkg is bring up16:07
PureTryOut[m]eekkelund: it won't compile with Qt 5.14 😜16:12
PureTryOut[m]Also right now to compile on Qt 5.12 it needs to revert a commit in Qt5Wayland16:13
PureTryOut[m]neochapay: ok will do. I can also write something about the current blockers for getting it merged into postmarketOS then16:13
malPureTryOut[m]: a small request, if you make more of those libsystemd related PRs to mer-core please also check the build requirements in the spec file and change those in case there are old ones16:13
PureTryOut[m]aa13q: I made https://git.sailfishos.org/mer-core/nemo-qml-plugin-systemsettings/merge_requests/135 in response to your comment16:13
PureTryOut[m]Oh sure will do16:15
PureTryOut[m]Did I already miss something?16:15
malPureTryOut[m]: https://git.sailfishos.org/mer-core/nemo-qml-plugin-devicelock/blob/master/rpm/nemo-qml-plugin-devicelock.spec#L1516:16
PureTryOut[m]Ah woops will fix that in a sec16:20
PureTryOut[m]mal: https://git.sailfishos.org/mer-core/nemo-qml-plugin-devicelock/merge_requests/4716:34
malthanks16:36
PureTryOut[m]neochapay: https://github.com/nemomobile-ux/nemomobile-ux.github.io/pull/217:16
malPureTryOut[m]: also in the future you can do multiple changes in same PR, related to the two PRs in nemo-qml-plugin-systemsettings, just do separate commits for those if they are unrelated, no need to change those now, just a suggestion for any new PRs17:27
malof course if the changes are big then separate PRs can be ok17:28
PureTryOut[m]I rather not, I keep changes separate from each other. Of course there are cases where you have multiple related commits, in which case I'll MR them together, but I'll definitely not MR unrelated changes together17:29
malok17:29
malin this case the other fix was a simple addition of header include which is why I thought it could have been in the same PR17:30
PureTryOut[m]Yeah I realize that, but I don't consider the size of a MR as a reason to combine any πŸ˜‰17:31
malhttps://git.sailfishos.org/mer-core/usb-moded/merge_requests/57/diffs is the #ifndef TEMP_FAILURE_RETRY intentionally outside the #ifndef  USB_MODED_MODESETTING_H_17:32
PureTryOut[m]Not really, I didn't know where to put it tbh17:32
T42<eekkelund> @neochapay some bugs still with glacier-home after rebasing. I think I will manage to sort those tomorrow and will send MR18:31
T42<neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/DPL60Sj0td.webp18:49
T42<neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/EIGYzcdsmd.webp18:49
T42<neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/Z6nRvET8PS.webp18:49
T42<neochapay> PureTryOut add logo plz18:59
PureTryOut[m]Uh, sure, I'll get it from somewhere19:05
* PureTryOut[m] uploaded an image: Screenshot_20200325_200530.png (305KB) < https://matrix.org/_matrix/media/r0/download/fam-ribbers.com/CgHmWFWqXawwLuuuhrYHJQYJ >19:05
PureTryOut[m]Uh, wrong room sorry19:06
PureTryOut[m]neochapay: where to put it on the post? πŸ€”19:10
T42<neochapay> PureTryOut look here https://github.com/nemomobile-ux/nemomobile-ux.github.io/commit/603ad5ffa903c69d4babf7b6d70170e4032aa8c319:19
PureTryOut[m]neochapay: ah thanks, done19:28
PureTryOut[m]I was afraid of that, the author doesn't show up 😒21:24
PureTryOut[m]Now it's not clear who has written the post21:25
T42<neochapay> I fix it tomorrow21:25
PureTryOut[m]https://github.com/nemomobile-ux/nemomobile-ux.github.io/pull/321:56
PureTryOut[m]That should fix it21:56
*** verin0x2 is now known as verin0x23:06

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!