Re: Updated: cp -t patch (w/ commentary)

2001-04-26 Thread Garance A Drosihn
At 6:08 PM -0700 4/25/01, Dima Dorfman wrote: Garance A Drosihn [EMAIL PROTECTED] writes: Dimi has written one or two different patches to xargs. Did ^^^ -- should be 'a', but that's okay. :-) Note that I also wrote: If you need an immediate fix, I'll be happy to change Dimi's

Re: Updated: cp -t patch (w/ commentary)

2001-04-26 Thread David O'Brien
On Wed, Apr 25, 2001 at 06:08:15PM -0700, Dima Dorfman wrote: Dima Dorfman [EMAIL PROTECTED] P.S. obrien: that's a very clever and unintrusive way of avoiding getting two copies of a message; much better than

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread David O'Brien
On Mon, Apr 23, 2001 at 11:33:24AM -0700, John W. De Boskey wrote: After some feedback, I have changed the patch slightly. Rename -d to -t and remove the requirement for the option to have a value. I thought people generally agreed the right fix was to add functionality to `xargs', not

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Brian Somers
On Mon, Apr 23, 2001 at 11:33:24AM -0700, John W. De Boskey wrote: After some feedback, I have changed the patch slightly. Rename -d to -t and remove the requirement for the option to have a value. I thought people generally agreed the right fix was to add functionality to `xargs',

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Sheldon Hearn
On Wed, 25 Apr 2001 10:01:18 -0400, John W. De Boskey wrote: I am dealing with a production process that currently runs approximately 10 hours. (on 28 866Mhz processors, 2 Netapps). This process fell into my lap about 2 months ago. Something to consider is that you're trying to solve a

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread John W. De Boskey
Hi David, Brian, Thank you for taking the time to reply. I hope you were able to review the patch also. I am dealing with a production process that currently runs approximately 10 hours. (on 28 866Mhz processors, 2 Netapps). This process fell into my lap about 2 months ago. After

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Maxim Sobolev
John W. De Boskey wrote: Hi David, Brian, Thank you for taking the time to reply. I hope you were able to review the patch also. I am dealing with a production process that currently runs approximately 10 hours. (on 28 866Mhz processors, 2 Netapps). This process fell into my lap

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Garance A Drosihn
At 10:01 AM -0400 4/25/01, John W. De Boskey wrote: Hi David, Brian, Thank you for taking the time to reply. I hope you were able to review the patch also. Every time you have asked for people's opinions, they have said that it seems wrong to made add a specific option to the 'cp' command

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Jordan Hubbard
However, a specific hack to cp(1) is what a lot of people don't like. If FreeBSD contained every little hack every committer had used to address specific problems, it'd be a mess. I was told that the hack everyone is referring to is already implemented in several other operating systems, but

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Brian Somers
[.] The xargs weenies have also offered an explicit patch that could be tried, but that patch is being ignored by you. It is not a matter of talking ourselves to death, it's a matter that we're looking for feedback from anyone who wants to respond to the proposed xargs changes. If

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Brian Somers
It is inconceivable that the proposed patch to 'xargs' would increase your running time. I don't mean the standard '-I' change, which would certainly destroy performance, but the proposed patch to 'xargs' which solves your specific problem in a general way. I'm still curious as to why

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Warner Losh
aIn message [EMAIL PROTECTED] John W. De Boskey writes: :I must say at this point, I tend to agree with him. Basically, : my review request was skipped over and folks simply went on to : discuss/argue the merits/demerits of various patchs to xargs. The : question of whether xargs is

Re: Updated: cp -t patch (w/ commentary)

2001-04-25 Thread Dima Dorfman
Garance A Drosihn [EMAIL PROTECTED] writes: At 10:01 AM -0400 4/25/01, John W. De Boskey wrote: I have reduced the runtime of the process so far by a solid hour. My change to cp is the lowest level/minimal change fix which allows me to maintain a O(1) time constraint. I've played with

Updated: cp -t patch (w/ commentary)

2001-04-23 Thread John W. De Boskey
Hello, After some feedback, I have changed the patch slightly. Rename -d to -t and remove the requirement for the option to have a value. -t aquire the target from *argv++ instead of argv[argc--] The patch can be found at: http://people.freebsd.org/~jwd/cp-t.patch Some