2024-09-12 16:00:50 <@james:fedora.im> !startmeeting fpc

2024-09-12 16:00:56 <@james:fedora.im> !topic Roll Call
2024-09-12 16:00:59 <@meetbot:fedora.im> Meeting started at 2024-09-12 16:00:50 UTC
2024-09-12 16:00:59 <@meetbot:fedora.im> The Meeting name is 'fpc'
2024-09-12 16:01:16 <@tibbs:fedora.im> Hey folks.
2024-09-12 16:01:18 <@limb:fedora.im> !hi
2024-09-12 16:01:19 <@zodbot:fedora.im> Gwyn Ciesla (limb) - she / her / hers
2024-09-12 16:01:30 <@james:fedora.im> !hi
2024-09-12 16:01:35 <@zodbot:fedora.im> James Antill (james)
2024-09-12 16:01:52 <@tibbs:fedora.im> In addition to having my house torn up, I am also sick.
2024-09-12 16:02:36 <@james:fedora.im> The joys of getting older ... :(
2024-09-12 16:03:04 <@tibbs:fedora.im> I think it's just a cold but it's tough to be sure.
2024-09-12 16:03:44 <@james:fedora.im> Been hit with outside allergy stuff the last month or so, which is unusual.
2024-09-12 16:05:36 <@decathorpe:fedora.im> !hi
2024-09-12 16:05:37 <@zodbot:fedora.im> Fabio Valentini (decathorpe) - he / him / his
2024-09-12 16:05:54 <@decathorpe:fedora.im> hello o/
2024-09-12 16:06:45 <@carlwgeorge:matrix.org> !hi
2024-09-12 16:06:47 <@zodbot:fedora.im> Carl George (carlwgeorge) - he / him / his
2024-09-12 16:08:49 <@james:fedora.im> Okay, nothing new ... but trying to get some closure on old things, so...
2024-09-12 16:09:00 <@james:fedora.im> !topic FPC PR#1014 https://pagure.io/packaging-committee/pull-request/1014
2024-09-12 16:09:47 <@james:fedora.im> Fabio Valentini 👀
2024-09-12 16:11:56 <@tibbs:fedora.im> I don't know enough about go, but with old things like this I have to wonder if it's still necessary.
2024-09-12 16:12:39 <@james:fedora.im> Yeh, it can't merge due to conflicts ... but I'm not sure that means it's fixed.
2024-09-12 16:13:13 <@decathorpe:fedora.im> hum hum 
2024-09-12 16:13:18 <@tibbs:fedora.im> Conflicts probably means little more than something getting line breaks
2024-09-12 16:13:30 <@decathorpe:fedora.im> I don't think this change is even correct
2024-09-12 16:13:44 <@decathorpe:fedora.im> probably we should ping gotmax23 who now maintains the Go macros.
2024-09-12 16:14:32 <@james:fedora.im> Fabio Valentini: I'm pretty happy to follow your lead here. If you think we should just close it I'll do it, obviously pinging anyone is fine ... just rather it not remain open for another 2 years :)
2024-09-12 16:14:42 <@carlwgeorge:matrix.org> with no reply from the submitter to the comments so far for four years, i think we should just close it.  someone that wants to address it can open a new pr.
2024-09-12 16:14:46 <@decathorpe:fedora.im> yeah
2024-09-12 16:15:17 <@decathorpe:fedora.im> it's a problem that should still be looked at (go compiling C stuff without inheriting proper flags), but I don't think it's as easy as setting CGO_FLAGS to CFLAGS
2024-09-12 16:15:41 <@decathorpe:fedora.im> probably some mechanism similar to extension_cflags that's used by Python will be needed, IIUC
2024-09-12 16:16:09 <@james:fedora.im> Makes sense.
2024-09-12 16:16:16 <@james:fedora.im> I closed it.
2024-09-12 16:16:28 <@james:fedora.im> !topic FPC PR#947 https://pagure.io/packaging-committee/pull-request/947
2024-09-12 16:17:12 <@decathorpe:fedora.im> hum, this sounds mostly like a no-brainer to me.
2024-09-12 16:17:13 <@james:fedora.im> I think we just close this and have someone open a new PR if there's anything that still needs changing ... but wanted to make sure.
2024-09-12 16:18:31 <@decathorpe:fedora.im> "Therefore any occurence of the old macros (...) MUST be replaced" is a bit strong for something that just deprecates things, but I agree with the spirit of the changes
2024-09-12 16:18:37 <@james:fedora.im> Some things were done, AIUI ... was the warning added?
2024-09-12 16:19:31 <@james:fedora.im> I don't see it at: https://docs.fedoraproject.org/en-US/packaging-guidelines/AutoProvidesAndRequiresFiltering/
2024-09-12 16:19:59 <@decathorpe:fedora.im> hm. yeah it looks like the docs for the old-style generators / filtering were just dropped
2024-09-12 16:20:05 <@decathorpe:fedora.im> so no warning necessary?
2024-09-12 16:20:34 <@tibbs:fedora.im> I'm surprised the old stuff even works at all.
2024-09-12 16:20:59 <@james:fedora.im> Yeh, I tried searching and we just refer to them and don't talk about them at all.
2024-09-12 16:21:16 <@james:fedora.im> I'll just close this as done.
2024-09-12 16:22:17 <@tibbs:fedora.im> It would be possible to redefine the underlying macros to warn or error out, I guess, but I think eventually RPM will just make it "not work".
2024-09-12 16:22:26 <@james:fedora.im> !topic FPC#1053 https://pagure.io/packaging-committee/issue/1053
2024-09-12 16:22:54 <@james:fedora.im> Yeh, I'm happy to do nothing and let people blame rpm ;)
2024-09-12 16:25:37 <@james:fedora.im> I don't know much about this ... but if things still haven't changed maybe we just close this and wait for a PR?
2024-09-12 16:26:31 <@decathorpe:fedora.im> I don't think anything about the situation has changed.
2024-09-12 16:27:07 <@tibbs:fedora.im> I still like my last suggestion (error and eventually fail the build if there's something to check but the validator wasn't installed).
2024-09-12 16:27:30 <@decathorpe:fedora.im> - the tool for validation is based on the library that's exclusively used for generating repository metadata
2024-09-12 16:27:30 <@decathorpe:fedora.im> - the tool that is *not* used for validation is based on the library that's now used by *all* software centers
2024-09-12 16:27:49 <@decathorpe:fedora.im> tibbs: yeah that's an interesting solution
2024-09-12 16:28:09 <@decathorpe:fedora.im> that would be a BRP script?
2024-09-12 16:28:30 <@decathorpe:fedora.im> I can file a Change Proposal to formalize this idea
2024-09-12 16:28:32 <@tibbs:fedora.im> Yes, I think it's entirely doable.
2024-09-12 16:28:40 <@james:fedora.im> But someone mentioned that the validator that is used is stricter than the one proposed (and reading the data) ... so it's all fine?
2024-09-12 16:28:46 <@tibbs:fedora.im> Assuming that you can easily determine that there's a file which needs validation.
2024-09-12 16:29:05 <@decathorpe:fedora.im> they're both doing some checks that the other one doesn't. so it's not a strict subset relationship
2024-09-12 16:29:14 <@james:fedora.im> ahh, of course.
2024-09-12 16:29:50 <@decathorpe:fedora.im> as far as I can tell, yes, it's just files in `%{_metainfodir}` and the old location (`%{_datadir}/appdata`)
2024-09-12 16:33:45 <@james:fedora.im> I added a comment about the BRP change proposal.
2024-09-12 16:34:01 <@tibbs:fedora.im> Can't promise anything in my current state but I will try to look over the existing BRP scripts and see whether there is anything simple that can be done.  Stuff can always be added later.
2024-09-12 16:34:44 <@tibbs:fedora.im> One nice thing about doing it that way is that you can largely isolate packagers from having to care about which tool actually does the validation.
2024-09-12 16:35:04 <@james:fedora.im> !topic FPC#1049 https://pagure.io/packaging-committee/issue/1049
2024-09-12 16:35:29 <@james:fedora.im> Last old one for this week ... one of the many things I don't know much about.
2024-09-12 16:36:38 <@decathorpe:fedora.im> oh, another one I filed 
2024-09-12 16:36:43 <@decathorpe:fedora.im> :sad face:
2024-09-12 16:37:12 <@decathorpe:fedora.im> I think we just need to write some basic guidelines. GIR belongs in -devel subpackage, .vapi too, .typelib belongs in main library package, etc.
2024-09-12 16:37:18 <@decathorpe:fedora.im> I can try to work on this.
2024-09-12 16:37:42 <@carlwgeorge:matrix.org> i ran into a similar thing this week, there was an rpmlint error in review about an explicit requires on libadwaita, which has the gi typelib file.
2024-09-12 16:38:07 <@carlwgeorge:matrix.org> https://bugzilla.redhat.com/show_bug.cgi?id=2311833#c2
2024-09-12 16:38:11 <@decathorpe:fedora.im> and it annoyed rpmlint because the package name has prefix "lib*" supposedly?
2024-09-12 16:38:37 <@decathorpe:fedora.im> we might also want to add dependency generators for typelib / gir / vapi stuff too while we're at it
2024-09-12 16:38:41 <@decathorpe:fedora.im> other distros have it
2024-09-12 16:38:41 <@carlwgeorge:matrix.org> i'm assuming so, because the same thing was true for the dep on gtk4, but it didn't complain about that one
2024-09-12 16:39:07 <@decathorpe:fedora.im> yeah I think rpmlint is stupid and just matches against (lib.*|.*-libs)
2024-09-12 16:39:16 <@decathorpe:fedora.im> yeah I think rpmlint is stupid and just matches against `(lib.*|.*-libs)`
2024-09-12 16:39:28 <@james:fedora.im> Can we copy and paste things?
2024-09-12 16:39:39 <@carlwgeorge:matrix.org> indeed i think a `Requires: typelib(Adw)` would have solved the problem here, if that was provided by libadwaita
2024-09-12 16:39:58 <@decathorpe:fedora.im> probably copy-paste would work - Conan Kudo knows this better than I do
2024-09-12 16:40:06 <@decathorpe:fedora.im> I'll put it on my list! :D
2024-09-12 16:40:11 <@carlwgeorge:matrix.org> even better if generated and not explicit
2024-09-12 16:41:08 <@decathorpe:fedora.im> generating Provides should be easy, I think. Requires, not so much, you'd need to parse Python code for `from gi.repository import Gtk_4.0` or something like that
2024-09-12 16:41:42 <@carlwgeorge:matrix.org> yeah was just about to say the generated dep might not be feasible, but the generated provides should work
2024-09-12 16:42:00 <@james:fedora.im> Mmmm, sweet sweet regexps ;)
2024-09-12 16:43:02 <@james:fedora.im> !topic Open Floor
2024-09-12 16:43:19 <@james:fedora.im> Okay, not trying to do all the old things at once ... but is there anything anyone wants to talk about?
2024-09-12 16:44:22 <@decathorpe:fedora.im> hey, ticket <del>archaeology</del> triage is fun
2024-09-12 16:45:49 <@decathorpe:fedora.im> I have just today had some ideas for improving the Versioning guidelines, but I don't want to step on anybody's toes since it's already in flux
2024-09-12 16:46:46 <@carlwgeorge:matrix.org> ping me about it after the meeting and i'll let you know if it's something i'm already incorporating, or if it makes sense to
2024-09-12 16:47:13 <@carlwgeorge:matrix.org> i started with the snapshot part, but it quickly grew to a total refresh as i couldn't unsee phrasing that didn't make sense
2024-09-12 16:47:34 <@decathorpe:fedora.im> (TL;DR is incorporating %autorelease usage, which is not documented yet, and adding more "do this in this case" style examples)
2024-09-12 16:48:46 <@carlwgeorge:matrix.org> %autorelease is already covered
2024-09-12 16:48:50 <@carlwgeorge:matrix.org> https://docs.fedoraproject.org/en-US/packaging-guidelines/Versioning/#_release_tag
2024-09-12 16:49:05 <@decathorpe:fedora.im> it is? hum, I must have missed that. I had an old TODO item for this
2024-09-12 16:49:38 <@decathorpe:fedora.im> it's just basic info, no examples for snapshot versioning etc. though
2024-09-12 16:51:06 <@carlwgeorge:matrix.org> i think that will only be needed for the traditional versioning style (which i'd like to move to an appendix)
2024-09-12 16:51:20 <@decathorpe:fedora.im> true!
2024-09-12 16:51:58 <@carlwgeorge:matrix.org> with the preferred snapshot info in the version, %autorelease is dead simple.  but you're right that examples of the advanced usage should exist somewhere in the guidelines.
2024-09-12 16:53:05 <@conan_kudo:matrix.org> !hi
2024-09-12 16:53:05 <@decathorpe:fedora.im> yeah, that situation seems to be what throws off most people
2024-09-12 16:53:07 <@zodbot:fedora.im> Neal Gompa (ngompa) - he / him / his
2024-09-12 16:55:40 <@james:fedora.im> Well, now Neal is here I guess we can end the meeting ;)
2024-09-12 16:57:28 <@conan_kudo:matrix.org> we can and should consider pulling in the openSUSE generator into Fedora for this
2024-09-12 16:57:38 <@conan_kudo:matrix.org> it's kind of silly that we haven't had this for 12+ years
2024-09-12 16:57:59 <@carlwgeorge:matrix.org> works for me, are you volunteering?  😀
2024-09-12 16:58:04 <@conan_kudo:matrix.org> sure why not
2024-09-12 16:59:58 <@zodbot:fedora.im> neil has already given cookies to ngompa during the F40 timeframe
2024-09-12 17:00:02 <@zodbot:fedora.im> james gave a cookie to ngompa. They now have 142 cookies, 29 of which were obtained in the Fedora 40 release cycle
2024-09-12 17:00:40 <@james:fedora.im> Anything else anybody wants to volunteer Neal for?
2024-09-12 17:01:11 <@james:fedora.im> Technically we are out of time, so I should close anyway ... but I think the next slot is free if we want to talk an extra couple of minutes.
2024-09-12 17:01:51 <@nhanlon:beeper.com> i'd love to be a fly on the wall and provide support if ya need
2024-09-12 17:06:03 <@james:fedora.im> Okay, thanks for coming everyone
2024-09-12 17:06:05 <@james:fedora.im> !endmeeting