All of lore.kernel.org
 help / color / mirror / Atom feed
* What's cooking in git.git (Apr 2012, #09; Mon, 23)
@ 2012-04-23 22:04 Junio C Hamano
  2012-04-24  9:14 ` Nelson Benitez Leon
  2012-04-25  2:14 ` What's cooking (incremental) Junio C Hamano
  0 siblings, 2 replies; 13+ messages in thread
From: Junio C Hamano @ 2012-04-23 22:04 UTC (permalink / raw)
  To: git

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 fourth batch of topics in this cycle is now in 'master'.

You can find the changes described here in the integration branches of the
repositories listed at

    http://git-blame.blogspot.com/p/git-public-repositories.html

--------------------------------------------------
[Graduated to "master"]

* bw/test-fix-grep-gnuism (2012-04-19) 1 commit
  (merged to 'next' on 2012-04-20 at ee7eec4)
 + t9400: fix gnuism in grep

Fix two places that were the only place in the test suite that gave "a\+"
to platform grep and expected it to mean one or more "a", which is a
blatant GNUism.

* jc/maint-blame-minimal (2012-04-11) 1 commit
  (merged to 'next' on 2012-04-20 at 2213284)
 + blame: accept --need-minimal

"git blame" started missing quite a few changes from the origin since we
stopped using the diff minimalization by default in v1.7.2 era.

* jh/apply-free-patch (2012-04-11) 9 commits
  (merged to 'next' on 2012-04-15 at 8cccf57)
 + apply: document buffer ownership rules across functions
 + apply: tighten constness of line buffer
 + apply: drop unused macro
 + apply: free unused fragments for submodule patch
 + apply: free patch->result
 + apply: release memory for fn_table
 + apply: free patch->{def,old,new}_name fields
 + apply: rename free_patch() to free_patch_list()
 + apply: do not leak patches and fragments

Valgrind reports quite a lot of discarded memory inside apply.

* jk/rebase-i-submodule-conflict-only (2012-04-09) 1 commit
  (merged to 'next' on 2012-04-15 at 94367a8)
 + rebase -i continue: don't skip commits that only change submodules

Giving "--continue" to a conflicted "rebase -i" session skipped a commit
that only results in changes to submodules.

* jl/maint-submodule-recurse-fetch (2012-04-14) 1 commit
  (merged to 'next' on 2012-04-20 at 2769d39)
 + submodules: recursive fetch also checks new tags for submodule commits

* jn/debian-customizes-default-editor (2012-04-10) 2 commits
  (merged to 'next' on 2012-04-15 at 0fc85c3)
 + var doc: advertise current DEFAULT_PAGER and DEFAULT_EDITOR settings
 + var doc: default editor and pager are configurable at build time

* jn/fast-import-doc (2012-04-15) 1 commit
  (merged to 'next' on 2012-04-20 at b9a7250)
 + fast-import doc: cat-blob and ls responses need to be consumed quickly

* jn/more-i18ncmp (2012-04-14) 3 commits
  (merged to 'next' on 2012-04-20 at b5548a3)
 + test: am of empty patch should not succeed
 + test: use test_i18ncmp for "Patch format detection failed" message
 + test: do not rely on US English tracking-info messages

* lp/maint-diff-three-dash-with-graph (2012-03-20) 3 commits
  (merged to 'next' on 2012-04-20 at b6cf0ff)
 + t4202: add test for "log --graph --stat -p" separator lines
 + log --graph: fix break in graph lines
 + log --graph --stat: three-dash separator should come after graph lines

* ms/remote-usage-string (2012-04-09) 1 commit
  (merged to 'next' on 2012-04-15 at 704be8f)
 + remote: update builtin usage

As an independent follow-up, we may want to rethink how the overall usage
string and subcommand usage strings are maintained.

* pw/git-p4 (2012-04-09) 3 commits
  (merged to 'next' on 2012-04-15 at 81ea41a)
 + git p4: use "git p4" directly in tests
 + git p4: update name in script
 + git-p4: move to toplevel
 (this branch is used by ld/git-p4-tags-and-labels.)

* rj/mingw-isguid (2012-04-18) 1 commit
  (merged to 'next' on 2012-04-20 at 9dbebca)
 + compat/mingw.h: Set S_ISUID to prevent a fast-import test failure

* rj/sh-setup-mswin-pwd (2012-04-18) 1 commit
  (merged to 'next' on 2012-04-20 at 6125f8d)
 + git-sh-setup.sh: Add an pwd() function for MinGW

* rj/submodule-mswin-path (2012-04-18) 1 commit
  (merged to 'next' on 2012-04-20 at fd9f8b3)
 + git-submodule.sh: Don't use $path variable in eval_gettext string

* rl/show-empty-prefix (2012-04-10) 1 commit
  (merged to 'next' on 2012-04-15 at b2e5938)
 + rev-parse --show-prefix: add in trailing newline

