git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* git archive --worktree-attributes doesn't exclude .gitattributes anymore
@ 2013-06-04 22:18 Gianfranco Costamagna
  2013-06-04 22:24 ` I: " Gianfranco Costamagna
  2013-06-05  3:13 ` Jeff King
  0 siblings, 2 replies; 6+ messages in thread
From: Gianfranco Costamagna @ 2013-06-04 22:18 UTC (permalink / raw)
  To: git@vger.kernel.org

git version 1.8.1.2
(please cc me, I'm not subscribed to this list)


Hi Developers, I write here because since my ubuntu update (quantal to raring)
and git update from 1.7.10.4-1ubuntu1 to 1.8.1.2

my export script doesn't work anymore.

I tried to put .gitattributes or .git/info/attributes, the file is the following
http://pastebin.com/irngA1L8

the git is
git clone http://boinc.berkeley.edu/git/boinc-v2.git

and the command is

git archive --prefix boinc-7.1.7+dfsg/ --format tgz -o ../boinc_7.1.7+dfsg.orig.tar.gz -9  client_release/7.1/7.1.7

The archive gets created, but every file is inside, no exclusions at all.

I suspect a regression between git 1.7 and 1.8

thanks for your time


Gianfranco

^ permalink raw reply	[flat|nested] 6+ messages in thread

* I: git archive --worktree-attributes doesn't exclude .gitattributes anymore
  2013-06-04 22:18 git archive --worktree-attributes doesn't exclude .gitattributes anymore Gianfranco Costamagna
@ 2013-06-04 22:24 ` Gianfranco Costamagna
  2013-06-05  3:13 ` Jeff King
  1 sibling, 0 replies; 6+ messages in thread
From: Gianfranco Costamagna @ 2013-06-04 22:24 UTC (permalink / raw)
  To: git@vger.kernel.org

Forgot to mention, also this command doesn't work


git archive --worktree-attributes  -v --format tgz -o ../boinc_7.1.7+dfsg.orig.tar.gz -9 --prefix boinc-7.1.7+dfsg/ client_release/7.1/7.1.7


Gianfranco



----- Messaggio inoltrato -----
> Da: Gianfranco Costamagna <costamagnagianfranco@yahoo.it>
> A: "git@vger.kernel.org" <git@vger.kernel.org>
> Cc: 
> Inviato: Mercoledì 5 Giugno 2013 0:18
> Oggetto: git archive --worktree-attributes doesn't exclude .gitattributes anymore
> 
>g it version 1.8.1.2
> (please cc me, I'm not subscribed to this list)
> 
> 
> Hi Developers, I write here because since my ubuntu update (quantal to raring)
> and git update from 1.7.10.4-1ubuntu1 to 1.8.1.2
> 
> my export script doesn't work anymore.
> 
> I tried to put .gitattributes or .git/info/attributes, the file is the following
> http://pastebin.com/irngA1L8
> 
> the git is
> git clone http://boinc.berkeley.edu/git/boinc-v2.git
> 
> and the command is
> 
> git archive --prefix boinc-7.1.7+dfsg/ --format tgz -o 
> ../boinc_7.1.7+dfsg.orig.tar.gz -9  client_release/7.1/7.1.7
> 
> The archive gets created, but every file is inside, no exclusions at all.
> 
> I suspect a regression between git 1.7 and 1.8
> 
> thanks for your time
> 
> 
> Gianfranco
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: git archive --worktree-attributes doesn't exclude .gitattributes anymore
  2013-06-04 22:18 git archive --worktree-attributes doesn't exclude .gitattributes anymore Gianfranco Costamagna
  2013-06-04 22:24 ` I: " Gianfranco Costamagna
@ 2013-06-05  3:13 ` Jeff King
       [not found]   ` <1370412601.77980.YahooMailAndroidMobile@web172702.mail.ir2.yahoo.com>
  1 sibling, 1 reply; 6+ messages in thread
From: Jeff King @ 2013-06-05  3:13 UTC (permalink / raw)
  To: Gianfranco Costamagna; +Cc: git@vger.kernel.org

On Tue, Jun 04, 2013 at 11:18:59PM +0100, Gianfranco Costamagna wrote:

> git version 1.8.1.2
> [...]
> I tried to put .gitattributes or .git/info/attributes, the file is the following
> http://pastebin.com/irngA1L8
> [...]
> The archive gets created, but every file is inside, no exclusions at all.
> 
> I suspect a regression between git 1.7 and 1.8

Yes, there was a regression with matching directory names in
gitattributes in v1.8.1.1. The regression is fixed in v1.8.1.6 and
above.

If upgrading is too hard, I think you can work around it by putting
"win_build/" instead of "win_build" in your .gitattributes (after
v1.8.1.6, both should work for your case).

-Peff

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: git archive --worktree-attributes doesn't exclude .gitattributes anymore
       [not found]   ` <1370412601.77980.YahooMailAndroidMobile@web172702.mail.ir2.yahoo.com>
@ 2013-06-05  7:10     ` Jeff King
  2013-06-05  7:20       ` Gianfranco Costamagna
  2013-06-05  7:22     ` I: " Gianfranco Costamagna
  1 sibling, 1 reply; 6+ messages in thread
From: Jeff King @ 2013-06-05  7:10 UTC (permalink / raw)
  To: Gianfranco Costamagna; +Cc: git@vger.kernel.org

On Wed, Jun 05, 2013 at 07:10:01AM +0100, Gianfranco Costamagna wrote:

> Just a question, is it possible to add a new test for handling this
> kind of regressions?

Yes, I added one in commit efa5f825.

> And second, is it possible to have a patch for this problem? I'll be
> glad to backport in debian, or to take care of updating the whole git
> package (it is already in experimental BTW)

The fix took a fair bit of refactoring; it's in commits
9db9eec..efa5f825. You can either cherry-pick that range, or just use
"git diff" to generate a patch.

However, for Debian, I don't see much point. Everything up through
unstable does not have the bug (it is all 1.7.10.x or older), and the
version in experimental is already 1.8.3. So as far as I can tell, no
Debian release is currently shipping a buggy version.

-Peff

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: git archive --worktree-attributes doesn't exclude .gitattributes anymore
  2013-06-05  7:10     ` Jeff King
@ 2013-06-05  7:20       ` Gianfranco Costamagna
  0 siblings, 0 replies; 6+ messages in thread
From: Gianfranco Costamagna @ 2013-06-05  7:20 UTC (permalink / raw)
  To: Jeff King; +Cc: git@vger.kernel.org

OH, I see...

Just ubuntu is syncing from debian experimental...
https://launchpad.net/ubuntu/+source/git




Gianfranco



>________________________________
> Da: Jeff King <peff@peff.net>
>A: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> 
>Cc: "git@vger.kernel.org" <git@vger.kernel.org> 
>Inviato: Mercoledì 5 Giugno 2013 9:10
>Oggetto: Re: git archive --worktree-attributes doesn't exclude .gitattributes anymore
> 
>
>On Wed, Jun 05, 2013 at 07:10:01AM +0100, Gianfranco Costamagna wrote:
>
>> Just a question, is it possible to add a new test for handling this
>> kind of regressions?
>
>Yes, I added one in commit efa5f825.
>
>> And second, is it possible to have a patch for this problem? I'll be
>> glad to backport in debian, or to take care of updating the whole git
>> package (it is already in experimental BTW)
>
>The fix took a fair bit of refactoring; it's in commits
>9db9eec..efa5f825. You can either cherry-pick that range, or just use
>"git diff" to generate a patch.
>
>However, for Debian, I don't see much point. Everything up through
>unstable does not have the bug (it is all 1.7.10.x or older), and the
>version in experimental is already 1.8.3. So as far as I can tell, no
>Debian release is currently shipping a buggy version.
>
>-Peff
>
>
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* I: git archive --worktree-attributes doesn't exclude .gitattributes  anymore
       [not found]   ` <1370412601.77980.YahooMailAndroidMobile@web172702.mail.ir2.yahoo.com>
  2013-06-05  7:10     ` Jeff King
@ 2013-06-05  7:22     ` Gianfranco Costamagna
  1 sibling, 0 replies; 6+ messages in thread
From: Gianfranco Costamagna @ 2013-06-05  7:22 UTC (permalink / raw)
  To: git@vger.kernel.org

Sending again, the list strips html

>Oh many thanks for your answer Jeff!
>I was wondering about a fault in my side!
>I was already building 1.8.3 when I sent the first mail...
>Anyway
 your software has a bunch of tests that make compilation a little slow 
(and no, I don't want to disable them, they are there for a really good 
reason).
>Just a question, is it possible to add a new test for handling this kind of regressions?
>And
 second, is it possible to have a patch for this problem? I'll be glad 
to backport in debian, or to take care of updating the whole git package
 (it is already in experimental BTW)
>Thanks
>G.


----- Messaggio inoltrato -----
>Da: Gianfranco Costamagna <costamagnagianfranco@yahoo.it>
>A: "peff@peff.net" <peff@peff.net>; "git@vger.kernel.org" <git@vger.kernel.org> 
>Inviato: Mercoledì 5 Giugno 2013 8:10
>Oggetto: Re: git archive --worktree-attributes doesn't exclude .gitattributes  anymore
> 
>
>
>Oh many thanks for your answer Jeff!
>I was wondering about a fault in my side!
>I was already building 1.8.3 when I sent the first mail...
>Anyway your software has a bunch of tests that make compilation a little slow (and no, I don't want to disable them, they are there for a really good reason).
>Just a question, is it possible to add a new test for handling this kind of regressions?
>And second, is it possible to have a patch for this problem? I'll be glad to backport in debian, or to take care of updating the whole git package (it is already in experimental BTW)
>Thanks
>G.
>Sent from Yahoo! Mail on Android 
>
>
>
>________________________________
> From:  Jeff King <peff@peff.net>; 
>To:  Gianfranco Costamagna <costamagnagianfranco@yahoo.it>; 
>Cc:  git@vger.kernel.org <git@vger.kernel.org>; 
>Subject:  Re: git archive --worktree-attributes doesn't exclude .gitattributes anymore 
>Sent:  Wed, Jun 5, 2013 3:13:49 AM 
>
>
>On Tue, Jun 04, 2013 at 11:18:59PM +0100, Gianfranco Costamagna wrote:
>
>> git version 1.8.1.2
>> [...]
>> I tried to put .gitattributes or .git/info/attributes, the file is the following
>> http://pastebin.com/irngA1L8
>> [...]
>> The archive gets created, but every file is inside, no exclusions at all.
>> 
>> I suspect a regression between git 1.7 and 1.8
>
>Yes, there was a regression with matching directory names in
>gitattributes in v1.8.1.1. The regression is fixed in v1.8.1.6 and
>above.
>
>If upgrading is too hard, I think you can work around it by putting
>"win_build/" instead of "win_build" in your .gitattributes (after
>v1.8.1.6, both should work for your case).
>
>-Peff
> 
>
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-06-05  7:22 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-04 22:18 git archive --worktree-attributes doesn't exclude .gitattributes anymore Gianfranco Costamagna
2013-06-04 22:24 ` I: " Gianfranco Costamagna
2013-06-05  3:13 ` Jeff King
     [not found]   ` <1370412601.77980.YahooMailAndroidMobile@web172702.mail.ir2.yahoo.com>
2013-06-05  7:10     ` Jeff King
2013-06-05  7:20       ` Gianfranco Costamagna
2013-06-05  7:22     ` I: " Gianfranco Costamagna

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).