On Thu, Nov 2, 2017 at 10:18 PM, Junio C Hamano wrote:
> Jacob Keller writes:
>
>> I think both questions are valuable, the first is "which commit last
>> had this blob", the second is "which commit first introduced this
>> blob", neither of which can always give a definitive answer. It really
>
Map the old address to the new, hopefully more permanent one.
Signed-off-by: Kaartic Sivaraam
---
.mailmap | 1 +
1 file changed, 1 insertion(+)
diff --git a/.mailmap b/.mailmap
index ab85e0d16..2634baef2 100644
--- a/.mailmap
+++ b/.mailmap
@@ -113,6 +113,7 @@ Junio C Hamano
Junio C Hamano
Jacob Keller writes:
> I think both questions are valuable, the first is "which commit last
> had this blob", the second is "which commit first introduced this
> blob", neither of which can always give a definitive answer. It really
> depends on what question you're asking up front.
Given that
On Friday 03 November 2017 12:12 AM, Stefan Beller wrote:
On Thu, Nov 2, 2017 at 1:39 AM, Kaartic Sivaraam
wrote:
On Thursday 02 November 2017 12:24 PM, Kaartic Sivaraam wrote:
Signed-off-by: Kaartic Sivaraam
Signed-off-by: Kaartic Sivaraam
I just now saw this small glitch as a consequen
On Thursday 02 November 2017 07:51 PM, Eric Sunshine wrote:
Nicely explained; easily understood.
Good to hear that.
Translators can correct me, but this smells like "sentence lego"[1],
which we'd like to avoid. Translators lack full context when presented
with bits and pieces of a sentenc
On Thu, Nov 2, 2017 at 6:46 PM, Junio C Hamano wrote:
> Jacob Keller writes:
>
>> I agree, "something" is better than "nothing", and we can work to
>> improve "something" in the future, especially after we get more real
>> use and see what people think. Only question would be how much do we
>> ne
On Thu, Nov 2, 2017 at 6:26 PM, SZEDER Gábor wrote:
> On Thu, Nov 2, 2017 at 7:10 PM, Stefan Beller wrote:
>
>> +blame.blankBoundary::
>> + Show blank SHA-1 for boundary commits in linkgit:git-blame[1].
>
> This is still SHA-1 instead of object id (or perhaps "commit object
> name" would be
Johannes Schindelin writes:
>> If I was correct in assuming that "2>&1" is just as foreign as
>> ">/dev/null", then we should be shunning "2>&1" just like we shun
>> ">/dev/null". That was all I meant to say.
>
> Did you know that `2>&1` works in Powershell?
No. And that makes me curious if ">
Simon Ruderich writes:
> On Thu, Nov 02, 2017 at 02:16:52PM +0900, Junio C Hamano wrote:
>> Junio C Hamano writes:
>>> This patch is incomplete without adjusting a handful of tests to
>>> expect the updated messages, no?
>>
>> I'll squash these in while queuing, but there might be more that I
>>
Jacob Keller writes:
> I agree, "something" is better than "nothing", and we can work to
> improve "something" in the future, especially after we get more real
> use and see what people think. Only question would be how much do we
> need to document the current behavior might be open for improvem
On Thu, Nov 2, 2017 at 7:10 PM, Stefan Beller wrote:
> +blame.blankBoundary::
> + Show blank SHA-1 for boundary commits in linkgit:git-blame[1].
This is still SHA-1 instead of object id (or perhaps "commit object
name" would be even better).
Not sure whether oversight or intentional.
Commit ce9b8aab5 (setup_git_directory_1(): avoid changing global state,
2017-03-13) changed how the git directory is discovered, and as a side
effect when the discovery reaches the root directory Git tries to
access paths like '//HEAD' and '//objects'. This interacts badly with
Cygwin, which inter
On Thu, Nov 02, 2017 at 11:45:55PM +, Andrew Baumann wrote:
> I have a workaround for this, but someone on stack overflow [1]
> suggested reporting it upstream, so here you go:
>
> I have a fancy shell prompt that executes "git rev-parse
> --is-inside-work-tree" to determine whether we're cur
On Thu, Nov 2, 2017 at 12:41 PM, Stefan Beller wrote:
> Thanks for the discussion on v2[1].
>
> Interdiff is below, just fixing minor things.
>
> We'll keep the original algorithm for now, deferring an improvement on
> the algorithm front towards a future developer.
>
I agree, "something" is bett
Hello,
Do you need a loan?
Regards
Chen Alex
Prime Funds
Hi,
I have a workaround for this, but someone on stack overflow [1] suggested
reporting it upstream, so here you go:
I have a fancy shell prompt that executes "git rev-parse --is-inside-work-tree"
to determine whether we're currently inside a working directory. This causes
git to walk up the d
On Thu, 2 Nov 2017 20:31:15 +
Jeff Hostetler wrote:
> From: Jeff Hostetler
>
> This is part 3 of 3 for partial clone.
> It assumes that part 1 [1] and part 2 [2] are in place.
>
> Part 3 is concerned with the commands: clone, fetch, upload-pack, fetch-pack,
> remote-curl, index-pack, and
On 2017-11-02 22:31:02, Thomas Adam wrote:
> On Thu, Nov 02, 2017 at 06:26:43PM -0400, Antoine Beaupré wrote:
>> On 2017-11-02 22:18:07, Thomas Adam wrote:
>> > Hi,
>> >
>> > On Thu, Nov 02, 2017 at 05:25:18PM -0400, Antoine Beaupré wrote:
>> >> +print {*STDERR} "$#{$mw_pages} found in name
On 2017-11-02 18:43:00, Eric Sunshine wrote:
> On Thu, Nov 2, 2017 at 5:25 PM, Antoine Beaupré wrote:
>> Virtual namespaces do not correspond to pages in the database and are
>> automatically generated by MediaWiki. It makes little sense,
>> therefore, to fetch pages from those namespaces and the
On Thu, Nov 2, 2017 at 5:25 PM, Antoine Beaupré wrote:
> Without this, the fetch process seems hanged while we fetch page
> listings across the namespaces. Obviously, it should be possible to
> silence this with -q, but that's an issue already present everywhere
> in the code and should be fixed s
On Thu, Nov 2, 2017 at 5:25 PM, Antoine Beaupré wrote:
> Ideally, we'd process them in numeric order since that is more
> logical, but we can't do that yet since this is where we find the
> numeric identifiers in the first place. Lexicographic order is a good
> compromise.
>
> Reviewed-by: Eric Su
On Thu, Nov 2, 2017 at 5:25 PM, Antoine Beaupré wrote:
> When we specify a list of namespaces to fetch from, by default the MW
> API will not fetch from the default namespace, refered to as "(Main)"
> in the documentation:
>
> https://www.mediawiki.org/wiki/Manual:Namespace#Built-in_namespaces
>
>
On Thu, Nov 2, 2017 at 5:25 PM, Antoine Beaupré wrote:
> Virtual namespaces do not correspond to pages in the database and are
> automatically generated by MediaWiki. It makes little sense,
> therefore, to fetch pages from those namespaces and the MW API doesn't
> support listing those pages.
>
>
On Thu, Nov 02, 2017 at 06:26:43PM -0400, Antoine Beaupré wrote:
> On 2017-11-02 22:18:07, Thomas Adam wrote:
> > Hi,
> >
> > On Thu, Nov 02, 2017 at 05:25:18PM -0400, Antoine Beaupré wrote:
> >> +print {*STDERR} "$#{$mw_pages} found in namespace
> >> $local_namespace ($namespace_id)\n";
>
On 2017-11-02 22:18:07, Thomas Adam wrote:
> Hi,
>
> On Thu, Nov 02, 2017 at 05:25:18PM -0400, Antoine Beaupré wrote:
>> +print {*STDERR} "$#{$mw_pages} found in namespace $local_namespace
>> ($namespace_id)\n";
>
> How is this any different to using warn()? I appreciate you're using a
>
On Thu, 2 Nov 2017 20:20:44 +
Jeff Hostetler wrote:
> From: Jeff Hostetler
>
> Introduce the ability to have missing objects in a repo. This
> functionality is guarded by new repository extension options:
> `extensions.partialcloneremote` and
> `extensions.partialclonefilter`.
Wi
Hi,
On Thu, Nov 02, 2017 at 05:25:18PM -0400, Antoine Beaupré wrote:
> +print {*STDERR} "$#{$mw_pages} found in namespace $local_namespace
> ($namespace_id)\n";
How is this any different to using warn()? I appreciate you're using a
globbed filehandle, but it seems superfluous to me.
Ki
On Thu, Nov 2, 2017 at 8:34 PM, Stefan Beller wrote:
> On Thu, Nov 2, 2017 at 1:30 AM, Orgad Shaneh wrote:
>> I can't reproduce this with a minimal example, but it happens in my project.
>>
>> What I tried to do for reproducing is:
>> rm -rf super sub
>> mkdir sub; cd sub; git init
>> git commit
my name is Mrs. Alice Walton, a business woman an America Citizen and the
heiress to the fortune of Walmart stores, born October 7, 1949. I have a
mission for you worth $100,000,000.00(Hundred Million United State Dollars)
which I intend using for CHARITY
Virtual namespaces do not correspond to pages in the database and are
automatically generated by MediaWiki. It makes little sense,
therefore, to fetch pages from those namespaces and the MW API doesn't
support listing those pages.
According to the documentation, those virtual namespaces are curren
This should be the final roll of patches for namespace support. I
included the undef check even though that problem occurs elsewhere in
the code. I also removed the needless "my" move.
Hopefully that should be the last in the queue!
From: Kevin
This introduces a new remote.origin.namespaces argument that is a
space-separated list of namespaces. The list of pages extract is then
taken from all the specified namespaces.
Reviewed-by: Antoine Beaupré
Signed-off-by: Antoine Beaupré
---
contrib/mw-to-git/git-remote-mediawiki.p
From: Ingo Ruhnke
we still want to use spaces as separators in the config, but we should
allow the user to specify namespaces with spaces, so we use underscore
for this.
Reviewed-by: Antoine Beaupré
Signed-off-by: Antoine Beaupré
---
contrib/mw-to-git/git-remote-mediawiki.perl | 1 +
1 file c
If we fail to find a requested namespace, we should tell the user
which ones we know about, since those were already fetched. This
allows users to fetch all namespaces by specifying a dummy namespace,
failing, then copying the list of namespaces in the config.
Eventually, we should have a flag tha
Ideally, we'd process them in numeric order since that is more
logical, but we can't do that yet since this is where we find the
numeric identifiers in the first place. Lexicographic order is a good
compromise.
Reviewed-by: Eric Sunshine
Signed-off-by: Antoine Beaupré
---
contrib/mw-to-git/git-
Without this, the fetch process seems hanged while we fetch page
listings across the namespaces. Obviously, it should be possible to
silence this with -q, but that's an issue already present everywhere
in the code and should be fixed separately:
https://github.com/Git-Mediawiki/Git-Mediawiki/issue
When we specify a list of namespaces to fetch from, by default the MW
API will not fetch from the default namespace, refered to as "(Main)"
in the documentation:
https://www.mediawiki.org/wiki/Manual:Namespace#Built-in_namespaces
I haven't found a way to address that "(Main)" namespace when getti
On 2017-11-02 10:24:40, Junio C Hamano wrote:
> Antoine Beaupré writes:
>
>> It might still worth fixing this, but I'm not sure what the process is
>> here - in the latest "what's cooking" Junio said this patchset would be
>> merged in "next". Should I reroll the patchset to fix this or not?
>
> T
On Thu, Nov 2, 2017 at 1:25 PM, Max Rothman wrote:
> No problem! Let me know if I've done something wrong with this patch,
> I'm new to git's contributor process.
Thanks for coming up with a patch!
Yeah, the contribution process has some initial road blocks;
I'll point them out below.
> completi
On 2017-11-01 15:56:51, Eric Sunshine wrote:
>> diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
>> b/contrib/mw-to-git/git-remote-mediawiki.perl
>> @@ -264,9 +264,14 @@ sub get_mw_tracked_categories {
>> sub get_mw_tracked_namespaces {
>> my $pages = shift;
>> foreach my $local
From: Jeff Hostetler
Teach upload-pack to negotiate object filtering over the protocol and
to send filter parameters to pack-objects. This is intended for partial
clone and fetch.
The idea to make upload-pack configurable using uploadpack.allowFilter
comes from Jonathan Tan's work in [1].
[1]
From: Jonathan Tan
Created tests to verify fetch-pack and upload-pack support
for excluding large blobs using --filter=blobs:limit=
parameter.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
t/t5500-fetch-pack.sh | 27 +++
1 file changed, 27 insertions(+)
From: Jonathan Tan
Separate out the calculation of remotes to be fetched from and the
actual fetching. This will allow us to include an additional step before
the actual fetching in a subsequent commit.
Signed-off-by: Jonathan Tan
---
builtin/fetch.c | 14 --
1 file changed, 8 inse
From: Jonathan Tan
In fetch-pack, the global variable save_commit_buffer is set to 0, but
not restored to its original value after use.
In particular, if show_log() (in log-tree.c) is invoked after
fetch_pack() in the same process, show_log() will return before printing
out the commit message (b
From: Jonathan Tan
As part of an effort to improve Git support for very large repositories
in which clients typically have only a subset of all version-controlled
blobs, test pack-objects support for --filter=blobs:limit=, packing only
blobs not exceeding that size unless the blob corresponds to
From: Jonathan Tan
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
t/t5500-fetch-pack.sh | 36
1 file changed, 36 insertions(+)
diff --git a/t/t5500-fetch-pack.sh b/t/t5500-fetch-pack.sh
index fdb98a8..7c8339f 100755
--- a/t/t5500-fetch-pack.
From: Jonathan Tan
When running checkout, first prefetch all blobs that are to be updated
but are missing. This means that only one pack is downloaded during such
operations, instead of one per missing blob.
This operates only on the blob level - if a repository has a missing
tree, they are stil
From: Jeff Hostetler
Teach index-pack to not complain about missing objects
when the --promisor flag is given. The assumption is that
index-pack is currently building the idx and promisor data
and the is_promisor_object() query would fail anyway.
Signed-off-by: Jeff Hostetler
---
builtin/inde
From: Jonathan Tan
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
t/t5500-fetch-pack.sh | 60 +++
1 file changed, 56 insertions(+), 4 deletions(-)
diff --git a/t/t5500-fetch-pack.sh b/t/t5500-fetch-pack.sh
index 7c8339f..86cf653 1
From: Jeff Hostetler
This is part 3 of 3 for partial clone.
It assumes that part 1 [1] and part 2 [2] are in place.
Part 3 is concerned with the commands: clone, fetch, upload-pack, fetch-pack,
remote-curl, index-pack, and the pack-protocol.
Jonathan and I independently started on this task. T
From: Jeff Hostetler
Signed-off-by: Jeff Hostetler
---
builtin/clone.c | 9 +
builtin/fetch-pack.c | 4
builtin/index-pack.c | 10 ++
fetch-pack.c | 13 +
fetch-pack.h | 2 ++
transport-helper.c | 5 +
transport.c | 4 +++
From: Jeff Hostetler
Teach fetch to use the list-objects filtering parameters
to allow a "partial fetch" following a "partial clone".
Signed-off-by: Jeff Hostetler
---
builtin/fetch.c | 66 -
1 file changed, 65 insertions(+), 1 deletion(-
From: Jonathan Tan
Teach fetch to use from-promisor and exclude-promisor-objects
parameters with sub-commands. Initialize fetch_if_missing
global variable.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
builtin/fetch.c | 9 ++---
connected.c | 3 +++
2 files changed, 9
From: Jonathan Tan
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
builtin/clone.c | 17 ++---
t/t5601-clone.sh | 49 +
2 files changed, 63 insertions(+), 3 deletions(-)
diff --git a/builtin/clone.c b/builtin/clone.c
i
From: Jeff Hostetler
Signed-off-by: Jeff Hostetler
---
Documentation/gitremote-helpers.txt | 4
remote-curl.c | 10 --
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/Documentation/gitremote-helpers.txt
b/Documentation/gitremote-helpers.txt
i
No problem! Let me know if I've done something wrong with this patch,
I'm new to git's contributor process.
completion: add missing completions for log, diff, show
* Add bash completion for the missing --no-* options on git log
* Add bash completion for --textconv and --indent-heuristic families
From: Jonathan Tan
Teach sha1_file to fetch objects from the remote configured in
extensions.partialcloneremote whenever an object is requested but missing.
The fetching of objects can be suppressed through a global variable.
This is used by fsck and index-pack.
However, by default, such fetchi
From: Jeff Hostetler
Introduce the ability to have missing objects in a repo. This
functionality is guarded by new repository extension options:
`extensions.partialcloneremote` and
`extensions.partialclonefilter`.
See the update to Documentation/technical/repository-version.txt
in this
From: Jonathan Tan
In a subsequent commit, index-pack will be taught to write ".promisor"
files which are similar to the ".keep" files it knows how to write.
Refactor the writing of ".keep" files, so that the implementation of
writing ".promisor" files becomes easier.
Signed-off-by: Jonathan Tan
From: Jonathan Tan
Teach fsck to not treat refs referring to missing promisor objects as an
error when extensions.partialclone is set.
For the purposes of warning about no default refs, such refs are still
treated as legitimate refs.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
-
From: Jonathan Tan
Currently, Git does not support repos with very large numbers of objects
or repos that wish to minimize manipulation of certain blobs (for
example, because they are very large) very well, even if the user
operates mostly on part of the repo, because Git is designed on the
assum
From: Jonathan Tan
Teach gc to stop traversal at promisor objects, and to leave promisor
packfiles alone. This has the effect of only repacking non-promisor
packfiles, and preserves the distinction between promisor packfiles and
non-promisor packfiles.
Signed-off-by: Jonathan Tan
Signed-off-by:
From: Jonathan Tan
Teach fsck to not treat missing promisor objects provided on the CLI as
an error when extensions.partialcloneremote is set.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
builtin/fsck.c | 2 ++
t/t0410-partial-clone.sh | 13 +
2 files c
From: Jonathan Tan
Introduce fetch-object, providing the ability to fetch one object from a
promisor remote.
This uses fetch-pack. To do this, the transport mechanism has been
updated with 2 flags, "from-promisor" to indicate that the resulting
pack comes from a promisor remote (and thus should
From: Jonathan Tan
Teach fsck to not treat missing promisor objects indirectly pointed to
by refs as an error when extensions.partialcloneremote is set.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
builtin/fsck.c | 11 +++
t/t0410-partial-clone.sh | 23 +
From: Jeff Hostetler
This is part 2 of a proposed 3 part sequence for partial clone.
Part 2 assumes part 1 [1] is in place.
Part 2 is concerned with fsck, gc, initial support for dynamic
object fetching, and tracking promisor objects. Jonathan Tan
originally developed this code. I have moved i
I've tested the code change locally and seems like it fixes the issue.
Yaroslav
On Thu, Nov 2, 2017 at 2:55 PM, Dennis Kaarsemaker
wrote:
> On Thu, 2017-11-02 at 11:35 -0700, Stefan Beller wrote:
>> On Thu, Nov 2, 2017 at 9:00 AM, Yaroslav Sapozhnyk
>> wrote:
>> > When using Git on Fedora with
On Thu, 2 Nov 2017 17:50:07 +
Jeff Hostetler wrote:
> From: Jeff Hostetler
>
> Here is V2 of the list-object filtering. It replaces [1]
> and reflect a refactoring and simplification of the original.
Thanks, overall this looks quite good. I reviewed patches 2-6 (skipping
1 since it's alre
The function `describe` has already a variable named `oid` declared at
the beginning of the function for an object id. Do not shadow that
variable with a pointer to an object id.
Signed-off-by: Stefan Beller
---
builtin/describe.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
d
Signed-off-by: Stefan Beller
---
t/t6120-describe.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t6120-describe.sh b/t/t6120-describe.sh
index 1c0e8659d9..c8b7ed82d9 100755
--- a/t/t6120-describe.sh
+++ b/t/t6120-describe.sh
@@ -304,7 +304,7 @@ test_expect_success 'descr
In the next patch we'll learn how to describe more than just commits,
so factor out describing commits into its own function. That will make
the next patches easy as we still need to describe a commit as part of
describing blobs.
While factoring out the functionality to describe_commit, make sure
The functionality to list tree objects in the order they were seen
while traversing the commits will be used in the next commit,
where we teach `git describe` to describe not only commits, but
trees and blobs, too.
Signed-off-by: Stefan Beller
---
Documentation/rev-list-options.txt | 5
li
For debuggers aid we'd want to print debug statements early, so
introduce a new line in the debug output that describes the whole
function, and then change the next debug output to describe why we
need to search. Conveniently drop the arg from the second line;
which will be useful in a follow up co
Sometimes users are given a hash of an object and they want to
identify it further (ex.: Use verify-pack to find the largest blobs,
but what are these? or [1])
"This is an interesting endeavor, because describing things is hard."
-- me, upon writing this patch.
When describing commits, we try t
Thanks for the discussion on v2[1].
Interdiff is below, just fixing minor things.
We'll keep the original algorithm for now, deferring an improvement on
the algorithm front towards a future developer.
Thanks,
Stefan
[1] https://public-inbox.org/git/20171031211852.13001-1-sbel...@google.com/
S
With traverse_trees_and_blobs factored out of the main traverse function,
the next patch can introduce an in-order revision walking with ease.
In the next patch we'll call `traverse_trees_and_blobs` from within the
loop walking the commits, such that we'll have one invocation of that
function per
On Thu, 2 Nov 2017 17:50:11 +
Jeff Hostetler wrote:
> +int parse_list_objects_filter(struct list_objects_filter_options
> *filter_options,
> + const char *arg)
Returning void is fine, I think. It seems that all your code paths
either return 0 or die.
> +{
> +
On Thu, 2017-11-02 at 11:35 -0700, Stefan Beller wrote:
> On Thu, Nov 2, 2017 at 9:00 AM, Yaroslav Sapozhnyk
> wrote:
> > When using Git on Fedora with locked password store
> > credential-libsecret asks for username/password instead of displaying
> > the unlock dialog.
>
> Git as packaged by Fed
Sorry, should have mentioned that. It's packaged by Fedora - 2.13.6.
Yaroslav
On Thu, Nov 2, 2017 at 2:35 PM, Stefan Beller wrote:
> On Thu, Nov 2, 2017 at 9:00 AM, Yaroslav Sapozhnyk
> wrote:
>> When using Git on Fedora with locked password store
>> credential-libsecret asks for username/passw
On Thu, Nov 2, 2017 at 1:39 AM, Kaartic Sivaraam
wrote:
> On Thursday 02 November 2017 12:24 PM, Kaartic Sivaraam wrote:
>>
>> Signed-off-by: Kaartic Sivaraam
>> Signed-off-by: Kaartic Sivaraam
>
>
> I just now saw this small glitch as a consequence of recently
> changing my email address. I wou
On Thu, Nov 2, 2017 at 9:00 AM, Yaroslav Sapozhnyk
wrote:
> When using Git on Fedora with locked password store
> credential-libsecret asks for username/password instead of displaying
> the unlock dialog.
Git as packaged by Fedora or upstream Git (which version)?
> If the store is unlocked crede
On Thu, Nov 2, 2017 at 1:30 AM, Orgad Shaneh wrote:
> I can't reproduce this with a minimal example, but it happens in my project.
>
> What I tried to do for reproducing is:
> rm -rf super sub
> mkdir sub; cd sub; git init
> git commit --allow-empty -m 'Initial commit'
> mkdir ../super; cd ../supe
On Thu, Nov 2, 2017 at 12:23 AM, Andreas Schwab wrote:
> On Nov 01 2017, Johannes Schindelin wrote:
>
>> Sure, but it is still a tricky thing. Imagine
>>
>> - A1 - B1 - A2 - B2 - B3
>>
>> where all the B* commits have the blob. Do you really want to report B1
>> rather than B2 as the commit intro
The options are currently only referenced by the git-blame man page,
also explain them in git-config, which is the canonical page to
contain all config options.
Signed-off-by: Stefan Beller
---
* correct option to blame.showRoot
* camelCased other options
* use linkgit:git-[1] instead of `git
I was a bit trigger happy posting this, after digging a bit more this is
a way more serious than I originally thought.
1) .gitignore exists in nested repo (either tracked or untracked)
2) .gitignore is excluded
This can be any file, including those commonly excluded such as *~.
Demonst
On Thu, Nov 2, 2017 at 1:01 AM, Jeff King wrote:
> On Thu, Nov 02, 2017 at 04:48:45PM +0900, Junio C Hamano wrote:
>
>> Jeff King writes:
>>
>> > If the clutter is too much, it could also go into a git-notes ref
>> > (that's not already implemented, but it seems like it would be pretty
>> > easy
From: Jeff Hostetler
Create traverse_commit_list_filtered() and add filtering
interface to allow certain objects to be omitted from the
traversal.
Update traverse_commit_list() to be a wrapper for the above
with a null filter to minimize the number of callers that
needed to be changed.
Object f
From: Jeff Hostetler
Teach rev-list to use the filtering provided by the
traverse_commit_list_filtered() interface to omit
unwanted objects from the result. This feature is
intended to help with partial clone.
Object filtering is only allowed when one of the "--objects*"
options are used.
When
From: Jeff Hostetler
Add the usual iterator methods to oidset.
Add oidset_remove().
Signed-off-by: Jeff Hostetler
---
oidset.c | 10 ++
oidset.h | 36
2 files changed, 46 insertions(+)
diff --git a/oidset.c b/oidset.c
index f1f874a..454c54f 100644
From: Jeff Hostetler
Teach pack-objects to use the filtering provided by the
traverse_commit_list_filtered() interface to omit unwanted
objects from the resulting packfile.
This feature is intended for partial clone/fetch.
Filtering requires the use of the "--stdout" option.
Add t5317 test.
S
From: Jeff Hostetler
Refactor add_excludes() to separate the reading of the
exclude file into a buffer and the parsing of the buffer
into exclude_list items.
Add add_excludes_from_blob_to_list() to allow an exclude
file be specified with an OID without assuming a local
worktree or index exists.
From: Jeff Hostetler
Here is V2 of the list-object filtering. It replaces [1]
and reflect a refactoring and simplification of the original.
After much discussion on the "list-object-filter-map" I've replaced
it with a regular oidset -- the only need for the map was to store
the first observed pa
From: Jeff Hostetler
Add the usual map iterator functions to oidmap.
Signed-off-by: Jeff Hostetler
---
oidmap.h | 22 ++
1 file changed, 22 insertions(+)
diff --git a/oidmap.h b/oidmap.h
index 18f54cd..d3cd2bb 100644
--- a/oidmap.h
+++ b/oidmap.h
@@ -65,4 +65,26 @@ extern
Am 02.11.2017 um 03:18 schrieb Junio C Hamano:
> René Scharfe writes:
>
>> @@ -807,6 +816,8 @@ static int get_cmd_result(struct imap_store *ctx, struct
>> imap_cmd *tcmd)
>> if (cmdp->cb.cont || cmdp->cb.data)
>> imap->literal_pending = 0;
>>
Hi all,
Since commit 6b1db4310 it is possible to make git clean -d to remove
nested git repositories if
1) .gitignore exists in nested repo (either tracked or untracked)
2) .gitignore is excluded
Regarding to 2) it doesn't matter if .gitignore is excluded from
(another) .gitignore or fro
Hi Junio,
On Thu, 2 Nov 2017, Junio C Hamano wrote:
> Johannes Schindelin writes:
>
> >> > I feel this is the wrong way round. `>/dev/null` may sound very intuitive
> >> > to you, but this feature is Windows only. Guess three times how intuitive
> >> > it sounds to Windows developers to write `
When using Git on Fedora with locked password store
credential-libsecret asks for username/password instead of displaying
the unlock dialog.
If the store is unlocked credential helper gets the credentials from
the store though.
--
Regards,
Yaroslav Sapozhnyk
Attn: Beneficiary,
I write to inform you that we already issued those documents to accompany your
$5,000 payment each day. But the only problem we are having right here is your
personal signatures which the Federal Administer of Fund Benin Republic
requested that you must sign those documents b
On Thu, Nov 2, 2017 at 2:54 AM, Kaartic Sivaraam
wrote:
> From: Kaartic Sivaraam
>
> When trying to rename an inexistent branch to with a name of a branch
> that already exists the rename failed specifying the new branch name
> exists rather than specifying that the branch trying to be renamed
>
On 2 November 2017 at 05:54, Junio C Hamano wrote:
> Martin Ågren writes:
>
>> `find_bisection()` rebuilds the commit list it is given by reversing it
>> and skipping uninteresting commits. The uninteresting list entries are
>> leaked. Free them to fix the leak.
>>
>> While we're here and underst
1 - 100 of 109 matches
Mail list logo
