On Sun, Sep 22, 2013 at 12:15:21PM -0700, Jonathon Mah wrote:
> A few days ago I came across the same "surprising" output of git-grep,
> tried to adjust the code to print "git show"-able object names, and
> ran into similar subtleties. I just found this thread, and Jeff's code
> handles more cases
On Tue, Sep 24, 2013 at 01:31:48AM -0500, Felipe Contreras wrote:
> > I don't think it is the end of the world if we say "upstream branch". I
> > was hoping to find a term that could be both friendly and accurate.
> >
> > And failing that, I hoped you might say "I see what you are saying, but
> >
On Tue, Sep 24, 2013 at 01:41:19AM -0500, Felipe Contreras wrote:
> > People who have different aliases for
> > the same terms are unaffected on existing boxes, but slightly hindered
> > on new boxes as the aliases do something else.
>
> Less hindered than in the current situation.
I do not agre
On Tue, Sep 24, 2013 at 1:18 AM, Jeff King wrote:
> On Tue, Sep 24, 2013 at 12:49:21AM -0500, Felipe Contreras wrote:
>
>> Anyway, if you are so worried about this hypothetical user not
>> noticing that 'git ci' didn't commit all the files, we could ma ci to
>> 'git commit -v' so we are being stra
On Tue, Sep 24, 2013 at 1:10 AM, Jeff King wrote:
> On Tue, Sep 24, 2013 at 12:57:36AM -0500, Felipe Contreras wrote:
>
>> No, I'm not. The users that know branch.*.remote exists know why it
>> exists. The part where it is explained, 'git config --help', is
>> perfectly clear: "When on branch , it
On Tue, Sep 24, 2013 at 12:49:21AM -0500, Felipe Contreras wrote:
> Anyway, if you are so worried about this hypothetical user not
> noticing that 'git ci' didn't commit all the files, we could ma ci to
> 'git commit -v' so we are being straightforward to the user as to what
> is being committed.
On Tue, Sep 24, 2013 at 12:57:36AM -0500, Felipe Contreras wrote:
> No, I'm not. The users that know branch.*.remote exists know why it
> exists. The part where it is explained, 'git config --help', is
> perfectly clear: "When on branch , it tells 'git fetch' and 'git
> push' which remote to fetch
On Tue, Sep 24, 2013 at 12:40 AM, Jeff King wrote:
> On Tue, Sep 24, 2013 at 12:36:38AM -0500, Felipe Contreras wrote:
>
>> > Yeah, it's not a term we use elsewhere, so it's not great. Probably
>> > "default remote" would be better, or even just say "branch.*.remote for
>> > the current branch" or
On Mon, Sep 23, 2013 at 02:37:29PM -0700, Jonathan Nieder wrote:
> Hi,
>
> Michael S. Tsirkin wrote:
> >> On Tue, Sep 17, 2013 at 04:56:16PM -0400, Jeff King wrote:
>
> > A problem with both schemes, though, is that they are not
> > backwards-compatible with existing git-patch-id implemen
On Tue, Sep 24, 2013 at 12:37 AM, Jeff King wrote:
> On Tue, Sep 24, 2013 at 12:32:46AM -0500, Felipe Contreras wrote:
>
>> > You are making things more consistent for people who already define
>> > those aliases in the same way (they are available everywhere, even if
>> > they have not moved thei
On Mon, Sep 23, 2013 at 02:37:29PM -0700, Jonathan Nieder wrote:
> >> Add --order-sensitive to get historical unstable behaviour.
>
> The --order-sensitive option seems confusing. How do I use it to
> replicate a historical patch-id? If I record all options that might
> have influenced ordering
On Tue, Sep 24, 2013 at 12:36:38AM -0500, Felipe Contreras wrote:
> > Yeah, it's not a term we use elsewhere, so it's not great. Probably
> > "default remote" would be better, or even just say "branch.*.remote for
> > the current branch" or something.
>
> Yeah, general users don't know what you a
On Tue, Sep 24, 2013 at 12:32:46AM -0500, Felipe Contreras wrote:
> > You are making things more consistent for people who already define
> > those aliases in the same way (they are available everywhere, even if
> > they have not moved their config to a new installation), but less so for
> > peopl
On Tue, Sep 24, 2013 at 12:30 AM, Jeff King wrote:
> On Tue, Sep 24, 2013 at 12:23:21AM -0500, Felipe Contreras wrote:
>
>> > Should this be "upstream remote" rather than "upstream branch"? I don't
>> > think we should be looking at branch.*.merge at all for git-fetch.
>>
>> As a general user, how
On Mon, Sep 23, 2013 at 11:53 PM, Jeff King wrote:
> On Sat, Sep 21, 2013 at 02:20:21PM -0500, Felipe Contreras wrote:
>
>> For now simply add a few common aliases.
>>
>> co = checkout
>> ci = commit
>> rb = rebase
>> st = status
>
> Are these the best definitions of those shortcuts? It se
On Fri, Sep 20, 2013 at 08:21:04AM +0200, Piotr Krukowiecki wrote:
> >I can't think off-hand of a way to do so using preprocessor tricks, and
> >even if we could, I suspect the result would end up quite ugly.
>
> What I meant was: can we add a test (in t/) which greps git source
> code and fails
On Tue, Sep 24, 2013 at 12:23:21AM -0500, Felipe Contreras wrote:
> > Should this be "upstream remote" rather than "upstream branch"? I don't
> > think we should be looking at branch.*.merge at all for git-fetch.
>
> As a general user, how do I configure the "upstream remote"?
Yeah, it's not a t
On Tue, Sep 24, 2013 at 12:03 AM, Jeff King wrote:
> On Sat, Sep 21, 2013 at 09:09:22AM -0500, Felipe Contreras wrote:
>
>> There's no mention of the 'origin' default, or the fact that the
>> upstream tracking branch remote is used.
>
> Sounds like a good thing to mention.
>
>> diff --git a/Docume
On Sat, Sep 21, 2013 at 09:09:23AM -0500, Felipe Contreras wrote:
> diff --git a/remote.c b/remote.c
> index efcba93..654e7f5 100644
> --- a/remote.c
> +++ b/remote.c
> @@ -480,7 +480,6 @@ static void read_config(void)
> int flag;
> if (default_remote_name) /* did this already */
>
On Sat, Sep 21, 2013 at 09:09:22AM -0500, Felipe Contreras wrote:
> There's no mention of the 'origin' default, or the fact that the
> upstream tracking branch remote is used.
Sounds like a good thing to mention.
> diff --git a/Documentation/git-fetch.txt b/Documentation/git-fetch.txt
> index e0
On Sun, Sep 22, 2013 at 12:04:24AM +0300, Ram Rachum wrote:
> I'm making a script `gm` which lets me merge one branch into another
> without having either checked out. It works for some cases but not
> all. I'm trying to make it work for more cases.
>
> I concluded that the best way to do it woul
On Sat, Sep 21, 2013 at 02:20:21PM -0500, Felipe Contreras wrote:
> For now simply add a few common aliases.
>
> co = checkout
> ci = commit
> rb = rebase
> st = status
Are these the best definitions of those shortcuts? It seems[1] that some
people define "ci" as "commit -a", and some pe
Does anyone care to help? I'd really like to solve this :(
On 22/9/2013 00:04, Ram Rachum wrote:
Hi everybody!
I need some help with Git.
I'm making a script `gm` which lets me merge one branch into another
without having either checked out. It works for some cases but not
all. I'm trying t
On Mon, Sep 23, 2013 at 08:54:35PM +0200, Steffen Prohaska wrote:
> Labeled lists require a double colon.
Thanks, this looks good.
While looking at the git.txt source, I noticed it is quite awkward to
read due to the size of the stalenotes section. The patch below moves
that section out to its o
On Mon, Sep 23, 2013 at 11:49:01AM -0700, Brandon Casey wrote:
> Brandon Casey (16):
> contrib/git-credential-gnome-keyring.c: remove unnecessary
> pre-declarations
> [...]
Thanks, I think this is a good change. There were patches[1] from
Philipp about a year ago that went in the opposite d
Травяной чай представляется мощным естественным антиоксидантом
http://jilc.net/fba5
On Sep 23, 2013, at 14:54, Jonathan Nieder wrote:
Stefan Beller wrote:
git fetch https://repo.or.cz/r/git/jrn.git
doesn't work, as it yields:
fatal: unable to access 'https://repo.or.cz/r/git/jrn.git/': server
certificate verification failed. CAfile: /etc/ssl/certs/ca-
certificates.c
Stefan Beller wrote:
> git fetch https://repo.or.cz/r/git/jrn.git
> doesn't work, as it yields:
> fatal: unable to access 'https://repo.or.cz/r/git/jrn.git/': server
> certificate verification failed. CAfile: /etc/ssl/certs/ca-certificates.crt
> CRLfile: none
Ah, figured it out. http://r
Stefan Beller wrote:
> On 09/23/2013 11:10 PM, Jonathan Nieder wrote:
>> https://repo.or.cz/r/git/jrn.git
[...]
> How would I get these changes?
> A plain
> git fetch https://repo.or.cz/r/git/jrn.git
> doesn't work, as it yields:
> fatal: unable to access 'https://repo.or.cz/r/git/jrn.g
Hi,
Michael S. Tsirkin wrote:
>> On Tue, Sep 17, 2013 at 04:56:16PM -0400, Jeff King wrote:
> A problem with both schemes, though, is that they are not
> backwards-compatible with existing git-patch-id implementations.
[...]
>>> It may be esoteric enough not to worry about, though.
Yeah,
On 09/23/2013 11:10 PM, Jonathan Nieder wrote:
> You can find the changes described here in the integration branches at
>
> https://repo.or.cz/r/git/jrn.git
>
> I am still catching up on patches sent since last week. If I have
> missed yours, do not despair, but feel free to send me a remind
The man page for `git svn` describes a situation in which "'git svn'
will not be able to rebuild" your .git/svn/**/.rev_map files, but no
mention is made of in what circumstances `git svn` *will* be able to do
so, or how to get `git svn` to do so.
This patch adds some language to the description o
What's cooking in git.git (Sep 2013, #07; Mon, 23)
--
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 fifth batch of topics are in 'master'
From: Junio C Hamano
Add a new, extensible style for tests that allows the addition of new
parameters other than the prerequitites
Signed-off-by: Junio C Hamano
Signed-off-by: Thomas Gummerer
---
t/README| 24 +++--
t/perf/README | 12 -
t/perf/pe
Add a --cleanup for the performance tests. This option can be used to
clean up the tested repository after each time the performance tests are
run. The option can be specified for normal tests too, although it will
not do anything for them. Use test_when_finished for those tests.
Signed-off-by:
From: Thomas Rast
Add a performance test for index version [23]/4 by using
git update-index --index-version=x, thus testing both the reader
and the writer speed of all index formats.
Signed-off-by: Thomas Rast
Signed-off-by: Thomas Gummerer
---
t/perf/p0003-index.sh | 56 +
This patch series comes out of the discussion at $gmane/234874, adding
a new (modern) form of writing tests. This form allows easier
extensibility of test cases. In the next patch a --cleanup option is
added for performance tests. The option does nothing for normal
tests, as test_when_finished i
On Fri, Sep 20, 2013 at 12:32:26AM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 17, 2013 at 04:56:16PM -0400, Jeff King wrote:
> > On Tue, Sep 17, 2013 at 11:38:07PM +0300, Michael S. Tsirkin wrote:
> >
> > > > A problem with both schemes, though, is that they are not
> > > > backwards-compatibl
On Mon, Sep 23, 2013 at 1:26 PM, Benoit Person wrote:
> diff --git a/contrib/mw-to-git/t/t9365-continuing-queries.sh
> b/contrib/mw-to-git/t/t9365-continuing-queries.sh
> new file mode 100755
> index 000..6fb5df4
> --- /dev/null
> +++ b/contrib/mw-to-git/t/t9365-continuing-queries.sh
> @@ -0,
On Mon, Sep 23, 2013 at 1:49 PM, Brandon Casey wrote:
> From: Brandon Casey
>
> Mostly unchanged.
>
> Inserts a patch to fix the style issues for block statements.
> i.e. use "if ()" instead of "if()"
>
> A couple early patches were reordered to improve logical flow.
>
> Updated the comment in th
On Mon, Sep 23, 2013 at 1:49 PM, Brandon Casey wrote:
> From: Brandon Casey
>
> The gnome-keyring lib distributed with RHEL 5.X is ancient and does
> not provide a few of the functions/defines that more recent versions
> do, but mostly the API is the same. Let's provide the missing bits
> via ma
On Mon, Sep 23, 2013 at 2:20 PM, Patrick Donnelly wrote:
> On Mon, Sep 23, 2013 at 2:53 PM, Felipe Contreras
> wrote:
>> Yes, but that's not what the words said, the words said 'lua
>> integration' and 'ruby integration' would take that much. Either way
>> it doesn't matter, shared libraries exis
git-prune-packed operates on GIT_OBJECT_DIRECTORY. See 'environment.c',
git_object_dir = getenv(DB_ENVIRONMENT);
and cache.h,
#define DB_ENVIRONMENT "GIT_OBJECT_DIRECTORY"
Signed-off-by: Steffen Prohaska
---
Documentation/git-prune-packed.txt | 2 +-
1 file changed, 1 insertion(+), 1
On Mon, Sep 23, 2013 at 2:53 PM, Felipe Contreras
wrote:
> Yes, but that's not what the words said, the words said 'lua
> integration' and 'ruby integration' would take that much. Either way
> it doesn't matter, shared libraries exist for a reason. We don't need
> to statically compile openssl do
From: Brandon Casey
If the correct arguments were not specified, this program should exit
non-zero. Let's do so.
Signed-off-by: Brandon Casey
---
contrib/credential/gnome-keyring/git-credential-gnome-keyring.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/credent
From: Brandon Casey
Mark global variable and functions as static.
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/contrib/credential/gnome-keyring/git-credential-gnome
Junio C Hamano wrote:
> [on vacaion, with only gmail webmail UI; please excuse me if this message
> comes
> out badly formatted or gets dropped by vger.kernel.org]
>
> On Sat, Sep 21, 2013 at 4:56 PM, brian m. carlson
> wrote:
> > On Sat, Sep 21, 2013 at 05:52:05PM -0500, Felipe Contreras wrote:
From: Brandon Casey
The gnome-keyring lib (0.4) distributed with RHEL 4.X is really ancient
and does not provide most of the synchronous functions that even ancient
releases do. Thankfully, we're only using one function that is missing.
Let's emulate gnome_keyring_item_delete_sync() by calling t
From: Brandon Casey
Since this is a Gnome application, let's set the application name to
something reasonable. This will be displayed in Gnome dialog boxes
e.g. the one that prompts for the user's keyring password.
We add an include statement for glib.h and add the glib-2.0 cflags and
libs to t
From: Brandon Casey
Rather than carefully allocating memory for sprintf() to write into,
let's make use of the glib helper function g_strdup_printf(), which
makes things a lot easier and less error-prone.
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c
From: Brandon Casey
Rather than roll our own, let's use the messaging functions provided
by glib.
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c | 33 +++---
1 file changed, 4 insertions(+), 29 deletions(-)
diff --git a/contrib/credential/g
From: Brandon Casey
These are all defined before they are used, so it is not necessary to
pre-declare them. Remove the pre-declarations.
Signed-off-by: Brandon Casey
---
.../credential/gnome-keyring/git-credential-gnome-keyring.c | 13 -
1 file changed, 13 deletions(-)
diff --git
Benoit Person writes:
> The latest API renamed and moved the "continuing" information in the
> response, necessary to build the next query. The code failed to retrieve
> that information but still detected that it was in a "continuing
> query". As a result, it launched the same query over and ove
From: Brandon Casey
Ensure buffer length is non-zero before attempting to access the last
element.
Signed-off-by: Brandon Casey
---
contrib/credential/gnome-keyring/git-credential-gnome-keyring.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/credential/gnome-keyri
From: Brandon Casey
Rather than roll our own, let's use the memory allocation/free routines
provided by glib.
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c | 48 --
1 file changed, 16 insertions(+), 32 deletions(-)
diff --git a/contrib
From: Brandon Casey
gnome-keyring provides functions to allocate non-pageable memory (if
possible). Let's use them to allocate memory that may be used to hold
secure data read from the keyring.
Signed-off-by: Brandon Casey
---
.../credential/gnome-keyring/git-credential-gnome-keyring.c | 12
From: Brandon Casey
Produce an error message when we fail to store a password to the keyring.
Signed-off-by: Brandon Casey
---
Difference from v1:
Additionally interpret GNOME_KEYRING_RESULT_CANCELLED as a successful exit
status, since that means that the user intentionally cancelled the
opera
From: Brandon Casey
Also, initialization is not necessary since it is assigned before it is
used.
Signed-off-by: Brandon Casey
---
contrib/credential/gnome-keyring/git-credential-gnome-keyring.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/credential/gnome-keyrin
From: Brandon Casey
The gnome-keyring lib distributed with RHEL 5.X is ancient and does
not provide a few of the functions/defines that more recent versions
do, but mostly the API is the same. Let's provide the missing bits
via macro definitions and function implementation.
Signed-off-by: Brand
From: Brandon Casey
Signed-off-by: Brandon Casey
---
.../credential/gnome-keyring/git-credential-gnome-keyring.c| 10 --
1 file changed, 10 deletions(-)
diff --git a/contrib/credential/gnome-keyring/git-credential-gnome-keyring.c
b/contrib/credential/gnome-keyring/git-credential-g
From: Brandon Casey
gnome-keyring provides functions for allocating non-pageable memory (if
possible) intended to be used for storing passwords. Let's use them.
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c| 21 ++---
1 file changed, 6 i
Labeled lists require a double colon.
Signed-off-by: Steffen Prohaska
---
Documentation/git.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/git.txt b/Documentation/git.txt
index 5d68d33..4c2757e 100644
--- a/Documentation/git.txt
+++ b/Documentation/gi
From: Brandon Casey
Signed-off-by: Brandon Casey
---
.../gnome-keyring/git-credential-gnome-keyring.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/contrib/credential/gnome-keyring/git-credential-gnome-keyring.c
b/contrib/credential/gnome-keyrin
On Mon, Sep 23, 2013 at 1:20 PM, Patrick Donnelly wrote:
> Hello Felipe,
>
> On Sun, Sep 22, 2013 at 4:29 AM, Felipe Contreras
> wrote:
>> On Sun, Sep 22, 2013 at 3:12 AM, Fredrik Gustafsson wrote:
>>> And see my humble test of what the speedup would be for git-submodule
>>> even with a faulty l
From: Brandon Casey
Mostly unchanged.
Inserts a patch to fix the style issues for block statements.
i.e. use "if ()" instead of "if()"
A couple early patches were reordered to improve logical flow.
Updated the comment in the last patch to hopefully improve clarity
wrt RHEL 4.X
The only functi
Hello Felipe,
On Sun, Sep 22, 2013 at 4:29 AM, Felipe Contreras
wrote:
> On Sun, Sep 22, 2013 at 3:12 AM, Fredrik Gustafsson wrote:
>> And see my humble test of what the speedup would be for git-submodule
>> even with a faulty lua integration (still forking... but huge
>> performance boost anywa
Matthieu Moy wrote:
> Benoit Person writes:
>> For now, if the tests suite is run without the fix, the new test
>> introduces an infinite loop. I am not sure if this should be handled ?
>> (a timeout of some kind maybe ?)
>
> If the patch fix this, then it's not a really big problem. The test
> f
Mediawiki introduces a new API for queries w/ more than 500 results in
version 1.21. That change triggered an infinite loop while cloning a
mediawiki with such a page.
The latest API renamed and moved the "continuing" information in the
response, necessary to build the next query. The code failed
On Mon, Sep 23, 2013 at 3:20 AM, John Szakmeister wrote:
> On Sun, Sep 22, 2013 at 10:07:56PM -0700, Brandon Casey wrote:
>> A few cleanups, followed by improved usage of the glib library (no need
>> to reinvent the wheel when glib provides the necessary functionality), and
>> then the addition of
Thanks.
On Sun, Sep 22, 2013 at 10:43 PM, Felipe Contreras
wrote:
> Brandon Casey wrote:
>> Ensure buffer length is non-zero before attempting to access the last
>> element.
>>
>> Signed-off-by: Brandon Casey
>> ---
>> contrib/credential/gnome-keyring/git-credential-gnome-keyring.c | 2 +-
>> 1
On Sun, Sep 22, 2013 at 10:07:56PM -0700, Brandon Casey wrote:
> A few cleanups, followed by improved usage of the glib library (no need
> to reinvent the wheel when glib provides the necessary functionality), and
> then the addition of support for RHEL 4.x and 5.x.
>
> Brandon Casey (15):
> con
Am 12.09.2013 06:10, schrieb Junio C Hamano:
> Karsten Blees writes:
>
>> +/*
>> + * Hashmap entry data structure, intended to be used as first member of user
>> + * data structures. Consists of a pointer and an int. Ideally it should be
>
> It is technically correct to say this is "intended to
Sorry for the delay, I've been on vacation...
Am 12.09.2013 01:56, schrieb Junio C Hamano:
> Karsten Blees writes:
>
>> +#define FNV32_BASE ((unsigned int) 0x811c9dc5)
>> +#define FNV32_PRIME ((unsigned int) 0x01000193)
>> + ...
>> +static inline unsigned int bucket(const hashmap *map, const has
On Fri, Sep 20, 2013 at 12:34 PM, Jeff King wrote:
> On Fri, Sep 20, 2013 at 07:38:17AM +0200, Francis Moreau wrote:
>
>> I'm using notes in my project. I'm wondering if it's possible to save
>> the state of the notes when I'm releasing/tagging a new version of my
>> project so I can restore the s
74 matches
Mail list logo