Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Christos Zoulas
In article 20150303071552.gc27...@britannica.bec.de, Joerg Sonnenberger jo...@britannica.bec.de wrote: You are missing the most important part of my mail: why did this not go to tech-userlevel first? If we want to make every single change to go through tech-userlevel, we should institute a rule

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Christos Zoulas
For actionable items: pwait: Should I remove pwait or leave it? Are there any features you want added? In general: Should we have more concrete commit rules, or do we prefer that current status quo which is to leave things to people's judgement and occasionally backout things when

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Joerg Sonnenberger
On Tue, Mar 03, 2015 at 03:58:56PM +, Christos Zoulas wrote: For actionable items: pwait: Should I remove pwait or leave it? Are there any features you want added? Bring it up on tech-userlevel. In general: Should we have more concrete commit rules, or do we prefer that current

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Greg Troxel
chris...@astron.com (Christos Zoulas) writes: If we want to make every single change to go through tech-userlevel, we should institute a rule to do so. To my knowledge we don't have yet such a rule. We already have the rest of the p* programs which originated in Solaris, we were missing this

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 14:35 schrieb Greg Troxel: chris...@astron.com (Christos Zoulas) writes: If we want to make every single change to go through tech-userlevel, we should institute a rule to do so. To my knowledge we don't have yet such a rule. We already have the rest of the p* programs

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 21:32 schrieb Greg Troxel: Marc Balmer m...@msys.ch writes: I meant that adding to base was discuss-worthy because there's a bloat or necessary question, not because of risk of breakage. Sure. So how much bloat is pwait? Is it a huge piece of software or a small

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 22:03 schrieb Greg Troxel: Marc Balmer m...@msys.ch writes: I think you contradict yourself, when you say a) new programs in base are pretty rare, and b) we have too much commit first, argue about appropriate later. Both are true; some/most commit first discuss later

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 21:22 schrieb Greg Troxel: Marc Balmer m...@msys.ch writes: Am 03.03.15 um 14:35 schrieb Greg Troxel: chris...@astron.com (Christos Zoulas) writes: If we want to make every single change to go through tech-userlevel, we should institute a rule to do so. To my

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 16:58 schrieb Christos Zoulas: For actionable items: pwait: Should I remove pwait or leave it? Are there any features you want added? no. In general: Should we have more concrete commit rules, or do we prefer that current status quo which is to leave things to

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Marc Balmer
Am 03.03.15 um 17:49 schrieb Joerg Sonnenberger: On Tue, Mar 03, 2015 at 03:58:56PM +, Christos Zoulas wrote: For actionable items: pwait: Should I remove pwait or leave it? Are there any features you want added? Bring it up on tech-userlevel. In general: Should we have more

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Greg Troxel
Marc Balmer m...@msys.ch writes: Am 03.03.15 um 14:35 schrieb Greg Troxel: chris...@astron.com (Christos Zoulas) writes: If we want to make every single change to go through tech-userlevel, we should institute a rule to do so. To my knowledge we don't have yet such a rule. We already

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Greg Troxel
Marc Balmer m...@msys.ch writes: I meant that adding to base was discuss-worthy because there's a bloat or necessary question, not because of risk of breakage. Sure. So how much bloat is pwait? Is it a huge piece of software or a small utility? I think that matters a bit. Posting a note

re: CVS commit: xsrc/external/mit/MesaLib/dist/src/mesa/main

2015-03-03 Thread matthew green
Christos Zoulas writes: Module Name: xsrc Committed By: christos Date: Tue Mar 3 21:32:27 UTC 2015 Modified Files: xsrc/external/mit/MesaLib/dist/src/mesa/main: imports.h Log Message: xmmintrin.h is a gcc-specific header it is? -rw-r--r-- 1 mrg wheel 27578 Feb 12

re: CVS commit: src/sys/external/bsd/drm2/dist/drm/radeon

