16:59:54 #startmeeting FESCo meeting 20100108 16:59:54 Meeting started Fri Jan 8 16:59:54 2010 UTC. The chair is nirik. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:59:54 Useful Commands: #action #agreed #halp #info #idea #link #topic. 17:00:06 #meetingname fesco 17:00:06 The meeting name has been set to 'fesco' 17:00:07 * skvidal awaits 17:00:14 #chair dgilmore notting nirik skvidal Kevin_Kofler ajax pjones cwickert mjg59 17:00:14 Current chairs: Kevin_Kofler ajax cwickert dgilmore mjg59 nirik notting pjones skvidal 17:00:16 oh geez, thx nirik :) 17:00:27 * dgilmore is here 17:00:27 #topic init process 17:00:29 * skvidal is here 17:00:37 * cwickert is here too 17:00:57 GOOD MORNING IRC 17:01:35 welcome new fesco peeps 17:01:36 zonk 17:02:29 Welcome to new fesco folks, and thanks to departing ones... 17:03:03 for the record: 17:03:08 Welcome New members - Adam Jackson, Christoph Wickert, Peter Jones and Matthew Garrett 17:03:08 Farewells to departing members - Jon Stanley, Dan Horák, Jarod Wilson, and David Woodhouse 17:04:24 sorry i'm late 17:04:45 ok, I guess lets go ahead and start in. 17:04:53 #topic New Chair 17:05:03 * skvidal nominates nirik 17:05:22 * notting seconds! 17:05:27 Works for me 17:05:28 subscribe 17:05:30 I'd be happy to do it unless someone else wants it... if so they are welcome to it. ;) 17:05:38 no, that's okay, you can do it ;) 17:05:50 +1 for nirik 17:06:22 ok, will try not to screw up too badly. ;) 17:06:35 no, that's fine 17:06:40 :) 17:06:52 #topic Meeting time 17:07:01 so, do we want to keep meeting at this time? or change? 17:07:06 I've a mild preference for it not to be at 12 17:07:09 (EST) 17:07:18 +1 for nirik 17:07:21 mjg59: lunch is for the weak -- or maybe the hungry 17:07:21 It interferes with lunch... 17:07:27 mjg59: what it this in UTC? 17:07:36 cwickert: Right now we're at 1700UTC 17:07:43 right 17:07:43 2pm (EST5EDT) on a thursday would be nice. 17:07:48 1800 would suit me better 17:07:51 hmm 17:07:55 2pm thursday would be fine 17:07:59 pjones: +1 17:08:09 friday meetings suck. as do meetings during or immediately after lunch time. 17:08:22 2pm thursday doesnt really work for me 17:08:22 mjg59: 1800 UTC is fine for me, even better than 1700, but both are ok 17:08:24 whats 2pm EST in UTC? :) 17:08:30 nirik: 1900 17:08:32 1900 I think/ 17:08:35 right now, anyway ;) 17:08:35 nirik: add 5. 17:08:41 (at least for now, stupid DST) 17:08:41 19:00 UTC is very late… 17:08:53 * cwickert likes 1900 UTC 17:08:56 It's dinner time or even after dinner time around here. 17:09:01 11am (1600) would also work? 17:09:13 pjones: not on thursdays 17:09:14 or even 10:30 (1530) 17:09:27 dgilmore: I'm open to any of three other days of the week as well ;) 17:09:28 pjones: not really, still working then 17:09:28 pjones: thats on the early side for me. ;( (MST/MDT) 17:09:35 11am is a conflict for me 17:09:46 1100EST thursdays that is 17:09:54 E*T really 17:09:57 how about 18:00 UTC on fridays? (ie, just push it out one hour from now)? 17:10:00 For the record, +1 from me for nirik as a chair (and sorry for joining in late). 17:10:13 Kevin_Kofler: thanks. No problem. 17:10:19 nirik: that works for me, but I would /prefer/ non-friday. 17:10:22 From my point of view, any time between 14:30UTC and 21:00UTC works for me, except for 18:00UTC 17:10:35 Sorry, 17:00UTC 17:10:37 * notting is ok with current, but would prefer slightly later 17:10:37 * dgilmore thinks we should use something to work out a suitable time 17:10:40 how about we use whenisgood? 17:10:48 +1 17:10:49 skvidal: good plan 17:10:49 skvidal: we should 17:10:51 yeah, we could use one of those things... 17:11:09 Ok. Do that, discuss results on list, enact next week? 17:11:19 * skvidal goes to open one 17:11:27 mjg59: sounds good to me. 17:11:41 I guess I can deal with many times, no go at the moment is Tuesday between 12:00 and 15:00 UTC, otherwise I could deal with most times (though some are more inconvenient than others). 17:11:45 note that moving the fesco meeting does affect some things like packaging comittee... 17:11:59 * skvidal waits 17:12:12 nirik: Then we simply need to abolish the packaging committee 17:12:19 mjg59: :) 17:12:38 #agreed Will try and come up with a better time/day for meeting and announce it early next week. 17:12:43 The current time range is a bit annoying because it's around dinner time (and it's around lunch time in the US, so everyone kept complaining). 17:12:58 ok, shall we move on to the next topic then? 17:13:06 okay. 17:13:07 go for it 17:13:54 #topic #298 Revoke Paul Johnsons pacakger access and put him on probation. - 17:13:59 .fesco 298 17:14:00 nirik: #298 (Revoke Paul Johnsons pacakger access and put him on probation.) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/298 17:14:02 this sounds like it's been resolved 17:14:13 Yeah, I think we should defer/not deal with this now. 17:14:17 or at least delayed. 17:14:19 He's having problems with his mail server. 17:14:28 so, on to the next... sorry about that. ;) 17:14:46 Yeah, -1 to the sledgehammer approach, let's try diplomacy first. :-) 17:15:00 +1 for "defer" 17:15:07 The next one we need to defer probibly too. 17:15:10 defer. 17:15:19 #topic #278 Better Hostname - https://fedoraproject.org/wiki/Features/BetterHostname 17:15:21 nevertheless there needs something to be done, he even deleted his own changelog entries 17:15:23 .fesco 278 17:15:24 nirik: #278 (Better Hostname - https://fedoraproject.org/wiki/Features/BetterHostname) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/278 17:15:34 cwickert: yeah, he said that was a script that he's not using now 17:15:48 this is waiting on the desktop folks for some more info... it's not high priority for them. 17:16:06 However, if any folks want to add new questions or discuss it now we can. 17:16:10 This keeps coming up every week. If we're going to keep deferring it, can we flag it as deferred until updated? 17:16:22 +1 17:16:36 Though really, it seems to have been scrutinised far more than the majority of features 17:16:46 * dgilmore thinks that we should reject it due to the proposerscontinually not providing the requested info 17:17:03 mjg59: I disagree - it's just been scrutinized the normal amount and questioned 17:17:09 and the questions have not been answered 17:17:23 I'd be fine with moving it off the agenda until there is new info. 17:17:41 nirik: I think that's preferable to rejecting it 17:17:53 mjg59: part of that is probably my fault - in that he's asking anaconda to add features that aren't specified in the proposal 17:17:59 yeah, defer until updated. 17:18:06 but yeah, defer until updated. 17:18:08 * nirik isn't against it, just would like more info about corner cases, etc. 17:18:09 It'll defacto get rejected if there's no progress in implementation 17:18:53 ok, I'll removing meeting keyword and move on then. 17:19:11 #topic #299 Feature: AtSpiTwo - https://fedoraproject.org/wiki/Features/AtSpiTwo 17:19:12 defer +1 17:19:16 .fesco 299 17:19:17 nirik: #299 (Feature: AtSpiTwo - https://fedoraproject.org/wiki/Features/AtSpiTwo) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/299 17:19:18 mchua: ticket now?! as you wish... 17:19:47 Looks sane, this code has needed reworking for a long time 17:19:58 Re AtSpiTwo, that one is cool as it may allow accessibility interoperability between GTK+ and Qt apps. 17:20:04 There's a Qt implementation of it. 17:20:06 And it's what upstream are doing 17:20:14 mjg59: the code looks fine. I can't figure out why it needs a feature proposal htough. 17:20:24 seems like it's just boring development. 17:20:25 +! for it it seems like a win, and can co-exist with whats there 17:20:29 +! 17:20:31 yeah, most users won't care much, but it might be nice to tout us doing it first 17:20:34 +1 17:20:37 pjones: If it were relnoted as "Accessibility will no longer make your desktop crash randomly"? 17:20:38 I guess. 17:20:55 I hope we can get the Qt part in, but that'll be a KDE SIG task. 17:20:56 * notting is +1 17:20:56 mjg59: relnoting it is fine - we seriously need to divorce that from being a "Feature". 17:20:57 i don't have any engineering objections to it. 17:21:02 The feature page is OK as is. 17:21:04 So +1 from me. 17:21:14 In the sense that it fits the current feature process, +1 17:21:18 +1 from me too, looks sane 17:21:19 right. 17:21:28 +1 here as well. 17:21:55 #agreed The AtSpiTwo feature is accepted. 17:22:05 #link https://labs.codethink.co.uk/index.php/p/qt-atspi2/ 17:22:08 #info (the Qt implementation) 17:22:27 #topic #300 Feature: BetterWebcamSupportF13 - https://fedoraproject.org/wiki/Features/BetterWebcamSupportF13 17:22:32 .fesco 300 17:22:33 nirik: #300 (Feature: BetterWebcamSupportF13 - https://fedoraproject.org/wiki/Features/BetterWebcamSupportF13) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/300 17:23:01 Incremental improvement in hardware support? Seems worth mentioning. 17:23:43 yeah 17:23:44 Right, +1. 17:23:55 yeah, this is a add on to the f10 feature that brough in a bunch of support. 17:24:05 as I own one of these cams I'm +1 17:24:14 nice doc, no engineering objections, approve. 17:24:18 * notting is +1 17:24:32 +1 17:24:33 +1 17:24:36 * skvidal sees no reason to -1 17:24:37 so +1 17:25:03 yeah, +1 here as well... hopefully folks to who webcam support is important will see this feature and give fedora a try. 17:25:18 #agreed The BetterWebcamSupportF13 feature is approved. 17:26:01 #topic #291: Man pages Packaging Guideline 17:26:04 .fesco 291 17:26:05 nirik: #291 (Man pages Packaging Guideline) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/291 17:26:19 This was approved at the last fesco meeting, but still needs to be written up. 17:26:42 does someone want to take the task of doing so, or bugging the packaging folks to do so? 17:27:03 I can write it up 17:27:23 it may need special powers in the Packaging space of the wiki (see abadger1999 or spot) 17:27:34 Will do 17:27:45 nirik: only packaging committe can update the packaging guidelines 17:28:08 Well, I'll submit a writeup to them 17:28:08 yeah 17:28:17 thanks mjg59 17:28:20 #topic Open Floor 17:28:25 thats all I had for meeting items. 17:28:32 Anyone have any topics for Open Floor? 17:28:49 just a reminder mailing lists are about to be migrated 17:28:59 dgilmore: thanks 17:29:13 nirik: might we want to discuss not having to approve every release note as a feature? ;) 17:29:43 I think it was pretty clear from the voting period that several of us had concerns about the current feature process 17:29:51 pjones: sort of hard to know whether a feature is "just a release note" or notwithout someone reviewing it. 17:29:55 Sure, proposals for change welcome. 17:30:03 But I'm not sure IRC is the best way to start this conversation 17:30:12 mjg59: fair enough 17:30:14 also note that "release note" doesn't really carry much weight in reviews or media writeups. 17:30:15 Uh, people can add whatever they want to the release note beats already. 17:30:18 umm 17:30:28 There's no requirement that everything has to go through the feature process. 17:30:30 I don't want fesco meetings to move to phone conversations 17:30:39 skvidal: me either. ;) 17:30:40 skvidal: +1 17:30:41 mjg59: if that's what you were suggesting 17:30:41 pjones: Perhaps better to come up with some strawmen proposals first? 17:30:44 skvidal: No 17:30:47 mjg59: okay, good 17:30:47 IRC is great. 17:31:08 What I meant was that it's easier to have this kind of discussion if we have a more concrete starting point 17:31:21 I think he was suggesting emailing some strawmen out 17:31:50 okay. that's fine 17:31:55 If people can formualte a list of their perceived issues and ways that they'd fix them, we can take a bash at that and see whether we end up with something that turns out to be the current process or not 17:32:03 * nirik nods. 17:32:10 other than there being a lot of features to review 17:32:20 I don't have any specific issues with the process 17:32:53 But I don't think we need a specific action point on this 17:33:16 The people with issues (me included) should just get on with it and come up with something to present to the community/committee 17:33:20 at times it seems strange to approve or nitpick something that someone is already going to do... 17:33:24 The main weirdness about the process is that features can get rejected for various reasons, but still implemented, just not advertised. 17:33:33 Kevin_Kofler: yeah. 17:34:11 I do see we have some outstanding non meeting tickets we could go over for status, etc if folks are willing... 17:34:14 nirik: yeah, there's certainly a bright line of "will it happen and just not be noticed if it wasn't a 'Feature'", and in that case there's a strong argument that it shouldn't need to be a Feature to get relnoted. 17:34:21 I think it would make more sense to document what actually got implemented, no matter whether it made some deadline or not, and on the other hand, if we reject something due to strong technical reasons, stricter enforcement of our "no". 17:34:37 I also think that we need a process to advertise features in updates. 17:34:38 but since we've already all agreed we should come up with lists and proposals on the mailing list, I think it's maybe time to move to the next subject. 17:34:57 Especially new packages are allowed in updates, they should also be advertised as such and not as a feature of the next release months later. 17:35:03 pjones: yeah, sounds good. 17:35:47 Kevin_Kofler: that can always be fixed by changing what goes in updates 17:35:56 But that's a bad solution. 17:36:05 The problem is not that we allow those updates, they are a GOOD thing. 17:36:10 I'd prefer to /mitigate/ features in updates. 17:36:14 The problem is that we don't advertise them enough. 17:36:15 they're not a good thing - they slow down development. 17:36:27 Not from the user's perspective, they don't. 17:36:33 For the user, they speed it up. 17:36:41 They get their new feature up to 6 months earlier. 17:36:44 Kevin_Kofler: we don't have a good channel for advertising them to our end users tho... Features get media coverage and reviews and our docs. 17:36:45 that sounds like a problem on it's own 17:37:35 With sufficient work on the developer side, PackageKit could do a better job of documenting those enhancement updates. 17:37:51 For example, we could already fill in a feature page for the update which the update notes could then link to. 17:38:05 It'd just needs a process. 17:38:06 the "up to six months earlier" argument is sortof a silly math problem 17:38:20 No, it's just basic Mathematics. :-) 17:38:29 if a developer is working on a feature in N-1, then yes, they'll get that feature earlier, but there's something else that they're getting reciprically later. 17:38:40 Not necessarily. 17:38:45 Is this discussion likely to lead to a useful conclusion that's within our field of responsibility? 17:38:59 shall we move this to the lists for strawman/proposals and move on? 17:39:05 sure. 17:39:07 You're assuming that the developers working on features are a saturated resource. 17:39:17 Kevin_Kofler: of this I am sure. 17:39:19 Maybe they are now. 17:39:23 #topic #225 Bugzilla 484855 - Mediawiki Fedora-only patch 17:39:26 But if you ban feature updates, no they won't be anymore. 17:39:26 .fesco 225 17:39:28 nirik: #225 (Bugzilla 484855 - Mediawiki Fedora-only patch) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/225 17:39:31 Kevin_Kofler: any news on this? 17:40:01 it seems to have just gone back to status quo. ;( 17:40:03 No. I'm starting to think it would make more sense to appoint another mediator, as I completely and utterly failed there. 17:40:28 Too much stuff to do, and right now a broken Internet connection at home to compound things. 17:40:47 Kevin_Kofler: :( Sorry to hear it... 17:41:18 I might have more time to look into this next week, but then again I might not. 17:41:20 Anyone willing to step up and mediate? or have ideas who we could appoint to do so that has no horse in the race? 17:41:34 that bug is way into tl;dr land. someone one-line it for me. 17:41:46 tl;dr? 17:41:51 "too long; didn't read" 17:41:54 ah 17:41:56 That's the main issue, the dispute is complex. 17:42:11 It's hard to figure out who's doing the right thing there. 17:42:11 ajax: the fedora mediawiki package is packaged in weird way that some say is broken. (some parts of upstream as well). 17:42:33 maintainer is unwilling to change. 17:42:36 The maintainer says it's required for FHS compliance, upstream says that's not true. 17:42:40 hrm, seems like kevin's right about it generally being the package maintainers call, but also seems like the patch is broken. 17:43:06 And they promised in August to sort it out with each other, but nothing happened since then. 17:43:18 at the same time, not having perfect FHS compliance (though it is against the policy) isn't the worst thing to ever happen 17:43:59 pjones: FHS is a SHOULD item... 17:44:04 "Any deviation from the FHS should be rationalized when the package is reviewed." 17:44:07 right. 17:44:48 * nirik notes also that fedora's mediawiki is NOT using the fedora/epel packaging, it's using the method ricky / upstream suggest. 17:45:27 well, i don't have a horse in the race, but i'm unlikely to be able to look at it for a good three weeks 17:45:29 So, anyone willing to be a moderator? Or do we want to propose some other solution? 17:45:31 nirik: by 'fedora's', you mean fedora infrastructure? 17:45:35 notting: yes, sorry. 17:46:45 * nirik hears the chirping crickets. ;) 17:47:00 ok, we can ask around and see if we can come up with a mediator before next meeting? 17:47:57 i'd be happy to look into it, as long as no one minds that not being soon 17:48:15 well, it's waited this long... and Kevin_Kofler might be able to look further as well... 17:48:49 * nirik would be perfectly happy for ajax to look at it. 17:48:55 The bug mentioned appears to have been fixed? 17:49:24 A good start would probably be to have a separate bug that just discusses this patch 17:50:08 Although the ticket seems to have taken on that role 17:50:27 mjg59: it's not clear that it is fixed... the maintainer says so, ricky/upstream say not, or only one case of the bug. 17:50:35 there's a lot of back and forth. ;( 17:50:58 nirik: The bug was opened with a pretty defined problem report, and nobody (in the bug) appears to say that it hasn't been resolved 17:51:03 right then, i'll take a look first week of february if no one beats me to it. 17:51:26 I think it's also the case that the FHS aspect is probably a red herring 17:51:33 mjg59: it certainly appears so 17:51:42 ajax: thanks. 17:51:44 In that FHS compliance can be achieved without patches upstream doesn't like 17:52:03 #agreed ajax will take a look at the issue for FESCo 17:52:03 But the maintainer believes that the benefits of his approach outweigh the disadvantages caused in terms of upstream divergence 17:52:16 But beyond that, yeah, if ajax is going to take a look then that would be great 17:52:54 We got anything else for today? 17:53:08 * nirik looks over the open tickets at https://fedorahosted.org/fesco/report/1 17:54:10 we could discuss #297 Please consider the idea of a security (privilege escalation) policy I suppose. 17:54:25 But again, looks like more something that should get hashed out on the lists/have a proposal attached. 17:54:35 #topic Open Floor (again) 17:55:00 There's some other things that that need to be closed... I can do that over the next week 17:55:14 Does anyone have anything they would like to bring up/discuss? 17:55:32 who's starting the whenisgood thing for scheduling? 17:55:50 skvidal: ? 17:56:07 seth seemed to indicate that he would 17:56:23 * cwickert can do it, I'm already logged in atm 17:56:26 ok, sorry, missed that 17:57:02 skvidal: are you going to do the whenisgood poll or should I? 17:59:04 cwickert / skvidal: You guys can work out who is making it and mail the link to the list? 17:59:14 * nirik can make sure everyone has been added to the list. 17:59:25 sorry my network dropped for a few minutes 17:59:31 cwickert: if you'd like to make it, that's fine 17:59:40 already working on it 17:59:44 cwickert: thanks 17:59:49 nirik: all worked out 18:00:08 groovy. ;) 18:01:17 #link http://whenisgood.net/fesco-meeting 18:01:28 is it still open floor? 18:01:32 Oxf13: yep 18:01:58 quick question for FESCo, how do you feel about packages using macros for things like description, when the macro definition exists in some other package? 18:02:14 %description 18:02:14 %{_mingw32_description} 18:02:22 where that description exists somewhere else 18:02:23 that initially sets off my 'ew' meter 18:02:25 is there a buildreq relationship? 18:02:43 skvidal: there is, however the srpm that gets published out of koji will not have that description filled in 18:02:47 it's ew, but in the case where it's a suite of programs that belong together and there's a buildreq, it /kindof/ makes sense. 18:02:57 because the initial srpm creation is done without the builddeps installed, and then passed around to the builders 18:03:09 If the SRPM doesn't get the macro filled in, that's an issue somewhere indeed. 18:03:12 that's very ew. 18:03:15 Oxf13: the srpm seems like an issue 18:03:22 Oxf13: i dont find it acceptable 18:03:32 I'd blame Koji, but on the other hand having the SRPM creation pull in lots of BRs would also suck. 18:03:33 Oxf13: maybe another question - what's the justification? 18:03:37 (this just landed on my plate as "mock" bug) 18:03:37 I mean it's a frelling description 18:03:38 yeah, if the srpm winds up not having the description, that's a problem. 18:03:45 is it really that big of a deal to dupe it? 18:03:48 The justification is not to have to copy and paste the same blurb all the time, I guess. 18:03:52 skvidal: in the case of mingw - a lot of typing. 18:03:54 i would only find that acceptable if the template macro ended up in redhat-rpm-macros or something 18:03:56 or copy+paste. 18:03:58 pjones: ? 18:03:58 And not to have to do the changes everywhere. 18:04:01 which admittedly isn't that compelling 18:04:10 does the descript change A LOT? 18:04:12 and if so, why? 18:04:17 I don't think it does 18:04:18 ajax: thats the only way it could possibly be acceptable 18:04:32 I think rjones is just trying to macro out everything he possibly can to make it 'fast' to churn out packages 18:04:34 -1 - this shouldn't be allowed and someone is "over optimiizing" 18:04:43 0xf13 squash it now before it grows 18:04:46 -1 - what seth said 18:04:48 we already have template macros in r-r-m (for debuginfo packages, but still) 18:04:57 skvidal: it's mingw so that shouldn't be surprising 18:06:06 yeah, -1 don't do this. 18:06:19 gedankenexperiment: is it reasonable to have some package metadata describing the srpm construction requirements for a package? 18:06:26 so the next question is how does FESCo fix this? 18:06:38 are there other scenarios where you'd want that kind of information passed in? 18:06:44 Oxf13: ask him to pretty please change it? 18:06:47 Oxf13: we refer you to the packaging group. 18:06:58 presumably they write a sed script. 18:07:07 there are rules. this probably ought to be among them ;) 18:07:15 ajax: I like that idea. 18:07:18 pjones: 'don't be a nimrod'? I think that's a rule of life 18:07:24 I think just banning the macros in the specfiles is suboptimal. 18:07:29 skvidal: sometimes, I find it useful to be a bit more specific. 18:07:41 pjones: don't be a nimrod in packaging 18:07:54 pjones: "if you're being that clever, go find a hobby" 18:07:55 "you cannot legislate common sense". ;) 18:07:56 Oxf13: If you can come up with a good Packaging Guideline that nixes this, I'll try to get it passed. 18:08:09 this might run afoul of one of the more "fuzzy" guidelines, about readability and what not 18:08:19 Kevin_Kofler: i can't think of a motivation against the extra metadata, but i can't come up with another instance where it'd be useful. 18:08:39 I think it encourages readaility :-) (Too many macros is unreadable) 18:09:04 ajax: well, BuildRequires are that metadata. They describe what is required to be installed in order to full create the srpm 18:09:20 I could imagine adding some macros to kde-filesystem for use at SRPM build time. 18:09:24 Though right now we don't have any. 18:09:27 "if your description doesn't show up right in the srpm, the srpm is broken" 18:09:27 abadger1999: "this" in my statement meant the macros 18:09:30 Oxf13: well, we could create the srpm twice ;) 18:09:33 All our macros are only needed at package build time. 18:09:47 build it, inspect its BRs, install them, build it again, toss the chroot 18:09:54 ajax: the problem with going down this route is that some of the macros may be arch specific 18:10:06 Oxf13: Ah -- well, I agree... but I also think clearly spelling this out would be better. 18:10:10 we've never made a promise that hte srpm you get is suitable in its exact form for every arch 18:10:16 Oxf13: ick, yes. 18:10:24 The main problem I see there is that it'll slow down SRPM creation a lot. :-( 18:10:31 and I really don't want to go back to having specific srpms for each arch, as the mirrors would hate us 18:10:37 the main problem I see here is that this is crazy. 18:10:43 pjones: +1 18:10:47 Koji takes quite some time to fetch BRs. 18:10:47 I don't think the "problem" here justifies some more code. 18:10:54 nirik: +1 18:10:55 pjones: yeah, forming a guideline around that is hard :/ 18:10:56 i agree, i'm just trying to elaborate the justification. 18:11:05 so that we can write it down 18:11:06 okay 18:11:11 Maybe we could just hardcode the mingw32 macros into the buildroot for SRPM creation? 18:11:13 does the packaging guidelines have anything tlike 18:11:17 now, the buildsystem /does/ rebuild the srpm before it uses it to build the binary 18:11:23 "anything fesco or the pkg commit says is bad"? 18:11:27 We'd allow only macro and -filesystem packages, on explicit request. 18:11:28 each builder takes the canonical srpm, rebuilds it, then binary builds it 18:11:28 and if not, why not 18:11:31 I don't think that'd hurt a lot. 18:11:33 Kevin_Kofler: maybe we could just put the text for the packages into the packages. 18:11:49 skvidal: we say it's bad today, what's to stop the next package from duplicating this ? 18:11:57 adding them to f-r-c sets a terrible precedent. 18:12:02 here's a proposal: 18:12:05 Oxf13: nothing orther than people checking things 18:12:12 Oxf13: and that's the point - we cannot stop all stupid 18:12:16 (or anywhere else, really) 18:12:16 but we can catch as catch can 18:12:20 skvidal: fair enough. 18:12:21 Oxf13: If it's only small -filesystem or -macros packages, I don't see that as a big issue at all. 18:12:22 i'll write up a description of how srpms get created that mortals can understand 18:12:29 Those packages are fast to install into the buildroot. 18:12:30 skvidal: Nope. It has something about common sense or whatnot but that's not so useful ... many of the Guidelines are created /c two people disagree on what is common sense. 18:12:38 Kevin_Kofler: doesn't solve the problem unless we start forcing all those packages to be pulled in at srpm creation time 18:12:43 abadger1999: which is why the committee exists 18:12:48 Kevin_Kofler: the /initial/ srpm creation time. 18:12:50 Yeah, we'd require an explicit request to get those pulled in. 18:12:55 to decide what is the commone sense 18:12:57 and in that, i'll point out the caveats packagers should be aware of (arch-specificity, etc) 18:13:00 And then we'd just hardcode the list. 18:13:07 including this bit about macro expansion 18:13:14 I don't think those will ever grow to a problematic size. 18:13:22 1 is a problematic size 18:13:22 Right now I think there's only the MinGW macros. 18:13:27 skvidal: true... but normally, we encode the decision into a new guideline rather than having a blanket, do what we say rule. 18:13:44 so if we then say "srpms emitted by koji must meet these requirements", people can know how to make that happen 18:13:47 abadger1999: I like a do what we say rule when the rule violation is 'over optimization and ugliness' 18:13:48 It takes what to install on the builder… 10 seconds? 18:13:52 do we have a way to detect all current cases of this issue? 18:13:53 :-) 18:14:06 abadger1999: think of it like declining a patch b/c of code style 18:14:07 nirik: a big grep across all the spec files 18:14:09 nirik: sure, grep for %{ in srpm descriptions 18:14:18 abadger1999: the Packaging committee and fesco are the arbiters of taste here 18:14:19 is it only description? 18:14:21 this is godawful ugly 18:14:25 or any other fields? 18:14:26 there are likely other fields where macros in srpms can get you in trouble 18:14:37 right. 18:14:44 ajax: That won't catch everything. 18:14:47 although due to how koji builds things we mitigate most of those 18:14:52 Or actually the opposite. 18:14:56 so, perhaps we should figure out how to find them / which cause problems and then base the guideline on that? 18:14:56 It'll catch a lot of stuff that's fine. 18:15:00 Kevin_Kofler: incremental improvement is not a bad thing. 18:15:17 A lot of rdieter's packages have a %description of: %{summary}. 18:15:20 That's valid. 18:15:27 we'd have to scrub out any macro that is defined within the spec itself, or within r-r-c 18:15:27 It doesn't require anything in the buildroot. 18:15:31 Kevin_Kofler: that'll be expanded though 18:15:37 i said srpm, not spec. 18:15:38 actually yea 18:15:47 ajax: OK, I guess you're right. :-) 18:15:50 ajax is right, query the pile of srpms for unexpanded macros 18:16:02 +1 to that 18:16:15 skvidal: So.... if we want that, I think it's fesco's juridiction to say so. It's kind of a "All power for FPC derives from fesco" so fesco can add the ability for fpc to rule on matters of style. 18:16:19 at least in description/summary 18:16:36 so, perhaps a "MUST: you must run a rpm -qlivp foo.src.rpm on your koji produced src.rpm and confirm that it has no undefined macros in it" ? 18:16:51 nirik: unexpanded 18:16:55 sorry, yeah. 18:16:58 that seems pretty reasonable. 18:17:03 * nirik is out of coffee. 18:17:11 nirik: that's a way to do it. i'd kind of like that automated by autoqa, but. 18:17:18 walk before we run 18:17:20 right 18:17:23 sure, this would be another great one for autoqa. 18:17:48 sounds like an rpmguard sub-test 18:17:53 So that's what to do... what are we preventing? 18:17:54 this is a great reason why we need to stab specfiles through the heart 18:17:58 and/or rpmlint 18:17:58 frelling language of doom 18:18:09 why -l ? 18:18:21 unexpanded macros in summary + description + ? 18:18:26 filenames? 18:18:48 abadger1999: it's not clear to me what fields this affects... 18:19:00 abadger1999: perhaps someone could go play with it some and come up with a proposal for packaging? 18:19:07 abadger1999: yes, that's what we're preventing. 18:19:16 I'd like to know that before approving a Guideline. 18:19:20 unexpanded macros in our shipped srpms 18:19:52 nirik: i'd start with "anything you can see in rpm -qip foo.src.rpm" 18:20:06 so, who will take this action item? test it and propose to packaging a guideline? 18:20:10 Oxf13: What about %{?not_at_srpm_buildtime} macros? 18:20:21 $ rpm -qlivp /pub/fedora/linux/development/source/SRPMS/mingw32-* 2>/dev/null|grep '%{' |wc -l 18:20:25 Oxf13: Do you want those tobelegalor illegal? 18:20:25 24 18:20:42 abadger1999: example? 18:20:50 grrr... spaces: to be legal or illegal 18:21:09 %description 18:21:14 %{?_mingw32_description} 18:21:24 %description doesn't show up in srpm... 18:21:24 does anyone like the idea of delegating style/taste decisions to the packaging committee and dispensing with this one? 18:21:37 yes it does 18:21:38 skvidal: I think I suggested that 10 minutes ago ;) 18:21:42 i am pro-delegation. 18:21:44 er shoops 18:21:59 abadger1999: I'm not sure what you're asking. 18:21:59 yes, er shoops. 18:22:19 Oxf13: Won't that expand to nothing in the srpm? 18:22:19 but if fpc has taste, they'll shoot this down. 18:22:26 abadger1999: that won't show up in an srpm, though it'll still have an empty description which is bad. 18:22:30 abadger1999: oh the ? 18:22:33 pjones: Exactly. 18:22:38 we have templates for this already, they're in rpmdev-newspec. 18:22:40 abadger1999: and thus it is wrong. 18:22:48 yeah, harder to catch, but all the same. 18:23:25 I don't care what body takes this issue on, I'm going to close it as WONTFIX in mock 18:23:41 Oxf13: yes 18:23:55 I say we give it to FPC since it's obviously theirs. 18:24:01 yes. 18:24:14 also, where's the autoqa todo list, because this belongs on it. 18:25:12 is there going to be someone on FPC that will be willing to run with this? 18:25:14 ajax: that should go into rpmguard 18:25:43 although I'm having difficulty finding where rpmguard is housed 18:25:48 in autoqa 18:25:49 tests/ 18:25:58 I thought it was it's own project 18:26:05 its 18:26:21 umm 18:26:28 recent commits to rpmguard in autoqa 18:26:31 I'd say its there 18:26:32 Oxf13: I'd close it as NOTABUG if you consider it not worth fixing, not WONTFIX. 18:26:40 But that's just splitting hairs. ;-) 18:26:49 it's 18:26:52 :) 18:27:08 All macros in summary and description need to be expandable at srpm buildtime. rpm -qpvi [SRPM] 2>/dev/null|grep '%{' can find cases where the macro is unconditionally included. Watch out for %{?Foo} which is also wrong but will just lead to the information being left out of srpms. 18:27:19 * abadger1999 will put that on the FPC agenda 18:27:24 Kevin_Kofler: NOTABUG means I don't think it's a bug. WONTFIX means "yeah, it could be a bug, but I'm not fixing it" 18:27:32 abadger1999: excellent. Thanks. 18:27:41 abadger1999: Yeah, that makes sense, go ahead! 18:27:52 anyone have any other items they would like to discuss? (either from our trac or whatever)? 18:28:13 abadger1999: you'll have to explain why this can happen due to uninstalled buildreqs 18:28:26 abadger1999: so perhaps add in there something about built with --nodeps 18:28:29 * nirik would like any feedback on his proposal to split the rawhide repos into a subpackage, but that could be on list just fine. 18:28:39 Okay. 18:28:52 nirik: seems so far the best solution is to have it as a sub-package, but still disabled by default 18:28:57 for complete nanny-state 18:29:26 Oxf13: yeah. I was hoping to avoid the disabled, but it seems it's going to have to be that way for some use cases. ;( 18:29:42 nirik: right, either you want to protect people, or you don't :/ 18:29:47 filed rpmguard ticket: https://fedorahosted.org/autoqa/ticket/108 18:29:56 thanks ajax 18:30:01 Yeah, it should be disabled by default even if it's not installed by default. 18:30:12 Installing packages changing system configuration is evil. 18:30:13 ajax: wow, that was like all responsible and stuff (: 18:30:27 It's part of what caused the "Big PackageKit Security Fiasco" in F12. 18:30:39 Kevin_Kofler: it's not 'changing configuration' because the configuration doesn't exist without the package 18:30:44 oh, while abadger1999 is around. Any news on ticket 275? 18:30:50 .fesco 275 18:30:51 nirik: #275 (Propose a soft-path via co-maintainer status to becoming sponsored) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/275 18:31:04 * Oxf13 glares at that proposal 18:31:47 I /really/ don't like the solution of adding more groups 18:32:11 nirik: jds2001 started working on that. We got a lot of it done byend of FudCon. Wehit some hangups but I'm not sure if that's because of optional features(turing theaclsinto fsacls)or corefunctionality. 18:32:22 I'll ping jds2001 about it this week. 18:32:42 Oxf13: does the additional group cause issues with the git migration? 18:32:45 I like the idea, but like Oxf13 I really don't like the implementation 18:33:22 nirik: I haven't fully explored it, but likely not. It certainly makes things more complicated when this really feels like a social problem, not a technical one 18:33:37 Perhaps add objections/other ideas to the ticket(s) and we can discuss more next meeting? 18:34:59 sounds reasonable to me 18:35:06 * nirik proposes we close out the meeting today. Will try and be more organized next week. 18:35:29 nirik: +1 18:35:44 second 18:36:07 Thanks for coming everyone. 18:36:22 \o/ 18:36:29 #endmeeting