* rs/commit-list-sort-in-batch (2012-04-17) 4 commits
  (merged to 'next' on 2012-04-17 at 21208e9)
 + mergesort: rename it to llist_mergesort()
  (merged to 'next' on 2012-04-15 at 39fe527)
 + revision: insert unsorted, then sort in prepare_revision_walk()
 + commit: use mergesort() in commit_list_sort_by_date()
 + add mergesort() for linked lists

* sl/test-wc-l-line-count (2012-04-11) 1 commit
  (merged to 'next' on 2012-04-15 at 40797d9)
 + tests: modernise style: more uses of test_line_count

--------------------------------------------------
[New Topics]

* cc/fix-missing-va-end-in-revert (2012-04-21) 1 commit
  (merged to 'next' on 2012-04-23 at b90eb70)
 + revert: add missing va_end

Will merge to 'master'.

* pw/t5800-import-race-fix (2012-04-22) 2 commits
 - fixup! git-remote-testgit: fix race when spawning fast-import
 - git-remote-testgit: fix race when spawning fast-import

Will merge to 'next' after squashing the fixup!

* nd/i18n (2012-04-23) 11 commits
 - i18n: bundle: mark strings for translation
 - i18n: index-pack: mark strings for translation
 - i18n: apply: update say_patch_name to give translators complete sentence
 - i18n: apply: mark strings for translation
 - i18n: remote: mark strings for translation
 - i18n: make warn_dangling_symref() automatically append \n
 - i18n: help: mark strings for translation
 - i18n: mark relative dates for translation
 - fixup! strbuf: convenience format functions with \n automatically appended
 - strbuf: convenience format functions with \n automatically appended
 - Makefile: feed all header files to xgettext

Will merge to 'next' after squashing the fixup!
We can address the pseudo-underline of section headers in "git help --all"
as a follow-up patch.

* rk/git-svn-ignore-sigpipe (2012-04-23) 3 commits
 - git-svn: drop redundant blocking of SIGPIPE
 - git-svn: ignore SIGPIPE
 - git-svn: use POSIX::sigprocmask to block signals

Waiting for an Ack from Eric.

--------------------------------------------------
[Stalled]

* jc/split-blob (2012-04-03) 7 commits
 - chunked-object: streaming checkout
 - chunked-object: fallback checkout codepaths
 - bulk-checkin: support chunked-object encoding
 - bulk-checkin: allow the same data to be multiply hashed
 - new representation types in the packstream
 - packfile: use varint functions
 + varint: make it available outside the context of pack
 (this branch is tangled with jc/index-v4.)

Not ready.

I finished the streaming checkout codepath, but as explained in
127b177 (bulk-checkin: support chunked-object encoding, 2011-11-30),
these are still early steps of a long and painful journey. At least
pack-objects and fsck need to learn the new encoding for the series
to be usable locally, and then index-pack/unpack-objects needs to
learn it to be used remotely.

