T42 | <neochapay> (Photo, 800x451) https://irc.thaodan.de/.imgstore/Vx3Y0CaKFA.png | 08:15 |
---|---|---|
PureTryOut[m] | Well fixed the libiodata problem I was having, a missing include | 09: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 account | 09:33 |
DylanVanAssche | AFAIK, you have to ask someone on IRC to create it for you? `lbt` manages the Mer repos | 09:33 |
lbt | yeah - let me know your username and email | 09: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 |
DylanVanAssche | bugs.sailfishos.org is a Bugzilla instance, same credentials as git.sailfishos.org | 09:37 |
PureTryOut[m] | Cool thanks | 09:37 |
DylanVanAssche | lbt: However, Bugzilla seems down atm | 09:37 |
lbt | No - the only bug reporting is at TJC | 09: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 |
locusf | np | 09: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 from | 09:45 |
T42 | <eekkelund> @neochapay [I still wait @eekkelund patches], Finally on it! :) | 09:51 |
T42 | <neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/PUbfmykGS1.webp | 09: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 still | 09: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.12 | 09: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 such | 09:54 |
PureTryOut[m] | In this case I think I already found it, seems there is a tzdata-timed | 09: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 it | 09:57 |
T42 | <eekkelund> PureTryOut: This is how Asteroid does it on OpenEmbedded side, tzdata-timed is not required for them | 09:57 |
T42 | <eekkelund> https://github.com/AsteroidOS/meta-asteroid/blob/03ad5c25ec1a7eee13498e4b5916c22a6b9166bc/recipes-nemomobile/timed/timed/0001-Fixes-build.patch | 09: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 know | 10:22 |
T42 | <abranson> is asteroid still going. it's been very quiet since mid-2019 | 10: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 from | 11: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_42842 | 11:10 |
PureTryOut[m] | Dropping that dep would be great | 11: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, etc | 11: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-systemsettings | 11:26 |
PureTryOut[m] | Ah, in that case dropping it shouldn't be that much effort | 11:26 |
T42 | Toby %lastname% was added by: Toby %lastname% | 11:39 |
T42 | <Sam %lastname%> 6 | 11:39 |
T42 | <neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/JGWti9ioPb.webp | 11: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 is | 12:02 |
PureTryOut[m] | But I'm not sure what "dirty" means here. <version>-dirty is an invalid version for pkconf anyhow | 12: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 libresource | 12:17 |
PureTryOut[m] | Will fix locally | 12: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#L993 | 12: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=1 | 12: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 false | 12: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 it | 12: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.spec | 12:47 |
T42 | <aa13q> https://git.sailfishos.org/mer-core/libresource/blob/master/COPYING | 12:47 |
PureTryOut[m] | You sure it's -or-later? Seems -only to me | 12:48 |
T42 | <aa13q> hm, my bad! | 12:53 |
PureTryOut[m] | But yeah it'd be nice if they used more specific SPDX id's | 12:54 |
T42 | ιη§ζ£ %lastname% was added by: ιη§ζ£ %lastname% | 13:18 |
PureTryOut[m] | Hmm seems https://git.sailfishos.org/mer-core/libmlocale/blob/master/configure#L339 doesn | 13:23 |
PureTryOut[m] | *doesn't work, warnings are still being treated as errors | 13:23 |
PureTryOut[m] | Instead I just fixed the deprecated function warnings lol. Will upstream | 13: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] | Definitely | 14:03 |
PureTryOut[m] | Although then I have to start keeping track of package updates | 14: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 already | 14: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 well | 14: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/32 | 14: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/alpm | 14:17 |
PureTryOut[m] | We don't really want to add Alpine support to packagekit tbh | 14:17 |
T42 | <aa13q> anyway, that dependency should be optional not hardcoded imho | 14:17 |
PureTryOut[m] | PackageKit is outdated and doesn't get new features/backends | 14:17 |
PureTryOut[m] | Instead the maintainer now recommends to add direct support to the individual packaging systems in applications that need it | 14:17 |
PureTryOut[m] | Yeah agreed | 14:17 |
PureTryOut[m] | For now I'm just seeing if I can remove it entirely, will try to make it optional later I guess | 14:18 |
PureTryOut[m] | Got my first Mer MR merged though π https://git.sailfishos.org/mer-core/libmlocale/merge_requests/10 | 14:19 |
T42 | <aa13q> cool! :) | 14:19 |
PureTryOut[m] | > tmw it's easier to fix the code rather than changing the build configuration | 14:20 |
T42 | * aa13q received some dejavu flashback since he definetly patched qGreater and qSort somewhere else too | 14:21 |
PureTryOut[m] | Any clue what sailfishusermanager is? | 14:22 |
T42 | <aa13q> sailfish-mdm packages probably | 14:24 |
T42 | <aa13q> https://sailfishos.org/develop/docs/sailfish-mdm/ | 14:24 |
PureTryOut[m] | > PKGCONFIG += sailfishpolicy sailfishmdm | 14:24 |
PureTryOut[m] | That doesn't seem to be it | 14:24 |
PureTryOut[m] | In this case it's PKGCONFIG += sailfishusermanager | 14:25 |
T42 | <aa13q> sailfishusermanager | 14: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 package | 14:28 |
PureTryOut[m] | Maybe, but so far no one is stepping up to do it | 14:29 |
PureTryOut[m] | Then again, as far as nemo-qml-plugin-systemsettings goes, the bit that needs PackageKit is really SailfishOS specific | 14:30 |
PureTryOut[m] | For enabling/installing the developer mode | 14: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 it | 14:36 |
PureTryOut[m] | Why is that not on https://git.sailfishos.org/mer-core? π€ | 14:36 |
DylanVanAssche | I guess that's not part of `mer`, but `sailfish os` specific? | 14:37 |
PureTryOut[m] | But a part of Mer is depending on it lol | 14: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 |
DylanVanAssche | I 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 nice | 14: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 least | 15: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 ssusysinfo | 15:12 |
T42 | <eekkelund> https://github.com/AsteroidOS/meta-asteroid/blob/master/recipes-nemomobile/nemo-qml-plugins/nemo-qml-plugin-systemsettings/0001-Disable-SSU-dependency.patch | 15: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-sysinfo | 15:12 |
T42 | <neochapay> @eekkelund [Fix for not packaging ssusysinfo | 15: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 MR | 15:23 |
T42 | <neochapay> hehe | 15:25 |
T42 | Queena %lastname% was added by: Queena %lastname% | 15:28 |
PureTryOut[m] | I don't get why they haven't upstreamed their changes | 15: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 lol | 15:51 |
PureTryOut[m] | For now I just completely patched out developer mode from systemsettings. It's very SailfishOS specific anyway | 15: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/mer | 15: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 atm | 16:04 |
PureTryOut[m] | Although I can make a WIP post for now maybe | 16:04 |
T42 | <eekkelund> What is wrong with lipstick? | 16:05 |
T42 | <neochapay> just write about progress - it's goot - see how somethinkg is bring up | 16: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 Qt5Wayland | 16:13 |
PureTryOut[m] | neochapay: ok will do. I can also write something about the current blockers for getting it merged into postmarketOS then | 16:13 |
mal | PureTryOut[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 ones | 16:13 |
PureTryOut[m] | aa13q: I made https://git.sailfishos.org/mer-core/nemo-qml-plugin-systemsettings/merge_requests/135 in response to your comment | 16:13 |
PureTryOut[m] | Oh sure will do | 16:15 |
PureTryOut[m] | Did I already miss something? | 16:15 |
mal | PureTryOut[m]: https://git.sailfishos.org/mer-core/nemo-qml-plugin-devicelock/blob/master/rpm/nemo-qml-plugin-devicelock.spec#L15 | 16:16 |
PureTryOut[m] | Ah woops will fix that in a sec | 16:20 |
PureTryOut[m] | mal: https://git.sailfishos.org/mer-core/nemo-qml-plugin-devicelock/merge_requests/47 | 16:34 |
mal | thanks | 16:36 |
PureTryOut[m] | neochapay: https://github.com/nemomobile-ux/nemomobile-ux.github.io/pull/2 | 17:16 |
mal | PureTryOut[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 PRs | 17:27 |
mal | of course if the changes are big then separate PRs can be ok | 17: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 together | 17:29 |
mal | ok | 17:29 |
mal | in this case the other fix was a simple addition of header include which is why I thought it could have been in the same PR | 17: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 |
mal | https://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 tbh | 17:32 |
T42 | <eekkelund> @neochapay some bugs still with glacier-home after rebasing. I think I will manage to sort those tomorrow and will send MR | 18:31 |
T42 | <neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/DPL60Sj0td.webp | 18:49 |
T42 | <neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/EIGYzcdsmd.webp | 18:49 |
T42 | <neochapay> (Sticker, 512x512) https://irc.thaodan.de/.imgstore/Z6nRvET8PS.webp | 18:49 |
T42 | <neochapay> PureTryOut add logo plz | 18:59 |
PureTryOut[m] | Uh, sure, I'll get it from somewhere | 19: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 sorry | 19: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/603ad5ffa903c69d4babf7b6d70170e4032aa8c3 | 19:19 |
PureTryOut[m] | neochapay: ah thanks, done | 19: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 post | 21:25 |
T42 | <neochapay> I fix it tomorrow | 21:25 |
PureTryOut[m] | https://github.com/nemomobile-ux/nemomobile-ux.github.io/pull/3 | 21:56 |
PureTryOut[m] | That should fix it | 21:56 |
*** verin0x2 is now known as verin0x | 23:06 |
Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!