2015-03-03 Thread matthew green
Taylor R Campbell writes: Module Name: src Committed By: riastradh Date: Tue Mar 3 13:57:20 UTC 2015 Modified Files: src/sys/external/bsd/drm2/dist/drm/radeon: radeon_fence.c Log Message: radeon_fence_wait returns 0, not positive, on success. i haven't confirmed yet,

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Greg Troxel
Marc Balmer m...@msys.ch writes: I think you contradict yourself, when you say a) new programs in base are pretty rare, and b) we have too much commit first, argue about appropriate later. Both are true; some/most commit first discuss later isn't about new programs. While in some cases it

Re: CVS commit: xsrc/external/mit/MesaLib/dist/src/mesa/main

2015-03-03 Thread Joerg Sonnenberger
On Wed, Mar 04, 2015 at 09:19:01AM +1100, matthew green wrote: Christos Zoulas writes: Module Name:xsrc Committed By: christos Date: Tue Mar 3 21:32:27 UTC 2015 Modified Files: xsrc/external/mit/MesaLib/dist/src/mesa/main: imports.h Log

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Robert Elz
Date:Wed, 4 Mar 2015 00:13:01 +0100 From:Joerg Sonnenberger jo...@britannica.bec.de Message-ID: 20150303231301.gb13...@britannica.bec.de | It comes with a maintainance price. Add something better a version later | and it is significantly harder to get rid of the

Re: CVS commit: src/sys/external/bsd/drm2/dist/drm/radeon

2015-03-03 Thread Taylor R Campbell
Date: Wed, 04 Mar 2015 09:24:22 +1100 from: matthew green m...@eterna.com.au Taylor R Campbell writes: Module Name: src Committed By: riastradh Date: Tue Mar 3 13:57:20 UTC 2015 Modified Files:

re: CVS commit: src/sys/external/bsd/drm2/dist/drm/radeon

2015-03-03 Thread matthew green
Taylor R Campbell writes: Date: Wed, 04 Mar 2015 09:24:22 +1100 from: matthew green m...@eterna.com.au Taylor R Campbell writes: Module Name: src Committed By:riastradh Date:Tue Mar 3 13:57:20 UTC 2015 Modified Files:

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Joerg Sonnenberger
On Tue, Mar 03, 2015 at 09:13:31PM +0100, Marc Balmer wrote: Am 03.03.15 um 17:49 schrieb Joerg Sonnenberger: On Tue, Mar 03, 2015 at 03:58:56PM +, Christos Zoulas wrote: For actionable items: pwait: Should I remove pwait or leave it? Are there any features you want added?

re: CVS commit: src/sys/external/bsd/drm2/dist/drm/radeon

2015-03-03 Thread matthew green
Taylor R Campbell writes: Module Name: src Committed By: riastradh Date: Mon Mar 2 17:53:00 UTC 2015 Modified Files: src/sys/external/bsd/drm2/dist/drm/radeon: radeon_fence.c Log Message: Return the error if there is one in radeon_fence_wait_seq. Don't just always

re: CVS commit: xsrc/external/mit/MesaLib/dist/src/mesa/main

2015-03-03 Thread Christos Zoulas
On Mar 4, 9:19am, m...@eterna.com.au (matthew green) wrote: -- Subject: re: CVS commit: xsrc/external/mit/MesaLib/dist/src/mesa/main | | Christos Zoulas writes: | Module Name:xsrc | Committed By: christos | Date: Tue Mar 3 21:32:27 UTC 2015 | | Modified Files:

Re: CVS commit: src/usr.bin/pwait

2015-03-03 Thread Christos Zoulas
On Mar 3, 8:35am, g...@ir.bbn.com (Greg Troxel) wrote: -- Subject: Re: CVS commit: src/usr.bin/pwait | We do sort of have a rule, which is that significant changes need | discussion. I would say adding programs to base always counts. Other | than that, it's trickier, but if there's a