Skip to content

Commit 9320262

Browse files
jeffhostetlerdscho
authored andcommitted
Merge first wave of gvfs-helper feature
Includes commits from these pull requests: #191 #205 #206 #207 #208 #215 #220 #221 Signed-off-by: Derrick Stolee <[email protected]>
2 parents 2cade4b + 017157a commit 9320262

29 files changed

+7215
-5
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@
7474
/git-gc
7575
/git-get-tar-commit-id
7676
/git-grep
77+
/git-gvfs-helper
7778
/git-hash-object
7879
/git-help
7980
/git-hook

Documentation/config.adoc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -448,6 +448,8 @@ include::config/gui.adoc[]
448448

449449
include::config/guitool.adoc[]
450450

451+
include::config/gvfs.adoc[]
452+
451453
include::config/help.adoc[]
452454

453455
include::config/http.adoc[]

Documentation/config/core.adoc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -793,6 +793,9 @@ core.gvfs::
793793
flag just blocks them from occurring at all.
794794
--
795795

796+
core.useGvfsHelper::
797+
TODO
798+
796799
core.sparseCheckout::
797800
Enable "sparse checkout" feature. See linkgit:git-sparse-checkout[1]
798801
for more information.

Documentation/config/gvfs.adoc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
gvfs.cache-server::
2+
TODO
3+
4+
gvfs.sharedcache::
5+
TODO

Documentation/lint-manpages.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ check_missing_docs () (
2727
git-init-db) continue;;
2828
git-remote-*) continue;;
2929
git-stage) continue;;
30+
git-gvfs-helper) continue;;
3031
git-legacy-*) continue;;
3132
git-?*--?* ) continue ;;
3233
esac

Makefile

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1047,6 +1047,7 @@ LIB_OBJS += gpg-interface.o
10471047
LIB_OBJS += graph.o
10481048
LIB_OBJS += grep.o
10491049
LIB_OBJS += gvfs.o
1050+
LIB_OBJS += gvfs-helper-client.o
10501051
LIB_OBJS += hash-lookup.o
10511052
LIB_OBJS += hashmap.o
10521053
LIB_OBJS += help.o
@@ -1697,6 +1698,9 @@ endif
16971698
endif
16981699
BASIC_CFLAGS += $(CURL_CFLAGS)
16991700

1701+
PROGRAM_OBJS += gvfs-helper.o
1702+
TEST_PROGRAMS_NEED_X += test-gvfs-protocol
1703+
17001704
REMOTE_CURL_PRIMARY = git-remote-http$X
17011705
REMOTE_CURL_ALIASES = git-remote-https$X git-remote-ftp$X git-remote-ftps$X
17021706
REMOTE_CURL_NAMES = $(REMOTE_CURL_PRIMARY) $(REMOTE_CURL_ALIASES)
@@ -2970,6 +2974,10 @@ scalar$X: scalar.o GIT-LDFLAGS $(GITLIBS)
29702974
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) \
29712975
$(filter %.o,$^) $(LIBS)
29722976

2977+
git-gvfs-helper$X: gvfs-helper.o http.o GIT-LDFLAGS $(GITLIBS) $(LAZYLOAD_LIBCURL_OBJ)
2978+
$(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) \
2979+
$(CURL_LIBCURL) $(EXPAT_LIBEXPAT) $(LIBS)
2980+
29732981
$(LIB_FILE): $(LIB_OBJS)
29742982
$(QUIET_AR)$(RM) $@ && $(AR) $(ARFLAGS) $@ $^
29752983

bin-wrappers/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,5 @@
66
/git-upload-pack
77
/scalar
88
/test-fake-ssh
9+
/test-gvfs-protocol
910
/test-tool

builtin/index-pack.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -892,7 +892,7 @@ static void sha1_object(const void *data, struct object_entry *obj_entry,
892892
read_lock();
893893
collision_test_needed =
894894
repo_has_object_file_with_flags(the_repository, oid,
895-
OBJECT_INFO_QUICK);
895+
OBJECT_INFO_FOR_PREFETCH);
896896
read_unlock();
897897
}
898898

config.c

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
#include "wildmatch.h"
4545
#include "ws.h"
4646
#include "write-or-die.h"
47+
#include "transport.h"
4748