Given that I heard a lot of noise that people want large files, and
that I was asked by somebody at GitTogether'11 privately for an
advice on how to pay developers (not me) to help adding necessary
support, I am somewhat dissapointed that the original patch series
that was sent almost two months ago still remains here without much
comments and updates from the developer community. I even made the
interface to the logic that decides where to split chunks easily
replaceable, and I deliberately made the logic in the original patch
extremely stupid to entice others, especially the "bup" fanboys, to
come up with a better logic, thinking that giving people an easy
target to shoot for, they may be encouraged to help out. The plan is
not working :-(.

--------------------------------------------------
[Cooking]

* nh/empty-rebase (2012-04-20) 7 commits
 - fixup! git-rebase: add keep_empty flag
 - git-rebase: add keep_empty flag
 - fixup! git-cherry-pick: Add test to validate new options
 - git-cherry-pick: Add test to validate new options
 - fixup! git-cherry-pick: Add keep-redundant-commits option
 - git-cherry-pick: Add keep-redundant-commits option
 - git-cherry-pick: add allow-empty option

Will merge to 'next' after squashing the fixup!

* rt/cherry-revert-conflict-summary (2012-04-19) 1 commit
  (merged to 'next' on 2012-04-23 at e7766c2)
 + sequencer: remove additional blank line

In the older days, the header "Conflicts:" in "cherry-pick" and
"merge" was separated by a blank line from the list of paths that
follow for readability, but when "merge" was rewritten in C, we lost
it by mistake. Remove the newline from "cherry-pick" to make them
match again.

Will merge to 'master'.

* js/fast-import-test-9300 (2012-04-20) 1 commit
  (merged to 'next' on 2012-04-23 at a9bd236)
 + t9300-fast-import: avoid 'exit' in test_expect_success snippets

Will merge to 'master'.

* jm/maint-strncpy-diff-no-index (2012-04-16) 1 commit
  (merged to 'next' on 2012-04-20 at 638ed9f)
 + diff: avoid stack-buffer-read-overrun for very long name

Will merge to 'master'.

* lp/diffstat-with-graph (2012-04-18) 6 commits
  (merged to 'next' on 2012-04-23 at 8de0747)
 + Prevent graph_width of stat width from falling below min
 + t4052: Test diff-stat output with minimum columns
 + t4052: Adjust --graph --stat output for prefixes
 + Adjust stat width calculations to take --graph output into account
 + Add output_prefix_length to diff_options
 + t4052: test --stat output with --graph

Looked reasonable.
Will merge to 'master'.

* jc/merge-reduce-parents-early (2012-04-18) 5 commits
  (merged to 'next' on 2012-04-23 at 16e7e53)
 + fmt-merge-msg: discard needless merge parents
 + builtin/merge.c: reduce parents early
 + builtin/merge.c: collect other parents early
 + builtin/merge.c: remove "remoteheads" global variable
 + merge tests: octopus with redundant parents

Will merge to 'master'.

* cb/cherry-pick-rev-path-confusion (2012-04-15) 1 commit
  (merged to 'next' on 2012-04-23 at f872ba5)
 + cherry-pick: do not expect file arguments

The command line parser choked "git cherry-pick $name" when $name can be
both revision name and a pathname, even though $name can never be a path
in the context of the command.

The issue the patch addresses is real, but the way it is implemented felt
unnecessarily invasive a bit.  It may be cleaner for this caller to add
the "--" to the end of the argv_array it passes to setup_revisions().

Will merge to 'master'.

* cb/daemon-test-race-fix (2012-04-19) 1 commit
 - git-daemon wrapper to wait until daemon is ready

Fix race condition between starting and waiting for git-daemon in the
test.

Will merge to 'next'.

* cb/maint-report-mount-point-correctly-in-setup (2012-04-15) 1 commit
  (merged to 'next' on 2012-04-23 at 9e0c8b4)
 + properly keep track of current working directory

The filesystem boundary was not correctly reported when .git directory
discovery stopped at a mount point.

Will merge to 'master'.

* rl/maint-stash-i18n-save-error (2012-04-14) 1 commit
  (merged to 'next' on 2012-04-23 at 3692fec)
 + stash: use eval_gettextln correctly

Will merge to 'master'.

* nd/columns (2012-04-20) 9 commits
 - tag: add --column
 - column: support piping stdout to external git-column process
 - status: add --column
 - branch: add --column
 - help: reuse print_columns() for help -a
 - column: add dense layout support
 - column: add columnar layout
 - Stop starting pager recursively
 - Add column layout skeleton and git-column

Will merge to 'next'.

* nl/http-proxy-more (2012-04-19) 7 commits
 - http: fix proxy password passing
 - http: fix proxy authentication
 - http: rename HTTP_REAUTH to HTTP_AUTH_RETRY
 - http: Avoid limit of retrying request only twice
 - http: handle proxy authentication failure (error 407)
 - http: handle proxy proactive authentication
 - http: try http_proxy env var when http.proxy config option is not set

The code to talk to http proxies learn to use the same credential
API used to talk to the final http destinations.  This still needs
to peek into $ENV{HTTPS_PROXY}.

* mb/fetch-call-a-non-branch-a-ref (2012-04-17) 2 commits
  (merged to 'next' on 2012-04-20 at df2c57e)
 + fetch: describe new refs based on where it came from
 + fetch: Give remote_ref to update_local_ref() as well

Will merge to 'master'.

* mh/ref-api (2012-04-10) 13 commits
  (merged to 'next' on 2012-04-20 at 94e9384)
 + do_for_each_ref(): only iterate over the subtree that was requested
 + refs: store references hierarchically
 + sort_ref_dir(): simplify logic
 + refs.c: rename ref_array -> ref_dir
 + struct ref_entry: nest the value part in a union
 + check_refname_component(): return 0 for zero-length components
 + free_ref_entry(): new function
 + names_conflict(): simplify implementation
 + repack_without_ref(): reimplement using do_for_each_ref_in_array()
 + do_for_each_ref_in_arrays(): new function
 + do_for_each_ref_in_array(): new function
 + refs: manage current_ref within do_one_ref()
 + refs.c: reorder definitions more logically

Allows walking only a sub-hierarchy in refs/*.

Will merge to 'master'.

* jk/repack-no-explode-objects-from-old-pack (2012-04-18) 4 commits
  (merged to 'next' on 2012-04-20 at 4eb2534)
 + gc: use argv-array for sub-commands
 + argv-array: add a new "pushl" method
 + argv-array: refactor empty_argv initialization
  (merged to 'next' on 2012-04-15 at 7586e50)
 + gc: do not explode objects which will be immediately pruned

* ld/git-p4-tags-and-labels (2012-04-11) 2 commits
  (merged to 'next' on 2012-04-15 at 1b1e9a1)
 + git p4: import/export of labels to/from p4
 + git p4: Fixing script editor checks

Waiting for a follow-up to address the review comments
by Pete Wyckoff Message-ID: <20120418113422.GB19994@padd.com>

* zj/upstream-error-message (2012-04-15) 5 commits
  (merged to 'next' on 2012-04-20 at 1ed9682)
 + i18n: mark @{upstream} error messages for translation
 + Be more specific if upstream branch is not tracked
 + Provide better message for barnhc_wiht_tpyo@{u}
 + Provide branch name in error message when using @{u}
 + t1507: add tests to document @{upstream} behaviour

Will merge to 'master'.

* fc/completion-tests (2012-04-21) 7 commits
  (merged to 'next' on 2012-04-23 at 9b42326)
 + completion: fix completion after 'git --option <TAB>'
 + completion: avoid trailing space for --exec-path
 + completion: add missing general options
 + completion: simplify by using $prev
 + completion: simplify __gitcomp_1
 + tests: add tests for the __gitcomp() completion helper function
 + tests: add initial bash completion tests

Will merge to 'master'.

* nd/threaded-index-pack (2012-04-19) 4 commits
 - index-pack: disable threading if NO_PREAD is defined
 - index-pack: support multithreaded delta resolving
 - index-pack: split second pass obj handling into own function
 - compat/win32/pthread.h: Add an pthread_key_delete() implementation

* cb/http-multi-curl-auth (2012-04-14) 4 commits
  (merged to 'next' on 2012-04-23 at b0259be)
 + http: use newer curl options for setting credentials
 + http: clean up leak in init_curl_http_auth
 + fix http auth with multiple curl handles
 + http auth fails with multiple curl handles

Fixes http authentication breakage when we keep multiple HTTP requests in
flight using curl-multi.

Will merge to 'master'.

* pw/fast-import-dataref-parsing (2012-04-10) 1 commit
  (merged to 'next' on 2012-04-20 at c299a63)
 + fast-import: tighten parsing of datarefs

Will merge to 'master'.

* tr/xdiff-fast-hash (2012-04-09) 1 commit
 - xdiff: load full words in the inner loop of xdl_hash_record

Will merge to 'next'.

* jc/index-v4 (2012-04-04) 10 commits
  (merged to 'next' on 2012-04-19 at 5e4b029)
 + update-index: upgrade/downgrade on-disk index version
 + read-cache.c: write prefix-compressed names in the index
 + read-cache.c: read prefix-compressed names in index on-disk version v4
 + read-cache.c: move code to copy incore to ondisk cache to a helper function
 + read-cache.c: move code to copy ondisk to incore cache to a helper function
 + read-cache.c: report the header version we do not understand
 + read-cache.c: make create_from_disk() report number of bytes it consumed
 + read-cache.c: allow unaligned mapping of the index file
 + cache.h: hide on-disk index details
 + varint: make it available outside the context of pack
 (this branch is tangled with jc/split-blob.)

Trivially shrinks the on-disk size of the index file to save both I/O and
checksum overhead.  The "v4" format represented here may not be what we
eventually want to have, but the preparatory refactoring steps and the way
how backward compatibility is retained should give plenty to learn to GSoC
student-hopefuls.

The bottom one makes use of "varint" encoding from the stalled
jc/split-blob topic.

* mk/gitweb-diff-hl (2012-04-11) 8 commits
  (merged to 'next' on 2012-04-20 at 54179a2)
 + gitweb: Refinement highlightning in combined diffs
 + gitweb: Highlight interesting parts of diff
 + gitweb: Push formatting diff lines to print_diff_chunk()
 + gitweb: Use print_diff_chunk() for both side-by-side and inline diffs
 + gitweb: Extract print_sidebyside_diff_lines()
 + gitweb: Pass esc_html_hl_regions() options to esc_html()
 + gitweb: esc_html_hl_regions(): Don't create empty <span> elements
 + gitweb: Use descriptive names in esc_html_hl_regions()

Will merge to 'master'.

* it/fetch-pack-many-refs (2012-04-10) 4 commits
  (merged to 'next' on 2012-04-20 at e2126d7)
 + remote-curl: main test case for the OS command line overflow
 + fetch-pack: test cases for the new --stdin option
 + remote-curl: send the refs to fetch-pack on stdin
 + fetch-pack: new --stdin option to read refs from stdin

Will merge to 'master'.

* hv/submodule-recurse-push (2012-03-30) 3 commits
  (merged to 'next' on 2012-04-20 at fe99e3d)
 + push: teach --recurse-submodules the on-demand option
 + Refactor submodule push check to use string list instead of integer
 + Teach revision walking machinery to walk multiple times sequencially

Will merge to 'master'.

* th/difftool-diffall (2012-04-23) 8 commits
 - difftool: print list of valid tools with '--tool-help'
 - difftool: teach difftool to handle directory diffs
 - difftool: eliminate setup_environment function
 - difftool: stop appending '.exe' to git
 - difftool: remove explicit change of PATH
 - difftool: exit(0) when usage is printed
 - difftool: add '--no-gui' option
 - difftool: parse options using Getopt::Long

Rolls the two-directory-diff logic from diffall script (in contrib/) into
"git difftool" framework. 

Still being reworked.

* jc/run-hook-env-1 (2012-03-11) 1 commit
 - run_hook(): enhance the interface to pass arbitrary environment

Not urgent.

Updates run_hook() API to be much less specific to "commit".  It would
only be useful if people start doing more interesting things with hooks.

* mm/push-default-switch-warning (2012-04-23) 10 commits
 - fixup! push: introduce new push.default mode "simple"
 - fixup! t5528-push-default.sh: add helper functions
 - push: suggested updates to push configuration documentation
 - push: start warning upcoming default change for push.default
 - push: document the future default change for push.default (matching -> simple)
 - t5570: use explicit push refspec
 - push: introduce new push.default mode "simple"
 - t5528-push-default.sh: add helper functions
 - Undocument deprecated alias 'push.default=tracking'
 - Documentation: explain push.default option a bit more

Introduces 'simple' mode to be used as the push.default in a future
version of git.  I think the series is almost there modulo some minor
nits.

This topic should be renamed to "mm/push-simple-method" and the last
step to start warning made into a separate topic that build on it, but
the review comments need to be addressed in a reroll before that

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: What's cooking in git.git (Apr 2012, #09; Mon, 23)
  2012-04-23 22:04 What's cooking in git.git (Apr 2012, #09; Mon, 23) Junio C Hamano
@ 2012-04-24  9:14 ` Nelson Benitez Leon
  2012-04-24 18:57   ` Junio C Hamano
  2012-04-25  2:14 ` What's cooking (incremental) Junio C Hamano
  1 sibling, 1 reply; 13+ messages in thread
From: Nelson Benitez Leon @ 2012-04-24  9:14 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

On 04/24/2012 12:04 AM, Junio C Hamano wrote:
> 
> * nl/http-proxy-more (2012-04-19) 7 commits
>  - http: fix proxy password passing
>  - http: fix proxy authentication
>  - http: rename HTTP_REAUTH to HTTP_AUTH_RETRY
>  - http: Avoid limit of retrying request only twice
>  - http: handle proxy authentication failure (error 407)
>  - http: handle proxy proactive authentication
>  - http: try http_proxy env var when http.proxy config option is not set
> 

I'm a bit lost with the state of this, I read your last discussion and
you sent patches to support a proxy url without the protocol part, I assume
that's done and apart from that there is a bug in the LIBCURL_VERSION which
is an easy fix I could send a patch for.

> The code to talk to http proxies learn to use the same credential
> API used to talk to the final http destinations.  This still needs
> to peek into $ENV{HTTPS_PROXY}.

My original patch already peeked in HTTPS_PROXY env var, I post the
relevant part:
 
+	if (!curl_http_proxy) {
+		const char *env_proxy, *no_proxy;
+		char *env_proxy_var;
+		int read_http_proxy;
+		struct strbuf buf = STRBUF_INIT;
+		credential_from_url(&cre_url, url);
+		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
+		env_proxy_var = strbuf_detach(&buf, NULL);
+		env_proxy = getenv(env_proxy_var);

                ^^ HERE ^^

+		if (env_proxy) {
+			read_http_proxy = 1;
+			no_proxy = getenv("no_proxy");
+			if (no_proxy && (!strcmp("*", no_proxy) || strstr(no_proxy, cre_url.host)))
+				read_http_proxy = 0;
+			
+			if (read_http_proxy)
+				curl_http_proxy = xstrdup(env_proxy);
+		}
+		free(env_proxy_var);
+	}
 	if (curl_http_proxy) {

so are you referring to something else ?

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: What's cooking in git.git (Apr 2012, #09; Mon, 23)
  2012-04-24  9:14 ` Nelson Benitez Leon
@ 2012-04-24 18:57   ` Junio C Hamano
  2012-04-26 13:16     ` [PATCH] Try an uppercase version of $prot_proxy env var Nelson Benitez Leon
  0 siblings, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2012-04-24 18:57 UTC (permalink / raw)
  To: Nelson Benitez Leon; +Cc: git

Nelson Benitez Leon <nelsonjesus.benitez@seap.minhap.es> writes:

> My original patch already peeked in HTTPS_PROXY env var, I post the
> relevant part:
>  
> +	if (!curl_http_proxy) {
> +		const char *env_proxy, *no_proxy;
> +		char *env_proxy_var;
> +		int read_http_proxy;
> +		struct strbuf buf = STRBUF_INIT;
> +		credential_from_url(&cre_url, url);
> +		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
> +		env_proxy_var = strbuf_detach(&buf, NULL);
> +		env_proxy = getenv(env_proxy_var);
>
>                 ^^ HERE ^^

When cre_url.protocol is "https", this looks at $ENV{"https_proxy"}, no?

    http://curl.haxx.se/docs/manpage.html#ENVIRONMENT

> so are you referring to something else ?

There may be some other things that break people's settings but
regarding PROXY, the above was wnat I was referring to.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* What's cooking (incremental)
  2012-04-23 22:04 What's cooking in git.git (Apr 2012, #09; Mon, 23) Junio C Hamano
  2012-04-24  9:14 ` Nelson Benitez Leon
@ 2012-04-25  2:14 ` Junio C Hamano
  2012-04-25  9:06   ` Thomas Rast
  1 sibling, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2012-04-25  2:14 UTC (permalink / raw)
  To: git

I've merged the following topics to 'next':

  * tr/xdiff-fast-hash (2012-04-09) 1 commit
  * cb/daemon-test-race-fix (2012-04-19) 1 commit
  * ld/git-p4-tags-and-labels (2012-04-24) 6 commits
  * nh/empty-rebase (2012-04-24) 4 commits
  * pw/t5800-import-race-fix (2012-04-24) 1 commit
  * nd/i18n (2012-04-24) 10 commits

Some of them are scary and somewhat iffy (I had to apply a last minute
fix and rebuild 'next' to one of them), so please test them throughly to
help us avoid breakages on 'master'.

On the 'master' front, these have graduated:

  * cc/fix-missing-va-end-in-revert (2012-04-21) 1 commit
  * fc/completion-tests (2012-04-21) 7 commits
  * hv/submodule-recurse-push (2012-03-30) 3 commits
  * it/fetch-pack-many-refs (2012-04-10) 4 commits
  * mh/ref-api (2012-04-10) 13 commits
  * mk/gitweb-diff-hl (2012-04-11) 8 commits
  * pw/fast-import-dataref-parsing (2012-04-10) 1 commit
  * zj/upstream-error-message (2012-04-15) 5 commits

Thanks.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: What's cooking (incremental)
  2012-04-25  2:14 ` What's cooking (incremental) Junio C Hamano
@ 2012-04-25  9:06   ` Thomas Rast
  2012-04-25 15:17     ` Junio C Hamano
  0 siblings, 1 reply; 13+ messages in thread
From: Thomas Rast @ 2012-04-25  9:06 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Junio C Hamano <gitster@pobox.com> writes:

> I've merged the following topics to 'next':
>
>   * tr/xdiff-fast-hash (2012-04-09) 1 commit
>   * cb/daemon-test-race-fix (2012-04-19) 1 commit
>   * ld/git-p4-tags-and-labels (2012-04-24) 6 commits
>   * nh/empty-rebase (2012-04-24) 4 commits
>   * pw/t5800-import-race-fix (2012-04-24) 1 commit
>   * nd/i18n (2012-04-24) 10 commits
>
> Some of them are scary and somewhat iffy (I had to apply a last minute
> fix and rebuild 'next' to one of them), so please test them throughly to
> help us avoid breakages on 'master'.

Umm, which one?

As the author of the first, I'm kind of curious whether I need to
scramble to fix (or at least further test) something :-)

-- 
Thomas Rast
trast@{inf,student}.ethz.ch

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: What's cooking (incremental)
  2012-04-25  9:06   ` Thomas Rast
@ 2012-04-25 15:17     ` Junio C Hamano
  0 siblings, 0 replies; 13+ messages in thread
From: Junio C Hamano @ 2012-04-25 15:17 UTC (permalink / raw)
  To: Thomas Rast; +Cc: git

Thomas Rast <trast@student.ethz.ch> writes:

> Junio C Hamano <gitster@pobox.com> writes:
>
>> I've merged the following topics to 'next':
>>
>>   * tr/xdiff-fast-hash (2012-04-09) 1 commit
>>   * cb/daemon-test-race-fix (2012-04-19) 1 commit
>>   * ld/git-p4-tags-and-labels (2012-04-24) 6 commits
>>   * nh/empty-rebase (2012-04-24) 4 commits
>>   * pw/t5800-import-race-fix (2012-04-24) 1 commit
>>   * nd/i18n (2012-04-24) 10 commits
>>
>> Some of them are scary and somewhat iffy (I had to apply a last minute
>> fix and rebuild 'next' to one of them), so please test them throughly to
>> help us avoid breakages on 'master'.
>
> Umm, which one?
>
> As the author of the first, I'm kind of curious whether I need to
> scramble to fix (or at least further test) something :-)

Authors do not need to "scramble"; they are trusted to have done as good
a job as they can possibly do before submitting anyway.

It is sufficient to just use the resulting Git in your daily work, but
paying extra attention to notice if there is any behaviour that feels
strange.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-26 13:16     ` [PATCH] Try an uppercase version of $prot_proxy env var Nelson Benitez Leon
@ 2012-04-26 13:08       ` Jeff King
  2012-04-26 15:18         ` Junio C Hamano
  2012-04-26 14:04       ` Junio C Hamano
  1 sibling, 1 reply; 13+ messages in thread
From: Jeff King @ 2012-04-26 13:08 UTC (permalink / raw)
  To: Nelson Benitez Leon; +Cc: Junio C Hamano, git

On Thu, Apr 26, 2012 at 03:16:51PM +0200, Nelson Benitez Leon wrote:

> Try an uppercase version of $prot_proxy env var when thelowercase version 
> is not found.
> [...]
> diff --git a/http.c b/http.c
> index 6a98195..bf9c20e 100644
> --- a/http.c
> +++ b/http.c
> @@ -329,6 +329,13 @@ static CURL *get_curl_handle(const char *url)
>  		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
>  		env_proxy_var = strbuf_detach(&buf, NULL);
>  		env_proxy = getenv(env_proxy_var);
> +		if (!env_proxy) {
> +			char *p;
> +			for (p = env_proxy_var; *p; p++) {
> +				*p = toupper(*p);
> +			}
> +			env_proxy = getenv(env_proxy_var);
> +		}

Don't we explicitly not want to do this when the protocol is http? Curl
doesn't respect HTTP_PROXY.

-Peff

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-24 18:57   ` Junio C Hamano
@ 2012-04-26 13:16     ` Nelson Benitez Leon
  2012-04-26 13:08       ` Jeff King
  2012-04-26 14:04       ` Junio C Hamano
  0 siblings, 2 replies; 13+ messages in thread
From: Nelson Benitez Leon @ 2012-04-26 13:16 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git


Try an uppercase version of $prot_proxy env var when thelowercase version 
is not found.

Signed-off-by: Nelson Benitez Leon <nbenitezl@gmail.com>
---
This is for pu branch where http-proxy-more is located.

 http.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/http.c b/http.c
index 6a98195..bf9c20e 100644
--- a/http.c
+++ b/http.c
@@ -329,6 +329,13 @@ static CURL *get_curl_handle(const char *url)
 		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
 		env_proxy_var = strbuf_detach(&buf, NULL);
 		env_proxy = getenv(env_proxy_var);
+		if (!env_proxy) {
+			char *p;
+			for (p = env_proxy_var; *p; p++) {
+				*p = toupper(*p);
+			}
+			env_proxy = getenv(env_proxy_var);
+		}
 		if (env_proxy) {
 			read_http_proxy = 1;
 			no_proxy = getenv("no_proxy");
-- 
1.7.7.6

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-26 13:16     ` [PATCH] Try an uppercase version of $prot_proxy env var Nelson Benitez Leon
  2012-04-26 13:08       ` Jeff King
@ 2012-04-26 14:04       ` Junio C Hamano
  1 sibling, 0 replies; 13+ messages in thread
From: Junio C Hamano @ 2012-04-26 14:04 UTC (permalink / raw)
  To: Nelson Benitez Leon; +Cc: git

Nelson Benitez Leon <nelsonjesus.benitez@seap.minhap.es> writes:

> Try an uppercase version of $prot_proxy env var when thelowercase version 
> is not found.

Yeah, something like that, but you should be careful not to try the
uppercase HTTP_PROXY which cURL explicitly ignores on purpose.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-26 13:08       ` Jeff King
@ 2012-04-26 15:18         ` Junio C Hamano
  2012-04-26 16:29           ` Nelson Benitez Leon
  2012-04-27  8:12           ` Jeff King
  0 siblings, 2 replies; 13+ messages in thread
From: Junio C Hamano @ 2012-04-26 15:18 UTC (permalink / raw)
  To: Nelson Benitez Leon, Jeff King; +Cc: git

Jeff King <peff@peff.net> writes:

> Don't we explicitly not want to do this when the protocol is http? Curl
> doesn't respect HTTP_PROXY.

Yes.  Here is what I'll queue.

-- >8 --
From: Nelson Benitez Leon <nelsonjesus.benitez@seap.minhap.es>
Date: Thu, 26 Apr 2012 15:16:51 +0200
Subject: [PATCH] http: try an uppercase version of $proto_proxy

Fall back to an uppercase version of $prot_proxy environment variable
when the lowercase version is not found (but do not do that for http).

Signed-off-by: Nelson Benitez Leon <nbenitezl@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 http.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/http.c b/http.c
index ad2dc36..262778b 100644
--- a/http.c
+++ b/http.c
@@ -320,6 +320,12 @@ static CURL *get_curl_handle(const char *url)
 		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
 		env_proxy_var = strbuf_detach(&buf, NULL);
 		env_proxy = getenv(env_proxy_var);
+		if (!env_proxy && strcmp("http_proxy", env_proxy_var)) {
+			char *p;
+			for (p = env_proxy_var; *p; p++)
+				*p = toupper(*p);
+			env_proxy = getenv(env_proxy_var);
+		}
 		if (env_proxy) {
 			read_http_proxy = 1;
 			no_proxy = getenv("no_proxy");
-- 
1.7.10.475.g8b959

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-26 15:18         ` Junio C Hamano
@ 2012-04-26 16:29           ` Nelson Benitez Leon
  2012-04-27  8:12           ` Jeff King
  1 sibling, 0 replies; 13+ messages in thread
From: Nelson Benitez Leon @ 2012-04-26 16:29 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Jeff King, git

On 04/26/2012 05:18 PM, Junio C Hamano wrote:
> Jeff King <peff@peff.net> writes:
> 
>> Don't we explicitly not want to do this when the protocol is http? Curl
>> doesn't respect HTTP_PROXY.
> 
> Yes.  Here is what I'll queue.

Fine. You beat me on time, anyway I send you my version just for the sake
of having done it.

>From 66c5e59f486088d12b48a2e624a98242e7ebce46 Mon Sep 17 00:00:00 2001
From: Nelson Benitez Leon <nbenitezl@gmail.com>
Date: Thu, 26 Apr 2012 14:44:03 +0200
Subject: [PATCH] http: try an uppercase version of $prot_proxy env var

If the lowercase version of $prot_proxy is not found
then try the uppercase one, excluding HTTP_PROXY case
as it is ignored by cURL.

Signed-off-by: Nelson Benitez Leon <nbenitezl@gmail.com>
---
 http.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/http.c b/http.c
index 6a98195..0ca5bba 100644
--- a/http.c
+++ b/http.c
@@ -329,6 +329,13 @@ static CURL *get_curl_handle(const char *url)
 		strbuf_addf(&buf, "%s_proxy", cre_url.protocol);
 		env_proxy_var = strbuf_detach(&buf, NULL);
 		env_proxy = getenv(env_proxy_var);
+		if (!env_proxy && strcmp("http", cre_url.protocol)) { /* skip HTTP_PROXY as cURL ignores it */
+			char *p;
+			for (p = env_proxy_var; *p; p++) {
+				*p = toupper(*p);
+			}
+			env_proxy = getenv(env_proxy_var);
+		}
 		if (env_proxy) {
 			read_http_proxy = 1;
 			no_proxy = getenv("no_proxy");
-- 
1.7.7.6

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-26 15:18         ` Junio C Hamano
  2012-04-26 16:29           ` Nelson Benitez Leon
@ 2012-04-27  8:12           ` Jeff King
  2012-04-27 15:03             ` Junio C Hamano
  1 sibling, 1 reply; 13+ messages in thread
From: Jeff King @ 2012-04-27  8:12 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Nelson Benitez Leon, git

On Thu, Apr 26, 2012 at 08:18:58AM -0700, Junio C Hamano wrote:

> Jeff King <peff@peff.net> writes:
> 
> > Don't we explicitly not want to do this when the protocol is http? Curl
> > doesn't respect HTTP_PROXY.
> 
> Yes.  Here is what I'll queue.

I had raised several other points in my (admittedly belated) review[1].
So I was kind of expecting a re-roll rather than a patch on top. It was
mostly readability issues, but there is also a potential segfault when
the url has no protocol. I'm not sure it can happen in normal git use,
but:

  git remote-https url-without-protocol

may dereference null (it's via snprintf, so on my system glibc magically
rewrites this as "(null)_proxy", but other systems will segfault).

-Peff

[1] http://article.gmane.org/gmane.comp.version-control.git/195443
    http://article.gmane.org/gmane.comp.version-control.git/195445

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH] Try an uppercase version of $prot_proxy env var
  2012-04-27  8:12           ` Jeff King
@ 2012-04-27 15:03             ` Junio C Hamano
  0 siblings, 0 replies; 13+ messages in thread
