16:00:04 <geppetto> #startmeeting fpc
16:00:04 <zodbot> Meeting started Thu Oct 10 16:00:04 2019 UTC.
16:00:04 <zodbot> This meeting is logged and archived in a public location.
16:00:04 <zodbot> The chair is geppetto. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:04 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
16:00:04 <zodbot> The meeting name has been set to 'fpc'
16:00:04 <geppetto> #meetingname fpc
16:00:04 <zodbot> The meeting name has been set to 'fpc'
16:00:04 <geppetto> #topic Roll Call
16:00:13 <tibbs> Hey.
16:00:15 * limburgher here
16:00:22 <mhroncok> will be late ~5 minutes, sorry
16:00:39 * decathorpe is here
16:00:41 <geppetto> #chair tibbs
16:00:41 <zodbot> Current chairs: geppetto tibbs
16:00:44 <geppetto> #chair limburgher
16:00:44 <zodbot> Current chairs: geppetto limburgher tibbs
16:00:46 <geppetto> #chair mhroncok
16:00:46 <zodbot> Current chairs: geppetto limburgher mhroncok tibbs
16:00:49 <geppetto> #chair decathorpe
16:00:49 <zodbot> Current chairs: decathorpe geppetto limburgher mhroncok tibbs
16:05:26 <mhroncok> here
16:05:36 <geppetto> ok, I'll start
16:05:58 <geppetto> #topic Schedule
16:06:01 <geppetto> #link https://lists.fedoraproject.org/archives/list/packaging@lists.fedoraproject.org/message/FLVWSJLJDX5SKXLJYFN53L3BUX535HSD/
16:06:13 <geppetto> #topic #814 Add SELinux Independent Policy Guidelines
16:06:23 <geppetto> #link https://pagure.io/packaging-committee/pull-request/814
16:06:33 <mhroncok> still haven't met with them
16:06:43 <geppetto> Ok
16:06:49 <limburgher> No a op-op for us today?
16:06:53 <limburgher> *So
16:07:04 <geppetto> #topic Open Floor
16:07:21 <geppetto> Pretty much … we have 3 tickets tagged with meeting, but they are all waiting on something AFAIK
16:07:43 <geppetto> So if anyone wants to talk about any of them or anything else, now's the time to bring it up
16:08:18 <tibbs> I don't have anything.  I think I have an open PR.
16:08:44 <tibbs> Actually there is an interesting question in there.
16:09:05 <tibbs> Currently the manpage guidelines say you should install manpages uncompressed.
16:09:29 <tibbs> But RPM will actually recompress already-compressed manpages into its preferred format.
16:10:04 <decathorpe> so ... should we drop that requirement, or just mention that "eh, no problem, RPM will do the right thing anyway"?
16:10:22 <tibbs> I don't know if it's worth mentioning, honestly.
16:11:21 <geppetto> But we should remove the old recommendation, right?
16:11:25 <geppetto> Or not?
16:11:29 <mhroncok> we still need to mention to use glob in %files, right?
16:11:36 <tibbs> Yes, you must still use the glob.
16:11:59 <tibbs> It's just that RPM doesn't care much about what you give it; it will decompress and then gzip in the end.
16:12:06 <mhroncok> and in the sentence that explains why, maybe we can mention that rpm will make sure to compress it with the right thing?
16:12:07 <decathorpe> so .. does the recompression not happen in flatpak builds then?
16:12:55 <tibbs> The brp-compress script which does this simply doesn't look wherever flatpak installs manpages.
16:13:12 <decathorpe> makes sense
16:13:34 <tibbs> I believe it's simple to fix that but I figure the flatpak folks would be the best ones to drive that since I don't really understand what flatpak actually does under the hood.
16:14:26 <tibbs> I guess I can report to RPM upstream and see what they say.
16:15:10 <decathorpe> I think it sets %_prefix to /app
16:15:14 * geppetto nods … so do we need/want to do anything?
16:16:03 <tibbs> Yes, and so I think %__os_install_post should pass %_prefix to brp-compress so it looks in the right place.
16:16:30 <decathorpe> that's what I was thinking too
16:16:34 <tibbs> The script already accepts an argument; rpm just doesn't pass one.
16:16:38 <tibbs> I'll file a ticket.
16:16:46 <geppetto> cool
16:16:51 <decathorpe> tibbs++
16:16:51 <zodbot> decathorpe: Karma for tibbs changed to 6 (for the current release cycle):  https://badges.fedoraproject.org/tags/cookie/any
16:17:17 <tibbs> And I'll amend my PR to adapt to the fact that brp-compress will recompress manpages into its preferred format.
16:17:35 <geppetto> +1 on that PR then
16:17:59 <tibbs> We actually have a number of pending PRs.
16:18:39 <tibbs> It's not always easy to see how they link to tickets.
16:19:47 <tibbs> I think it may be worth going over the open PRs occasionally in a meeting just to make sure we're not forgetting anything.
16:19:56 <geppetto> yeh, I only include PRs on the schedule that are marked meeting
16:20:15 <geppetto> I don't mind doing that … did you want to do it today?
16:20:41 <tibbs> It really just needs someone to do it; doesn't have to be all of us.
16:21:18 * geppetto nods … having a quick look they are mostly last modified 3 months ago
16:21:24 <tibbs> https://pagure.io/packaging-committee/pull-request/919 doesn't seem to have an associated ticket.
16:21:35 <tibbs> It's a cleanup of the R stuff.
16:22:02 <tibbs> Though it is related to https://pagure.io/packaging-committee/issue/914
16:22:07 <geppetto> yeh, that's linked to ticket 914
16:22:12 * geppetto nods
16:22:30 <tibbs> So I think we actually have something to discuss here, since it was updated according to the feedback from the meeting two weeks ago.
16:23:48 <tibbs> The PR does talk about filtering errant Suggests: as we requested.
16:24:12 <tibbs> I think conceptually it's mergeable but I haven't gone over everything.
16:26:31 <geppetto> Is there an easy way to see a merged diff?
16:26:38 <tibbs> https://pagure.io/packaging-committee/pull-request/919#request_diff
16:26:59 <tibbs> Or did you want to see the rendered file after the merge?
16:29:15 <decathorpe> the changes look sane from my POV, though not everything has been updated to use sembr
16:29:27 <geppetto> No, that
16:30:38 <tibbs> Yeah, I don't have a problem with them not converting everything to sembr.  We can do that in a separate commit later.
16:31:06 <geppetto> So that PR looks fine to me (not knowing R)
16:31:30 <mhroncok> looks reasonabel to me as well, but untested
16:31:47 <tibbs> The main hangup was the issue about having Suggests: for things which aren't in the repository.
16:32:14 <tibbs> I have to assume it's been tested by those who are driving this.
16:32:22 <decathorpe> I hope so :D
16:32:29 <tibbs> Some of it simply reflects the current reality.
16:32:54 * geppetto nods
16:33:11 <tibbs> One question I have is if the "Automatically generated dependencies" section is actually true for all Fedora releases.
16:34:32 <tibbs> I don't think it works on F29, for example.
16:34:36 <tibbs> I'll ask that in the ticket.
16:35:13 <decathorpe> R-rpm-macros seems to only be available on f31+ and epel-8
16:35:49 <decathorpe> (there are koji builds for older fedoras, but no bodhi updates)
16:36:02 <smooge> ... is wondering why it is in epel-8 because R can't build for EPEL-8 ..
16:36:05 <tibbs> It's possible that they're waiting for us in some way.
16:37:05 <decathorpe> smooge: what about this then? https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2019-16908d432e
16:38:19 <smooge> I stand corrected
16:38:31 <smooge> older R needed objc and EL-8 doesn't ship with that
16:38:49 <smooge> to the people who fixed that.. I am in awe and thanks
16:39:00 <decathorpe> looks like it was spot :)
16:39:14 <smooge> so never mind and I should research more
16:44:24 <geppetto> Ok, I think we are good now :)
16:44:32 <geppetto> going to end the not so quick meeting
16:45:02 <geppetto> spot++
16:45:02 <zodbot> geppetto: Karma for spot changed to 5 (for the current release cycle):  https://badges.fedoraproject.org/tags/cookie/any
16:45:21 <geppetto> #endmeeting