17:00:02 #startmeeting FESCO (2022-08-30) 17:00:03 Meeting started Tue Aug 30 17:00:02 2022 UTC. 17:00:03 This meeting is logged and archived in a public location. 17:00:03 The chair is mhayden. Information about MeetBot at https://fedoraproject.org/wiki/Zodbot#Meeting_Functions. 17:00:03 Useful Commands: #action #agreed #halp #info #idea #link #topic. 17:00:03 The meeting name has been set to 'fesco_(2022-08-30)' 17:00:08 #meetingname fesco 17:00:08 The meeting name has been set to 'fesco' 17:00:12 #chair nirik, decathorpe, zbyszek, sgallagh, mhroncok, dcantrell, music, mhayden, Conan_Kudo, Pharaoh_Atem, Son_Goku, King_InuYasha, Sir_Gallantmon, Eighth_Doctor 17:00:12 Current chairs: Conan_Kudo Eighth_Doctor King_InuYasha Pharaoh_Atem Sir_Gallantmon Son_Goku dcantrell decathorpe mhayden mhroncok music nirik sgallagh zbyszek 17:00:17 #topic init process 17:00:19 .hello mhayden 17:00:20 mhayden: mhayden 'Major Hayden' 17:00:24 .hello2 17:00:24 👋🏻 17:00:24 dcantrell: dcantrell 'David Cantrell' 17:00:35 howdy David! 17:00:37 morning 17:00:47 howdy nirik! 17:01:02 we're 3/5 17:01:03 .hello gotmax23 17:01:04 gotmax[m]: gotmax23 'Maxwell G' 17:01:04 Matrix is lagging quite badly 17:01:06 hi, I've been out for a while. had PTO then travel to Brno and then got back from Brno and tested positive for COVID 17:01:26 dcantrell: oh man, i hope you're feeling better 17:01:33 COVID is not so fun 17:02:04 .hello music 17:02:05 music[m]: music 'Benjamin Beasley' 17:02:07 .hello2 17:02:08 zbyszek: zbyszek 'Zbigniew Jędrzejewski-Szmek' 17:02:12 I am now, thanks. I had Paxlovid too, which I'm not entirely convinced was necessary, but I just do what the doctor says. This was my first covid. 17:02:20 hello music[m] and zbyszek 17:02:31 i think we're 5/5 on quorum if my counting is accurate 🤔 17:03:18 i'll give it til :04 after and we will get rolling 17:03:29 you, dcantrell, nirik, music[m], zbyszek[m] 17:03:36 gotmax: you! 17:03:45 oh hah 17:03:58 i see... i need more coffee as the afternoon drags on 17:04:04 it's :04 after, let's go 17:04:11 just 3 things i found for today 17:04:19 #topic #2855 Improve package orphaning process 17:04:23 .fexco 2855 17:04:26 .fesco 2855 17:04:27 mhayden: Issue #2855: Improve package orphaning process - fesco - Pagure.io - https://pagure.io/fesco/issue/2855 17:04:41 re-visiting this one from last week to see if we wanted to talk through anything else 17:05:09 * zbyszek has no new ideas 17:05:12 igor has been responding in his ticket -- thanks so much to igor for helping out there and with all of that packaging work 17:05:21 I think mhroncok already opened https://pagure.io/releng/pull-request/10986 and maintainers were mentioned on the Igor ticket 17:05:40 #link https://pagure.io/releng/pull-request/10986 17:05:40 It should be possible to resolve groups and mention all of the maintainers, but I'm not sure we want that 17:05:50 that would be... a lot 17:06:06 mhroncok++ 17:06:12 A lot because you think it'd be difficult or too many people would be mentioned? 17:06:14 Yeah, @python-sig would be bit too much. 17:06:31 the email servers would never forgive us 17:06:38 anything else on this topic or is it pretty well covered? 17:06:54 Seems covered, although the PR wasn't yet merged 17:07:22 looks like nirik gave it a thumbs up -- do we need more reviews on it? 17:07:42 It looked reasonable to me 17:07:59 Apparently python-sig has 31 members, and go-sig has 32. 17:08:08 Less than I expected in both cases. 17:08:22 Yeah, although I'm not sure how many of the go-sig members are actually active... 17:08:23 looks like i should apply for python-sig so we can outdo gotmax 😉 17:08:50 Or you could apply to the go-sig :p 17:09:01 okay, mhroncok should be good to merge that i assume 17:09:10 gotmax: you don't want me on any more golang packages 🤣 17:09:22 alrighty, let's move along 17:09:26 #topic #2860 Periodically check BLAS/LAPACK guidelines 17:09:29 .fesco 2860 17:09:30 mhayden: Issue #2860: Periodically check BLAS/LAPACK guidelines - fesco - Pagure.io - https://pagure.io/fesco/issue/2860 17:09:55 This feels like a slippery slope. Are we going to do this for all of the packaging guidelines? 17:10:17 good question -- could this be rolled into fedora-review? 17:10:35 .hi 17:10:35 decathorpe: decathorpe 'Fabio Valentini' 17:10:36 sorry, I lost track of time. 17:10:36 i guess fedora-review wouldn't help much with an existing package 17:10:43 decathorpe: that's okay, we're glad you're here 17:10:49 mhayden: Yup, only for new ones 17:10:52 we're talking about #2860 17:11:03 gotmax: I think the rub is that this was correctly implemented, and compliance has regressed. So I don't think the slippery slope argument applies. We generally expect that guidelines which were fully implemented remain so. 17:11:22 we do very little to check packages after initial review... 17:11:40 nirik: very true 17:11:43 no meeting? 17:12:01 salimma: we're underway for the Fesco meeting -- welcome 17:12:13 nirik: Agreed. If zuul is enabled, it runs rpmlint when PRs are submitted, but that relies on people not pushing directly to distgit and actually looking at the output... 17:12:31 .hi 17:12:32 * salimma had a stuck Matrix client 17:12:32 salimma: salimma 'Michel Alexandre Salim' 17:12:46 salimma: no problem, we're talking about #2860 17:12:50 mhayden: adding support for fedora-review re-checking an existing package would be nice 17:12:50 at least in this case checking is relatively easy (i.e. verifying that applications + libraries link against the correct library) 17:12:57 nirik: do you thing releng wants to take this on to run the script periodically? 17:13:21 not really. 17:13:37 Because if the answer is negative, I think it's fine if we close this and ask iucar (or other contributors) to run the thing. 17:13:51 Yeah. No special permissions are needed. 17:13:58 this could be a gating test for those packages? 17:14:57 Proposal: contributors/interested parties are encouraged to do periodic checks and submit bugs as necessary. No special privileges are required. 17:15:15 +1 17:15:24 It could be. In my experience, gating has been a nuisance that barely works. And it's not enabled by default. 17:15:32 * decathorpe fears that packagers would just drop gating tests rather than fix their packages 17:15:44 decathorpe: i would never! 🤭 17:15:51 zbyszek[m]: If you mark it with #idea it shows up in the minutes 17:16:10 meh, +1 17:16:15 perhaps we combine zbyszek's idea with a suggestion to roll it into fedora-review and/or rpmlint? 17:16:17 +1 to the proposal... but perhaps more discussion on list could lead to some more generic solution 17:16:41 Is this a general proposal about this sort of scenario or just about this specific case? 17:16:43 mhayden: but that'd require somebody implementing this. I don't think we should require this. 17:16:59 right 17:17:21 nirik: yeah it would make sense to have at least some mechanism in place to make sure we don't regress wrt/ some implemented features 17:17:22 If somebody wants to do that, that'd be great of course. 17:17:30 As someone who maintains several packages that link flexiblas, I would be perfectly happy for someone to set up a gating test for correct linking, but I wouldn’t want to have to set it up myself. 17:18:07 Enhancements to fedora-review and rpmlint are welcome but don’t seem to address the issue of regressions since they’re rarely run on existing packages. 17:18:14 Just as an example, we could do this for rust packages bundling by checking for `bundled(crate(*))` Provides 17:18:16 * nirik wonders if we could setup a way to do commit time tests... but thats likely a ton of work 17:18:17 +1 to zbyszek’s proposal 17:18:31 Although it seems like decathorpe is already aware of at least some of the violations 17:18:54 #idea zbyszek proposes: contributors/interested parties are encouraged to do periodic checks and submit bugs as necessary. No special privileges are required 17:19:09 mhayden: please use #agree 17:19:34 oops, thanks zbyszek 17:19:47 so i'm a +1 on zbyszek's proposal, too 17:19:58 i think we're +5, 0, -0 if i counted correctly 17:20:30 #agree Contributors/interested parties are encouraged to do periodic checks and submit bugs as necessary since no special permissions are required (+5, 0, -0) 17:20:47 I think it's #agreed? 17:21:00 #commands 17:21:00 Available commands: #accept #accepted #action #agree #agreed #chair #commands #endmeeting #halp #help #idea #info #link #lurk #meetingname #meetingtopic #nick #rejected #restrictlogs #save #startmeeting #topic #unchair #undo #unlurk 17:21:08 perhaps it's aliased? 17:21:34 i've got it in the notes -- i'll manually call it out if needed 😉 17:21:42 anything more on this topic? one more left 17:21:42 It is. (Sorry, just realized the time...) 17:21:54 Maybe zodbot is slow? 17:21:55 howdy sgallagh! 17:22:00 * gotmax waves to sgallagh 17:22:14 .hello 17:22:14 jonathanspw: (hello ) -- Alias for "hellomynameis $1". 17:22:18 okay, last topic! 17:22:18 .hi 17:22:19 jonathanspw: jonathanspw 'Jonathan Wright' 17:22:20 howdy jonathanspw 17:22:25 #topic #2857 Change proposal: Minizip Renaming 17:22:31 .fesco 2857 17:22:32 meetbot doesn't provide any confirmation for its commands 17:22:32 mhayden: Issue #2857: Change proposal: Minizip Renaming - fesco - Pagure.io - https://pagure.io/fesco/issue/2857 17:22:38 Zodbot doesn't respond to agree[d] statements. It just takes them 17:22:49 this seems like a fairly standard rename as i read over it 17:23:38 > Obsoletes: (minizip < 3.0.2-7 and minizip > 3.0.0-1) 17:23:43 I'm not sure that works 17:23:51 i think there are some good suggestions for the author in the devel thread 17:24:08 gotmax: I'm pretty sure it doesn't 17:24:26 gotmax: those numbers look backwards 17:24:49 Is there any reason not to use the standard approach described in Packaging Guidelines? 17:25:10 I think the problem is that Obsoletes don't work with boolean dependencies. But maybe two Obsolete statements would work. 17:25:19 it would be great if the change proposal included adding Provides for the new names to the existing packages, that would make it almost backwards compatible ... not sure if I remembered to post that on devel@ :) 17:25:21 #info https://docs.fedoraproject.org/en-US/packaging-guidelines/#renaming-or-replacing-existing-packages 17:25:42 #link https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/I5WYNPVD3ZOKO3X76LQ5SIRUUXDFUF2Y/ 17:25:44 ^^ devel thread 17:26:31 oh, looks like I did in fact post that suggestion. 17:26:35 just fixed broken devel thread link in the issue and added a link to that original discussion 17:26:41 thanks, music[m] 17:26:43 music[m]++ 17:27:10 thanks 17:27:11 would someone volunteer to give some feedback on that one for improving it? 17:27:21 gotmax: I'm confused why `minizip > 3.0.0-1` at all 17:27:25 That's... never going to work right 17:28:12 Hmm, actually Obsoletes is all wrong. 17:28:27 Because we want the packages to be parallel-installable. 17:30:17 Yeah, I think it'd need to be a with instead of an and, and you can't use either. It would work if they used `minizip < 3.0.2-7` and epoch'd the new minizip-compat. 17:30:51 Meh, but why all this bother? 17:30:56 who can reply on that ticket to give the change proposer some suggestions? (i haven't been through this process with my packages yet) 17:31:06 Because they don 17:31:09 oops 17:31:23 I can reply. 17:31:29 zbyszek++ 17:31:29 mhayden: Karma for zbyszek changed to 2 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:31:42 #action zbyszek to reply on #2857 with suggestions 17:31:45 thank you, zbyszek 17:32:05 anything else on this for now? 17:32:25 Because they don't want to the older minizip to be replaced with the renamed minizip-ng 17:34:17 okay, it got quiet, so i'll move on 17:34:23 #topic Next week's chair 17:34:36 * mhayden has been in the driver's seat for two weeks and is happy to pass the baton 😜 17:34:47 whoa, i crossed my metaphors/idioms therer 17:35:05 anybody excited to do next week's meeting? 17:36:38 * gotmax hears crickets 17:36:52 I might not be available. I can take it three weeks from now. 17:37:40 hah, Monday is a holiday in the USA next week 17:38:23 well, if no one else will I guess I can... 17:38:31 nirik++ 17:38:31 zbyszek: Karma for kevin changed to 7 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:38:33 nirik++ 17:38:35 mhayden: Karma for kevin changed to 8 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:38:50 #action nirik to chair next week's meeting 🪑 17:38:59 uh oh, i hope zodbot does emojis 17:39:07 #topic Open Floor 17:39:22 #link https://pagure.io/fesco/fesco-docs/pull-request/68 17:39:30 Happy to be here :) 17:39:32 nirki++ 17:39:38 That's my open floor contribution. 17:39:40 good ol' nirki 17:39:49 jonathanspw: happy you're here, too! 17:39:54 jonathanspw++ 17:39:56 zbyszek: Karma for jonathanspw changed to 1 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:39:59 jonathanspw++ 17:39:59 mhayden: Karma for jonathanspw changed to 2 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:40:01 My first cookie! 17:40:07 Oh back to back cookies! 17:40:12 gotmax: looking at that PR! 17:40:22 So far only zbyszek has voted 17:40:36 oh this latest version looks ✅ to me 17:41:26 mhayden: please vote in the ticket. It's easier to keep track this way. 17:41:39 zbyszek: just did, but thought i'd make gotmax feel good in here too 17:41:40 He did already 17:41:56 Oh, indeed. 17:42:02 * mhayden prefers to share his appreciation and approvals all over the place 17:42:19 * mhayden also makes a note to have less coffee before this meeting 17:42:36 :) 17:42:38 mhayden++ 17:42:38 gotmax: Karma for mhayden changed to 3 (for the current release cycle): https://badges.fedoraproject.org/tags/cookie/any 17:42:51 since we're doing the open floor thing, i just want to say thanks to gotmax and music[m] for all their help with my packaging -- i've learned a ton and i think i'm making much fewer errors now 17:43:00 🍵 17:43:13 and good gosh thanks to mhroncok and friends for the pyproject-rpm-macros 🫂 17:43:27 :) 17:43:59 okay, i've got another $dayjob meeting rapidly approaching, so let's wrap this up! 17:44:17 thanks everyone for coming today. see you next week. hopefully all the USA friends have a good holiday weekend. 17:44:20 mhayden, thanks for chairing. 17:44:33 thanks mhayden 17:44:44 you're welcome! 17:44:47 #endmeeting