4849
struct config_source {
4950
struct config_source *prev;
@@ -1628,6 +1629,11 @@ int git_default_core_config(const char *var, const char *value,
16281629
return 0;
16291630
}
16301631

1632+
if (!strcmp(var, "core.usegvfshelper")) {
1633+
core_use_gvfs_helper = git_config_bool(var, value);
1634+
return 0;
1635+
}
1636+
16311637
if (!strcmp(var, "core.sparsecheckout")) {
16321638
/* virtual file system relies on the sparse checkout logic so force it on */
16331639
if (core_virtualfilesystem)
@@ -1779,6 +1785,37 @@ static int git_default_mailmap_config(const char *var, const char *value)
17791785
return 0;
17801786
}
17811787

1788+
static int git_default_gvfs_config(const char *var, const char *value)
1789+
{
1790+
if (!strcmp(var, "gvfs.cache-server")) {
1791+
char *v2 = NULL;
1792+
1793+
if (!git_config_string(&v2, var, value) && v2 && *v2) {
1794+
free(gvfs_cache_server_url);
1795+
gvfs_cache_server_url = transport_anonymize_url(v2);
1796+
}
1797+
free(v2);
1798+
return 0;
1799+
}
1800+
1801+
if (!strcmp(var, "gvfs.sharedcache") && value && *value) {
1802+
strbuf_setlen(&gvfs_shared_cache_pathname, 0);
1803+
strbuf_addstr(&gvfs_shared_cache_pathname, value);
1804+
if (strbuf_normalize_path(&gvfs_shared_cache_pathname) < 0) {
1805+
/*
1806+
* Pretend it wasn't set. This will cause us to
1807+
* fallback to ".git/objects" effectively.
1808+
*/
1809+
strbuf_release(&gvfs_shared_cache_pathname);
1810+
return 0;
1811+
}
1812+
strbuf_trim_trailing_dir_sep(&gvfs_shared_cache_pathname);
1813+
return 0;
1814+
}
1815+
1816+
return 0;
1817+
}
1818+
17821819
static int git_default_attr_config(const char *var, const char *value)
17831820
{
17841821
if (!strcmp(var, "attr.tree")) {
@@ -1846,6 +1883,9 @@ int git_default_config(const char *var, const char *value,
18461883
if (starts_with(var, "sparse."))
18471884
return git_default_sparse_config(var, value);
18481885

1886+
if (starts_with(var, "gvfs."))
1887+
return git_default_gvfs_config(var, value);
1888+
18491889
/* Add other config variables here and to Documentation/config.txt. */
18501890
return 0;
18511891
}

contrib/buildsystems/CMakeLists.txt

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -635,7 +635,7 @@ if(NOT CURL_FOUND)
635635
add_compile_definitions(NO_CURL)
636636
message(WARNING "git-http-push and git-http-fetch will not be built")
637637
else()
638-
list(APPEND PROGRAMS_BUILT git-http-fetch git-http-push git-imap-send git-remote-http)
638+
list(APPEND PROGRAMS_BUILT git-http-fetch git-http-push git-imap-send git-remote-http git-gvfs-helper)
639639
if(CURL_VERSION_STRING VERSION_GREATER_EQUAL 7.34.0)
640640
add_compile_definitions(USE_CURL_FOR_IMAP_SEND)
641641
endif()
@@ -823,6 +823,9 @@ if(CURL_FOUND)
823823
add_executable(git-http-push ${CMAKE_SOURCE_DIR}/http-push.c)
824824
target_link_libraries(git-http-push http_obj common-main ${CURL_LIBRARIES} ${EXPAT_LIBRARIES})
825825
endif()
826+
827+
add_executable(git-gvfs-helper ${CMAKE_SOURCE_DIR}/gvfs-helper.c)
828+
target_link_libraries(git-gvfs-helper http_obj common-main ${CURL_LIBRARIES} )
826829
endif()
827830

828831
parse_makefile_for_executables(git_builtin_extra "BUILT_INS")
@@ -1116,6 +1119,20 @@ set(wrapper_scripts
11161119
set(wrapper_test_scripts
11171120
test-fake-ssh test-tool)
11181121

1122+
if(CURL_FOUND)
1123+
list(APPEND wrapper_test_scripts test-gvfs-protocol)
1124+
1125+
add_executable(test-gvfs-protocol ${CMAKE_SOURCE_DIR}/t/helper/test-gvfs-protocol.c)
1126+
target_link_libraries(test-gvfs-protocol common-main)
1127+
1128+
if(MSVC)
1129+
set_target_properties(test-gvfs-protocol
1130+
PROPERTIES RUNTIME_OUTPUT_DIRECTORY_DEBUG ${CMAKE_BINARY_DIR}/t/helper)
1131+
set_target_properties(test-gvfs-protocol
1132+
PROPERTIES RUNTIME_OUTPUT_DIRECTORY_RELEASE ${CMAKE_BINARY_DIR}/t/helper)
1133+
endif()
1134+
endif()
1135+
11191136

11201137
foreach(script ${wrapper_scripts})
11211138
file(STRINGS ${CMAKE_SOURCE_DIR}/bin-wrappers/wrap-for-bin.sh content NEWLINE_CONSUME)

credential.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -456,6 +456,8 @@ static int run_credential_helper(struct credential *c,
456456
else
457457
helper.no_stdout = 1;
458458

459+
helper.trace2_child_class = "cred";
460+
459461
if (start_command(&helper) < 0)
460462
return -1;
461463

environment.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,9 @@ int protect_hfs = PROTECT_HFS_DEFAULT;
9999
#define PROTECT_NTFS_DEFAULT 1
100100
#endif
101101
int protect_ntfs = PROTECT_NTFS_DEFAULT;
102+
int core_use_gvfs_helper;
103+
char *gvfs_cache_server_url;
104+
struct strbuf gvfs_shared_cache_pathname = STRBUF_INIT;
102105

103106
/*
104107
* The character that begins a commented line in user-editable file

environment.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,9 @@ extern int core_gvfs;
175175
extern int precomposed_unicode;
176176
extern int protect_hfs;
177177
extern int protect_ntfs;
178+
extern int core_use_gvfs_helper;
179+
extern char *gvfs_cache_server_url;
180+
extern struct strbuf gvfs_shared_cache_pathname;
178181

179182
extern int core_apply_sparse_checkout;
180183
extern int core_sparse_checkout_cone;

0 commit comments

Comments
 (0)