Eric Sunshine writes:
> On Wed, May 8, 2019 at 6:56 AM Junio C Hamano wrote:
>> Since e67a228cd8a ("send-email: automatically determine
>> transfer-encoding"), the value of sendmail.transferencoding in the
>> configuration file is ignored, because $target_xfer_encoding is
>
> s/,//
(also all ot
Denton Liu writes:
> In eea9c1e78f (tag: advise on nested tags, 2019-04-04), tag was taught
> to hint at the user if a nested tag is made. However, this message had a
> typo and it said "The object referred to by your new is...", which was
> missing a "tag" after "new". Fix this message by adding
Ævar Arnfjörð Bjarmason writes:
> With commit-graph- all these unlink() race conditions go away,
> partial reads due to concurrent graph writing becomes a non-issue (we'd
> just leave the old files, "gc" deals with them later..), no need to
> carefully fsync() files/dirs etc as we need to careful
Ævar Arnfjörð Bjarmason writes:
> On Sat, May 04 2019, SZEDER Gábor wrote:
> ...
>> And don't forget that the commit-graph progress bar still prints
>> nonsense :)
>
> I'm inclined to just wait until Derrick's refactorings land post-2.22.0
Fine by me. Thanks.
Git Gadget writes:
> Forwarding this mail to the Git mailing list, as the original did not
> make it there (for reasons unknown).
It seems that the forwarding mechansim (if this weren't manual---I
cannot tell) mangles whitespaces? No need to resend, as the
receiving end manually fixed them up.
Dial Direct Loan SA
Consolidate your debts with Dial Direct Loan SA for your peace of
mind at a fixed interest rate of 4.75%,personal and business loans
are also welcome.For details file in your applications by sending an email
to: infodialdirectlo...@mail2consultant.com.
Yours in Service,
S
On Wed, May 08, 2019 at 07:10:13PM -0400, Drew DeVault wrote:
> I want to gather some thoughts about this. Say you've written a patch
> series and are getting ready to send a -v2. If you set
> --in-reply-to=ask, it'll show you a list of emails you've recently sent,
> and their subject lines, and as
On Wed, May 08, 2019 at 08:39:17AM +, opal hart wrote:
> On Tue, 7 May 2019 03:33:22 -0400
> Jeff King wrote:
>
> > The "security settings" toggle for the site on Cloudflare is already set
> > to "low", and there are no other restrictions. Is it actually blocking?
> > Or is it asking for a C
On Wed, May 08, 2019 at 07:56:07PM +0900, Junio C Hamano wrote:
> diff --git a/git-send-email.perl b/git-send-email.perl
> index 98bc295c6e..b9dd775b63 100755
> --- a/git-send-email.perl
> +++ b/git-send-email.perl
> @@ -231,7 +231,7 @@ sub do_edit {
> my (@suppress_cc);
> my ($auto_8bit_encoding
I want to gather some thoughts about this. Say you've written a patch
series and are getting ready to send a -v2. If you set
--in-reply-to=ask, it'll show you a list of emails you've recently sent,
and their subject lines, and ask you to pick one to use the message ID
from. It'll set the In-Reply-T
On Wed, May 08, 2019 at 01:45:25PM +0200, Johannes Schindelin wrote:
> Hi René,
>
> On Thu, 2 May 2019, René Scharfe wrote:
>
> > Am 27.04.19 um 01:27 schrieb Johannes Schindelin via GitGitGadget:
> > > From: Johannes Schindelin
> > >
> > > They really are unsigned, and we are using e.g. BLOCKS
On Wed, May 08, 2019 at 08:36:10AM -0400, Denton Liu wrote:
> > %.cocci.patch: %.cocci $(COCCI_SOURCES)
> > @echo '' SPATCH $<; \
> > - if ! echo $(COCCI_SOURCES) | xargs -n $(SPATCH_BATCH_SIZE) \
> > + if test $(SPATCH_BATCH_SIZE) = 0; then \
> > + limit=; \
> > + else \
On Sun, May 05 2019, Eric Sunshine wrote:
> On Sun, May 5, 2019 at 3:43 PM Jeffrey Walton wrote:
>> I have a PowerMac I use for testing. [...]
>> make -C t/ all
>> readline() on unopened filehandle test_must_fail
>> run_sub_test_lib_test at check-non-portable-shell.pl line 34.
>> Modific
On Wed, May 08, 2019 at 06:13:58PM +0200, Ævar Arnfjörð Bjarmason wrote:
> > I do wonder if this really needs to be part of the progress bar. The
> > goal of the progress bar is to give the user a sense that work is
> > happening, and (if possible, but not for "enumerating") an idea of when
> > it
Hi,
Git-cinnabar is a git remote helper to interact with mercurial
repositories. It allows to clone, pull and push from/to mercurial remote
repositories, using git.
Code on https://github.com/glandium/git-cinnabar
This release on
https://github.com/glandium/git-cinnabar/releases/tag/0.5.1
What's
From: Johannes Schindelin
In `--no-index` mode, we now no longer require a worktree nor a
repository. But some code paths in `difftool` expect those to be
present.
The most notable such code path is the `--dir-diff` one: we use the
existing checkout machinery to copy the files, and that machiner
While debugging why a straight-forward backport of the js/difftool-no-index
branch onto Git for Windows' master does not work as expected (spoiler: it's
due to git diff --no-index now implying --ext-diff, a change that is not yet
in Git for Windows' master), I ran into quite a few segmentation fau
On Wed, May 8, 2019 at 6:56 AM Junio C Hamano wrote:
> Since e67a228cd8a ("send-email: automatically determine
> transfer-encoding"), the value of sendmail.transferencoding in the
> configuration file is ignored, because $target_xfer_encoding is
s/,//
> already defined read_config sub parses the
On Sat, May 04 2019, SZEDER Gábor wrote:
> On Sat, May 04, 2019 at 08:52:01AM +0200, Ævar Arnfjörð Bjarmason wrote:
>> As an aside this is the Nth time I notice how crappy that "Enumerating
>> objects" progress bar is.
>
> And don't forget that the commit-graph progress bar still prints
> nonsen
> > And "official" is a phrase I have trouble with in this context. A
> > mirror does not have to be blessed as official; that's the point of
> > a mirror---anybody can make one to help users with better
> > connectivity or availability, as long as its users trust the mirror
> > maintainer for mir
Hey Thomas
Thanks for the heartwarming welcome and cheers Matheus for getting
selected to GSoC '19. May all our code gets merged (in respective
projects ofc :p).
Best
Rohit
On Wed, May 08 2019, Derrick Stolee via GitGitGadget wrote:
> This patch series is marked as RFC quality because it is missing some key
> features and tests, but hopefully starts a concrete discussion of how the
> incremental commit-graph writes can work. If this is a good direction, then
> it w
On 5/8/2019 2:43 PM, Denton Liu wrote:
> In the documentation for pull.rebase, all of the valid values are
> surrounded with backticks except for "true". Surround "true" with these
> missing backticks.
This would make the rendered docs [1] look a bit better, but I think this series
is incomplete.
In eea9c1e78f (tag: advise on nested tags, 2019-04-04), tag was taught
to hint at the user if a nested tag is made. However, this message had a
typo and it said "The object referred to by your new is...", which was
missing a "tag" after "new". Fix this message by adding the "tag".
Signed-off-by: D
On Wed, May 08, 2019 at 12:46:47PM +0900, Junio C Hamano wrote:
> Emily Shaffer writes:
>
> > +=== Clone the Git Repository
> > +
> > +Git is mirrored in a number of locations. Clone the repository from one of
> > them;
> > +https://git-scm.com/downloads suggests one of the best places to clone
On 5/8/2019 1:20 PM, SZEDER Gábor wrote:
> On Wed, May 08, 2019 at 08:53:57AM -0700, Derrick Stolee via GitGitGadget
> wrote:
>> From: Derrick Stolee
> Consider the following sequence of events:
>
> 1. There are three commit-graph files in the repository.
>
> 2. A git process opens the base
In the documentation for pull.rebase, all of the valid values are
surrounded with backticks except for "true". Surround "true" with these
missing backticks.
Signed-off-by: Denton Liu
---
Documentation/config/pull.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentati
On Wed, May 8, 2019 at 10:23 AM Junio C Hamano wrote:
> * en/unicode-in-refnames (2019-04-26) 1 commit
> - Honor core.precomposeUnicode in more places
>
> The names of the refs stored as filesystem entities may become
> different from what the end-user expects, just like files in the
> workin
On 08.05.19 19:31, Denton Liu wrote:
> Hi Beat,
>
> On Wed, May 08, 2019 at 07:18:18PM +0200, Beat Bolli wrote:
>> On 08.05.19 17:02, Denton Liu wrote:
>>> In git-format-patch, notes can be appended with the `--notes` option.
>>> However, this must be specified by the user on an
>>> invocation-by-
Hi Junio,
On Thu, May 09, 2019 at 02:23:24AM +0900, Junio C Hamano wrote:
> Here are the topics that have been cooking. Commits prefixed with
> '-' are only in 'pu' (proposed updates) while commits prefixed with
> '+' are in 'next'. The ones marked with '.' do not appear in any of
> the integrat
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'. The ones marked with '.' do not appear in any of
the integration branches, but I am still holding onto them.
The 8th batch of topics, which Hop
Hi Beat,
On Wed, May 08, 2019 at 07:18:18PM +0200, Beat Bolli wrote:
> On 08.05.19 17:02, Denton Liu wrote:
> > In git-format-patch, notes can be appended with the `--notes` option.
> > However, this must be specified by the user on an
> > invocation-by-invocation basis. If a user is not careful,
On 08.05.19 17:02, Denton Liu wrote:
> In git-format-patch, notes can be appended with the `--notes` option.
> However, this must be specified by the user on an
> invocation-by-invocation basis. If a user is not careful, it's possible
> that they may forget to include it and generate a patch series
On Wed, May 08, 2019 at 08:53:57AM -0700, Derrick Stolee via GitGitGadget wrote:
> From: Derrick Stolee
>
> The design for the split commit-graphs uses file names to force
> a "stack" of commit-graph files. This allows incremental writes
> without updating the file format.
>
> Signed-off-by: Der
As in past years, Git is participating in Google Summer of Code this
year. The student slots have just been announced, and we're going to
have 2 students and 4 (co-)mentors this year.
Please join me in congratulating Rohit Ashiwal and Matheus Tavares
Bernardino for being accepted into the program
Hi Junio,
Understood. Thanks for the clarification.
Best regards,
Boxuan Li
On Wed, May 8, 2019 at 1:49 PM Junio C Hamano wrote:
>
> "LI, BO XUAN" writes:
>
> > Thanks for your review! I can understand your point, but I've got a
> > quick question:
> >
> > What if format-patch really breaks an
On Wed, May 08 2019, Jeff King wrote:
> On Tue, May 07, 2019 at 10:12:06AM +0200, Ævar Arnfjörð Bjarmason wrote:
>
>> > I think we'd want a way to tell the bitmap code to update our progress
>> > meter as it traverses (both single objects, but also taking into account
>> > when it finds a bitmap
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
Extract copy_oids_to_commits(), which fills the commits list
with the distinct commits from the oids list. During this loop,
it also counts the number of "extra" edges from octopus merge
From: Derrick Stolee
Signed-off-by: Derrick Stolee
---
commit-graph.c | 248 +++-
commit-graph.h | 1 +
t/t5318-commit-graph.sh | 2 +-
3 files changed, 244 insertions(+), 7 deletions(-)
diff --git a/commit-graph.c b/commit-graph.c
inde
From: Derrick Stolee
Signed-off-by: Derrick Stolee
---
builtin/commit-graph.c | 10 +++---
t/t5318-commit-graph.sh | 26 ++
2 files changed, 33 insertions(+), 3 deletions(-)
diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c
index 828b1a713f..c2c07d3917 1
This patch series is marked as RFC quality because it is missing some key
features and tests, but hopefully starts a concrete discussion of how the
incremental commit-graph writes can work. If this is a good direction, then
it would replace ds/commit-graph-format-v2.
The commit-graph is a valuable
From: Derrick Stolee
The write_commit_graph() method is too large and complex. To simplify
it, we should extract several small methods. However, we will risk
repeating a lot of declarations related to progress incidators and
object id or commit lists.
Create a new write_commit_graph_context stru
From: Derrick Stolee
The design for the split commit-graphs uses file names to force
a "stack" of commit-graph files. This allows incremental writes
without updating the file format.
Signed-off-by: Derrick Stolee
---
Documentation/technical/commit-graph.txt | 142 +++
1 fil
From: Derrick Stolee
The write_commit_graph() and write_commit_graph_reachable() methods
currently take two boolean parameters: 'append' and 'report_progress'.
We will soon expand the possible options to send to these methods, so
instead of complicating the parameter list, first simplify it.
Col
From: Derrick Stolee
The write_commit_graph() method uses die() to report failure and
exit when confronted with an unexpected condition. This use of
die() in a library function is incorrect and is now replaced by
error() statements and an int return type.
Now that we use 'goto cleanup' to jump t
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
Extract count_distinct_commits(), which sorts the oids list, then
iterates through to find duplicates.
Signed-off-by: Derrick Stolee
---
commit-graph.c | 35 ++
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
Extract fill_oids_from_all_packs() that reads all pack-files
for commits and fills the oid list in the context.
Signed-off-by: Derrick Stolee
---
commit-graph.c | 26 +++--
From: Derrick Stolee
The commit-graph feature began with a long list of planned
benefits, most of which are now complete. The future work
section has only a few items left.
As for making more algorithms aware of generation numbers,
some are only waiting for generation number v2 to ensure the
per
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
Extract write_commit_graph_file() that takes all of the information
in the context struct and writes the data to a commit-graph file.
Signed-off-by: Derrick Stolee
---
commit-graph.c
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
This extracts fill_oids_from_packs() that reads the given
pack-file list and fills the oid list in the context.
Signed-off-by: Derrick Stolee
---
commit-graph.c | 83 +
From: Derrick Stolee
The write_commit_graph() method is too complex, so we are
extracting methods one by one.
Extract fill_oids_from_commit_hex() that reads the given commit
id list and fille the oid list in the context.
Signed-off-by: Derrick Stolee
---
commit-graph.c | 72 ++
From: Derrick Stolee
Signed-off-by: Derrick Stolee
---
commit-graph.c | 66 --
commit-graph.h | 4 +++
2 files changed, 62 insertions(+), 8 deletions(-)
diff --git a/commit-graph.c b/commit-graph.c
index 7723156964..f790f44a9c 100644
--- a/commi
From: Derrick Stolee
Signed-off-by: Derrick Stolee
---
builtin/fetch.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/builtin/fetch.c b/builtin/fetch.c
index b620fd54b4..cf0944bad5 100644
--- a/builtin/fetch.c
+++ b/builtin/fetch.c
@@ -23,6 +23,7 @@
#include "packfile.h
From: Derrick Stolee
Starting with commit-graph, load commit-graph files in a
sequence as follows:
commit-graph
commit-graph-1
commit-graph-2
...
commit-graph-N
This creates N + 1 files in order.
Signed-off-by: Derrick Stolee
---
commit-graph.c | 39
From: Derrick Stolee
The parse_commit_buffer() method takes a repository pointer, so it
should not refer to the_repository anymore.
Signed-off-by: Derrick Stolee
---
commit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/commit.c b/commit.c
index a5333c7ac6..e4d1233226 10
On 5/8/2019 10:41 AM, Duy Nguyen wrote:
> On Wed, May 8, 2019 at 9:07 PM Derrick Stolee wrote:
>>
>> On 5/8/2019 7:12 AM, Nguyễn Thái Ngọc Duy wrote:
>>> Bitfield addresses cannot be passed around in a pointer. This makes it
>>> hard to use parse-options to set/unset them. Turn this struct to
>>>
In git-format-patch, notes can be appended with the `--notes` option.
However, this must be specified by the user on an
invocation-by-invocation basis. If a user is not careful, it's possible
that they may forget to include it and generate a patch series without
notes.
Teach git-format-patch the `
Hi Junio,
Thanks for the review. I made format.notes accept one (or more) refs
now.
Changes since v1:
* Made format.notes accept a notes ref instead of a boolean
Denton Liu (2):
git-format-patch.txt: document --no-notes option
format-patch: teach format.notes config option
Documentation/
Internally, git-format-patch uses the `handle_revision_opt` parser. The
parser handles the `--no-notes` option to negate an earlier `--notes`
option, but it isn't documented. Document this option so that users are
aware of it.
Signed-off-by: Denton Liu
---
Documentation/git-format-patch.txt | 4
On Wed, May 8, 2019 at 9:07 PM Derrick Stolee wrote:
>
> On 5/8/2019 7:12 AM, Nguyễn Thái Ngọc Duy wrote:
> > Bitfield addresses cannot be passed around in a pointer. This makes it
> > hard to use parse-options to set/unset them. Turn this struct to
> > normal integers. This of course increases th
On 5/7/2019 4:31 AM, Junio C Hamano wrote:
> "Derrick Stolee via GitGitGadget" writes:
>
>> From: Derrick Stolee
>>
>> Much of the multi-pack-index code focuses on the multi_pack_index
>> struct, and so we only pass a pointer to the current one. However,
>> we will insert a dependency on the pac
On 5/8/2019 3:11 AM, Jeff King wrote:
> On Tue, May 07, 2019 at 10:12:06AM +0200, Ævar Arnfjörð Bjarmason wrote:
>
>>> I think we'd want a way to tell the bitmap code to update our progress
>>> meter as it traverses (both single objects, but also taking into account
>>> when it finds a bitmap and
On 5/8/2019 7:12 AM, Nguyễn Thái Ngọc Duy wrote:
> Bitfield addresses cannot be passed around in a pointer. This makes it
> hard to use parse-options to set/unset them. Turn this struct to
> normal integers. This of course increases the size of this struct
> multiple times, but since we only have a
Hi Jeff,
On Wed, May 08, 2019 at 03:07:54AM -0400, Jeff King wrote:
> On Tue, May 07, 2019 at 01:52:32PM +0900, Junio C Hamano wrote:
>
> > Jeff King writes:
> >
> > > Yes, 2^31-1 is probably a better number, but it's harder to write out. :)
> > >
> > > Here's what a patch might look like to im
Hi René,
On Thu, 2 May 2019, René Scharfe wrote:
> Am 27.04.19 um 01:27 schrieb Johannes Schindelin via GitGitGadget:
> > From: Johannes Schindelin
> >
> > They really are unsigned, and we are using e.g. BLOCKSIZE as `size_t`
> > parameter to pass to `write_or_die()`.
>
> True, but the compiler
Hi Brian,
On Wed, 1 May 2019, brian m. carlson wrote:
> On Wed, May 01, 2019 at 08:39:22PM +0200, Alban Gruin wrote:
> > -Og also exists to debug[0], even if it’s far less known. Perhaps it’s
> > better to check for -g (and its variants[1]) as the user clearly states
> > their intent to debug th
Hi Alban,
On Wed, 1 May 2019, Alban Gruin wrote:
> Le 01/05/2019 à 00:41, Johannes Schindelin a écrit :
> >
> > On Tue, 30 Apr 2019, Johannes Sixt wrote:
> >
> >> [had to add Dscho as recipient manually, mind you]
> >
> > I usually pick up responses to GitGitGadget patch series even if I am not
>
These two techniques make it harder to come up with exploits, by reducing
what is commonly called the "attack surface" in security circles: by making
the addresses less predictable, and by making it harder to inject data that
is then (mis-)interpreted as code, this hardens Git's executables on
Wind
From: =?UTF-8?q?=C4=B0smail=20D=C3=B6nmez?=
Enable DEP (Data Execution Prevention) and ASLR (Address Space Layout
Randomization) support. This applies to both 32bit and 64bit builds
and makes it substantially harder to exploit security holes in Git by
offering a much more unpredictable attack sur
From: =?UTF-8?q?=C4=B0smail=20D=C3=B6nmez?=
This is the first step for enabling ASLR (Address Space Layout
Randomization) support. We want to enable ASLR for better protection
against exploiting security holes in Git: it makes it harder to attack
software by making code addresses unpredictable.
Hi Jonathan & Peff,
On Wed, 1 May 2019, Jonathan Nieder wrote:
> Jeff King wrote:
>
> > I wonder if this points to this patch touching the wrong level. These
> > compiler flags are a thing that _some_ builds want (i.e., production
> > builds where people care most about security and not about deb
This patch is essentially no-op. It allows to parse_options() to handle
some options. But the new option list remains empty. The option will be
moved one by one from the old manual parsing code to this list.
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 44
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/revision.c b/revision.c
index d34e17984d..6efa9bee1e 100644
--- a/revision.c
+++ b/revision.c
@@ -2452,6 +2452,18 @@ static int rev_opt_glob(const struct opt
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 15 ---
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/revision.c b/revision.c
index f04eb7f140..dd22ac5c39 100644
--- a/revision.c
+++ b/revision.c
@@ -2601,6 +2601,9 @@ static void make_pseudo_options(struct rev_in
This block at the top of rev_info is "Starting list" and repo is
obviously not one. Move it to the bottom since it's not that important
to stay on top.
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/revision.h b/revision.h
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 34 +-
1 file changed, 25 insertions(+), 9 deletions(-)
diff --git a/revision.c b/revision.c
index 5183cdf66d..bcfca8856f 100644
--- a/revision.c
+++ b/revision.c
@@ -2425,6 +2425,27 @@ static int rev_opt_branch
Besides preparing an empty option table to be added in. The table is
also concatenated with diff option table so we don't need
diff_opt_parse() anymore. Merging with pseudo-opt table though will
not happen until we kill parse_revision_opt().
--abbrev has to be converted right away to override the
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 34 +-
1 file changed, 25 insertions(+), 9 deletions(-)
diff --git a/revision.c b/revision.c
index 0d34f81716..5183cdf66d 100644
--- a/revision.c
+++ b/revision.c
@@ -2425,6 +2425,27 @@ static int rev_opt_branch
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 41 ++---
revision.h | 2 ++
2 files changed, 32 insertions(+), 11 deletions(-)
diff --git a/revision.c b/revision.c
index 65d40c9255..9a346577f6 100644
--- a/revision.c
+++ b/revision.c
@@ -30,6 +30,10 @@
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/revision.c b/revision.c
index 6efa9bee1e..9cfa4dc151 100644
--- a/revision.c
+++ b/revision.c
@@ -2438,6 +2438,18 @@ static int rev_opt_exclude(const struct
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 47 ++-
1 file changed, 30 insertions(+), 17 deletions(-)
diff --git a/revision.c b/revision.c
index b02cb4660b..f04eb7f140 100644
--- a/revision.c
+++ b/revision.c
@@ -2464,6 +2464,29 @@ static int
Following the conversion in diff.c to use parse_options, this time it's
revision.c. There are about 76 patches to convert all options, split in
4 parts. After that there are about 10 more to convert rev users to use
parse_options() directly.
Full 76 patches are available here [1]. I'm not quite do
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 28 +---
1 file changed, 21 insertions(+), 7 deletions(-)
diff --git a/revision.c b/revision.c
index bcfca8856f..013b8ec19f 100644
--- a/revision.c
+++ b/revision.c
@@ -30,6 +30,9 @@
#include "prio-queue.h"
#include "
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/revision.c b/revision.c
index 013b8ec19f..d34e17984d 100644
--- a/revision.c
+++ b/revision.c
@@ -2428,6 +2428,16 @@ static int rev_opt_branches(const struct
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 28 +---
1 file changed, 21 insertions(+), 7 deletions(-)
diff --git a/revision.c b/revision.c
index 7db1109b57..0d34f81716 100644
--- a/revision.c
+++ b/revision.c
@@ -2387,6 +2387,23 @@ static int rev_opt_all(const st
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 38 +-
1 file changed, 29 insertions(+), 9 deletions(-)
diff --git a/revision.c b/revision.c
index 9a346577f6..7db1109b57 100644
--- a/revision.c
+++ b/revision.c
@@ -34,6 +34,10 @@
OPT_CALLBACK_F(s,
Bitfield addresses cannot be passed around in a pointer. This makes it
hard to use parse-options to set/unset them. Turn this struct to
normal integers. This of course increases the size of this struct
multiple times, but since we only have a handful of rev_info variables
around, memory consumption
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/revision.c b/revision.c
index 9cfa4dc151..b02cb4660b 100644
--- a/revision.c
+++ b/revision.c
@@ -2464,6 +2464,17 @@ static int rev_opt_glob(const struct opti
Signed-off-by: Nguyễn Thái Ngọc Duy
---
parse-options-cb.c | 8
parse-options.h| 4
revision.c | 10 --
3 files changed, 16 insertions(+), 6 deletions(-)
diff --git a/parse-options-cb.c b/parse-options-cb.c
index 6e2e8d6273..7cdbbf5f6d 100644
--- a/parse-optio
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/revision.c b/revision.c
index c53347d362..42d466cd08 100644
--- a/revision.c
+++ b/revision.c
@@ -1996,6 +1996,9 @@ static void make_rev_options(struct rev_info *revs)
Signed-off-by: Nguyễn Thái Ngọc Duy
---
revision.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/revision.c b/revision.c
index f15aa3e62d..c53347d362 100644
--- a/revision.c
+++ b/revision.c
@@ -1976,12 +1976,26 @@ static int rev_opt_abbrev(co
Hi folks,
please find a patch attached to properly compile and link Git 2.21.0 on
HP-UX IA64.
The patch includes basically two changes:
* Detection of big endian on HP-UX otherwise SHA1 calc is broken
* Detection of linking style with HP aCC
export PREFIX=/opt/ports
export LIBDIR=$PREFIX/li
Johannes Schindelin writes:
> How about this:
>
> The offical CVS for Windows (called CVS NT) produces DOS line
> endings in its `cvs status` output. Let's teach our own
> `cvsexportcommit` command to handle that gracefully.
>
> It is unlikely that anybody wants to spend time "f
Hi Junio,
On Tue, 7 May 2019, Junio C Hamano wrote:
> "Dustin Spicuzza via GitGitGadget" writes:
>
> > From: Dustin Spicuzza
> >
> > When using cvsnt + msys + git, it seems like the output of cvs status
> > had \r\n in it, and caused the command to fail.
>
> This is a bit under-explained in tha
The program has a good mechanism to specify the fallback default
values for boolean configuration variables after two invocations of
read_config() for "sendmail.$ident.$var" and "sendemail.$var" have
not found any configuration. Imitate it so that we can set the
default values for non-boolean vari
Since e67a228cd8a ("send-email: automatically determine
transfer-encoding"), the value of sendmail.transferencoding in the
configuration file is ignored, because $target_xfer_encoding is
already defined read_config sub parses the configuration file.
Instead of initializing variable $target_xfer_en
Earlier I sent a series of "oops that was wrong" which was rather
embarrassing. This is what I have to replace Heinrich's patch that
was not quite right, while making it harder to make the same mistake
while defining fallback default values to configuration variables.
Junio C Hamano (2):
send-e
Forwarding this mail to the Git mailing list, as the original did not
make it there (for reasons unknown).
-- Forwarded message -
From: Chris. Webster via GitGitGadget
Date: Wed, Oct 31, 2018 at 11:58 PM
Subject: [PATCH v2 1/1] diff-highlight: Use correct /dev/null for UNIX
and Wi
On Wed, May 8, 2019 at 1:31 AM Emily Shaffer wrote:
> > > I found myself in a situation where I had accidentally
> > > staged all my changes to tracked files (I think resulting from a stash
> > > pop which generated a merge conflict?) and didn't see a good way to
> > > unstage everything using res
Junio C Hamano writes:
> So, here is a two-patch series that tries to do so, primarily done
> to gauge if there still is the level of interest needed to make it
> worth for us to pursue this topic. Here is the first one; I'll send
> the second one that takes advantage of this change separately (
1 - 100 of 108 matches
Mail list logo
