Re: [PATCH v2] init-db: migrate to parse-options

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Junio C Hamano <gitster@xxxxxxxxx> wrote:

> Michał Kiedrowicz <michal.kiedrowicz@xxxxxxxxx> writes:
> 
> > Also add missing --bare to init-db synopsis.
> >
> > Signed-off-by: Michał Kiedrowicz <michal.kiedrowicz@xxxxxxxxx>
> > ---
> >  Documentation/git-init-db.txt |    2 +-
> >  builtin-init-db.c             |   51
> > ++++++++++++++++++++++++----------------- 2 files changed, 31
> > insertions(+), 22 deletions(-)
> 
> The subject says patch v2 but there is no description since the
> previous round, nor ...
> 
> > diff --git a/Documentation/git-init-db.txt
> > b/Documentation/git-init-db.txt index 1fd0ff2..eba3cb4 100644
> > --- a/Documentation/git-init-db.txt
> > +++ b/Documentation/git-init-db.txt
> > @@ -8,7 +8,7 @@ git-init-db - Creates an empty git repository
> >  ...
> > diff --git a/builtin-init-db.c b/builtin-init-db.c
> > index 4a56006..d68f61b 100644
> > --- a/builtin-init-db.c
> > +++ b/builtin-init-db.c
> > @@ -6,6 +6,7 @@
> > ...
> 
> ... I see any change in the blob object name recorded on the index
> line.
> 
> What is going on?

I just added "PARSE_OPT_NONEG" flag to "shared" option in reply to
Stephen's comment:

Stephen Boyd <bebarino@xxxxxxxxx> wrote:
> Or you can add PARSE_OPT_NONEG here and avoid the above comment.

Sorry I haven't described the change, I just thought it was obvious.


BTW: There is a difference:

> diff --git a/builtin-init-db.c b/builtin-init-db.c
> index 4a56006..9b1ce45 100644

and 

> diff --git a/builtin-init-db.c b/builtin-init-db.c
> index 4a56006..d68f61b 100644


-- 
Michał Kiedrowicz
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]