D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-18 Thread khanchi97 (Sushil khanchi)
khanchi97 added a comment.


  Okay, I will take care of that.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

To: khanchi97, #hg-reviewers, yuja
Cc: pulkit, yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-18 Thread pulkit (Pulkit Goyal)
pulkit added a comment.


  For future bug fixes: we don't append [bugzilla] at the end of commits which 
fixes bug, we only mention the issue number. We only use bugzilla for 
bugs/issues so an issue number in the commit message refers to the issue on 
bugzilla.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

To: khanchi97, #hg-reviewers, yuja
Cc: pulkit, yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-17 Thread khanchi97 (Sushil khanchi)
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGff36116f30f3: bundle: updates the help text for hg bundle 
(issue5744) (authored by khanchi97, committed by ).

CHANGED PRIOR TO COMMIT
  https://phab.mercurial-scm.org/D2278?vs=5794=5809#toc

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2278?vs=5794=5809

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

AFFECTED FILES
  mercurial/commands.py

CHANGE DETAILS

diff --git a/mercurial/commands.py b/mercurial/commands.py
--- a/mercurial/commands.py
+++ b/mercurial/commands.py
@@ -1157,13 +1157,15 @@
 def bundle(ui, repo, fname, dest=None, **opts):
 """create a bundle file
 
-Generate a bundle file containing data to be added to a repository.
+Generate a bundle file containing data to be transferred to another
+repository.
 
 To create a bundle containing all changesets, use -a/--all
 (or --base null). Otherwise, hg assumes the destination will have
 all the nodes you specify with --base parameters. Otherwise, hg
 will assume the repository has all the nodes in destination, or
-default-push/default if no destination is specified.
+default-push/default if no destination is specified, where destination
+is the repository you provide through DEST option.
 
 You can change bundle format with the -t/--type option. See
 :hg:`help bundlespec` for documentation on this format. By default,



To: khanchi97, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-17 Thread yuja (Yuya Nishihara)
yuja added a comment.


  Queued, thanks. Dropped `[` `]` from `[DEST]` because `[]` is the notation
  meaning that the `DEST` argument may be omitted.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

To: khanchi97, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-16 Thread yuja (Yuya Nishihara)
yuja requested changes to this revision.
yuja added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> commands.py:1165
>  (or --base null). Otherwise, hg assumes the destination will have
> -all the nodes you specify with --base parameters. Otherwise, hg
> +all the nodes you specify with --base parameters, where destination is
> +the repository you provide through [DEST] option. Otherwise, hg

where the destination repository isn't DEST. :)
`--base` and `--all` are the options incompatible with DEST.

Can you move this to the last sentence?

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

To: khanchi97, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-16 Thread khanchi97 (Sushil khanchi)
khanchi97 updated this revision to Diff 5778.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2278?vs=5760=5778

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

AFFECTED FILES
  mercurial/commands.py

CHANGE DETAILS

diff --git a/mercurial/commands.py b/mercurial/commands.py
--- a/mercurial/commands.py
+++ b/mercurial/commands.py
@@ -1157,11 +1157,13 @@
 def bundle(ui, repo, fname, dest=None, **opts):
 """create a bundle file
 
-Generate a bundle file containing data to be added to a repository.
+Generate a bundle file containing data to be transferred to another
+repository.
 
 To create a bundle containing all changesets, use -a/--all
 (or --base null). Otherwise, hg assumes the destination will have
-all the nodes you specify with --base parameters. Otherwise, hg
+all the nodes you specify with --base parameters, where destination is
+the repository you provide through [DEST] option. Otherwise, hg
 will assume the repository has all the nodes in destination, or
 default-push/default if no destination is specified.
 



To: khanchi97, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


D2278: bundle: updates the help text for hg bundle (issue5744) [bugzilla]

2018-02-16 Thread yuja (Yuya Nishihara)
yuja requested changes to this revision.
yuja added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> commands.py:1161
> +Generate a bundle file containing data to be added to the
> +destination/[DEST] which is probably a repository.
>  

"probably a repository" ?

If `--all` given, the generated bundle isn't intended to be added to the DEST 
repository.

> commands.py:1167
>  will assume the repository has all the nodes in destination, or
>  default-push/default if no destination is specified.
>  

Here the destination repository is described. I think we can instead refine
this paragraph to clarify what the DEST is.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2278

To: khanchi97, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel