hi
this is an electronics shop
bike,brand guitar,camera,tv,samsung product free shipping
www .slooone .com
if you do not want to receive our email,please reply to us, we will never send
email to you
I'd like to announce a project I've been working on for a while:
git-series provides a tool for managing patch series with git, tracking
the "history of history". git series tracks changes to the patch series
over time, including rebases and other non-fast-forwarding changes. git
series also track
On Fri, Jul 29, 2016 at 12:15:24AM -0400, Jeff King wrote:
> Note that this reordering does have a potential impact on
> the final pack, as we store only a single "found" pack for
> each object, even if it is present in multiple packs. In
> principle, any copy is acceptable, as they all refer to t
In the original implementation of want_object_in_pack(), we
always looked for the object in every pack, so the order did
not matter for performance.
As of the last few patches, however, we can now often break
out of the loop early after finding the first instance, and
avoid looking in the other pa
In want_object_in_pack(), we can exit early from our loop if
neither "local" nor "ignore_pack_keep" are set. If they are,
however, we must examine each pack to see if it has the
object and is non-local or has a ".keep".
It's quite common for there to be no non-local or .keep
packs at all, in which
When pack-objects collects the list of objects to pack
(either from stdin, or via its internal rev-list), it
filters each one through want_object_in_pack().
This function loops through each existing packfile, looking
for the object. When we find it, we mark the pack/offset
combo for later use. How
Each pack has an index for looking up entries in O(log n)
time, but if we have multiple packs, we have to scan through
them linearly. This can produce a measurable overhead for
some operations.
We dealt with this long ago in f7c22cc (always start looking
up objects in the last used pack first, 200
There are a few places in Git that would benefit from a fast
most-recently-used cache (e.g., the list of packs, which we
search linearly but would like to order based on locality).
This patch introduces a generic list that can be used to
store arbitrary pointers in most-recently-used order.
The im
The point of this function is to drop an entry from the
"packed_git" cache that points to a file we might be
overwriting, because our contents may not be the same (and
hence the only caller was pack-objects as it moved a
temporary packfile into place).
In older versions of git, this could happen b
Git's pack storage does efficient (log n) lookups in a
single packfile's index, but if we have multiple packfiles,
we have to linearly search each for a given object. This
patch introduces some timing tests for cases where we have a
large number of packs, so that we can measure any
improvements we
This is a follow-up to the patches in
http://public-inbox.org/git/20160725184938.ga12...@sigill.intra.peff.net/
that are currently queued in jk/pack-objects-optim-skimming. Roughly,
they try to optimize a loop that is O(nr_objects * nr_packs) by breaking
out early in some cases.
I had written
On Thu, Jul 28, 2016 at 08:16:19PM -0400, Jeff King wrote:
> The question in my mind is whether people actually use format-patch for
> things besides emailing, and if the final destination is something other
> than "git am". It is a handy format because it is the least-lossy way
> to move commits
On Thu, Jul 28, 2016 at 9:22 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>>> Anyway, I'll apply the "addition to the completion" patch.
>>>
>>> Thanks.
>>
>> Thanks for this patch!
>>
>> Note: if we ever decide to resurrect sb/submodule-default-path,
>> we run into a merge conflict. The r
In a later patch we want to enhance the logic for the branch selection.
Rewrite the current logic to be in C, so we can directly use C when
we enhance the logic.
Signed-off-by: Stefan Beller
---
builtin/submodule--helper.c | 28 +++-
git-submodule.sh| 2 +-
2
Signed-off-by: Stefan Beller
---
git-submodule.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/git-submodule.sh b/git-submodule.sh
index 174f4ea..0d4021f 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh
@@ -591,7 +591,6 @@ cmd_update()
name=$(git submo
The branch field will be used in a later patch by `submodule update`.
Signed-off-by: Stefan Beller
---
submodule-config.c | 11 ++-
submodule-config.h | 1 +
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/submodule-config.c b/submodule-config.c
index 077db40..ebee1e4 100
Gerrit has a "superproject subscription" feature[1], that triggers a
commit in a superproject that is subscribed to its submodules.
Conceptually this Gerrit feature can be done on the client side with
Git via (except for raciness, error handling etc):
while [ true ]; do
git -C submodule upd
The prior hard coded depth was chosen to be exactly the length from the
recorded gitlink to the tip of the remote, so if you add more commits
to the remote before, this test will not test its intention any more.
Signed-off-by: Stefan Beller
---
t/t7406-submodule-update.sh | 19 +++---
Internally we call the underscore version of relative_path, but externally
we present an API with no underscores.
Signed-off-by: Stefan Beller
---
builtin/submodule--helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--help
This got bigger than expected, but I am happier with the results.
The meat is found in the last patch. (At least what I am interested in; others
may be more interested in the second patch which could be argued to be a real
bug fix to be merged down to maint.)
Thanks Junio for the thourough review
When depth is given the user may have a reasonable expectation that
any remote operation is using the given depth. Add a test to demonstrate
we still get the desired sha1 even if the depth is too short to
include the actual commit.
Signed-off-by: Stefan Beller
---
git-submodule.sh|
On Thu, Jul 28, 2016 at 5:29 PM, Jeff King wrote:
>
> I guess we want something like:
>
> +void reset_ident_date(void)
> +{
> + strbuf_reset(&git_default_date);
> +}
Looks sane.
> and then to sprinkle calls liberally through builtin-ified programs when
> they move from one unit of work to
On Thu, Jul 28, 2016 at 5:21 PM, Jeff King wrote:
>
> I do wonder if you would be happier giving each commit a "fake"
> monotonically increasing time, so they are correctly ordered by commit
> date.
No, that would be nasty, partly for the corner case you mention, but
partly because I just think i
On Thu, Jul 28, 2016 at 04:47:17PM -0700, Junio C Hamano wrote:
> Also makes me wonder if "git cherry-pick A..B" shares the same
> breakage.
Probably.
I guess we want something like:
diff --git a/ident.c b/ident.c
index 139c528..e20a772 100644
--- a/ident.c
+++ b/ident.c
@@ -184,6 +184,11 @@ st
On Thu, Jul 28, 2016 at 04:35:58PM -0700, Linus Torvalds wrote:
> Now, it would be lovely if the new builtin git-am really was *so* fast
> that it applies a 100+-patch series in under a second, but no, that's
> not it. It's just that it only looks up the current time once.
>
> That seems entirely
On Thu, Jul 28, 2016 at 04:53:16PM -0700, Josh Triplett wrote:
> I can think of aesthetic reasons to want the non-"--from" format (for
> instance, sticking patch files into a non-git-based tool like quilt or a
> distribution packaging system, and not wanting your own email address
> included), but
On Thu, Jul 28, 2016 at 03:14:48PM -0700, Junio C Hamano wrote:
> > I think the original reason I did not make "--from" the default is that
> > I was worried about breaking consumers which do not know how to handle
> > in-body headers.
>
> That's a fair concern.
>
> So going back to Josh's origi
On Thu, Jul 28, 2016 at 02:37:04PM -0700, Junio C Hamano wrote:
> Josh Triplett writes:
>
> > I'd like to propose changing the default behavior of git-format-patch to
> > --from (and adding a --from-author option to override, and perhaps a
> > config setting). This will not change the output *ex
On Thu, Jul 28, 2016 at 05:56:03PM -0400, Jeff King wrote:
> Another way to think about it is that "--from" is a no-brainer when you
> really are going to email the patches (and that's why it is has always
> been the default behavior in git-send-email). But if you _aren't_ going
> to mail the patch
On Thu, Jul 28, 2016 at 03:14:48PM -0700, Junio C Hamano wrote:
> Jeff King writes:
> > I think the original reason I did not make "--from" the default is that
> > I was worried about breaking consumers which do not know how to handle
> > in-body headers.
>
> That's a fair concern.
>
> So going
On Thu, Jul 28, 2016 at 4:35 PM, Linus Torvalds
wrote:
> Ok, it's no longer *that* new, but I only now noticed..
>
> So I noticed that when I applied the last patch-bomb series from
> Andrew, all the commit date-stamps are idential.
> ...
> That seems entirely accidental, I think that what happene
Ok, it's no longer *that* new, but I only now noticed..
So I noticed that when I applied the last patch-bomb series from
Andrew, all the commit date-stamps are idential.
Now, it would be lovely if the new builtin git-am really was *so* fast
that it applies a 100+-patch series in under a second, b
On 29 July 2016 at 00:38, Junio C Hamano wrote:
> Øyvind A. Holm writes:
> > Jakub Narębski wrote:
> > > I wonder if using `git worktree` instead of `git clone` (well,
> > > local clone uses hardlinks, so it is not that costly as it looks
> > > like) would be a better solution.
> >
> > That's an
On 28 July 2016 at 21:31, Jakub Narębski wrote:
> W dniu 2016-07-28 o 18:56, Øyvind A. Holm pisze:
> > Øyvind A. Holm writes:
> > > This is a script I created some weeks ago, and I've found it to be
> > > immensely useful. Here is a snippet from git-testadd --help:
> > > [...]
> > > This script
I'm just a random Git user, not very familiar with the Git development process
and so on, so please by gentle.
Setup:
- OS X 10.11.6
- Git 2.9.2 via Homebrew
Curiosity triggered me to enable 'commit.verbose' globally with 'git config
--global commit.verbose true'. I have not configured it on th
Jeff King writes:
>> ... What is most worrysome is the latter
>> half of the last sentence. Is it really "should not be", or is it
>> merely "use of this option is just a waste of time, as you would get
>> exactly the same result anyway"? If it is the latter, that is fine.
>
> It does what you
Junio C Hamano wrote:
> Subject: [PATCH] t9100: portability fix
>
> Do not say "export VAR=VAL"; "VAR=VAL && export VAR" is always more
> portable.
Oops, sorry I should've caught that :x
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.
Duy Nguyen wrote:
> I read this and thought "temporarily" but apparently it's not [1]. A
> lot of our links in the mail archive are gmane's :(
>
> [1] https://lars.ingebrigtsen.no/2016/07/28/the-end-of-gmane/
I may not have time to integrate this extensibly into the public-inbox
search engine to
On Thu, Jul 28, 2016 at 02:37:04PM -0700, Junio C Hamano wrote:
> Josh Triplett writes:
>
> > I'd like to propose changing the default behavior of git-format-patch to
> > --from (and adding a --from-author option to override, and perhaps a
> > config setting). This will not change the output *e
Josh Triplett writes:
> I'd like to propose changing the default behavior of git-format-patch to
> --from (and adding a --from-author option to override, and perhaps a
> config setting). This will not change the output *except* when
> formatting patches authored by someone else. git-am and git-
Junio C Hamano writes:
> Eric Wong writes:
>
>> Repushed my master in case it's a convenient time to pull.
>>
>> The following changes since commit 29493589e97a2de0c4c1c314f61ccafaee3b5caf:
>>
>> archive-tar: huge offset and future timestamps would not work on 32-bit
>> (2016-07-15 10:51:55 -
Kirill Smelkov writes:
> I'm waiting so long for main patch to be at least queued to pu, that I'm
> now a bit frustrated and ready to do something not related to main goal :)
Perhaps the first step would be to stop putting multiple patches in
a single e-mail buried after a few pages of discussio
When git-format-patch formats a patch authored by someone other than
yourself, it defaults to filling in the "From:" field of the email from
the commit author. If you explicitly pass the --from option,
git-format-patch will instead use your own committer identity as the
"From:", and then put a "Fr
Eric Wong wrote:
> Code is AGPL-3.0+: git clone https://public-inbox.org/
>
>
> Additional mirrors or forks (perhaps different UIs) are very welcome,
Btw, it's possible to do quote highlighting with user-side CSS:
https://public-inbox.org/meta/20160709-user-side-css-example@11/
Will probably
Duy Nguyen writes:
>> 4. Fsck complains about missing blobs. Should be fairly easy to fix.
>
> Not really. You'll have to associate path information with blobs
> before you decide that a blob should exist or not.
Also the same blob or the tree can exist both inside and outside the
narrowed area,
Jakub Narębski writes:
> On the gripping hand, the number of currently processed commits
> (instructions) in an interactive rebase is a number, and arithmetic
> expansion can be understood as shell equivalent of casting to integer.
I get that argument; it is a bit too cute a justification for my
Junio, first of all thanks for feedback,
On Wed, Jul 27, 2016 at 01:40:36PM -0700, Junio C Hamano wrote:
> Kirill Smelkov writes:
>
> > > From: Kirill Smelkov
> > Subject: [PATCH 1/2] pack-objects: Make sure use_bitmap_index is not active
> > under
> > --local or --honor-pack-keep
> >
> > Sin
Eric Wong writes:
> Repushed my master in case it's a convenient time to pull.
>
> The following changes since commit 29493589e97a2de0c4c1c314f61ccafaee3b5caf:
>
> archive-tar: huge offset and future timestamps would not work on 32-bit
> (2016-07-15 10:51:55 -0700)
>
> are available in the git
Stefan Beller writes:
> Well I wanted to express:
>
> The .gitmodules used in these Gerrit projects do not conform
> to Gits understanding of how .gitmodules should look like.
> Let's make Git understand this Gerrit corner case (which you
> would only need when using Gerrit).
>
> Adding
Stefan Beller writes:
> The depth of 2 I chose originally turns out to be a lucky
> accident too, as the depth from "Commit 2" is 2,
> so that we would observe the same depth no matter if
> a --depth 2 was given and working or not.
>
> I'll redo this test (as 2 tests, one is testing the situation
On Thu, Jul 28, 2016 at 12:10 PM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> Gerrit has a "superproject subscription" feature[1], that triggers a
>> commit in a superproject that is subscribed to its submodules.
>> Conceptually this Gerrit feature can be done on the client side with
>> Gi
W dniu 2016-07-28 o 18:56, Øyvind A. Holm pisze:
> On 28 July 2016 at 18:37, Junio C Hamano wrote:
>> Øyvind A. Holm writes:
>>> This is a script I created some weeks ago, and I've found it to be
>>> immensely useful. Here is a snippet from git-testadd --help:
>>>
>>> If you have lots of unrel
On Thu, Jul 28, 2016 at 11:47 AM, Stefan Beller wrote:
> On Thu, Jul 28, 2016 at 11:39 AM, Junio C Hamano wrote:
>> Stefan Beller writes:
>>
>>> We used to ask for 3 changes and tested for having 1, so the test
>>> seems broken.
>>
>> I am not sure what to think of "seems broken".
>
> When askin
Stefan Beller writes:
> Gerrit has a "superproject subscription" feature[1], that triggers a
> commit in a superproject that is subscribed to its submodules.
> Conceptually this Gerrit feature can be done on the client side with
> Git via:
>
> git -C submodule update --remote --force
> g
Duy Nguyen wrote:
> On Thu, Jul 28, 2016 at 11:11 AM, Lars Schneider
> wrote:
> > Hi,
> >
> > gmane is down
At least the following should work in case public-inbox fails:
https://mid.mail-archive.com/%s
https://marc.info/?i=%s
public-inbox falls back to them on missing MIDs,
al
On Thu, Jul 28, 2016 at 11:39 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> We used to ask for 3 changes and tested for having 1, so the test
>> seems broken.
>
> I am not sure what to think of "seems broken".
When asking for depth 3, I would expect a result of 1,2, or 3 commits.
But w
W dniu 2016-07-28 o 20:22, Eric Sunshine pisze:
> On Thu, Jul 28, 2016 at 1:47 PM, Johannes Sixt wrote:
>> Interactive rebase uses 'wc -l' to write the current patch number
>> in a progress report. Some implementations of 'wc -l' produce spaces
>> before the number, leading to ugly output such as
Stefan Beller writes:
> We used to ask for 3 changes and tested for having 1, so the test
> seems broken.
I am not sure what to think of "seems broken".
Asking for 3 and having 1 is broken in what way? Should we be
expecting for 3 because we asked for that many? Should we expect
less than thr
On Thu, Jul 28, 2016 at 1:47 PM, Johannes Sixt wrote:
> Interactive rebase uses 'wc -l' to write the current patch number
> in a progress report. Some implementations of 'wc -l' produce spaces
> before the number, leading to ugly output such as
>
> Rebasing ( 3/8)
>
> Remove the spaces using
Gerrit has a "superproject subscription" feature[1], that triggers a
commit in a superproject that is subscribed to its submodules.
Conceptually this Gerrit feature can be done on the client side with
Git via:
git -C submodule update --remote --force
git -C commit -a -m "Update submodule
Interactive rebase uses 'wc -l' to write the current patch number
in a progress report. Some implementations of 'wc -l' produce spaces
before the number, leading to ugly output such as
Rebasing ( 3/8)
Remove the spaces using a trivial arithmetic evaluation.
Before 9588c52 (i18n: rebase-int
On 28/07/16 14:02, Remi Galan Alfonso wrote:
> Hi Phillip,
>
> Phillip Wood writes:
>> When running ‘git rebase -i --autostash’ if the editor fails then the
>> rebase fails but stash is not applied so the working files are not
>> restored. Running ‘git rebase --abort’ replies that there’s no reba
We used to ask for 3 changes and tested for having 1, so the test
seems broken.
Correct the test by using test_line_count that exists in the test suite.
Signed-off-by: Stefan Beller
---
t/t7406-submodule-update.sh | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/t/t7406-
The meat is in patch 2 and helps Git and Gerrit work well together.
patch 1 looks unrelated but it is not, as when left out the broken test, breaks
with patch 2. This is because we add more commits in the submodule.
Thanks,
Stefan
Stefan Beller (2):
t7406: correct depth test in shallow test
Gerrit has a "superproject subscription" feature[1], that triggers a
commit in a superproject that is subscribed to its submodules.
Conceptually this Gerrit feature can be done on the client side with
Git via:
git -C submodule update --remote --force
git -C commit -a -m "Update submodule
On Thu, Jul 28, 2016 at 11:11 AM, Lars Schneider
wrote:
> Hi,
>
> gmane is down
I read this and thought "temporarily" but apparently it's not [1]. A
lot of our links in the mail archive are gmane's :(
[1] https://lars.ingebrigtsen.no/2016/07/28/the-end-of-gmane/
--
Duy
--
To unsubscribe from th
Corrections..
On Thu, Jul 28, 2016 at 6:59 PM, Duy Nguyen wrote:
> Ah.. this is what I call narrow checkout [1] (but gmane is down at the moment)
s/checkout/clone/
> [2] https://github.com/pclouds/git/commits/lanh/narrow-checkout
s,lanh/,,
--
Duy
--
To unsubscribe from this list: send the lin
On Thu, Jul 28, 2016 at 6:02 PM, Robin Ruede wrote:
> This patch series adds a `--sparse-prefix=` option to multiple commands,
> allowing fetching repository contents from only a subdirectory of a remote.
>
> This works along with sparse-checkout, and is especially useful for
> repositories
> whe
On 28 July 2016 at 18:37, Junio C Hamano wrote:
> Øyvind A. Holm writes:
> > This is a script I created some weeks ago, and I've found it to be
> > immensely useful. Here is a snippet from git-testadd --help:
> >
> > If you have lots of unrelated uncommitted changes in the current
> > reposit
Stefan Beller writes:
>> Anyway, I'll apply the "addition to the completion" patch.
>>
>> Thanks.
>
> Thanks for this patch!
>
> Note: if we ever decide to resurrect sb/submodule-default-path,
> we run into a merge conflict. The reasoning for using
> "--recurse-submodules" instead of a plain "--r
Hi, I've been working with Jeff King on this patch, but he encouraged me to
email the list.
I recently learned about the diff-highlight tool and find it very helpful,
however, I frequently use the --graph option to git log which breaks the tool.
This patch looks to fix this.
I wanted to try and a
On Thu, Jul 28, 2016 at 5:49 AM, Heiko Voigt wrote:
> Commit 959b5455 (submodule: implement a config API for lookup of
> .gitmodules values, 2015-08-18) implemented the initial version of the
> submodule config cache. During development of that initial version we
> extracted the function gitmodule
This is a script I created some weeks ago, and I've found it to be
immensely useful. Here is a snippet from git-testadd --help:
If you have lots of unrelated uncommitted changes in the current
repository and want to split up the commit, how can you easily check
if the changes passes the test
On Thu, Jul 28, 2016 at 01:14:03PM +, Vasco Almeida wrote:
> static void die_bad_number(const char *name, const char *value)
> {
> - const char *reason = errno == ERANGE ?
> - "out of range" :
> - "invalid unit";
> if (!value)
>
Vasco Almeida writes:
> Unfold the message into several templates for each known origin_type.
> That would result in better translation at the expense of code
> verbosity.
Looks good now, except one minor nit I'll tweak out while queuing.
> if (cf->die_on_error)
> - die(_("bad
Vasco Almeida writes:
> Mark comment displayed when editing a note for translation.
>
> Signed-off-by: Vasco Almeida
> ---
>
> This patch follows the original output and Ævar Arnfjörð Bjarmason
> sugestion to remove \n from the source string in order to assure that the
> ouput layout is not chan
also pass sparse-prefix option from fetch to rev-list while checking
connectivity
Signed-off-by: Robin Ruede
---
builtin/fetch.c | 19 ++-
connected.c | 7 ++-
transport.c | 4
transport.h | 4
4 files changed, 28 insertions(+), 6 deletions(-)
diff -
For example
git fetch-pack --sparse-prefix=/contrib/ origin HEAD
Should fetch a pack that is generated on the remote by
echo HEAD | git pack-objects --revs --stdout --sparse-prefix=/contrib/
Signed-off-by: Robin Ruede
---
builtin/fetch-pack.c | 6 ++
fetch-pack.c | 4 +++
Signed-off-by: Robin Ruede
---
remote-curl.c | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/remote-curl.c b/remote-curl.c
index e181e62..b9f7cf1 100644
--- a/remote-curl.c
+++ b/remote-curl.c
@@ -20,6 +20,7 @@ static struct strbuf url = STRBUF_INIT;
struct op
For example
git clone git@remote:repo --sparse-prefix=/contrib/
would create a repository, cloning only the relevant files for the
contrib subdirectory, and also setting the sparse-checkout option, so
only those files will be checked out.
Signed-off-by: Robin Ruede
---
builtin/clone.c | 27
(this might be wrong, not sure if this is the right place)
Signed-off-by: Robin Ruede
---
cache-tree.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/cache-tree.c b/cache-tree.c
index f28b1f4..ab01ae5 100644
--- a/cache-tree.c
+++ b/cache-tree.c
@@ -354,7 +354,8 @@ static
this allows creating packs that contain only the blobs relevant to a
specific subdirectory, e.g.
echo HEAD | git pack-objects --revs --sparse-prefix=/contrib/
will create a pack containing the complete history of HEAD, including
all commits, all trees, and the files in the contrib directory.
this skips all blob objects who's path does not begin with the specified
prefix
Signed-off-by: Robin Ruede
---
list-objects.c | 4 +++-
revision.c | 4
revision.h | 1 +
3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/list-objects.c b/list-objects.c
index f3ca6aa..91a6
This patch series adds a `--sparse-prefix=` option to multiple commands,
allowing fetching repository contents from only a subdirectory of a remote.
This works along with sparse-checkout, and is especially useful for repositories
where a subdirectory has meaning when standing alone.
* Motivation
On Thu, Jul 28, 2016 at 8:38 AM, Junio C Hamano wrote:
> Heiko Voigt writes:
>
>> @@ -121,6 +121,11 @@ its behaviour. Try to make sure your explanation can
>> be understood
>> without external resources. Instead of giving a URL to a mailing list
>> archive, summarize the relevant points of th
Heiko Voigt writes:
> @@ -121,6 +121,11 @@ its behaviour. Try to make sure your explanation can be
> understood
> without external resources. Instead of giving a URL to a mailing list
> archive, summarize the relevant points of the discussion.
>
> +If you want to reference a previous commit
Michael Haggerty writes:
> On 07/27/2016 08:03 PM, Duy Nguyen wrote:
>>>
>>> https://github.com/mhagger/git-when-merged ?
>>
>> Beautiful. If it had an option to show a topic (i.e. git-log from
>> merge base to merge point) I would be ecstatic.
>
> That's a good idea. I just created a pull reque
Heiko Voigt writes:
>> Apparently we put the subject first and then the date. I always did it
>> the other way
>> round, to there is no strict coding guide line,...
Please don't say "this is not spelled out in the guidelines, so I
can do whatever I like, *EVEN* *THOUGH* I know that is different
On Thu, Jul 28, 2016 at 11:11:54AM +0200, Lars Schneider wrote:
> gmane is down and I wonder if there are alternatives to
> find a message based on the message ID in the Git
> mailing list archive?
Try:
https://public-inbox.org/git/20160710004813.ga20...@dcvr.yhbt.net
-Peff
--
To unsubscribe
On 07/27/2016 08:03 PM, Duy Nguyen wrote:
> On Wed, Jul 27, 2016 at 7:50 PM, Stefan Beller wrote:
>> On Wed, Jul 27, 2016 at 10:42 AM, Duy Nguyen wrote:
>>> Before I start doing anything silly because I don't know it can
>>> already be done without waving my C wand like a mad man...
>>>
>>> I oft
On Thu, Jul 28, 2016 at 02:10:12PM +0200, Lars Schneider wrote:
> > I think that's orthogonal. I just mean that using zero for success puts
> > you in our usual style, and then accumulating errors can be done with
> > "|=".
>
> Ah. I guess I was misguided by the way errors are currently handled
>
On Thu, Jul 28, 2016 at 09:16:18AM +0200, Lars Schneider wrote:
> But Peff ($gmane/299902), Duy, and Eric, seemed to prefer the pkt-line
> solution (gmane is down - otherwise I would have given you the links).
FWIW, I think there are arguments for transmitting size + content
(namely, that it is s
Introduced in 473166b ("config: add 'origin_type' to config_source
struct", 2016-02-19), Git can inform the user about the origin of a
config error, but the implementation does not allow translators to
translate the keywords 'file', 'blob, 'standard input', and
'submodule-blob'. Moreover, for the s
To reference previous commits people used to put just the abbreviated
SHA-1 into commit messages. This is what has evolved as a more
stable format for referencing commits. So lets document it for everyone
to lookup when needed.
Signed-off-by: Heiko Voigt
---
On Thu, Jul 28, 2016 at 01:16:36PM +02
Since arg[0] will be NULL without any argument here and starts_with()
does not like NULL-pointers.
Signed-off-by: Heiko Voigt
---
A small fix I found while developing the test.
test-submodule-config.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test-submodule-config.c b/
So we have simpler return handling code and all the cleanup code in
almost one place.
Signed-off-by: Heiko Voigt
---
This is an updated cleanup patch. Now with goto so we have simpler code.
submodule-config.c | 31 ---
1 file changed, 12 insertions(+), 19 deletions(-
Commit 959b5455 (submodule: implement a config API for lookup of
.gitmodules values, 2015-08-18) implemented the initial version of the
submodule config cache. During development of that initial version we
extracted the function gitmodule_sha1_from_commit(). During that process
we missed that the s
Hi Phillip,
Phillip Wood writes:
> When running ‘git rebase -i --autostash’ if the editor fails then the
> rebase fails but stash is not applied so the working files are not
> restored. Running ‘git rebase --abort’ replies that there’s no rebase
> in progress. If you notice what’s happened it’s n
> On 27 Jul 2016, at 20:11, Jeff King wrote:
>
> On Wed, Jul 27, 2016 at 07:31:26PM +0200, Lars Schneider wrote:
>
+ strbuf_grow(sb, size + 1); // we need one extra byte for the
packet flush
>>>
>>> What happens if size is the maximum for size_t here (i.e., 4GB-1 on a
>>> 32-b
Mark comment displayed when editing a note for translation.
Signed-off-by: Vasco Almeida
---
This patch follows the original output and Ævar Arnfjörð Bjarmason
sugestion to remove \n from the source string in order to assure that the
ouput layout is not change by one translator forgetting to add
1 - 100 of 108 matches
Mail list logo