FESCo-2009-03-03

--- nirik has changed the topic to: FESCo special session for features
nirikwho all is here for the special session?
* j-rod here
sharkcz here
nirik is here.
j-rodmay have to manually poke some folks who didn't notice the channel change
j-rodbpepple: poke, prod
* j-rod pokes jwb in #fedora-kernel
jwbi'm 1/2 here.  on phone
nirikbpepple was going to be a few late.
j-rodjust poked dennis
* nirik notes we need at least one more to have enough people I think.
* bpepple is gets here finally.
nirikcool... so we have 4.5 out of 9?
j-rodyep, that's what I came up w/ too
didn't see any votes from jds come in via trac
nirikyeah, me either.
bpeppleSo, where does that leave us?
nirikwell, it would be nice to get at least one more person here IMHO
j-rodI can't remember... do we need >50% or >60% for quorum?
jwbyou need at least 5
bpepplejwb: correct.  more than 50% of FESCo.
j-rodok, that's what I thought. and then to approve anything, all 5 would have to vote +1
well, we got 5...
j-rodjust start through stuff, and anything that doesn't get +5 gets postponed?
bpeppleyup.  Lets' try to get through what we can.  Features we can't agree upon we can then bring to the rest of FESCo on the mailing list.
j-rodhey! 6
nirikyep. Lets go...
nirik.fesco 90
zodbotnirik: #90 (FEATURE: http://fedoraproject.org/wiki/Features/DBusPolicy) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/90
nirikok, so is this a feature?
j-rod+1, seems like a kinda nasty security hole
well, +1 for doing it
bpeppleI'd normally say this isn't a feature, but it's a pretty big security hole.
j-rodbut yeah, I question whether this is really a feature
nirikI guess it does require a lot of coordination...
sharkczand testing ...
bpeppleI'd lean towards it is a feature, just due to how big a part of the distro dbus is.
nirikyeah, guess so.
jwb+1
nirik+1 here
bpepple+1
sharkcz+1 here too
* bpepple thinks we have five +1's.
dgilmoreim +1
sharkcznow it's 6 +1
nirikok, this feature passes.
nirik.fesco 91
zodbotnirik: #91 (FEATURE: http://fedoraproject.org/wiki/Features/DRI2) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/91
j-rod+1
sharkczyes, +1
nirikrelease notes need fleshing out.
bpepple+1 here also.
nirik+1 here tho with that noted.
jwb+1
dgilmore+1
nirikI see +6, so this is approved
nirik.fesco 92
zodbotnirik: #92 (FEATURE: http://tinyurl.com/bxazrx) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/92
j-rod+1
nirikthis is cool, but the low % has me a bit worried.
dgilmore-1  65% done and it needs to land today
j-rodyeah, there's that
* j-rod pings halfline
j-rodbut if the 65% is 'multiple stacks patches enabled and you can use the fingerprint stuff' and the remaining 35% is mostly the smartcard stuff, that's not bad
dgilmoreif one stack works thats enough i think
nirikyeah, if halfline is around some info would be good.
j-rodI'm guessing he's down lunching. table it for now, come back to it after we go through others?
niriksure.
nirik.fesco 93
zodbotnirik: #93 (FEATURE: http://fedoraproject.org/wiki/Features/NetworkManagerIPv6) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/93
nirikoh, that got pulled.
nirik.fesco 94
zodbotnirik: #94 (FEATURE: http://fedoraproject.org/wiki/Features/Radeon3DUpdate) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/94
j-rodthis is the big source of all the people complaining about 3D performance regressions, isn't it?
jwbyeah
i'm +1 on this
sharkcz+1
bpepple+1
nirik+1 sounds good.
dgilmoreim +1
j-rodI'm mostly for it, but would like to know that we're going to be addressing the performance hit
dgilmoreid like it at 90%
j-rodI guess that's at least somewhat covered...
dgilmore: I'm wondering if the remaining 20% is mostly "fix regressions"
* j-rod pokes airlied...
dgilmorej-rod: its 2am thee
there
well 2:31am
realistically it should be at 100% today
sharkczisn't the 100% deadline next week?
nirikwell, it should be "testable" now.
j-rod+1 from me, on the assumption that the merged code is testable right now, and the remaining percentage points are 'fixing regressions'
nirikNew features must be feature complete or close enough to completion by Beta freeze that a majority of its functionality can be suitably tested--the "feature is testable"
ok, so thats +6, this feature is approved.
nirik.fesco 95
zodbotnirik: #95 (FEATURE: http://fedoraproject.org/wiki/Features/TigerVNC) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/95
sharkczI have some details
dgilmoredidnt we have a different VNC feature
j-roddgilmore: yep, it was TightVNC, feature page got renamed to TigerVNC
i.e., this is it
nirikwhich is a fork
sharkczas I have already wrote in the mail, the feature owner is one the developers, and he paired together with few others to create a fork
j-rodbut we're revisiting because of the change
* dgilmore says -1 to this version
j-rodsharkcz: so I'm assuming nothing major really changed wrt what the actual shipped code will be
bpeppleIsn't this fork pretty much exactly what we approved the first time?
j-rodwe'd just be using a 'tigervnc' tarball instead of tightvnc + patches
sharkczthe package is exactly the same, only with s/tightvnc/tigervnc/g
nirikwell, upstream has not yet had a formal release it seems... and the package is still under review. So is is testable now?
sharkczyes, it is testable
bpepplej-rod: right, if that is the case I don't have any issues with the rename.
sharkcztighvnc was a post-release snapshot
j-rodwell, technically, its not testable if you can't 'yum install tigervnc', right?
dgilmorej-rod: correct
sharkczj-rod: I am going to do the review today
dgilmorewhy was a fork created?
j-rodcuttin' it close, eh? ;)
the wiki page says "TigerVNC is RealVNC fork"... the tigervnc site says its a tightvnc fork...
sharkczdgilmore: https://sourceforge.net/mailarchive/forum.php?thread_name=alpine.LFD.2.00.0902271116020.25749%40maggie.lkpg.cendio.se&forum_name=tigervnc-users
unfortunately I wasn't able to catch the developer today
dgilmoresharkcz: to me thats really not a good enough reason
but the details may be missing
nirikso it sounds like upstream plans no release based on trunk...
dgilmore: yeah, they might have been diplomatic in there and there might be more going on.
dgilmoreI guess we are missing some details that that email doesnt cover
j-rodI still don't quite understand... what's the link between TigerVNC and RealVNC and TightVNC?
dgilmorej-rod: its a fork on TightVNC
j-rodbut then... why does the wiki page say its a fork of RealVNC?
or is TightVNC a fork of RealVNC
and the wiki page didn't get updated correctly?
dgilmorej-rod: the email did say TightVNC was a fork of something
niriknotting: we are talking about http://fedoraproject.org/wiki/Features/TigerVNC now
sharkczAFAIK TigerVNC is a fork of TightVNC and TightVNC is a fork of RealVNC
jwbso if you fork a fork, it's really not a fork of anything is it?
nirikif you fork a fork, you're forked? ;)
j-roddevil's pitchfork
jwbsomething like that :)
nirikin any case, I think this feature is really really close to the wire... I suppose if the package gets approved/built today it could still make it.
* dgilmore still says -1
nirikcould this package parallel install with the existing vnc?
sharkczno
j-rodI'm of the mind that since this isn't really any different than the tightvnc feature other than in name, its fine if it actually get in today
nirikit would be nice if it could, then I would say punt it to next release and get it more testing before it became default.
bpepplej-rod: +1
nottingdidn't we approve tightvnc?
niriknotting: yeah, this is a fork of that. They renamed the feature based on the new name
sharkczthe already present tighvnc obsoleted vnc from F-10
* dgilmore says we use the existing VNC feature and strongly suggest that this fork be disbanded. and that they try resolve the issues upstream to fix the issues
sharkczbut the existing one is tighvnc
nirikright, so tightvnc is already in, but it's a release that upstream tightvnc will never make?
sharkczprobably, but I don't know the details
j-rodand in most ways, save name, is identical to tigervnc, aiui
I have to assume the tigervnc folks have *tried* to resolves the issues w/the tightvnc upstream
could well end up being something like compiz->compiz & beryl->compiz
nirikwell, given that we approved the old one and it's already in, I guess I would say +1 if it's testable by today. But it would be good to hear more about why a fork is happening. ;(
j-rodditto
bpepple+1 here also.
nottingthe reading of the tigervnc announcement is that tightvnc isn't ever releasing anything?
j-rod+1 for me
sharkcz+1
notting+1
nirikjwb?
jwbi'm ok with this one, but i dislike the confusion it has caused.  +1
nirikok, 6 +1 and 1 -1. It passes... but note that it must finish review and be testable.
sharkczI was telling him to put an explanation paragraph to the feature page ...
nirikback to:
nirik.fesco 92
zodbotnirik: #92 (FEATURE: http://tinyurl.com/bxazrx) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/92
nirikFeatures/MultiplePAMStacksInGDM
halfline: we had some questions about how far along/completed this is...
j-rodhalfline: so there was some concern about it being listed as 65% complete
heh
i.e., how testable is it right now?
what's missing, etc
halflineso i actually bumped it up to 85% today
dgilmorewhat is remaining?
halflineit's not testable yet. right now it's all sitting on a git branch
basically i need to finish up a few odds an ends in the gdm code
dgilmorehalfline: it needs to be testable today
halflinewrite the authconfig code
get the authconfig code okay'd by t8m
and build packages
dgilmoreToday is Feature Freeze
nirik"New features must be feature complete or close enough to completion by Beta freeze that a majority of its functionality can be suitably tested--the "feature is testable""
halflinewell it's close, but it's not there yet
dgilmorehalfline: so even a single working stack would suffice
halflinewell what's in rawhide right now uses a single stack
but that's not the new code
dgilmorehalfline: single stack using the new code
can you land that today?
halflineugh, maybe
I mean i could pull a mega patch from git
apply it to the srpm
build a plugin into the main package (without doing a subpackage)
by today pretty easily
that's a little groady though
dgilmorethat would make the feature testable
you then get a week to do it right
if that can happen im +1
halflineokay i can do that
bpepple+1
notting+1
j-rod+1
sharkcz+1
nirik+1
I see 6 +1's, so this feature is approved, provided it can be testable. ;)
I think thats all the ones we had on the docket for today. Does anyone see any more or anything else we should go over?
* bpepple doesn't have anything.
* dgilmore has nothing
nirikok, lets call it a meeting then. Who can do the minutes/summary?
sharkczI already did 2 rounds, so the next one please
bpeppleI can do it.
j-rodI haven't done any yet, guess I could...
niriksharkcz: I'm gonna run the cvs queue later this afternoon... ping me when that review is done so I can process it after it's approved... so it's sure to get in today.
bpepplej-rod: you want it?  I'm fine with letting you. ;)
j-rodbpepple: want? no... but you've done more than enough of your share :)
* mclasen finds the sekrit meeting room
j-rodso yeah, I'll do it
bpepplej-rod: cool, thanks!
nirikthanks j-rod!
ok, thanks everyone.
---MEETING END---
bpepplemclasen: we just finished the meeting.  was there anything you wanted to discuss?
nirik: thanks for running the meeting.

Generated by irclog2html.py 2.5 by Marius Gedminas - find it at mg.pov.lt!