bb|hcb | blueness: 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 (it | 00:12 |
---|---|---|
bb|hcb | would be too much of a boilerplate to share)? | 00:12 |
bb|hcb | And I think that it is a good idea to keep it, even just as a redirect to the current forge | 00:12 |
golinux | IMO opinion, a website is premature and a distraction at this point. But a good idea to keep the domain registered among those in this group | 00:22 |
Arsen | I'd not have it be more than a redirect | 00:30 |
Arsen | for the time being, anyway | 00:30 |
Arsen | at HTTP level | 00:30 |
sam_ | yep | 00:35 |
bb|hcb | It may create distraction by not keeping it, and if we keep it, it should only redirect, so near 0 effort | 00:39 |
golinux | Exactly. | 00:45 |
Arsen | for some reason man/make.sh is absurdly slow on devuan hm | 01:30 |
sam_ | not riced | 01:31 |
sam_ | need -O3'd bash | 01:31 |
Arsen | hm, it's not that actually | 01:33 |
Arsen | gotta add -funroll-loops to the parser to get it to go fast | 01:34 |
lu_zero | I hope somehow a neobash appears sometimes | 10:23 |
Arsen | bb|hcb: quick review of https://github.com/eudev-project/eudev/pull/206? | 12:11 |
Arsen | hm, does this let PR users steal the token | 12:12 |
Arsen | let's not merge it yet | 12:12 |
Arsen | ah, it won't run on PRs at all | 12:14 |
Arsen | I'll see to make that possible | 12:15 |
prometheanfire | bb|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 instead | 17:15 |
prometheanfire | it's not exactly a hard thing to maintain lol | 17:15 |
Arsen | prometheanfire: redirect to the repo yeah | 19:32 |
bb|hcb | Arsen: getting the token will not be good | 21:06 |
Arsen | bb|hcb: I'm decently sure the token is not from our own repo | 21:06 |
Arsen | not 100% sure | 21:07 |
Arsen | I'm almost 100% certain, actually | 21:08 |
Arsen | since the token doesn't work like a secret | 21:08 |
Arsen | if you wanna try it, PR into my fork | 21:08 |
bb|hcb | 1 workflow awaiting approval | 21:12 |
bb|hcb | First-time contributors need a maintainer to approve running workflows. | 21:12 |
bb|hcb | I have looked into the changes, everything looks OK; but since am not good with this stuff, I may miss something | 21:13 |
bb|hcb | BTW. The s1 branch is all PRs that I made, merged in one place | 21:14 |
bb|hcb | Let us see how that goes, then I will merge it, ok? | 21:17 |
bb|hcb | I 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 important | 21:18 |
Arsen | bb|hcb: the date/author info should be incorporated into the commit | 21:40 |
Arsen | not part of the message | 21:40 |
Arsen | hm, this does run on my token | 21:41 |
Arsen | I think I can remove that step entirely maybe | 21:41 |
bb|hcb | Changes 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 that | 21:45 |
bb|hcb | Also 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|hcb | Did you see why it is giving 403 on updating the alpine builder? | 21:55 |
Arsen | dyne/devuan has no latest tag | 22:06 |
Arsen | bb|hcb: sure, but there's meant to be a commit discipline | 22:07 |
Arsen | #207 also doesn't respect it | 22:07 |
Arsen | imperative mood, column limit, I'd prefer to add a component prefix to them too, no trailing period, etc | 22:07 |
Arsen | commit messages being Author: A... Author: B... etc says really nothing about the chance | 22:09 |
Arsen | change* | 22:09 |
bb|hcb | Maybe we can add that to CONTRIBUTING? | 22:46 |
Arsen | that'd be good | 22:47 |
Arsen | should check whether upstream has some to inherit first | 22:47 |
Arsen | we don't want cherry-picked or other pulled in commits to be wrong | 22:48 |
bb|hcb | Definitely, 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 different | 22:51 |
bb|hcb | Is 206 ready to merge? | 22:52 |
Arsen | yeah it should be now | 22:52 |
Arsen | i got rid of the token | 22:52 |
Arsen | I don't wanna mess with gh's shenanigans right now and would like PRs to get compiled | 22:53 |
bb|hcb | Ok, merged. Maybe we have to replace Build status in README.md too | 22:58 |
bb|hcb | About 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 thing | 23:10 |
Arsen | it's probably around time to rewrite the readme anywayh | 23:15 |
Arsen | (I'd like to lobby for restructuredtext or asciidoc at this moment) | 23:15 |
bb|hcb | And commit the result of whatever to markdown as README.md? | 23:27 |
bb|hcb | Any way that is convenient is OK | 23:27 |
Arsen | nah just put it in README.rst instead of README.md | 23:28 |
Arsen | or README.adoc | 23:28 |
Arsen | all three are supported as READMEs | 23:28 |
bb|hcb | Are those automagically parsed? | 23:28 |
Arsen | yes, on gh | 23:28 |
bb|hcb | Then OK :) | 23:28 |
Arsen | on sr.ht it's a bit less magic but pretty easy | 23:28 |
Arsen | https://git.sr.ht/~arsen/alsa_rnnoise/tree/master/item/.builds/readme.yml | 23:29 |
bb|hcb | pretty much dark-magic-foo for a readme ;) | 23:37 |
bb|hcb | https://github.com/eudev-project/eudev-cauldron/blob/main/README.md | 23:50 |
Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!