2025-11-13 17:02:33 <@james:fedora.im> !startmeeting fpc

2025-11-13 17:02:34 <@meetbot:fedora.im> Meeting started at 2025-11-13 17:02:33 UTC
2025-11-13 17:02:34 <@meetbot:fedora.im> The Meeting name is 'fpc'
2025-11-13 17:02:40 <@james:fedora.im> !topic Roll Call
2025-11-13 17:02:47 <@tibbs:fedora.im> Hello.
2025-11-13 17:02:52 <@carlwgeorge:fedora.im> !hi
2025-11-13 17:02:54 <@zodbot:fedora.im> Carl George (carlwgeorge) - he / him / his
2025-11-13 17:03:20 <@conan_kudo:matrix.org> !hi
2025-11-13 17:03:33 <@zodbot:fedora.im> Neal Gompa (ngompa) - he / him / his
2025-11-13 17:03:38 <@james:fedora.im> Hey
2025-11-13 17:04:04 <@limb:fedora.im> !hi
2025-11-13 17:04:05 <@zodbot:fedora.im> Gwyn Ciesla (limb) - she / her / hers
2025-11-13 17:07:07 <@james:fedora.im> Okay, we have 5 and I think this one is easy...
2025-11-13 17:07:19 <@james:fedora.im> !topic FPC PR#1501 https://pagure.io/packaging-committee/pull-request/1501
2025-11-13 17:07:42 <@limb:fedora.im> +1.
2025-11-13 17:08:44 <@decathorpe:fedora.im> !hi
2025-11-13 17:08:46 <@zodbot:fedora.im> Fabio Valentini (decathorpe) - he / him / his
2025-11-13 17:08:49 <@decathorpe:fedora.im> sorry for being late
2025-11-13 17:09:26 <@decathorpe:fedora.im> +1 for the PR 
2025-11-13 17:09:29 <@tibbs:fedora.im> +1 (I know zero about ocaml but keping -devel stuff out of regular packages is always good)
2025-11-13 17:11:46 <@carlwgeorge:fedora.im> +1
2025-11-13 17:11:55 <@salimma:fedora.im> +1
2025-11-13 17:12:27 <@james:fedora.im> Seems like a merge :)
2025-11-13 17:13:09 <@james:fedora.im> This one is a little older...
2025-11-13 17:13:18 <@james:fedora.im> !topic FPC#1489 https://pagure.io/packaging-committee/issue/1489
2025-11-13 17:14:39 <@tibbs:fedora.im> Slightly less relevant as F43 is actually released, but my experience is that the page is wrong.
2025-11-13 17:15:03 <@salimma:fedora.im> Zbyszek worked on this right?
2025-11-13 17:15:47 <@decathorpe:fedora.im> yeah it would be good to get his input on this. IIRC this *should* be working on Fedora 42+? 
2025-11-13 17:17:18 <@tibbs:fedora.im> Well the %sysusers_*_compat macros definitely do not evaluate to empty.
2025-11-13 17:17:23 <@tibbs:fedora.im> In F42.
2025-11-13 17:18:30 <@james:fedora.im> I feel like the obvious cheap change is to just s/f42/f43/ ... but I'm open to something else.
2025-11-13 17:18:46 <@tibbs:fedora.im> My problem is that I don't know what the expected behavior is.
2025-11-13 17:19:01 <@salimma:fedora.im> it might need to be fixed though even for f42, if the current behavior is a bug
2025-11-13 17:19:49 <@tibbs:fedora.im> I have a package that cannot work with the compat scriptlets in the guidelines, and I ran into that discrepancy while trying to figure out what to do.
2025-11-13 17:19:51 <@decathorpe:fedora.im> should they?
2025-11-13 17:20:11 <@decathorpe:fedora.im> not all packages were / could be converted, so making it a noop would break them, no?
2025-11-13 17:20:14 <@tibbs:fedora.im> The guidelines page says explicitly that they will evaluate as empty in F42.
2025-11-13 17:20:26 <@tibbs:fedora.im> That's the source of my confusion.
2025-11-13 17:21:11 <@conan_kudo:matrix.org> we probably should ask EPSCo about how to reconcile this for EPEL too
2025-11-13 17:21:41 <@conan_kudo:matrix.org> I've been kind of wary about this change because I find it difficult to reason how to handle all the release cases that common packagers have to deal with
2025-11-13 17:22:24 <@tibbs:fedora.im> The change was a mess; those PRs should not have been merged.
2025-11-13 17:22:41 <@conan_kudo:matrix.org> I think I wound up reverting or blocking some that affected KDE
2025-11-13 17:23:02 <@conan_kudo:matrix.org> ah yes, I never merged the one for sddm
2025-11-13 17:24:11 <@conan_kudo:matrix.org> I still don't know what I'm actually supposed to do
2025-11-13 17:24:31 <@conan_kudo:matrix.org> and I have to revisit this as part of packaging plasma-login-manager
2025-11-13 17:25:43 <@decathorpe:fedora.im> I'm all for getting rid of scriptlets, but this one was rushed 
2025-11-13 17:25:52 <@tibbs:fedora.im> If you don't use one spec across several releases then it isn't a big issue, as the stuff does appear to work as expected in F43.  But it doesn't work in F42 without using the compat stuff, and the guidelines say directly that it will.
2025-11-13 17:26:28 <@conan_kudo:matrix.org> I'd rather just make the scriptlets evaluate to nothing in cases where it isn't needed
2025-11-13 17:26:33 <@conan_kudo:matrix.org> that's what we did for ldconfig scriptlets
2025-11-13 17:27:06 <@salimma:fedora.im> right. but that's what the change proposal said
2025-11-13 17:27:11 <@salimma:fedora.im> the scriptlet *should* evaluate to nothing
2025-11-13 17:27:27 <@salimma:fedora.im> should we raise this to fesco? or ask Zbyszek directly first
2025-11-13 17:27:27 <@decathorpe:fedora.im> s/F42/F43/ in the guidelines should be easy enough, but I'd rather ask zbyszek whether that's actually *supposed* to be this way 
2025-11-13 17:27:34 <@salimma:fedora.im> maybe a F42 update was just missed
2025-11-13 17:27:46 <@salimma:fedora.im> if we defer to f43 we should ask what's the best way to fix things on f42
2025-11-13 17:27:54 <@tibbs:fedora.im> It may just be a bug, or maybe I'm just not understanding everything that's supposed to happen.
2025-11-13 17:28:03 <@decathorpe:fedora.im> I don't think it *hurts* that it doesn't evaluate to noop, right? it should be a noop at runtime instead?
2025-11-13 17:28:34 <@tibbs:fedora.im> Maybe?
2025-11-13 17:29:03 <@tibbs:fedora.im> At best it's just a source of confusion, as it was for me.
2025-11-13 17:29:17 <@conan_kudo:matrix.org> same
2025-11-13 17:29:29 <@conan_kudo:matrix.org> at this point I genuinely don't know what I'm supposed to do
2025-11-13 17:29:42 <@james:fedora.im> Anyone else want to comment and ping zbyszek ... if not I can do it.
2025-11-13 17:30:38 <@tibbs:fedora.im> I wasn't sure who to ping when I filed that.  I also managed to forget entirely that I filed it.
2025-11-13 17:30:52 <@tibbs:fedora.im> I can investigate and ping.
2025-11-13 17:31:05 <@james:fedora.im> Yeh, n/p ... I had a quick look today and didn't remember seeing it before ;)
2025-11-13 17:33:51 <@james:fedora.im> !topic Open Floor
2025-11-13 17:34:11 <@tibbs:fedora.im> We will need to discuss the R guidelines thing at some point.
2025-11-13 17:34:12 <@james:fedora.im> Those were the two things I'd seen ... does anyone else have anything to bring up or discuss?
2025-11-13 17:34:24 <@tibbs:fedora.im> And we will need to talk about the forge migration.
2025-11-13 17:34:48 <@james:fedora.im> https://pagure.io/packaging-committee/issue/1499
2025-11-13 17:34:49 <@tibbs:fedora.im> I'm happy to do the latter after other teams have migrated.
2025-11-13 17:35:30 <@tibbs:fedora.im> So the R thing seems conceptually reasonable but I haven't dug into the details.
2025-11-13 17:36:40 <@tibbs:fedora.im> It's also odd to see the old wiki workflow back from the dead.
2025-11-13 17:37:20 <@tibbs:fedora.im> There is also the question of whether we need to complete our discussion before the inclusion of R-srpm-macros to the buildroot.
2025-11-13 17:37:58 <@tibbs:fedora.im> Since PRs have already been filed to do that (in rawhide and EPEL9/10 but not any release of Fedora, for whatever reason).
2025-11-13 17:39:49 <@tibbs:fedora.im> One thing I see is ```Version:        %R_rpm_version 1.2-3```
2025-11-13 17:40:04 <@tibbs:fedora.im> Doesn't macros in version conflict with some other work being done?
2025-11-13 17:40:33 <@james:fedora.im> autospec?
2025-11-13 17:40:55 <@tibbs:fedora.im> Yes, weren't they switching away from using rpm to parse the spec?  This would probably break.
2025-11-13 17:41:31 <@james:fedora.im> Yeh, although I'm not sure how many R packages use autospec
2025-11-13 17:41:32 <@carlwgeorge:fedora.im> yeah that was https://github.com/praiskup/norpm i believe
2025-11-13 17:43:28 <@tibbs:fedora.im> I will ask that question, and obviously we will need a PR instead of a wiki page.
2025-11-13 17:45:01 <@tibbs:fedora.im> But I'm not sure what else to ask at this point.  The document looks clean.  There may be some slight issues around how license files are done in R.
2025-11-13 17:45:48 <@decathorpe:fedora.im> *ahem* I have a pending PR where I've been waiting for feedback for 2 months :)  
2025-11-13 17:47:32 <@tibbs:fedora.im> Feel free to link it....
2025-11-13 17:48:28 <@james:fedora.im> Guessing this one: https://pagure.io/packaging-committee/pull-request/1483
2025-11-13 17:48:39 <@decathorpe:fedora.im> yes, this one 
2025-11-13 17:49:12 <@james:fedora.im> Looks fine to me, I commented with the really minor thing and you explained
2025-11-13 17:50:25 <@tibbs:fedora.im> +1
2025-11-13 17:51:53 <@carlwgeorge:fedora.im> +1
2025-11-13 17:53:33 <@michel-slm:matrix.org> +1
2025-11-13 17:57:26 <@james:fedora.im> Okay, I'll merge
2025-11-13 17:57:29 <@james:fedora.im> Anything else?
2025-11-13 17:58:14 <@tibbs:fedora.im> Nothing from me.
2025-11-13 17:59:09 <@limb:fedora.im> Nope.
2025-11-13 17:59:23 <@limb:fedora.im> And +1. Sorry, brief AFK.
2025-11-13 18:00:42 <@james:fedora.im> Okay, normally we'd meet in two weeks ... but that's Thanksgiving day in the US ... so not sure if we want to just skip, or try to bump to a different week?
2025-11-13 18:02:10 <@carlwgeorge:fedora.im> skipping is fine by me
2025-11-13 18:02:18 <@limb:fedora.im> same.
2025-11-13 18:02:48 <@james:fedora.im> Okay, see y'all in four weeks.
2025-11-13 18:02:50 <@james:fedora.im> !endmeeting