2005-06-06 08:59:54 +04:00
|
|
|
#include "cache.h"
|
2006-04-02 16:44:09 +04:00
|
|
|
#include "blob.h"
|
2018-05-16 02:42:15 +03:00
|
|
|
#include "object-store.h"
|
2009-01-10 15:07:50 +03:00
|
|
|
#include "dir.h"
|
2011-05-13 01:31:08 +04:00
|
|
|
#include "streaming.h"
|
2017-03-15 00:46:40 +03:00
|
|
|
#include "submodule.h"
|
2017-08-20 18:47:20 +03:00
|
|
|
#include "progress.h"
|
2017-09-22 19:35:40 +03:00
|
|
|
#include "fsmonitor.h"
|
2021-03-23 17:19:32 +03:00
|
|
|
#include "entry.h"
|
2005-06-06 08:59:54 +04:00
|
|
|
|
2009-02-09 23:54:08 +03:00
|
|
|
static void create_directories(const char *path, int path_len,
|
|
|
|
const struct checkout *state)
|
2005-06-06 08:59:54 +04:00
|
|
|
{
|
2016-02-23 01:44:28 +03:00
|
|
|
char *buf = xmallocz(path_len);
|
2009-02-09 23:54:08 +03:00
|
|
|
int len = 0;
|
|
|
|
|
|
|
|
while (len < path_len) {
|
|
|
|
do {
|
|
|
|
buf[len] = path[len];
|
|
|
|
len++;
|
|
|
|
} while (len < path_len && path[len] != '/');
|
|
|
|
if (len >= path_len)
|
|
|
|
break;
|
2005-06-06 08:59:54 +04:00
|
|
|
buf[len] = 0;
|
Do not expect unlink(2) to fail on a directory.
When "git checkout-index" checks out path A/B/C, it makes sure A
and A/B are truly directories; if there is a regular file or
symlink at A, we prefer to remove it.
We used to do this by catching an error return from mkdir(2),
and on EEXIST did unlink(2), and when it succeeded, tried
another mkdir(2).
Thomas Glanzmann found out the above does not work on Solaris
for a root user, as unlink(2) was so old fashioned there that it
allowed to unlink a directory.
As pointed out, this still doesn't guarantee that git won't call
"unlink()" on a directory (race conditions etc), but that's
fundamentally true (there is no "funlink()" like there is
"fstat()"), and besides, that is in no way git-specific (ie it's
true of any application that gets run as root).
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-18 09:58:28 +04:00
|
|
|
|
2009-01-18 18:14:52 +03:00
|
|
|
/*
|
|
|
|
* For 'checkout-index --prefix=<dir>', <dir> is
|
|
|
|
* allowed to be a symlink to an existing directory,
|
|
|
|
* and we set 'state->base_dir_len' below, such that
|
|
|
|
* we test the path components of the prefix with the
|
|
|
|
* stat() function instead of the lstat() function.
|
|
|
|
*/
|
2009-02-09 23:54:06 +03:00
|
|
|
if (has_dirs_only_path(buf, len, state->base_dir_len))
|
Do not expect unlink(2) to fail on a directory.
When "git checkout-index" checks out path A/B/C, it makes sure A
and A/B are truly directories; if there is a regular file or
symlink at A, we prefer to remove it.
We used to do this by catching an error return from mkdir(2),
and on EEXIST did unlink(2), and when it succeeded, tried
another mkdir(2).
Thomas Glanzmann found out the above does not work on Solaris
for a root user, as unlink(2) was so old fashioned there that it
allowed to unlink a directory.
As pointed out, this still doesn't guarantee that git won't call
"unlink()" on a directory (race conditions etc), but that's
fundamentally true (there is no "funlink()" like there is
"fstat()"), and besides, that is in no way git-specific (ie it's
true of any application that gets run as root).
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-18 09:58:28 +04:00
|
|
|
continue; /* ok, it is already a directory. */
|
|
|
|
|
|
|
|
/*
|
2009-01-18 18:14:52 +03:00
|
|
|
* If this mkdir() would fail, it could be that there
|
|
|
|
* is already a symlink or something else exists
|
|
|
|
* there, therefore we then try to unlink it and try
|
|
|
|
* one more time to create the directory.
|
Do not expect unlink(2) to fail on a directory.
When "git checkout-index" checks out path A/B/C, it makes sure A
and A/B are truly directories; if there is a regular file or
symlink at A, we prefer to remove it.
We used to do this by catching an error return from mkdir(2),
and on EEXIST did unlink(2), and when it succeeded, tried
another mkdir(2).
Thomas Glanzmann found out the above does not work on Solaris
for a root user, as unlink(2) was so old fashioned there that it
allowed to unlink a directory.
As pointed out, this still doesn't guarantee that git won't call
"unlink()" on a directory (race conditions etc), but that's
fundamentally true (there is no "funlink()" like there is
"fstat()"), and besides, that is in no way git-specific (ie it's
true of any application that gets run as root).
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-18 09:58:28 +04:00
|
|
|
*/
|
2005-07-06 12:21:46 +04:00
|
|
|
if (mkdir(buf, 0777)) {
|
Do not expect unlink(2) to fail on a directory.
When "git checkout-index" checks out path A/B/C, it makes sure A
and A/B are truly directories; if there is a regular file or
symlink at A, we prefer to remove it.
We used to do this by catching an error return from mkdir(2),
and on EEXIST did unlink(2), and when it succeeded, tried
another mkdir(2).
Thomas Glanzmann found out the above does not work on Solaris
for a root user, as unlink(2) was so old fashioned there that it
allowed to unlink a directory.
As pointed out, this still doesn't guarantee that git won't call
"unlink()" on a directory (race conditions etc), but that's
fundamentally true (there is no "funlink()" like there is
"fstat()"), and besides, that is in no way git-specific (ie it's
true of any application that gets run as root).
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-18 09:58:28 +04:00
|
|
|
if (errno == EEXIST && state->force &&
|
2009-04-30 01:22:56 +04:00
|
|
|
!unlink_or_warn(buf) && !mkdir(buf, 0777))
|
Do not expect unlink(2) to fail on a directory.
When "git checkout-index" checks out path A/B/C, it makes sure A
and A/B are truly directories; if there is a regular file or
symlink at A, we prefer to remove it.
We used to do this by catching an error return from mkdir(2),
and on EEXIST did unlink(2), and when it succeeded, tried
another mkdir(2).
Thomas Glanzmann found out the above does not work on Solaris
for a root user, as unlink(2) was so old fashioned there that it
allowed to unlink a directory.
As pointed out, this still doesn't guarantee that git won't call
"unlink()" on a directory (race conditions etc), but that's
fundamentally true (there is no "funlink()" like there is
"fstat()"), and besides, that is in no way git-specific (ie it's
true of any application that gets run as root).
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-07-18 09:58:28 +04:00
|
|
|
continue;
|
2009-06-27 19:58:47 +04:00
|
|
|
die_errno("cannot create directory at '%s'", buf);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
}
|
|
|
|
free(buf);
|
|
|
|
}
|
|
|
|
|
2014-03-13 13:19:08 +04:00
|
|
|
static void remove_subtree(struct strbuf *path)
|
2005-06-06 08:59:54 +04:00
|
|
|
{
|
2014-03-13 13:19:08 +04:00
|
|
|
DIR *dir = opendir(path->buf);
|
2005-06-06 08:59:54 +04:00
|
|
|
struct dirent *de;
|
2014-03-13 13:19:08 +04:00
|
|
|
int origlen = path->len;
|
2007-06-07 11:04:01 +04:00
|
|
|
|
2005-06-06 08:59:54 +04:00
|
|
|
if (!dir)
|
2014-03-13 13:19:08 +04:00
|
|
|
die_errno("cannot opendir '%s'", path->buf);
|
2005-06-06 08:59:54 +04:00
|
|
|
while ((de = readdir(dir)) != NULL) {
|
|
|
|
struct stat st;
|
2014-03-13 13:19:08 +04:00
|
|
|
|
2009-01-10 15:07:50 +03:00
|
|
|
if (is_dot_or_dotdot(de->d_name))
|
2005-06-06 08:59:54 +04:00
|
|
|
continue;
|
2014-03-13 13:19:08 +04:00
|
|
|
|
|
|
|
strbuf_addch(path, '/');
|
|
|
|
strbuf_addstr(path, de->d_name);
|
|
|
|
if (lstat(path->buf, &st))
|
|
|
|
die_errno("cannot lstat '%s'", path->buf);
|
2005-06-06 08:59:54 +04:00
|
|
|
if (S_ISDIR(st.st_mode))
|
2014-03-13 13:19:08 +04:00
|
|
|
remove_subtree(path);
|
|
|
|
else if (unlink(path->buf))
|
|
|
|
die_errno("cannot unlink '%s'", path->buf);
|
|
|
|
strbuf_setlen(path, origlen);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
closedir(dir);
|
2014-03-13 13:19:08 +04:00
|
|
|
if (rmdir(path->buf))
|
|
|
|
die_errno("cannot rmdir '%s'", path->buf);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
|
2005-07-14 20:58:45 +04:00
|
|
|
static int create_file(const char *path, unsigned int mode)
|
2005-06-06 08:59:54 +04:00
|
|
|
{
|
|
|
|
mode = (mode & 0100) ? 0777 : 0666;
|
2006-01-05 11:58:06 +03:00
|
|
|
return open(path, O_WRONLY | O_CREAT | O_EXCL, mode);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:33 +03:00
|
|
|
void *read_blob_entry(const struct cache_entry *ce, unsigned long *size)
|
2007-04-13 20:26:04 +04:00
|
|
|
{
|
|
|
|
enum object_type type;
|
sha1_file: convert read_sha1_file to struct object_id
Convert read_sha1_file to take a pointer to struct object_id and rename
it read_object_file. Do the same for read_sha1_file_extended.
Convert one use in grep.c to use the new function without any other code
change, since the pointer being passed is a void pointer that is already
initialized with a pointer to struct object_id. Update the declaration
and definitions of the modified functions, and apply the following
semantic patch to convert the remaining callers:
@@
expression E1, E2, E3;
@@
- read_sha1_file(E1.hash, E2, E3)
+ read_object_file(&E1, E2, E3)
@@
expression E1, E2, E3;
@@
- read_sha1_file(E1->hash, E2, E3)
+ read_object_file(E1, E2, E3)
@@
expression E1, E2, E3, E4;
@@
- read_sha1_file_extended(E1.hash, E2, E3, E4)
+ read_object_file_extended(&E1, E2, E3, E4)
@@
expression E1, E2, E3, E4;
@@
- read_sha1_file_extended(E1->hash, E2, E3, E4)
+ read_object_file_extended(E1, E2, E3, E4)
Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-03-12 05:27:53 +03:00
|
|
|
void *blob_data = read_object_file(&ce->oid, &type, size);
|
2007-04-13 20:26:04 +04:00
|
|
|
|
2018-02-14 21:59:41 +03:00
|
|
|
if (blob_data) {
|
2007-04-13 20:26:04 +04:00
|
|
|
if (type == OBJ_BLOB)
|
2018-02-14 21:59:41 +03:00
|
|
|
return blob_data;
|
|
|
|
free(blob_data);
|
2007-04-13 20:26:04 +04:00
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
Convert "struct cache_entry *" to "const ..." wherever possible
I attempted to make index_state->cache[] a "const struct cache_entry **"
to find out how existing entries in index are modified and where. The
question I have is what do we do if we really need to keep track of on-disk
changes in the index. The result is
- diff-lib.c: setting CE_UPTODATE
- name-hash.c: setting CE_HASHED
- preload-index.c, read-cache.c, unpack-trees.c and
builtin/update-index: obvious
- entry.c: write_entry() may refresh the checked out entry via
fill_stat_cache_info(). This causes "non-const struct cache_entry
*" in builtin/apply.c, builtin/checkout-index.c and
builtin/checkout.c
- builtin/ls-files.c: --with-tree changes stagemask and may set
CE_UPDATE
Of these, write_entry() and its call sites are probably most
interesting because it modifies on-disk info. But this is stat info
and can be retrieved via refresh, at least for porcelain
commands. Other just uses ce_flags for local purposes.
So, keeping track of "dirty" entries is just a matter of setting a
flag in index modification functions exposed by read-cache.c. Except
unpack-trees, the rest of the code base does not do anything funny
behind read-cache's back.
The actual patch is less valueable than the summary above. But if
anyone wants to re-identify the above sites. Applying this patch, then
this:
diff --git a/cache.h b/cache.h
index 430d021..1692891 100644
--- a/cache.h
+++ b/cache.h
@@ -267,7 +267,7 @@ static inline unsigned int canon_mode(unsigned int mode)
#define cache_entry_size(len) (offsetof(struct cache_entry,name) + (len) + 1)
struct index_state {
- struct cache_entry **cache;
+ const struct cache_entry **cache;
unsigned int version;
unsigned int cache_nr, cache_alloc, cache_changed;
struct string_list *resolve_undo;
will help quickly identify them without bogus warnings.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2013-07-09 19:29:00 +04:00
|
|
|
static int open_output_fd(char *path, const struct cache_entry *ce, int to_tempfile)
|
2011-05-13 08:36:42 +04:00
|
|
|
{
|
|
|
|
int symlink = (ce->ce_mode & S_IFMT) != S_IFREG;
|
|
|
|
if (to_tempfile) {
|
2015-09-25 00:06:53 +03:00
|
|
|
xsnprintf(path, TEMPORARY_FILENAME_LENGTH, "%s",
|
|
|
|
symlink ? ".merge_link_XXXXXX" : ".merge_file_XXXXXX");
|
2011-05-13 08:36:42 +04:00
|
|
|
return mkstemp(path);
|
|
|
|
} else {
|
|
|
|
return create_file(path, !symlink ? ce->ce_mode : 0666);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:33 +03:00
|
|
|
int fstat_checkout_output(int fd, const struct checkout *state, struct stat *st)
|
2011-05-13 08:36:42 +04:00
|
|
|
{
|
|
|
|
/* use fstat() only when path == ce->name */
|
|
|
|
if (fstat_is_reliable() &&
|
|
|
|
state->refresh_cache && !state->base_dir_len) {
|
2020-07-09 05:10:39 +03:00
|
|
|
return !fstat(fd, st);
|
2011-05-13 08:36:42 +04:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
Convert "struct cache_entry *" to "const ..." wherever possible
I attempted to make index_state->cache[] a "const struct cache_entry **"
to find out how existing entries in index are modified and where. The
question I have is what do we do if we really need to keep track of on-disk
changes in the index. The result is
- diff-lib.c: setting CE_UPTODATE
- name-hash.c: setting CE_HASHED
- preload-index.c, read-cache.c, unpack-trees.c and
builtin/update-index: obvious
- entry.c: write_entry() may refresh the checked out entry via
fill_stat_cache_info(). This causes "non-const struct cache_entry
*" in builtin/apply.c, builtin/checkout-index.c and
builtin/checkout.c
- builtin/ls-files.c: --with-tree changes stagemask and may set
CE_UPDATE
Of these, write_entry() and its call sites are probably most
interesting because it modifies on-disk info. But this is stat info
and can be retrieved via refresh, at least for porcelain
commands. Other just uses ce_flags for local purposes.
So, keeping track of "dirty" entries is just a matter of setting a
flag in index modification functions exposed by read-cache.c. Except
unpack-trees, the rest of the code base does not do anything funny
behind read-cache's back.
The actual patch is less valueable than the summary above. But if
anyone wants to re-identify the above sites. Applying this patch, then
this:
diff --git a/cache.h b/cache.h
index 430d021..1692891 100644
--- a/cache.h
+++ b/cache.h
@@ -267,7 +267,7 @@ static inline unsigned int canon_mode(unsigned int mode)
#define cache_entry_size(len) (offsetof(struct cache_entry,name) + (len) + 1)
struct index_state {
- struct cache_entry **cache;
+ const struct cache_entry **cache;
unsigned int version;
unsigned int cache_nr, cache_alloc, cache_changed;
struct string_list *resolve_undo;
will help quickly identify them without bogus warnings.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2013-07-09 19:29:00 +04:00
|
|
|
static int streaming_write_entry(const struct cache_entry *ce, char *path,
|
2011-05-21 01:33:31 +04:00
|
|
|
struct stream_filter *filter,
|
2011-05-13 01:31:08 +04:00
|
|
|
const struct checkout *state, int to_tempfile,
|
|
|
|
int *fstat_done, struct stat *statbuf)
|
|
|
|
{
|
2013-03-26 01:49:36 +04:00
|
|
|
int result = 0;
|
2012-03-07 14:54:15 +04:00
|
|
|
int fd;
|
2011-05-13 01:31:08 +04:00
|
|
|
|
|
|
|
fd = open_output_fd(path, ce, to_tempfile);
|
2013-03-26 01:49:36 +04:00
|
|
|
if (fd < 0)
|
|
|
|
return -1;
|
|
|
|
|
2016-09-05 23:07:59 +03:00
|
|
|
result |= stream_blob_to_fd(fd, &ce->oid, filter, 1);
|
2021-03-23 17:19:33 +03:00
|
|
|
*fstat_done = fstat_checkout_output(fd, state, statbuf);
|
2013-03-26 01:49:36 +04:00
|
|
|
result |= close(fd);
|
|
|
|
|
|
|
|
if (result)
|
2011-05-13 01:31:08 +04:00
|
|
|
unlink(path);
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
2017-06-30 23:41:28 +03:00
|
|
|
void enable_delayed_checkout(struct checkout *state)
|
|
|
|
{
|
|
|
|
if (!state->delayed_checkout) {
|
|
|
|
state->delayed_checkout = xmalloc(sizeof(*state->delayed_checkout));
|
|
|
|
state->delayed_checkout->state = CE_CAN_DELAY;
|
|
|
|
string_list_init(&state->delayed_checkout->filters, 0);
|
|
|
|
string_list_init(&state->delayed_checkout->paths, 0);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static int remove_available_paths(struct string_list_item *item, void *cb_data)
|
|
|
|
{
|
|
|
|
struct string_list *available_paths = cb_data;
|
|
|
|
struct string_list_item *available;
|
|
|
|
|
|
|
|
available = string_list_lookup(available_paths, item->string);
|
|
|
|
if (available)
|
|
|
|
available->util = (void *)item->string;
|
|
|
|
return !available;
|
|
|
|
}
|
|
|
|
|
2018-11-13 21:28:00 +03:00
|
|
|
int finish_delayed_checkout(struct checkout *state, int *nr_checkouts)
|
2017-06-30 23:41:28 +03:00
|
|
|
{
|
|
|
|
int errs = 0;
|
2017-08-20 18:47:20 +03:00
|
|
|
unsigned delayed_object_count;
|
|
|
|
off_t filtered_bytes = 0;
|
2017-06-30 23:41:28 +03:00
|
|
|
struct string_list_item *filter, *path;
|
2017-08-20 18:47:20 +03:00
|
|
|
struct progress *progress;
|
2017-06-30 23:41:28 +03:00
|
|
|
struct delayed_checkout *dco = state->delayed_checkout;
|
|
|
|
|
|
|
|
if (!state->delayed_checkout)
|
|
|
|
return errs;
|
|
|
|
|
|
|
|
dco->state = CE_RETRY;
|
2017-08-20 18:47:20 +03:00
|
|
|
delayed_object_count = dco->paths.nr;
|
2017-09-10 11:08:22 +03:00
|
|
|
progress = start_delayed_progress(_("Filtering content"), delayed_object_count);
|
2017-06-30 23:41:28 +03:00
|
|
|
while (dco->filters.nr > 0) {
|
|
|
|
for_each_string_list_item(filter, &dco->filters) {
|
|
|
|
struct string_list available_paths = STRING_LIST_INIT_NODUP;
|
2017-08-20 18:47:20 +03:00
|
|
|
display_progress(progress, delayed_object_count - dco->paths.nr);
|
2017-06-30 23:41:28 +03:00
|
|
|
|
|
|
|
if (!async_query_available_blobs(filter->string, &available_paths)) {
|
|
|
|
/* Filter reported an error */
|
|
|
|
errs = 1;
|
|
|
|
filter->string = "";
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (available_paths.nr <= 0) {
|
|
|
|
/*
|
|
|
|
* Filter responded with no entries. That means
|
|
|
|
* the filter is done and we can remove the
|
|
|
|
* filter from the list (see
|
|
|
|
* "string_list_remove_empty_items" call below).
|
|
|
|
*/
|
|
|
|
filter->string = "";
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* In dco->paths we store a list of all delayed paths.
|
|
|
|
* The filter just send us a list of available paths.
|
|
|
|
* Remove them from the list.
|
|
|
|
*/
|
|
|
|
filter_string_list(&dco->paths, 0,
|
|
|
|
&remove_available_paths, &available_paths);
|
|
|
|
|
|
|
|
for_each_string_list_item(path, &available_paths) {
|
|
|
|
struct cache_entry* ce;
|
|
|
|
|
|
|
|
if (!path->util) {
|
|
|
|
error("external filter '%s' signaled that '%s' "
|
|
|
|
"is now available although it has not been "
|
|
|
|
"delayed earlier",
|
|
|
|
filter->string, path->string);
|
|
|
|
errs |= 1;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Do not ask the filter for available blobs,
|
|
|
|
* again, as the filter is likely buggy.
|
|
|
|
*/
|
|
|
|
filter->string = "";
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
ce = index_file_exists(state->istate, path->string,
|
|
|
|
strlen(path->string), 0);
|
2017-08-20 18:47:20 +03:00
|
|
|
if (ce) {
|
2018-11-13 21:28:00 +03:00
|
|
|
errs |= checkout_entry(ce, state, NULL, nr_checkouts);
|
2017-08-20 18:47:20 +03:00
|
|
|
filtered_bytes += ce->ce_stat_data.sd_size;
|
|
|
|
display_throughput(progress, filtered_bytes);
|
|
|
|
} else
|
|
|
|
errs = 1;
|
2017-06-30 23:41:28 +03:00
|
|
|
}
|
|
|
|
}
|
|
|
|
string_list_remove_empty_items(&dco->filters, 0);
|
|
|
|
}
|
2017-08-20 18:47:20 +03:00
|
|
|
stop_progress(&progress);
|
2017-06-30 23:41:28 +03:00
|
|
|
string_list_clear(&dco->filters, 0);
|
|
|
|
|
|
|
|
/* At this point we should not have any delayed paths anymore. */
|
|
|
|
errs |= dco->paths.nr;
|
|
|
|
for_each_string_list_item(path, &dco->paths) {
|
|
|
|
error("'%s' was not filtered properly", path->string);
|
|
|
|
}
|
|
|
|
string_list_clear(&dco->paths, 0);
|
|
|
|
|
|
|
|
free(dco);
|
|
|
|
state->delayed_checkout = NULL;
|
|
|
|
|
|
|
|
return errs;
|
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:34 +03:00
|
|
|
void update_ce_after_write(const struct checkout *state, struct cache_entry *ce,
|
|
|
|
struct stat *st)
|
|
|
|
{
|
|
|
|
if (state->refresh_cache) {
|
|
|
|
assert(state->istate);
|
|
|
|
fill_stat_cache_info(state->istate, ce, st);
|
|
|
|
ce->ce_flags |= CE_UPDATE_IN_BASE;
|
|
|
|
mark_fsmonitor_invalid(state->istate, ce);
|
|
|
|
state->istate->cache_changed |= CE_ENTRY_CHANGED;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:35 +03:00
|
|
|
/* Note: ca is used (and required) iff the entry refers to a regular file. */
|
|
|
|
static int write_entry(struct cache_entry *ce, char *path, struct conv_attrs *ca,
|
|
|
|
const struct checkout *state, int to_tempfile)
|
2005-06-06 08:59:54 +04:00
|
|
|
{
|
2009-02-09 23:54:50 +03:00
|
|
|
unsigned int ce_mode_s_ifmt = ce->ce_mode & S_IFMT;
|
2017-10-09 20:48:52 +03:00
|
|
|
struct delayed_checkout *dco = state->delayed_checkout;
|
2009-02-09 23:54:51 +03:00
|
|
|
int fd, ret, fstat_done = 0;
|
2018-02-14 21:59:41 +03:00
|
|
|
char *new_blob;
|
2009-02-09 23:54:50 +03:00
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
unsigned long size;
|
2017-09-13 20:16:28 +03:00
|
|
|
ssize_t wrote;
|
|
|
|
size_t newsize = 0;
|
2009-02-09 23:54:51 +03:00
|
|
|
struct stat st;
|
2017-03-15 00:46:40 +03:00
|
|
|
const struct submodule *sub;
|
2020-03-16 21:05:03 +03:00
|
|
|
struct checkout_metadata meta;
|
|
|
|
|
|
|
|
clone_checkout_metadata(&meta, &state->meta, &ce->oid);
|
2009-02-09 23:54:50 +03:00
|
|
|
|
2011-05-21 01:33:31 +04:00
|
|
|
if (ce_mode_s_ifmt == S_IFREG) {
|
2021-03-23 17:19:35 +03:00
|
|
|
struct stream_filter *filter = get_stream_filter_ca(ca, &ce->oid);
|
2011-05-21 01:33:31 +04:00
|
|
|
if (filter &&
|
|
|
|
!streaming_write_entry(ce, path, filter,
|
|
|
|
state, to_tempfile,
|
|
|
|
&fstat_done, &st))
|
|
|
|
goto finish;
|
|
|
|
}
|
2011-05-13 01:31:08 +04:00
|
|
|
|
2009-02-09 23:54:50 +03:00
|
|
|
switch (ce_mode_s_ifmt) {
|
|
|
|
case S_IFLNK:
|
2018-02-14 21:59:41 +03:00
|
|
|
new_blob = read_blob_entry(ce, &size);
|
|
|
|
if (!new_blob)
|
2010-11-28 07:36:38 +03:00
|
|
|
return error("unable to read sha1 file of %s (%s)",
|
2021-02-16 17:06:51 +03:00
|
|
|
ce->name, oid_to_hex(&ce->oid));
|
2007-08-14 12:41:02 +04:00
|
|
|
|
2017-10-09 20:50:05 +03:00
|
|
|
/*
|
|
|
|
* We can't make a real symlink; write out a regular file entry
|
|
|
|
* with the symlink destination as its contents.
|
|
|
|
*/
|
|
|
|
if (!has_symlinks || to_tempfile)
|
|
|
|
goto write_file_entry;
|
|
|
|
|
2018-02-14 21:59:41 +03:00
|
|
|
ret = symlink(new_blob, path);
|
|
|
|
free(new_blob);
|
2017-10-09 20:50:05 +03:00
|
|
|
if (ret)
|
|
|
|
return error_errno("unable to create symlink %s", path);
|
|
|
|
break;
|
|
|
|
|
|
|
|
case S_IFREG:
|
2017-10-09 20:48:52 +03:00
|
|
|
/*
|
|
|
|
* We do not send the blob in case of a retry, so do not
|
|
|
|
* bother reading it at all.
|
|
|
|
*/
|
2017-10-09 20:50:05 +03:00
|
|
|
if (dco && dco->state == CE_RETRY) {
|
2018-02-14 21:59:41 +03:00
|
|
|
new_blob = NULL;
|
2017-10-09 20:48:52 +03:00
|
|
|
size = 0;
|
|
|
|
} else {
|
2018-02-14 21:59:41 +03:00
|
|
|
new_blob = read_blob_entry(ce, &size);
|
|
|
|
if (!new_blob)
|
2017-10-09 20:48:52 +03:00
|
|
|
return error("unable to read sha1 file of %s (%s)",
|
2021-02-16 17:06:51 +03:00
|
|
|
ce->name, oid_to_hex(&ce->oid));
|
2009-02-09 23:54:50 +03:00
|
|
|
}
|
|
|
|
|
2007-08-14 12:41:02 +04:00
|
|
|
/*
|
|
|
|
* Convert from git internal format to working tree format
|
|
|
|
*/
|
2017-10-09 20:50:05 +03:00
|
|
|
if (dco && dco->state != CE_NO_DELAY) {
|
2021-03-23 17:19:35 +03:00
|
|
|
ret = async_convert_to_working_tree_ca(ca, ce->name,
|
|
|
|
new_blob, size,
|
|
|
|
&buf, &meta, dco);
|
2017-10-09 20:50:05 +03:00
|
|
|
if (ret && string_list_has_string(&dco->paths, ce->name)) {
|
2018-02-14 21:59:41 +03:00
|
|
|
free(new_blob);
|
2017-10-09 20:50:05 +03:00
|
|
|
goto delayed;
|
2017-06-30 23:41:28 +03:00
|
|
|
}
|
2021-03-23 17:19:35 +03:00
|
|
|
} else {
|
|
|
|
ret = convert_to_working_tree_ca(ca, ce->name, new_blob,
|
|
|
|
size, &buf, &meta);
|
|
|
|
}
|
2017-10-09 20:50:05 +03:00
|
|
|
|
|
|
|
if (ret) {
|
2018-02-14 21:59:41 +03:00
|
|
|
free(new_blob);
|
|
|
|
new_blob = strbuf_detach(&buf, &newsize);
|
2017-10-09 20:50:05 +03:00
|
|
|
size = newsize;
|
2007-08-14 12:41:02 +04:00
|
|
|
}
|
2017-10-09 20:50:05 +03:00
|
|
|
/*
|
|
|
|
* No "else" here as errors from convert are OK at this
|
|
|
|
* point. If the error would have been fatal (e.g.
|
|
|
|
* filter is required), then we would have died already.
|
|
|
|
*/
|
2007-08-14 12:41:02 +04:00
|
|
|
|
2017-10-09 20:50:05 +03:00
|
|
|
write_file_entry:
|
2011-05-13 08:36:42 +04:00
|
|
|
fd = open_output_fd(path, ce, to_tempfile);
|
2005-06-06 08:59:54 +04:00
|
|
|
if (fd < 0) {
|
2018-02-14 21:59:41 +03:00
|
|
|
free(new_blob);
|
2016-05-08 12:47:44 +03:00
|
|
|
return error_errno("unable to create file %s", path);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
Lazy man's auto-CRLF
It currently does NOT know about file attributes, so it does its
conversion purely based on content. Maybe that is more in the "git
philosophy" anyway, since content is king, but I think we should try to do
the file attributes to turn it off on demand.
Anyway, BY DEFAULT it is off regardless, because it requires a
[core]
AutoCRLF = true
in your config file to be enabled. We could make that the default for
Windows, of course, the same way we do some other things (filemode etc).
But you can actually enable it on UNIX, and it will cause:
- "git update-index" will write blobs without CRLF
- "git diff" will diff working tree files without CRLF
- "git checkout" will write files to the working tree _with_ CRLF
and things work fine.
Funnily, it actually shows an odd file in git itself:
git clone -n git test-crlf
cd test-crlf
git config core.autocrlf true
git checkout
git diff
shows a diff for "Documentation/docbook-xsl.css". Why? Because we have
actually checked in that file *with* CRLF! So when "core.autocrlf" is
true, we'll always generate a *different* hash for it in the index,
because the index hash will be for the content _without_ CRLF.
Is this complete? I dunno. It seems to work for me. It doesn't use the
filename at all right now, and that's probably a deficiency (we could
certainly make the "is_binary()" heuristics also take standard filename
heuristics into account).
I don't pass in the filename at all for the "index_fd()" case
(git-update-index), so that would need to be passed around, but this
actually works fine.
NOTE NOTE NOTE! The "is_binary()" heuristics are totally made-up by yours
truly. I will not guarantee that they work at all reasonable. Caveat
emptor. But it _is_ simple, and it _is_ safe, since it's all off by
default.
The patch is pretty simple - the biggest part is the new "convert.c" file,
but even that is really just basic stuff that anybody can write in
"Teaching C 101" as a final project for their first class in programming.
Not to say that it's bug-free, of course - but at least we're not talking
about rocket surgery here.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2007-02-13 22:07:23 +03:00
|
|
|
|
2018-02-14 21:59:41 +03:00
|
|
|
wrote = write_in_full(fd, new_blob, size);
|
2011-05-13 08:36:42 +04:00
|
|
|
if (!to_tempfile)
|
2021-03-23 17:19:33 +03:00
|
|
|
fstat_done = fstat_checkout_output(fd, state, &st);
|
2005-06-06 08:59:54 +04:00
|
|
|
close(fd);
|
2018-02-14 21:59:41 +03:00
|
|
|
free(new_blob);
|
2017-09-13 20:16:28 +03:00
|
|
|
if (wrote < 0)
|
2010-11-28 07:36:38 +03:00
|
|
|
return error("unable to write file %s", path);
|
2005-06-06 08:59:54 +04:00
|
|
|
break;
|
2017-10-09 20:50:05 +03:00
|
|
|
|
2007-05-22 00:08:28 +04:00
|
|
|
case S_IFGITLINK:
|
2007-04-13 20:26:04 +04:00
|
|
|
if (to_tempfile)
|
2021-02-16 17:06:51 +03:00
|
|
|
return error("cannot create temporary submodule %s", ce->name);
|
2007-04-13 20:26:04 +04:00
|
|
|
if (mkdir(path, 0777) < 0)
|
2013-07-18 16:26:55 +04:00
|
|
|
return error("cannot create submodule directory %s", path);
|
2017-03-15 00:46:40 +03:00
|
|
|
sub = submodule_from_ce(ce);
|
|
|
|
if (sub)
|
|
|
|
return submodule_move_head(ce->name,
|
2017-04-19 00:37:22 +03:00
|
|
|
NULL, oid_to_hex(&ce->oid),
|
|
|
|
state->force ? SUBMODULE_MOVE_HEAD_FORCE : 0);
|
2007-04-13 20:26:04 +04:00
|
|
|
break;
|
2017-10-09 20:50:05 +03:00
|
|
|
|
2005-06-06 08:59:54 +04:00
|
|
|
default:
|
2021-02-16 17:06:51 +03:00
|
|
|
return error("unknown file mode for %s in index", ce->name);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
|
2011-05-13 01:31:08 +04:00
|
|
|
finish:
|
2005-06-06 10:15:40 +04:00
|
|
|
if (state->refresh_cache) {
|
2021-03-23 17:19:34 +03:00
|
|
|
if (!fstat_done && lstat(ce->name, &st) < 0)
|
|
|
|
return error_errno("unable to stat just-written file %s",
|
|
|
|
ce->name);
|
|
|
|
update_ce_after_write(state, ce , &st);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
2017-10-05 13:44:06 +03:00
|
|
|
delayed:
|
2005-06-06 08:59:54 +04:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2009-07-30 07:22:25 +04:00
|
|
|
/*
|
|
|
|
* This is like 'lstat()', except it refuses to follow symlinks
|
2009-08-17 10:53:12 +04:00
|
|
|
* in the path, after skipping "skiplen".
|
2009-07-30 07:22:25 +04:00
|
|
|
*/
|
2010-01-12 09:27:31 +03:00
|
|
|
static int check_path(const char *path, int len, struct stat *st, int skiplen)
|
2009-07-30 07:22:25 +04:00
|
|
|
{
|
2009-08-17 10:53:12 +04:00
|
|
|
const char *slash = path + len;
|
|
|
|
|
|
|
|
while (path < slash && *slash != '/')
|
|
|
|
slash--;
|
|
|
|
if (!has_dirs_only_path(path, slash - path, skiplen)) {
|
2009-07-30 07:22:25 +04:00
|
|
|
errno = ENOENT;
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
return lstat(path, st);
|
|
|
|
}
|
|
|
|
|
2018-08-17 21:00:39 +03:00
|
|
|
static void mark_colliding_entries(const struct checkout *state,
|
|
|
|
struct cache_entry *ce, struct stat *st)
|
|
|
|
{
|
|
|
|
int i, trust_ino = check_stat;
|
|
|
|
|
clone: fix colliding file detection on APFS
Commit b878579ae7 (clone: report duplicate entries on case-insensitive
filesystems - 2018-08-17) adds a warning to user when cloning a repo
with case-sensitive file names on a case-insensitive file system. The
"find duplicate file" check was doing by comparing inode number (and
only fall back to fspathcmp() when inode is known to be unreliable
because fspathcmp() can't cover all case folding cases).
The inode check is very simple, and wrong. It compares between a
32-bit number (sd_ino) and potentially a 64-bit number (st_ino). When
an inode is larger than 2^32 (which seems to be the case for APFS), it
will be truncated and stored in sd_ino, but comparing with itself will
fail.
As a result, instead of showing a pair of files that have the same
name, we show just one file (marked before the beginning of the
loop). We fail to find the original one.
The fix could be just a simple type cast (*)
dup->ce_stat_data.sd_ino == (unsigned int)st->st_ino
but this is no longer a reliable test, there are 4G possible inodes
that can match sd_ino because we only match the lower 32 bits instead
of full 64 bits.
There are two options to go. Either we ignore inode and go with
fspathcmp() on Apple platform. This means we can't do accurate inode
check on HFS anymore, or even on APFS when inode numbers are still
below 2^32.
Or we just to to reduce the odds of matching a wrong file by checking
more attributes, counting mostly on st_size because st_xtime is likely
the same. This patch goes with this direction, hoping that false
positive chances are too small to be seen in practice.
While at there, enable the test on Cygwin (verified working by Ramsay
Jones)
(*) this is also already done inside match_stat_data()
Reported-by: Carlo Arenas <carenas@gmail.com>
Helped-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-20 19:28:53 +03:00
|
|
|
#if defined(GIT_WINDOWS_NATIVE) || defined(__CYGWIN__)
|
2018-08-17 21:00:39 +03:00
|
|
|
trust_ino = 0;
|
|
|
|
#endif
|
|
|
|
|
|
|
|
ce->ce_flags |= CE_MATCHED;
|
|
|
|
|
|
|
|
for (i = 0; i < state->istate->cache_nr; i++) {
|
|
|
|
struct cache_entry *dup = state->istate->cache[i];
|
|
|
|
|
|
|
|
if (dup == ce)
|
|
|
|
break;
|
|
|
|
|
|
|
|
if (dup->ce_flags & (CE_MATCHED | CE_VALID | CE_SKIP_WORKTREE))
|
|
|
|
continue;
|
|
|
|
|
clone: fix colliding file detection on APFS
Commit b878579ae7 (clone: report duplicate entries on case-insensitive
filesystems - 2018-08-17) adds a warning to user when cloning a repo
with case-sensitive file names on a case-insensitive file system. The
"find duplicate file" check was doing by comparing inode number (and
only fall back to fspathcmp() when inode is known to be unreliable
because fspathcmp() can't cover all case folding cases).
The inode check is very simple, and wrong. It compares between a
32-bit number (sd_ino) and potentially a 64-bit number (st_ino). When
an inode is larger than 2^32 (which seems to be the case for APFS), it
will be truncated and stored in sd_ino, but comparing with itself will
fail.
As a result, instead of showing a pair of files that have the same
name, we show just one file (marked before the beginning of the
loop). We fail to find the original one.
The fix could be just a simple type cast (*)
dup->ce_stat_data.sd_ino == (unsigned int)st->st_ino
but this is no longer a reliable test, there are 4G possible inodes
that can match sd_ino because we only match the lower 32 bits instead
of full 64 bits.
There are two options to go. Either we ignore inode and go with
fspathcmp() on Apple platform. This means we can't do accurate inode
check on HFS anymore, or even on APFS when inode numbers are still
below 2^32.
Or we just to to reduce the odds of matching a wrong file by checking
more attributes, counting mostly on st_size because st_xtime is likely
the same. This patch goes with this direction, hoping that false
positive chances are too small to be seen in practice.
While at there, enable the test on Cygwin (verified working by Ramsay
Jones)
(*) this is also already done inside match_stat_data()
Reported-by: Carlo Arenas <carenas@gmail.com>
Helped-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-20 19:28:53 +03:00
|
|
|
if ((trust_ino && !match_stat_data(&dup->ce_stat_data, st)) ||
|
2018-08-17 21:00:39 +03:00
|
|
|
(!trust_ino && !fspathcmp(ce->name, dup->name))) {
|
|
|
|
dup->ce_flags |= CE_MATCHED;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:36 +03:00
|
|
|
int checkout_entry_ca(struct cache_entry *ce, struct conv_attrs *ca,
|
|
|
|
const struct checkout *state, char *topath,
|
|
|
|
int *nr_checkouts)
|
2005-06-06 08:59:54 +04:00
|
|
|
{
|
2014-03-13 13:19:07 +04:00
|
|
|
static struct strbuf path = STRBUF_INIT;
|
2006-03-05 11:24:15 +03:00
|
|
|
struct stat st;
|
2021-03-23 17:19:36 +03:00
|
|
|
struct conv_attrs ca_buf;
|
2005-06-06 08:59:54 +04:00
|
|
|
|
2018-12-20 16:48:15 +03:00
|
|
|
if (ce->ce_flags & CE_WT_REMOVE) {
|
|
|
|
if (topath)
|
|
|
|
/*
|
|
|
|
* No content and thus no path to create, so we have
|
|
|
|
* no pathname to return.
|
|
|
|
*/
|
|
|
|
BUG("Can't remove entry to a path");
|
|
|
|
unlink_entry(ce);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-23 17:19:35 +03:00
|
|
|
if (topath) {
|
2021-03-23 17:19:36 +03:00
|
|
|
if (S_ISREG(ce->ce_mode) && !ca) {
|
2021-03-23 17:19:35 +03:00
|
|
|
convert_attrs(state->istate, &ca_buf, ce->name);
|
|
|
|
ca = &ca_buf;
|
|
|
|
}
|
|
|
|
return write_entry(ce, topath, ca, state, 1);
|
|
|
|
}
|
2006-03-05 11:24:15 +03:00
|
|
|
|
2014-03-13 13:19:07 +04:00
|
|
|
strbuf_reset(&path);
|
|
|
|
strbuf_add(&path, state->base_dir, state->base_dir_len);
|
|
|
|
strbuf_add(&path, ce->name, ce_namelen(ce));
|
2005-06-06 08:59:54 +04:00
|
|
|
|
2014-03-13 13:19:07 +04:00
|
|
|
if (!check_path(path.buf, path.len, &st, state->base_dir_len)) {
|
2017-03-15 00:46:40 +03:00
|
|
|
const struct submodule *sub;
|
2018-08-13 19:14:32 +03:00
|
|
|
unsigned changed = ie_match_stat(state->istate, ce, &st,
|
|
|
|
CE_MATCH_IGNORE_VALID | CE_MATCH_IGNORE_SKIP_WORKTREE);
|
2017-03-15 00:46:40 +03:00
|
|
|
/*
|
|
|
|
* Needs to be checked before !changed returns early,
|
|
|
|
* as the possibly empty directory was not changed
|
|
|
|
*/
|
|
|
|
sub = submodule_from_ce(ce);
|
|
|
|
if (sub) {
|
|
|
|
int err;
|
|
|
|
if (!is_submodule_populated_gently(ce->name, &err)) {
|
|
|
|
struct stat sb;
|
|
|
|
if (lstat(ce->name, &sb))
|
|
|
|
die(_("could not stat file '%s'"), ce->name);
|
|
|
|
if (!(st.st_mode & S_IFDIR))
|
|
|
|
unlink_or_warn(ce->name);
|
|
|
|
|
|
|
|
return submodule_move_head(ce->name,
|
2017-04-19 00:37:22 +03:00
|
|
|
NULL, oid_to_hex(&ce->oid), 0);
|
2017-03-15 00:46:40 +03:00
|
|
|
} else
|
|
|
|
return submodule_move_head(ce->name,
|
|
|
|
"HEAD", oid_to_hex(&ce->oid),
|
2017-04-19 00:37:22 +03:00
|
|
|
state->force ? SUBMODULE_MOVE_HEAD_FORCE : 0);
|
2017-03-15 00:46:40 +03:00
|
|
|
}
|
|
|
|
|
2005-06-06 08:59:54 +04:00
|
|
|
if (!changed)
|
|
|
|
return 0;
|
|
|
|
if (!state->force) {
|
|
|
|
if (!state->quiet)
|
2014-03-13 13:19:07 +04:00
|
|
|
fprintf(stderr,
|
|
|
|
"%s already exists, no checkout\n",
|
|
|
|
path.buf);
|
2005-10-03 23:44:48 +04:00
|
|
|
return -1;
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
|
|
|
|
2018-08-17 21:00:39 +03:00
|
|
|
if (state->clone)
|
|
|
|
mark_colliding_entries(state, ce, &st);
|
|
|
|
|
2005-06-06 08:59:54 +04:00
|
|
|
/*
|
|
|
|
* We unlink the old file, to get the new one with the
|
|
|
|
* right permissions (including umask, which is nasty
|
|
|
|
* to emulate by hand - much easier to let the system
|
|
|
|
* just do the right thing)
|
|
|
|
*/
|
2005-07-14 20:58:45 +04:00
|
|
|
if (S_ISDIR(st.st_mode)) {
|
2007-04-13 20:26:04 +04:00
|
|
|
/* If it is a gitlink, leave it alone! */
|
2008-01-15 03:03:17 +03:00
|
|
|
if (S_ISGITLINK(ce->ce_mode))
|
2007-04-13 20:26:04 +04:00
|
|
|
return 0;
|
2014-03-13 13:19:08 +04:00
|
|
|
remove_subtree(&path);
|
2014-03-13 13:19:07 +04:00
|
|
|
} else if (unlink(path.buf))
|
2016-05-08 12:47:44 +03:00
|
|
|
return error_errno("unable to unlink old '%s'", path.buf);
|
2006-03-05 11:24:15 +03:00
|
|
|
} else if (state->not_new)
|
2005-06-06 08:59:54 +04:00
|
|
|
return 0;
|
2014-03-13 13:19:07 +04:00
|
|
|
|
|
|
|
create_directories(path.buf, path.len, state);
|
2021-03-23 17:19:35 +03:00
|
|
|
|
2018-11-13 21:28:00 +03:00
|
|
|
if (nr_checkouts)
|
|
|
|
(*nr_checkouts)++;
|
2021-03-23 17:19:35 +03:00
|
|
|
|
2021-03-23 17:19:36 +03:00
|
|
|
if (S_ISREG(ce->ce_mode) && !ca) {
|
2021-03-23 17:19:35 +03:00
|
|
|
convert_attrs(state->istate, &ca_buf, ce->name);
|
|
|
|
ca = &ca_buf;
|
|
|
|
}
|
|
|
|
|
|
|
|
return write_entry(ce, path.buf, ca, state, 0);
|
2005-06-06 08:59:54 +04:00
|
|
|
}
|
2018-12-20 16:48:14 +03:00
|
|
|
|
|
|
|
void unlink_entry(const struct cache_entry *ce)
|
|
|
|
{
|
|
|
|
const struct submodule *sub = submodule_from_ce(ce);
|
|
|
|
if (sub) {
|
|
|
|
/* state.force is set at the caller. */
|
|
|
|
submodule_move_head(ce->name, "HEAD", NULL,
|
|
|
|
SUBMODULE_MOVE_HEAD_FORCE);
|
|
|
|
}
|
checkout: don't follow symlinks when removing entries
At 1d718a5108 ("do not overwrite untracked symlinks", 2011-02-20),
symlink.c:check_leading_path() started returning different codes for
FL_ENOENT and FL_SYMLINK. But one of its callers, unlink_entry(), was
not adjusted for this change, so it started to follow symlinks on the
leading path of to-be-removed entries. Fix that and add a regression
test.
Note that since 1d718a5108 check_leading_path() no longer differentiates
the case where it found a symlink in the path's leading components from
the cases where it found a regular file or failed to lstat() the
component. So, a side effect of this current patch is that
unlink_entry() now returns early in all of these three cases. And
because we no longer try to unlink such paths, we also don't get the
warning from remove_or_warn().
For the regular file and symlink cases, it's questionable whether the
warning was useful in the first place: unlink_entry() removes tracked
paths that should no longer be present in the state we are checking out
to. If the path had its leading dir replaced by another file, it means
that the basename already doesn't exist, so there is no need for a
warning. Sure, we are leaving a regular file or symlink behind at the
path's dirname, but this file is either untracked now (so again, no
need to warn), or it will be replaced by a tracked file during the next
phase of this checkout operation.
As for failing to lstat() one of the leading components, the basename
might still exist only we cannot unlink it (e.g. due to the lack of the
required permissions). Since the user expect it to be removed
(especially with checkout's --no-overlay option), add back the warning
in this more relevant case.
Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-03-18 21:43:47 +03:00
|
|
|
if (check_leading_path(ce->name, ce_namelen(ce), 1) >= 0)
|
2018-12-20 16:48:14 +03:00
|
|
|
return;
|
|
|
|
if (remove_or_warn(ce->ce_mode, ce->name))
|
|
|
|
return;
|
|
|
|
schedule_dir_for_removal(ce->name, ce_namelen(ce));
|
|
|
|
}
|