Re: Add + Status patches
Dear diary, on Tue, Apr 19, 2005 at 12:18:12AM CEST, I got a letter where David Greaves <[EMAIL PROTECTED]> told me that... > Junio C Hamano wrote: > > >DG> It allows: > >DG> find src -type f | git add - > > > >I am slow today, but have you considered using xargs? > > > > > > > yep thanks :) > I know you _could_ do it with xargs - but you _could_ use the raw git > commands too. This is a "be nice to the user" layer and I was > 'surprised' that neither > git add . > nor > git add -r . > worked. Actually, when I saw your patch, the xargs solution rushed through my mind but I thought that '-' might be practical too. Thinking about it, I couldn't come up with anything. So, what about instead making git add . to work? ;-) -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ C++: an octopus made by nailing extra legs onto a dog. -- Steve Taylor - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Add + Status patches
> "DG" == David Greaves <[EMAIL PROTECTED]> writes: DG> ... neither DG> git add . DG> nor DG> git add -r . DG> worked. These would be much much much nicer than pipe the list of filenames from stdin which reminds me of cpio ;-). - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Add + Status patches
Junio C Hamano wrote: DG> It allows: DG> find src -type f | git add - I am slow today, but have you considered using xargs? yep thanks :) I know you _could_ do it with xargs - but you _could_ use the raw git commands too. This is a "be nice to the user" layer and I was 'surprised' that neither git add . nor git add -r . worked. That meant that I had to fix it so I started with the ability to handle a list and, since I got a friendly response, I can hopefully move on to help make git nicer to use for mere mortals. David - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Add + Status patches
> "DG" == David Greaves <[EMAIL PROTECTED]> writes: DG> Hi Petr DG> Thankyou for the help earlier - problem resolved. DG> I have a trivial patch (attached). DG> It allows: DG> find src -type f | git add - I am slow today, but have you considered using xargs? - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: Add + Status patches
Dear diary, on Mon, Apr 18, 2005 at 10:39:00PM CEST, I got a letter where David Greaves <[EMAIL PROTECTED]> told me that... > Hi Petr Hi, > Thankyou for the help earlier - problem resolved. > > I have a trivial patch (attached). > > It allows: > find src -type f | git add - > > and fixes git status not reporting added files properly (on my debian > system it only reported the first file in .git/add-queue) Thanks. Could you please send the patches signed off and either with content-disposition: inline or in the mail body? I think it would be cleaner to do the testing for the dash in the for loop, so that I can do git add foo bar -. Also, don't forget to update git add's documentation at the top. For the usage string, I'd probably prefer (-|FILE)... thanks, -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ C++: an octopus made by nailing extra legs onto a dog. -- Steve Taylor - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html