From: Junio C Hamano @ 2012-04-27 15:03 UTC (permalink / raw)
  To: Jeff King; +Cc: Nelson Benitez Leon, git

Jeff King <peff@peff.net> writes:

> On Thu, Apr 26, 2012 at 08:18:58AM -0700, Junio C Hamano wrote:
>
>> Jeff King <peff@peff.net> writes:
>> 
>> > Don't we explicitly not want to do this when the protocol is http? Curl
>> > doesn't respect HTTP_PROXY.
>> 
>> Yes.  Here is what I'll queue.
>
> I had raised several other points in my (admittedly belated) review[1].
> So I was kind of expecting a re-roll rather than a patch on top. It was
> mostly readability issues, but there is also a potential segfault when
> the url has no protocol. I'm not sure it can happen in normal git use,
> but:
>
>   git remote-https url-without-protocol
>
> may dereference null (it's via snprintf, so on my system glibc magically
> rewrites this as "(null)_proxy", but other systems will segfault).
>
> -Peff
>
> [1] http://article.gmane.org/gmane.comp.version-control.git/195443
>     http://article.gmane.org/gmane.comp.version-control.git/195445

Indeed.  Thanks for reminding me not to advance this beyond 'pu'.

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2012-04-27 15:03 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-23 22:04 What's cooking in git.git (Apr 2012, #09; Mon, 23) Junio C Hamano
2012-04-24  9:14 ` Nelson Benitez Leon
2012-04-24 18:57   ` Junio C Hamano
2012-04-26 13:16     ` [PATCH] Try an uppercase version of $prot_proxy env var Nelson Benitez Leon
2012-04-26 13:08       ` Jeff King
2012-04-26 15:18         ` Junio C Hamano
2012-04-26 16:29           ` Nelson Benitez Leon
2012-04-27  8:12           ` Jeff King
2012-04-27 15:03             ` Junio C Hamano
2012-04-26 14:04       ` Junio C Hamano
2012-04-25  2:14 ` What's cooking (incremental) Junio C Hamano
2012-04-25  9:06   ` Thomas Rast
2012-04-25 15:17     ` Junio C Hamano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.