17:01:48 <geppetto> #startmeeting fpc
17:01:48 <zodbot> Meeting started Thu Feb  3 17:01:48 2022 UTC.
17:01:48 <zodbot> This meeting is logged and archived in a public location.
17:01:48 <zodbot> The chair is geppetto. Information about MeetBot at https://fedoraproject.org/wiki/Zodbot#Meeting_Functions.
17:01:48 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
17:01:48 <zodbot> The meeting name has been set to 'fpc'
17:01:48 <geppetto> #meetingname fpc
17:01:48 <zodbot> The meeting name has been set to 'fpc'
17:01:48 <geppetto> #topic Roll Call
17:02:53 <tibbs> Hello.
17:02:58 <geppetto> #chair tibbs
17:02:58 <zodbot> Current chairs: geppetto tibbs
17:03:00 <geppetto> hey
17:04:36 <decathorpe> hello o/
17:05:04 <geppetto> #chair decathorpe
17:05:04 <zodbot> Current chairs: decathorpe geppetto tibbs
17:05:13 * GwynCieslasheher here
17:05:26 <geppetto> #chair GwynCieslasheher
17:05:26 <zodbot> Current chairs: GwynCieslasheher decathorpe geppetto tibbs
17:07:38 <carlwgeorge> .hi
17:07:39 <zodbot> carlwgeorge: carlwgeorge 'Carl George' <carl@redhat.com>
17:08:44 <geppetto> #chair carlwgeorge
17:08:45 <zodbot> Current chairs: GwynCieslasheher carlwgeorge decathorpe geppetto tibbs
17:11:40 <geppetto> #topic Schedule
17:11:43 <geppetto> #link https://lists.fedoraproject.org/archives/list/packaging@lists.fedoraproject.org/message/CCEMBTZ5BQPMNSO5HV65YAJXWE6SCPXU/
17:12:36 <geppetto> #topic #1159 Ban use of %configure in %prep
17:12:36 <geppetto> .fpc 1159
17:12:37 <zodbot> geppetto: Issue #1159: Ban use of %configure in %prep - packaging-committee - Pagure.io - https://pagure.io/packaging-committee/issue/1159
17:12:59 <geppetto> Is anyone against this?
17:13:18 <decathorpe> Noope. I think this is pretty uncontroversially bad :)
17:13:22 <tibbs> Not me.
17:14:05 <GwynCieslasheher> +1 to ban.
17:14:33 <carlwgeorge> +1
17:15:05 <tibbs> I can work on a PR as soon as this hellish week is over.
17:15:12 <geppetto> #action Ban use of %configure in %prep (+1:∞, 0:0, -1:0)
17:15:37 <geppetto> #topic #pr-1157 Drop redundant Source and Patch numbers
17:15:38 <geppetto> https://pagure.io/packaging-committee/pull-request/1157
17:16:53 <decathorpe> I'm confused by some uses of Patch0 %patch0 are replaced by Patch1 / %patch1
17:17:40 <GwynCieslasheher> That seems...silly.
17:18:06 <decathorpe> yeah, I don't think that is intentional.
17:18:22 <geppetto> I can see this being useful if you have a bunch of patches, and don't refer to them individually
17:18:59 <geppetto> As some wording where we say "this is allowed, but if you want to refer to specific patches/sources you should still number things"
17:19:43 <geppetto> Not sure if it was intentional or not, but it's too late to change now so w/e
17:22:32 <tibbs> In general I think this is fine, but:
17:23:18 <tibbs> I think we invite confusion when we don't number but later refer to something by number.
17:23:45 <geppetto> yeh, I think that's what I said ;)
17:24:03 <tibbs> I think we should mention this explicitly, say that in general the guidelines won't number patches but that we will if there is a need to refer to things by number.
17:24:07 <geppetto> In the FPC drinking game, I'mnot sure which of us should take a shot now ;)
17:24:09 <GwynCieslasheher> C barfs when you do that.
17:24:19 <tibbs> And then just be consistent.
17:24:20 <GwynCieslasheher> All of us, just for goo d measure.
17:24:24 <GwynCieslasheher> s/goo/good/, s/d//
17:24:28 * geppetto hi5s GwynCieslasheher
17:25:42 <GwynCieslasheher> 🍸
17:26:24 <tibbs> I think https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections is a good place to hang this.
17:27:51 <geppetto> Sure
17:29:52 <geppetto> #topic Open Floor
17:29:59 <geppetto> Anyting else anyone wants to talk about?
17:30:54 <carlwgeorge> i sent in my pr for the explicit lists thing
17:31:02 <carlwgeorge> https://pagure.io/packaging-committee/pull-request/1160
17:31:12 <carlwgeorge> i guess i should have tagged that for the meeting
17:32:56 <GwynCieslasheher> It's got +2 in the ticket,.
17:32:58 <GwynCieslasheher> s/,././
17:33:08 <decathorpe> you can have +1 from me too
17:33:45 <carlwgeorge> i think it's ready to merge unless anyone has suggestions for phrasing tweaks
17:34:02 <geppetto> Seems clear to me
17:34:26 <GwynCieslasheher> More professinal than OMG FFS don't
17:34:35 <GwynCieslasheher> s/professinal/professional/
17:35:07 <geppetto> I'd prefer more automation, and have some other way to catch problems … and I worry that people will just blindly add any commands that appear in bindir anyway … but would also like $666 million :)
17:35:09 <carlwgeorge> i give miro most of the credit as i shamelessly copied most of his phrasing
17:38:02 <geppetto> You want someone else to click merge button?
17:39:06 <geppetto> Ok, if nobody speaks up I'll close in another minute
17:39:24 <geppetto> Enjoy your bonus 20 minutes :)
17:39:34 <decathorpe> It seems people are mad about the package notes "feature"? But that's probably for FESCo and not for FPC?
17:39:37 <GwynCieslasheher> Yissssss
17:39:47 <geppetto> yeh, I looked at that
17:39:55 <geppetto> I don't think we did anything
17:40:03 <tibbs> Yeah, that probably shouldn't have gone in.
17:40:04 <geppetto> and I don't think we can fix it
17:40:20 <GwynCieslasheher> It has the word package in it ergo it's our fault.
17:40:21 <tibbs> Aside from backing it out, there isn't much we could do.
17:41:00 <tibbs> I don't think the general sentiment has gone so far as to start blaming people.
17:41:23 <tibbs> But FESCo should probably talk about whether they want it backed out.
17:41:31 <decathorpe> yeah :(
17:41:42 <decathorpe> sorry for the diversion, feel free to # endmeeting now :)
17:41:51 <geppetto> no problem
17:41:58 <tibbs> If they do, then there are plenty of people who can actually back it out.
17:42:00 <geppetto> #endmeeting