libera/#eudev/ Thursday, 2021-09-30

bb|hcbblueness: I really don't know how to proceed with that - we do not have anything set up that can handle it administratively. It is not a problem for me to do that myself as an individual (I already have a domain reseller account, so its a couple of clicks+couple of bucks) but that does not seem the proper way for the project. I'd prefer to hear suggestions - maybe Devuan or Alpine sponsoring it (it00:12
bb|hcbwould be too much of a boilerplate to share)?00:12
bb|hcbAnd I think that it is a good idea to keep it, even just as a redirect to the current forge00:12
golinuxIMO opinion, a website is premature and a distraction at this point.  But a good idea to keep the domain registered among those in this group00:22
ArsenI'd not have it be more than a redirect00:30
Arsenfor the time being, anyway00:30
Arsenat HTTP level00:30
sam_yep00:35
bb|hcbIt may create distraction by not keeping it, and if we keep it, it should only redirect, so near 0 effort00:39
golinuxExactly.00:45
Arsenfor some reason man/make.sh is absurdly slow on devuan hm01:30
sam_not riced01:31
sam_need -O3'd bash01:31
Arsenhm, it's not that actually01:33
Arsengotta add -funroll-loops to the parser to get it to go fast01:34
lu_zeroI hope somehow a neobash appears sometimes10:23
Arsenbb|hcb: quick review of https://github.com/eudev-project/eudev/pull/206?12:11
Arsenhm, does this let PR users steal the token12:12
Arsenlet's not merge it yet12:12
Arsenah, it won't run on PRs at all12:14
ArsenI'll see to make that possible12:15
prometheanfirebb|hcb: right now (and since I had it) I've been redirecting it to the gentoo wiki, I could keep it and change the redirect to github or something instead17:15
prometheanfireit's not exactly a hard thing to maintain lol17:15
Arsenprometheanfire: redirect to the repo yeah19:32
bb|hcbArsen: getting the token will not be good21:06
Arsenbb|hcb: I'm decently sure the token is not from our own repo21:06
Arsennot 100% sure21:07
ArsenI'm almost 100% certain, actually21:08
Arsensince the token doesn't work like a secret21:08
Arsenif you wanna try it, PR into my fork21:08
bb|hcb1 workflow awaiting approval21:12
bb|hcbFirst-time contributors need a maintainer to approve running workflows.21:12
bb|hcbI have looked into the changes, everything looks OK; but since am not good with this stuff, I may miss something21:13
bb|hcbBTW. The s1 branch is all PRs that I made, merged in one place21:14
bb|hcbLet us see how that goes, then I will merge it, ok?21:17
bb|hcbI am wondering about the basename PR - normally its better for the person who proposed the change to finish it, but they look like unresponsive. I see that change as important21:18
Arsenbb|hcb: the date/author info should be incorporated into the commit21:40
Arsennot part of the message21:40
Arsenhm, this does run on my token21:41
ArsenI think I can remove that step entirely maybe21:41
bb|hcbChanges are done manually, because the original commit is really unrelated to the current code base; I have also referenced the original commit in the message, but I see no other way to do that21:45
bb|hcbAlso this way of doing the sync (file by file) implies that a change may come from more than one systemd commits, then the message will be much longer :(21:51
bb|hcbDid you see why it is giving 403 on updating the alpine builder?21:55
Arsendyne/devuan has no latest tag22:06
Arsenbb|hcb: sure, but there's meant to be a commit discipline22:07
Arsen#207 also doesn't respect it22:07
Arsenimperative mood, column limit, I'd prefer to add a component prefix to them too, no trailing period, etc22:07
Arsencommit messages being Author: A... Author: B... etc says really nothing about the chance22:09
Arsenchange*22:09
bb|hcbMaybe we can add that to CONTRIBUTING?22:46
Arsenthat'd be good22:47
Arsenshould check whether upstream has some to inherit first22:47
Arsenwe don't want cherry-picked or other pulled in commits to be wrong22:48
bb|hcbDefinitely, that is why the review is there... From what I have seen already, I think it would be best to focus on behaviour/features, then review; so cherry-picking commits from upstream is plain impossible, because API for most of the stuff is different22:51
bb|hcbIs 206 ready to merge?22:52
Arsenyeah it should be now22:52
Arseni got rid of the token22:52
ArsenI don't wanna mess with gh's shenanigans right now and would like PRs to get compiled22:53
bb|hcbOk, merged. Maybe we have to replace Build status in README.md too22:58
bb|hcbAbout PR 207 - I believe that a single link to BUILD should be enough. My reasoning is that it is not expected a user to rebuild and install eudev; OTOH that will help people who want to join the dev effort - for e newbie just getting the build-deps installed would be a problem, so a document helping in that regard is a good thing23:10
Arsenit's probably around time to rewrite the readme anywayh23:15
Arsen(I'd like to lobby for restructuredtext or asciidoc at this moment)23:15
bb|hcbAnd commit the result of whatever to markdown as README.md?23:27
bb|hcbAny way that is convenient is OK23:27
Arsennah just put it in README.rst instead of README.md23:28
Arsenor README.adoc23:28
Arsenall three are supported as READMEs23:28
bb|hcbAre those automagically parsed?23:28
Arsenyes, on gh23:28
bb|hcbThen OK :)23:28
Arsenon sr.ht it's a bit less magic but pretty easy23:28
Arsenhttps://git.sr.ht/~arsen/alsa_rnnoise/tree/master/item/.builds/readme.yml23:29
bb|hcbpretty much dark-magic-foo for a readme ;)23:37
bb|hcbhttps://github.com/eudev-project/eudev-cauldron/blob/main/README.md23:50

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