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

2025-02-13 17:00:09 <@meetbot:fedora.im> Meeting started at 2025-02-13 17:00:07 UTC
2025-02-13 17:00:09 <@meetbot:fedora.im> The Meeting name is 'fpc'
2025-02-13 17:00:11 <@james:fedora.im> !topic Roll Call
2025-02-13 17:00:30 <@tibbs:fedora.im> Hey.
2025-02-13 17:00:42 <@decathorpe:fedora.im> hello
2025-02-13 17:00:48 <@conan_kudo:matrix.org> !hi
2025-02-13 17:00:51 <@james:fedora.im> !hi
2025-02-13 17:00:51 <@zodbot:fedora.im> Neal Gompa (ngompa) - he / him / his
2025-02-13 17:00:53 <@zodbot:fedora.im> James Antill (james)
2025-02-13 17:01:08 <@salimma:fedora.im> !hi
2025-02-13 17:01:09 <@zodbot:fedora.im> Michel Lind (salimma) - he / him / his
2025-02-13 17:01:18 <@salimma:fedora.im> Fabio Valentini: you're no longer sick! yay
2025-02-13 17:01:39 <@decathorpe:fedora.im> at least no longer sick enough :D
2025-02-13 17:02:12 <@salimma:fedora.im> paraphrasing Princess Bride, mostly well is barely sick :P
2025-02-13 17:08:05 <@carlwgeorge:fedora.im> !hi
2025-02-13 17:08:06 <@zodbot:fedora.im> Carl George (carlwgeorge) - he / him / his
2025-02-13 17:08:12 <@nhanlon:beeper.com> !hi
2025-02-13 17:08:14 <@zodbot:fedora.im> Neil Hanlon (neil) - he / him / his
2025-02-13 17:09:26 <@james:fedora.im> Neil Hanlon: Hey, you here for anything specific or just to hang out with us awesome people (who arne't even that sick anymore ;)
2025-02-13 17:10:42 <@james:fedora.im> !topic FPC PR#1435 review: drop obsolete recommendation for file path dependencies
2025-02-13 17:10:54 <@james:fedora.im> https://pagure.io/packaging-committee/pull-request/1435
2025-02-13 17:10:58 <@nhanlon:beeper.com> Just hanging out, as a fellow one who is not as sick anymore
2025-02-13 17:11:08 <@nhanlon:beeper.com> /me sips tea
2025-02-13 17:11:37 <@tibbs:fedora.im> The whole review guidelines page is such a mess.
2025-02-13 17:12:14 <@tibbs:fedora.im> We could certainly drop anything from it that's incorrect, but I have no hope of it actually being an adequate list of what a review should look for.
2025-02-13 17:12:23 <@james:fedora.im> This one seems pretty easy merge, and hopefully getting other people to do the patches for us will make them or others more likely to fix the rest of it ;)
2025-02-13 17:12:35 <@decathorpe:fedora.im> yeah this one looks straightforward 
2025-02-13 17:13:26 <@salimma:fedora.im> yeah
2025-02-13 17:14:58 <@james:fedora.im> !topic FPC PR#1436 UsersAndGroups: fully switch to sysusers.d
2025-02-13 17:15:02 <@james:fedora.im> https://pagure.io/packaging-committee/pull-request/1436
2025-02-13 17:15:17 <@james:fedora.im> Fabio Valentini: Already +1'd the new version
2025-02-13 17:15:35 <@tibbs:fedora.im> PRs were already filed for this, which seems way premature to me.
2025-02-13 17:17:29 <@decathorpe:fedora.im> why? the Change Proposal was already accepted:
2025-02-13 17:17:29 <@decathorpe:fedora.im> https://fedoraproject.org/wiki/Changes/RPMSuportForSystemdSysusers
2025-02-13 17:17:32 <@tibbs:fedora.im> But +1 to the guidelines change.  I always thought the rationale section was kind of pointless anyway.
2025-02-13 17:18:03 <@tibbs:fedora.im> If someone merges one of those package PRs back to F41, doesn't it break?
2025-02-13 17:18:26 <@decathorpe:fedora.im> I suppose it does
2025-02-13 17:18:53 <@tibbs:fedora.im> The PRs do not mention that at all.
2025-02-13 17:19:26 <@decathorpe:fedora.im> hm, I would have expected that to be mentioned in the PRs, yeah :(
2025-02-13 17:19:32 <@conan_kudo:matrix.org> tbh, I don't understand why we can't have guidance that is backward compatible
2025-02-13 17:19:52 <@conan_kudo:matrix.org> we need this for people maintaining across fedora releases, across epel, etc.
2025-02-13 17:20:06 <@conan_kudo:matrix.org> this seems kind of ham-fisted :/
2025-02-13 17:20:08 <@tibbs:fedora.im> I don't personally have an issue with picking changes back instead of just merging, but many people do expect to keep the same specfile across releases.
2025-02-13 17:20:44 <@tibbs:fedora.im> And in general, sure, people can't expect to always be able to do that without horrible spaghetti conditionals but this isn't a super important thing here.
2025-02-13 17:21:01 <@james:fedora.im> I thought the latest version is compatible between 41 and 42, it still includes macros that are noops on 42 but needed on 41.
2025-02-13 17:21:06 <@tibbs:fedora.im> It just makes use of some new RPM behavior. I don't think we have ever forced that kind of thing on people.
2025-02-13 17:21:42 <@salimma:fedora.im> even if it's compatible between 41 and 42... 40 is still supported
2025-02-13 17:21:43 <@decathorpe:fedora.im> hm, I do seem to remember some previous effort around dropping Scriptlets en masse 
2025-02-13 17:22:10 <@salimma:fedora.im> I think documenting which releases you can use it for is a must. there's also EPEL, and many of us maintain a single spec for all releases so this might break
2025-02-13 17:23:41 <@tibbs:fedora.im> This is a PR for a package I maintain:
2025-02-13 17:23:45 <@tibbs:fedora.im> https://src.fedoraproject.org/rpms/amanda/pull-request/10#request_diff
2025-02-13 17:24:16 <@tibbs:fedora.im> There's no backwards anything.  It just deletes the calls to useradd and drops in a sysusers file.
2025-02-13 17:24:24 <@conan_kudo:matrix.org> that PR would break it for anything older than F42
2025-02-13 17:24:24 <@decathorpe:fedora.im> so just to be clear: is this critique regarding the latest version of the packaging-guidelines PR?
2025-02-13 17:25:27 <@tibbs:fedora.im> To be clear, I think we should change the guideliens.
2025-02-13 17:25:32 <@conan_kudo:matrix.org> my critique specifically is that the PR deletes guidance that would make a package backward compatible
2025-02-13 17:25:50 <@salimma:fedora.im> yeah, I think we should do it like the Python packaging guidelines
2025-02-13 17:26:05 <@decathorpe:fedora.im> it doesn't though?
2025-02-13 17:26:08 <@salimma:fedora.im> if we want to keep the page clean, have a big sign saying "for releases older than X please see this other page"
2025-02-13 17:26:55 <@tibbs:fedora.im> I don't think it's really fair to make the submitter do all that, though.  We could merge and then fix up the document a bit if needed.
2025-02-13 17:26:55 <@carlwgeorge:fedora.im> we do have a section in the epel docs for "packaging things that only apply to epel branches and no active fedora branches"
2025-02-13 17:27:23 <@tibbs:fedora.im> I don't think it deletes any information which is still needed on older releases.  The big deletion is a bunch of rationale which didn't really need to be there in the first place.
2025-02-13 17:27:27 <@salimma:fedora.im> it says the macro will evaluate as empty on < F42 but that means the new users are not created as expected, right?
2025-02-13 17:27:51 <@conan_kudo:matrix.org> right
2025-02-13 17:28:14 <@decathorpe:fedora.im> https://pagure.io/packaging-committee/pull-request/1436#_1__31-36
2025-02-13 17:28:23 <@james:fedora.im> Fabio Valentini: There are no macro calls in the PR, right? So it doesn't work on 41 ... or am I missig something?
2025-02-13 17:28:58 <@decathorpe:fedora.im> but the PR also doesn't *drop* macro calls that are documented 
2025-02-13 17:29:28 <@decathorpe:fedora.im> and it explicitly adds that the creation needs to be manual and the file needs to be a separate source for Fedora < 42 
2025-02-13 17:30:02 <@salimma:fedora.im> it will be nice to be able to see the full rendered page with this change but I guess we need to locally apply this PR to do that
2025-02-13 17:30:40 <@decathorpe:fedora.im> this is the full text from the PR:
2025-02-13 17:30:40 <@decathorpe:fedora.im> https://pagure.io/fork/zbyszek/packaging-committee/raw/sysusers-switch/f/guidelines/modules/ROOT/pages/UsersAndGroups.adoc
2025-02-13 17:30:55 <@decathorpe:fedora.im> if you have the AsciiDoctor browser extension installed, it will render it
2025-02-13 17:31:34 <@salimma:fedora.im> ah. yeah it does still have the scriptlet method
2025-02-13 17:31:40 <@decathorpe:fedora.im> (yes the documentation for the `%sysusers_create_compat` macro is still there)
2025-02-13 17:32:33 <@james:fedora.im>   * New: sysusers, with automagic rpm stuff and the macros do nothing.
2025-02-13 17:32:33 <@james:fedora.im> 
2025-02-13 17:32:33 <@james:fedora.im>   * Current: sysusers, but add macros that do magic things in %pre.
2025-02-13 17:32:33 <@james:fedora.im> Wait ... maybe I'm wrong but I thought this was the case:
2025-02-13 17:32:33 <@james:fedora.im>   * Old: Just call useradd/etc. in %pre
2025-02-13 17:32:33 <@james:fedora.im> ...the policy PR documents New and Current ... but not Old. Jason ティビツ 's PR for his package goes directly from Old to New (and thus. breaks on 41)
2025-02-13 17:32:41 <@salimma:fedora.im> TIL, nice
2025-02-13 17:32:56 <@zodbot:fedora.im> salimma has already given cookies to decathorpe during the F41 timeframe
2025-02-13 17:33:25 <@decathorpe:fedora.im> James Antill: ah, so you're saying you want *three* versions to be documented? :(
2025-02-13 17:34:01 <@james:fedora.im> No, just Current and New documented ... but don't break packages by going from Old to New (instead of Old to current).
2025-02-13 17:34:04 <@conan_kudo:matrix.org> well, the "current" way was backported to EPEL a long time ago
2025-02-13 17:34:17 <@conan_kudo:matrix.org> so we just need the two, and packages being adapted should handle both modes
2025-02-13 17:34:21 <@carlwgeorge:fedora.im> even epel8?
2025-02-13 17:34:27 <@decathorpe:fedora.im> https://pagure.io/fork/zbyszek/packaging-committee/c/1e70fd534bc97ca15ab4a80eed3b420d69478ad1
2025-02-13 17:34:27 <@decathorpe:fedora.im> I'm pretty sure that the old stuff that's removed is explained here:
2025-02-13 17:34:49 <@decathorpe:fedora.im> i.e. the "really old" stuff has already been dropped in 2020
2025-02-13 17:34:56 <@conan_kudo:matrix.org> yes
2025-02-13 17:35:01 <@decathorpe:fedora.im> i.e. the "really old" stuff has already been dropped in 2020, and this is just cleanup of leftovers
2025-02-13 17:35:05 <@conan_kudo:matrix.org> https://src.fedoraproject.org/rpms/epel-rpm-macros/c/d733fbe4762ba4c43de897127128400caaf8f469?branch=epel8
2025-02-13 17:36:43 <@carlwgeorge:fedora.im> nice
2025-02-13 17:38:02 <@james:fedora.im> Okay, so is everyone fine with the policy PR?
2025-02-13 17:38:11 <@conan_kudo:matrix.org> I suppose so, yes
2025-02-13 17:38:36 <@decathorpe:fedora.im> +1 from me
2025-02-13 17:38:47 <@salimma:fedora.im> +1
2025-02-13 17:38:56 <@carlwgeorge:fedora.im> lgtm
2025-02-13 17:39:32 <@james:fedora.im> Okay, I'm going to merge ... feel free to complain about PRs to your packages to the people making them ;)
2025-02-13 17:40:18 <@tibbs:fedora.im> I just wanted to get it logged that it wasn't us trying to push this into the various packages.
2025-02-13 17:40:55 <@james:fedora.im> !topic FPC#1440 document that there are better alternatives to alternatives?
2025-02-13 17:41:01 <@james:fedora.im>  https://pagure.io/packaging-committee/issue/1440
2025-02-13 17:41:31 <@james:fedora.im> Conan Kudo: You want say anything?
2025-02-13 17:42:44 <@decathorpe:fedora.im> basically this would be about 1) marking the *Alternatives* page of the guidelines as "deprecated" or at least adding a banner that they are old and creaky and don't work everywhere, and 2) documenting the "modern" RPM based way to handle this kind of thing
2025-02-13 17:43:02 <@carlwgeorge:fedora.im> is the "modern" way just conflicts?
2025-02-13 17:43:08 <@conan_kudo:matrix.org> the main advantage of "alternatives" is that they can be coinstalled
2025-02-13 17:43:23 <@conan_kudo:matrix.org> Provides+Conflicts with a virtual name
2025-02-13 17:43:23 <@james:fedora.im> The problem I see is that none of the solutions really work the same way?
2025-02-13 17:44:07 <@carlwgeorge:fedora.im> not sure what's modern about that, but ok
2025-02-13 17:44:13 <@decathorpe:fedora.im> Carl George: essentially instead of running `sudo alternatives --set foo bar` you run `sudo dnf swap foo bar` to swap out the links 
2025-02-13 17:44:19 <@james:fedora.im> Like you have to swap in the correct packages ... I was also a little confused because fedora-release-identity-* has a cinnamon package, but I'm running cinnamon but have the workstation package.
2025-02-13 17:44:37 <@conan_kudo:matrix.org> an "rpm-ostree"/"bootc" compatible alternatives system does exist, we just don't use it in fedora
2025-02-13 17:45:01 <@conan_kudo:matrix.org> the opensuse folks made alts, but switching to it would take time and it's not packaged in fedora yet anyway
2025-02-13 17:45:36 <@carlwgeorge:fedora.im> fwiw, i do like the provides/conflicts approach, but i know some people don't like the fact that trying to install a package that conflicts with something you already have installed gives you a message that says "error"
2025-02-13 17:45:55 <@carlwgeorge:fedora.im> at least, that was the rationale i was given a long time ago for why software collections was invented
2025-02-13 17:46:50 <@decathorpe:fedora.im> in some cases you can still parallel-install multiple providers, while only having one "active", with this approach
2025-02-13 17:47:29 <@carlwgeorge:fedora.im> what would that case look like?  surely this is primarily for packages with conflicting files, right?
2025-02-13 17:47:57 <@james:fedora.im> I guess I'm fine if it's "this isn't better for everyone, but it actually works in ostree images so we need to do it" ... but it would be nice if someone had a look at the major packages using alternatives now and said "this is the way they should be done, and it works well enough"
2025-02-13 17:48:08 <@decathorpe:fedora.im> yes, the conflicting files would need to be in the subpackages that determine which provider is "active" 
2025-02-13 17:49:01 <@tibbs:fedora.im> Is there a documented approach that leaves everything parallel-installable with some tiny conflicting packages that just install simlinks in /usr/bin?
2025-02-13 17:49:17 <@decathorpe:fedora.im> no, but that's why I filed this ticket :D
2025-02-13 17:49:20 <@carlwgeorge:fedora.im> that seems not worth the trouble imo
2025-02-13 17:49:29 <@carlwgeorge:fedora.im> just have the main packages conflict
2025-02-13 17:49:33 <@tibbs:fedora.im> In other words, get rid of alternatives and instead package the symlinks that it would create.
2025-02-13 17:49:46 <@tibbs:fedora.im> Having the main packages conflict prevents parallel installability, which is often useful.
2025-02-13 17:50:02 <@decathorpe:fedora.im> basically, yes
2025-02-13 17:50:31 <@james:fedora.im> Carl George: Pretty sure there are alternatives usecases where you need to install (and use) more than one, but you still need an alternative type thing for what is the default.
2025-02-13 17:50:36 <@decathorpe:fedora.im> btw, I'm fine if nothing results from this ticket, other than that we should document that Alternatives no longer work universally 
2025-02-13 17:50:49 <@tibbs:fedora.im> The guidelines used to forbid conflicting packages with the goal of making the entire package set installable at the same time.
2025-02-13 17:51:17 <@james:fedora.im> That was a long time ago ;)
2025-02-13 17:51:28 <@carlwgeorge:fedora.im> i think it's questionable usefulness if users have to run a full path out of say /usr/libexec instead of just running the command in PATH
2025-02-13 17:52:20 <@carlwgeorge:fedora.im> i went through this semi-recently with singularity-ce and apptainer, and they ultimately went with simple conflicts and a virtual provides rather than figuring out the alternatives incantation
2025-02-13 17:53:06 <@tibbs:fedora.im> I would say that's a net loss and unfortunate.
2025-02-13 17:53:18 <@tibbs:fedora.im> But yes, alternatives is not great.
2025-02-13 17:53:35 <@tibbs:fedora.im> I'll have to look into what opensuse cooked ip.
2025-02-13 17:53:49 <@decathorpe:fedora.im> ... is that something that we *should* do?
2025-02-13 17:54:00 <@conan_kudo:matrix.org> tbh, I think so
2025-02-13 17:54:06 <@conan_kudo:matrix.org> co-installable runtime switching is useful
2025-02-13 17:54:06 <@carlwgeorge:fedora.im> there's nothing wrong with the "subpackage the symlink" approach, i just think it's not worth the effort
2025-02-13 17:54:17 <@tibbs:fedora.im> Feel free to drop a reference if you have one.
2025-02-13 17:54:31 <@conan_kudo:matrix.org> I'll put it in the ticket
2025-02-13 17:54:38 <@conan_kudo:matrix.org> and I'll try to get alts building on fedora again
2025-02-13 17:54:47 <@tibbs:fedora.im> https://en.opensuse.org/openSUSE:Migrating_to_libalternatives_with_alts
2025-02-13 17:54:52 <@james:fedora.im> Was just about to ask you to do that, so 👍️
2025-02-13 17:55:26 <@tibbs:fedora.im> I can't imagine it's difficult to build.
2025-02-13 17:56:52 <@carlwgeorge:fedora.im> i'm at step one of the goal section and this already seems vastly more complicated that provides+conflicts
2025-02-13 17:57:08 <@decathorpe:fedora.im> heh, this sounds similar to the approach used in the new x86_64-vX binary providing change proposal 
2025-02-13 17:57:13 <@carlwgeorge:fedora.im> maybe it's worthwhile in some cases, but perhaps not as the default recommendation
2025-02-13 17:58:36 <@decathorpe:fedora.im> yeah, I'm pretty sure many of the remaining uses of `alternatives` could be dropped entirely
2025-02-13 17:58:53 <@decathorpe:fedora.im> but for those that can't, moving to a "modern" implementation would seem worthwhile 
2025-02-13 17:59:35 <@carlwgeorge:fedora.im> i think realistically the modern implementation of parallel installable is separate containers on the same host
2025-02-13 18:00:03 <@decathorpe:fedora.im> ew, containers 🙅 😆
2025-02-13 18:00:22 <@carlwgeorge:fedora.im> i didn't say it was my preference, just reality 😛
2025-02-13 18:00:44 <@decathorpe:fedora.im> I don't think `ld                  	auto  	/usr/bin/ld.bfd` should be in a container though
2025-02-13 18:00:54 <@decathorpe:fedora.im> ugh formatting (
2025-02-13 18:01:11 <@conan_kudo:matrix.org> I believe a variation of this is leveraged for this in opensuse
2025-02-13 18:01:33 <@conan_kudo:matrix.org> at least before they went to the "provide RPMs and let rpm upgrade you as needed"
2025-02-13 18:01:40 <@conan_kudo:matrix.org> at least before they went to the "provide RPMs and let rpm upgrade you as needed" approach
2025-02-13 18:01:53 <@conan_kudo:matrix.org> (which I still think is the right approach for that)
2025-02-13 18:02:53 <@james:fedora.im> We are officially over time
2025-02-13 18:03:16 <@james:fedora.im> Anyone want to comment in the ticket, and then we can ignore it for a year or two? ;)
2025-02-13 18:03:26 <@conan_kudo:matrix.org> lol
2025-02-13 18:03:38 <@conan_kudo:matrix.org> I'll try to put some info in the ticket
2025-02-13 18:03:43 <@conan_kudo:matrix.org> we'll see how it goes from there
2025-02-13 18:03:54 <@farchord:fedora.im> Wait, you guys don't get overtime pay?
2025-02-13 18:03:56 <@zodbot:fedora.im> james gave a cookie to ngompa. They now have 158 cookies, 11 of which were obtained in the Fedora 41 release cycle
2025-02-13 18:04:07 <@zodbot:fedora.im> decathorpe gave a cookie to ngompa. They now have 159 cookies, 12 of which were obtained in the Fedora 41 release cycle
2025-02-13 18:04:16 <@conan_kudo:matrix.org> if only
2025-02-13 18:04:19 <@james:fedora.im> Steve Cossette [Farchord]: 10x pay ... alas. 10x 0 = 0.
2025-02-13 18:04:29 <@farchord:fedora.im> Haha XD
2025-02-13 18:05:28 <@james:fedora.im> Okay, everyone enjoy your -5m of extra time ;)
2025-02-13 18:05:35 <@james:fedora.im> !endmeeting