git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* git fast-import: how to prevent incremental commit with no changes
@ 2014-05-05  8:53 Timo Teras
  2014-05-06 19:18 ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Timo Teras @ 2014-05-05  8:53 UTC (permalink / raw)
  To: git

Hi,

I'm trying to script a setup that would periodically import a tarball
to git with fast-import. But things do not always change, so I'd like
fast-import to be able to not do the commit in case there is no change.

That is, I'm constructing the commit with "deleteall" + importing each
object by mark after that. Now, in case nothing changed, fast-import
will happily create an empty commit for me.

Would it be possible to add some flag that would make commit fail in
case nothing changed?

Any suggestions how to do this?

Thanks,
Timo

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

* Re: git fast-import: how to prevent incremental commit with no changes
  2014-05-05  8:53 git fast-import: how to prevent incremental commit with no changes Timo Teras
@ 2014-05-06 19:18 ` Junio C Hamano
  2014-05-07  5:01   ` Timo Teras
  0 siblings, 1 reply; 3+ messages in thread
From: Junio C Hamano @ 2014-05-06 19:18 UTC (permalink / raw)
  To: Timo Teras; +Cc: git

Timo Teras <timo.teras@iki.fi> writes:

> I'm trying to script a setup that would periodically import a tarball
> to git with fast-import. But things do not always change, so I'd like
> fast-import to be able to not do the commit in case there is no change.
>
> That is, I'm constructing the commit with "deleteall" + importing each
> object by mark after that. Now, in case nothing changed, fast-import
> will happily create an empty commit for me.
>
> Would it be possible to add some flag that would make commit fail in
> case nothing changed?
>
> Any suggestions how to do this?

I am not sure if such a conditional logically belongs to what
fast-import does.  Would it be an option for your script to rewind
the HEAD after the import is done and it finds that the tarball did
not have anything interesting new?

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

* Re: git fast-import: how to prevent incremental commit with no changes
  2014-05-06 19:18 ` Junio C Hamano
@ 2014-05-07  5:01   ` Timo Teras
  0 siblings, 0 replies; 3+ messages in thread
From: Timo Teras @ 2014-05-07  5:01 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

On Tue, 06 May 2014 12:18:16 -0700
Junio C Hamano <gitster@pobox.com> wrote:

> Timo Teras <timo.teras@iki.fi> writes:
> 
> > I'm trying to script a setup that would periodically import a
> > tarball to git with fast-import. But things do not always change,
> > so I'd like fast-import to be able to not do the commit in case
> > there is no change.
> >
> > That is, I'm constructing the commit with "deleteall" + importing
> > each object by mark after that. Now, in case nothing changed,
> > fast-import will happily create an empty commit for me.
> >
> > Would it be possible to add some flag that would make commit fail in
> > case nothing changed?
> >
> > Any suggestions how to do this?
> 
> I am not sure if such a conditional logically belongs to what
> fast-import does.  Would it be an option for your script to rewind
> the HEAD after the import is done and it finds that the tarball did
> not have anything interesting new?

Yes, this is what I ended up with for now. I wanted to avoid this mostly
so that I would not need to run "git gc --prune=now" or similar after
each import (or at least not "often").

Thanks.

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

end of thread, other threads:[~2014-05-07  5:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-05  8:53 git fast-import: how to prevent incremental commit with no changes Timo Teras
2014-05-06 19:18 ` Junio C Hamano
2014-05-07  5:01   ` Timo Teras

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 NNTP newsgroup(s).