On Wed, Apr 4, 2018 at 2:07 AM, Taylor Blau wrote:
> `git config` has long allowed the ability for callers to provide a 'type
> specifier', which instructs `git config` to (1) ensure that incoming
> values are satisfiable under that type, and (2) that outgoing values are
> canonicalized under that
>From the output of ls-files, we remove all but the leftmost path
component and then we eliminate duplicates. We do this in a while loop,
which is a performance bottleneck when the number of iterations is large
(e.g. for 6 files in linux.git).
$ COMP_WORDS=(git status -- ar) COMP_CWORD=3; time
On Wed, Apr 4, 2018 at 2:07 AM, Taylor Blau wrote:
> [...]
> In fact, `git config` will not accept multiple type specifiers at a
> time, as indicated by:
>
> $ git config --int --bool some.section
> error: only one type at a time.
>
> This patch uses `OPT_SET_INT` to prefer the _last_ mentione
On Tue, Apr 3, 2018 at 10:47 AM, Kaartic Sivaraam
wrote:
> From: Eric Sunshine
>
> "git branch --list" shows an in-progress rebase as:
>
> * (no branch, rebasing )
> master
> ...
>
> However, if the rebase is started from a detached HEAD, then there is no
> , and it would attempt to pri
Gruß
In einer kurzen Einleitung bin ich der Anwalt Vladimir Volf aus
zamberk Tschechische Republik, aber jetzt lebe ich in London, ich habe
dir eine E-Mail über meinen verstorbenen Klienten geschickt, aber ich
habe keine Antwort von dir erhalten, der Verstorbene ist ein Bürger
Von deinem Land mit
Apply for a loan at 3% reply to this Email for more Info
On 4/3/2018 4:20 PM, Jeff King wrote:
On Tue, Apr 03, 2018 at 09:14:50AM -0400, Derrick Stolee wrote:
I'm not sure what the exact solution would be, but I imagine something
like variable-sized "struct commit"s with the parent pointers embedded,
with some kind of flag to indicate the number of p
The containment algorithm for 'git branch --contains' is different
from that for 'git tag --contains' in that it uses is_descendant_of()
instead of contains_tag_algo(). The expensive portion of the branch
algorithm is computing merge bases.
When a commit-graph file exists with generation numbers c
A commit A can reach a commit B only if the generation number of A
is strictly larger than the generation number of B. This condition
allows significantly short-circuiting commit-graph walks.
Use generation number for '--contains' type queries.
On a copy of the Linux repository where HEAD is cont
On 4/4/2018 11:45 AM, Derrick Stolee wrote:
The containment algorithm for 'git branch --contains' is different
from that for 'git tag --contains' in that it uses is_descendant_of()
instead of contains_tag_algo(). The expensive portion of the branch
algorithm is computing merge bases.
When a comm
Hi drizzd,
On Wed, 4 Apr 2018, Clemens Buchacher wrote:
> From the output of ls-files, we remove all but the leftmost path
> component and then we eliminate duplicates. We do this in a while loop,
> which is a performance bottleneck when the number of iterations is large
> (e.g. for 6 files i
Hi team,
I found myself in dear need to quickly look up mails in the public-inbox
mail archive corresponding to any given commit in git.git. Some time ago,
I wrote a shell script to help me with that, and I found myself using it a
couple of times, so I think it might be useful for others, too.
Th
03.04.2018, 23:04, "Jacob Keller" :
> On Tue, Apr 3, 2018 at 11:53 AM, Alex Ivanov wrote:
>> Hi.
>> I want to use systemd as fastcgi spawner for gitweb + nginx.
>> The traffic is low and number of users is limited + traversal bots. For
>> that reason I've decided to use following mimimal ser
On 04/04, Derrick Stolee wrote:
> On 4/4/2018 11:45 AM, Derrick Stolee wrote:
> > The containment algorithm for 'git branch --contains' is different
> > from that for 'git tag --contains' in that it uses is_descendant_of()
> > instead of contains_tag_algo(). The expensive portion of the branch
> >
I built git on a mac osx laptop and got some errors when testing.
I ran ./ci/run-build-and-tests.sh and three of the tests had failures
that appear to be associated with character encoding:
...
BUILTIN git-whatchanged
SUBDIR git-gui
SUBDIR gitk-git
SUBDIR templates
+ make --quiet te
Create a '--sort' option for ls-remote, based on the one from
for-each-ref. This e.g. allows ref names to be sorted by version
semantics, so that v1.2 is sorted before v1.10.
Signed-off-by: Harald Nordgren
---
Notes:
Started using 'ALLOC_GROW'
Documentation/git-ls-remote.txt | 12 +
I updated the code to use 'ALLOC_GROW'. I makes sense, I now I realize
why array.alloc is there ;)
Jeff, you are right that 'git ls-remote --sort=committerdate' will not
work. Do you think we need to do something about this, or it's fine
that it fails the way you showed?
On Wed, Apr 4, 2018 at 7:
Am 03.04.2018 um 15:12 schrieb Johannes Schindelin:
On Fri, 30 Mar 2018, Junio C Hamano wrote:
* js/runtime-prefix-windows (2018-03-27) 2 commits
- mingw/msvc: use the new-style RUNTIME_PREFIX helper
- exec_cmd: provide a new-style RUNTIME_PREFIX helper for Windows
(this branch uses dj/ru
Links to previous revisions:
[1]
https://public-inbox.org/git/20180402174614.ga28...@sigill.intra.peff.net/T/#m108fe8c83f3558afaea8e317e680f7eaa136e9a9
[2]
https://public-inbox.org/git/20180402211920.ga32...@sigill.intra.peff.net/T/#ma9ec4e0ce664160086e535c012e20d76822c60e5
...
[4]
https://publ
On Wed, Apr 04, 2018 at 11:45:53AM -0400, Derrick Stolee wrote:
> @@ -1615,8 +1619,20 @@ static enum contains_result contains_tag_algo(struct
> commit *candidate,
> struct contains_cache *cache)
> {
> struct contains_stack contains_stack = { 0, 0
On Wed, Apr 04, 2018 at 11:48:42AM -0400, Derrick Stolee wrote:
> > diff --git a/commit.c b/commit.c
> > index 858f4fdbc9..2566cba79f 100644
> > --- a/commit.c
> > +++ b/commit.c
> > @@ -1059,12 +1059,19 @@ int in_merge_bases_many(struct commit *commit, int
> > nr_reference, struct commit *
> >
On Wed, Apr 04, 2018 at 06:35:59PM +0200, Johannes Schindelin wrote:
> Hi team,
>
> I found myself in dear need to quickly look up mails in the public-inbox
> mail archive corresponding to any given commit in git.git. Some time ago,
> I wrote a shell script to help me with that, and I found mysel
On Tue, Apr 3, 2018 at 3:26 PM, Eddy Petrișor wrote:
> Notes:
> I am aware this test is not probably the best one and maybe I
> should have first one test that does a one level non-default, before
> trying a test with 2 levels of submodules, but I wanted to express the
> goal of the patch.
Th
On 4/4/2018 2:24 PM, Jeff King wrote:
On Wed, Apr 04, 2018 at 11:48:42AM -0400, Derrick Stolee wrote:
diff --git a/commit.c b/commit.c
index 858f4fdbc9..2566cba79f 100644
--- a/commit.c
+++ b/commit.c
@@ -1059,12 +1059,19 @@ int in_merge_bases_many(struct commit *commit, int
nr_reference, stru
On Wed, Apr 04, 2018 at 07:11:53PM +0200, Harald Nordgren wrote:
> @@ -60,6 +60,16 @@ OPTIONS
> upload-pack only shows the symref HEAD, so it will be the only
> one shown by ls-remote.
>
> +--sort=::
> + Sort based on the key given. Prefix `-` to sort in
> + descending order
On Wed, Apr 04, 2018 at 07:18:42PM +0200, Harald Nordgren wrote:
> Jeff, you are right that 'git ls-remote --sort=committerdate' will not
> work. Do you think we need to do something about this, or it's fine
> that it fails the way you showed?
It's a reasonable-sized footgun, but one that I think
On Wed, Apr 04, 2018 at 02:53:45PM -0400, Derrick Stolee wrote:
> > I'd have to do some timings, but I suspect we may want to switch to the
> > "tag --contains" algorithm anyway. This still does N independent
> > merge-base operations, one per ref. So with enough refs, you're still
> > better off
On 4/4/2018 2:22 PM, Jeff King wrote:
On Wed, Apr 04, 2018 at 11:45:53AM -0400, Derrick Stolee wrote:
@@ -1615,8 +1619,20 @@ static enum contains_result contains_tag_algo(struct
commit *candidate,
struct contains_cache *cache)
{
struct co
On Wed, Apr 04, 2018 at 03:06:26PM -0400, Derrick Stolee wrote:
> > > @@ -1615,8 +1619,20 @@ static enum contains_result
> > > contains_tag_algo(struct commit *candidate,
> > > struct contains_cache
> > > *cache)
> > > {
> > > struct co
On 4/4/2018 3:16 PM, Jeff King wrote:
On Wed, Apr 04, 2018 at 03:06:26PM -0400, Derrick Stolee wrote:
@@ -1615,8 +1619,20 @@ static enum contains_result contains_tag_algo(struct
commit *candidate,
struct contains_cache *cache)
{
struct c
On Wed, Apr 04, 2018 at 03:22:01PM -0400, Derrick Stolee wrote:
> > I don't know that the reachability property is explicitly promised by
> > your work, but it seems like it would be a natural fallout (after all,
> > you have to know the generation of each ancestor in order to compute the
> > late
On 4/4/2018 3:42 PM, Jeff King wrote:
On Wed, Apr 04, 2018 at 03:22:01PM -0400, Derrick Stolee wrote:
That is the idea. I should make this clearer in all of my commit messages.
Yes, please. :) And maybe in the documentation of the file format, if
it's not there (I didn't check). It's a very us
On Wed, Apr 04, 2018 at 03:45:30PM -0400, Derrick Stolee wrote:
> On 4/4/2018 3:42 PM, Jeff King wrote:
> > On Wed, Apr 04, 2018 at 03:22:01PM -0400, Derrick Stolee wrote:
> >
> > > That is the idea. I should make this clearer in all of my commit messages.
> > Yes, please. :) And maybe in the doc
On Wed, Apr 4, 2018 at 12:35 PM, Johannes Schindelin
wrote:
> Hi team,
>
> I found myself in dear need to quickly look up mails in the public-inbox
> mail archive corresponding to any given commit in git.git. Some time ago,
> I wrote a shell script to help me with that, and I found myself using it
Updating git-completion.bash to include instructions to change the permissions
on the file when sourcing it in your .bashrc or .zshrc file.
---
contrib/completion/git-completion.bash | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/contrib/completion/git-completion.bash
On Wed, Apr 04 2018, Stephon Harris wrote:
> Updating git-completion.bash to include instructions to change the
> permissions on the file when sourcing it in your .bashrc or .zshrc
> file.
But why is this needed? Files sourced by shells don't need to be
executable, and quoting the bash manual "T
2018-04-04 21:36 GMT+03:00 Stefan Beller :
> On Tue, Apr 3, 2018 at 3:26 PM, Eddy Petrișor wrote:
>> Notes:
>> I am aware this test is not probably the best one and maybe I
>> should have first one test that does a one level non-default, before
>> trying a test with 2 levels of submodules, but
On Sat, Mar 31, 2018 at 08:41:11PM +0200, Ævar Arnfjörð Bjarmason wrote:
> > [...]
> > * jk/drop-ancient-curl (2017-08-09) 5 commits
> > - http: #error on too-old curl
> > - curl: remove ifdef'd code never used with curl >=7.19.4
> > - http: drop support for curl < 7.19.4
> > - http: drop supp
Hi Peff,
On Wed, 4 Apr 2018, Jeff King wrote:
> On Wed, Apr 04, 2018 at 06:35:59PM +0200, Johannes Schindelin wrote:
>
> > I found myself in dear need to quickly look up mails in the public-inbox
> > mail archive corresponding to any given commit in git.git. Some time ago,
> > I wrote a shell sc
Hi Mike,
as I said here:
On Wed, 4 Apr 2018, Mike Rappazzo wrote:
> On Wed, Apr 4, 2018 at 12:35 PM, Johannes Schindelin
> wrote:
> >
> > [...]
> >
> > With --open, it opens the mail in a browser (macOS support is missing,
> > mainly because I cannot test: just add an `open` alternative to
> >
On Mon, Apr 02, 2018 at 03:05:35PM -0700, Junio C Hamano wrote:
> "git help credential" mentions protocol, host, path, username and
> password (and also url which is a short-hand for setting protocol
> and host), but not "port". And common sense tells me, when a system
> allows setting host but n
On Tue, Apr 03, 2018 at 12:23:48AM +0100, Michal Nazarewicz wrote:
> First of all, ‘git credential fill’ does not store credentials
> but is used to *read* them. The command which adds credentials
> to the helper’s store is ‘git credential approve’.
Yep, makes sense (I wish we had just called th
On Wed, Apr 4, 2018 at 1:37 PM, Eddy Petrișor wrote:
>> This patch only contains the test, I presume this goes on top of
>> https://public-inbox.org/git/20180403222053.23132-1-eddy.petri...@codeaurora.org/
>> which you plan to later submit as one patch including both the change as
>> well as
>>
On 03.04.2018 16:32, Jeff King wrote:
> I'm tempted to say we should just scrap this test. It was added
> relatively recently and only shows the fix for a pretty minor bug.
> It's probably not worth the contortions to make it race-proof.
Thanks for your reply Jeff.
It appears race condition in re
We have a great news about your E-mail address!!!
You Won $950,500.00 USD on Amnesty International
Kenya online E-mail Promotion. For more details
about your prize claims, file with the following;
Names:
Country:
Tel:
Regards,
Mr. David Ford
We have a great news about your E-mail address!!!
You Won $950,500.00 USD on Amnesty International
Kenya online E-mail Promotion. For more details
about your prize claims, file with the following;
Names:
Country:
Tel:
Regards,
Mr. David Ford
Create a '--sort' option for ls-remote, based on the one from
for-each-ref. This e.g. allows ref names to be sorted by version
semantics, so that v1.2 is sorted before v1.10.
Signed-off-by: Harald Nordgren
---
Notes:
Partial fixes from Jeff King's comments
Documentation/git-ls-remote.txt |
Without digging to much into the `ref-filter` code itself, it seems
like there is an opportunity to generalize and unfify the logic
between these two cases. As well as using `ALLOC_GROW`. But maybe that
is best left as a follow-up task? Especially since this patch focuses
on `ls-remote`. Seems poss
Create a '--sort' option for ls-remote, based on the one from
for-each-ref. This e.g. allows ref names to be sorted by version
semantics, so that v1.2 is sorted before v1.10.
Signed-off-by: Harald Nordgren
---
Notes:
Create 'ref_array_push' function in ref-filter
Documentation/git-ls-remot
Create a '--sort' option for ls-remote, based on the one from
for-each-ref. This e.g. allows ref names to be sorted by version
semantics, so that v1.2 is sorted before v1.10.
Signed-off-by: Harald Nordgren
---
Notes:
Use 'ref_array_item_push' to fix 'REALLOC_ARRAY' call in ref-filter
Docum
On Wed, Apr 04, 2018 at 03:27:48AM -0400, Eric Sunshine wrote:
> On Wed, Apr 4, 2018 at 2:07 AM, Taylor Blau wrote:
> > In this patch, we prefer `--type=[int|bool|bool-or-int|...]` over
> > `--int`, `--bool`, and etc. This allows the aforementioned other patch
> > to add `--color` (in the non-trad
On Wed, Apr 04, 2018 at 03:57:12AM -0400, Eric Sunshine wrote:
> On Wed, Apr 4, 2018 at 2:07 AM, Taylor Blau wrote:
> > [...]
> > In fact, `git config` will not accept multiple type specifiers at a
> > time, as indicated by:
> >
> > $ git config --int --bool some.section
> > error: only one ty
Hi,
I have attached a fourth re-roll of my series to introduce
"--type=" in "git config", and prefer it to "--".
In particular, since the last update, I have changed the following:
- Clearer wording in the second patch per Eric's suggestion.
- Stopped spelling the required argument to "--ty
Internally, we represent `git config`'s type specifiers as a bitset
using OPT_BIT. 'bool' is 1<<0, 'int' is 1<<1, and so on. This technique
allows for the representation of multiple type specifiers in the `int
types` field, but this multi-representation is left unused.
In fact, `git config` will n
`git config` has long allowed the ability for callers to provide a 'type
specifier', which instructs `git config` to (1) ensure that incoming
values are satisfiable under that type, and (2) that outgoing values are
canonicalized under that type.
In another series, we propose to add extend this fun
`git config` has long allowed the ability for callers to provide a 'type
specifier', which instructs `git config` to (1) ensure that incoming
values are satisfiable under that type, and (2) that outgoing values are
canonicalized under that type.
In another series, we propose to add extend this fun
I've been working on converting all of git stash to be a
builtin, however it's hard to get it all working at once with
limited time, so I've moved around half of it to a new
stash--helper builtin and called these functions from the shell
script. Once this is stabalized, it should be easier to conve
In preparation for converting the stash command incrementally to
a builtin command, this patch improves test coverage of the option
parsing. Both for having too many parameters, or too few.
Signed-off-by: Joel Teichroeb
---
t/t3903-stash.sh | 35 +++
1 file change
Add stash branch to the helper and delete the apply_to_branch
function from the shell script.
Signed-off-by: Joel Teichroeb
---
builtin/stash--helper.c | 51 +
git-stash.sh| 17 ++---
2 files changed, 53 insertions(+), 15 de
Add a bulitin helper for performing stash commands. Converting
all at once proved hard to review, so starting with just apply
let conversion get started without the other command being
finished.
The helper is being implemented as a drop in replacement for
stash so that when it is complete it can s
Add stash pop to the helper and delete the pop_stash, drop_stash,
assert_stash_ref and pop_stash functions from the shell script
now that they are no longer needed.
Signed-off-by: Joel Teichroeb
---
builtin/stash--helper.c | 41
git-stash.sh|
Add the drop and clear commands to the builtin helper. These two
are each simple, but are being added together as they are quite
related.
We have to unfortunately keep the drop and clear functions in the
shell script as functions are called with parameters internally
that are not valid when the co
On Fri, Mar 30, 2018 at 11:06:22AM -0700, Junio C Hamano wrote:
> Taylor Blau writes:
>
> > For some use cases, callers of the `git-config(1)` builtin would like to
> > fallback to default values when the slot asked for does not exist. In
> > addition, users would like to use existing type specifi
On Fri, Mar 30, 2018 at 04:23:56PM -0400, Eric Sunshine wrote:
> On Wed, Mar 28, 2018 at 9:16 PM, Taylor Blau wrote:
> > This commit (and those following it in this series) aim to eventually
> > replace `--get-color` with a consistent alternative. By introducing
> > `--default`, we allow the `--ge
On Fri, Mar 30, 2018 at 04:26:09PM -0400, Eric Sunshine wrote:
> On Wed, Mar 28, 2018 at 9:16 PM, Taylor Blau wrote:
> > In preparation for adding `--color` to the `git-config(1)` builtin,
> > let's introduce a color parsing utility, `git_config_color` in a similar
> > fashion to `git_config_`.
>
On Fri, Mar 30, 2018 at 11:09:43AM -0700, Junio C Hamano wrote:
> Taylor Blau writes:
>
> > @@ -184,6 +183,7 @@ Valid `[type]`'s include:
> > --bool-or-int::
> > --path::
> > --expiry-date::
> > +--color::
> >Historical options for selecting a type specifier. Prefer instead
> > `--type`,
>
Hi,
Attached is a fourth re-roll of my series to add "--default" and
"--type=color" to "git config" in order to replace:
$ git config --get-color [default]
with
$ git config [--default=] --type=color
Since the last revision, I have:
- Rebased it upon the latest changes from my series
In preparation for adding `--type=color` to the `git-config(1)` builtin,
let's introduce a color parsing utility, `git_config_color` in a similar
fashion to `git_config_`.
Signed-off-by: Taylor Blau
---
config.c | 10 ++
config.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/co
For some use cases, callers of the `git-config(1)` builtin would like to
fallback to default values when the variable asked for does not exist.
In addition, users would like to use existing type specifiers to ensure
that values are parsed correctly when they do exist in the
configuration.
For exam
As of this commit, the canonical way to retreive an ANSI-compatible
color escape sequence from a configuration file is with the
`--get-color` action.
This is to allow Git to "fall back" on a default value for the color
should the given section not exist in the specified configuration(s).
With the
Fixes issue with seeing `sed: RE error: illegal byte sequence` when running
git-completion.bash
---
contrib/completion/git-completion.bash | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/completion/git-completion.bash
b/contrib/completion/git-completion.bash
index b09
We have a great news about your E-mail address!!!
You Won $950,500.00 USD on Amnesty International
Kenya online E-mail Promotion. For more details
about your prize claims, file with the following;
Names:
Country:
Tel:
Regards,
Mr. David Ford
On Thu, Apr 05 2018, Stephon Harris wrote:
> Fixes issue with seeing `sed: RE error: illegal byte sequence` when running
> git-completion.bash
> ---
> contrib/completion/git-completion.bash | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/contrib/completion/git-completio
73 matches
Mail list logo