Commit graph

40529 commits

Author SHA1 Message Date
Joey Hess
e51a587c57
comment 2021-09-03 11:41:38 -04:00
aurelf
7961c5a984 Added a comment: Automatically adding metadata can be very slow 2021-09-03 12:46:07 +00:00
Joey Hess
1d3f59a9db
close dup 2021-09-02 18:50:05 -04:00
Joey Hess
ec12537774
defer write permissions checking in import until after copy to repo
This should complete the fix started in
6329997ac4, fixing the actual cause of the
test suite failure this time.

Sponsored-by: Dartmouth College's Datalad project
2021-09-02 13:45:21 -04:00
Joey Hess
7b26222cbe
comment 2021-09-02 12:58:46 -04:00
Joey Hess
2241c7e879
reopen 2021-09-02 12:24:03 -04:00
Joey Hess
bd5494bb9c
fix windows build 2021-09-02 12:21:25 -04:00
jwodder
40adf643de 2021-09-02 14:48:00 +00:00
scy
ddd7d1d110 Added a comment 2021-09-02 08:17:03 +00:00
jkniiv
2bbfe30226 Added a comment 2021-09-02 05:24:14 +00:00
jkniiv
86b5b4d805 add author meta 2021-09-02 04:39:12 +00:00
jkniiv
f34bc033a1 commit 9595a247a doesn't compile on Windows without reverting a small previous change 2021-09-02 04:29:16 +00:00
Joey Hess
7319e4a458
Merge branch 'master' of ssh://git-annex.branchable.com 2021-09-01 15:35:32 -04:00
Joey Hess
4f42292b13
improve url download failure display
* When downloading urls fail, explain which urls failed for which
  reasons.
* web: Avoid displaying a warning when downloading one url failed
  but another url later succeeded.

Some other uses of downloadUrl use urls that are effectively internal use,
and should not all be displayed to the user on failure. Eg, Remote.Git
tries different urls where content could be located depending on how the
remote repo is set up. Exposing those urls to the user would lead to wild
goose chases. So had to parameterize it to control whether it displays urls
or not.

A side effect of this change is that when there are some youtube urls
and some regular urls, it will try regular urls first, even if the
youtube urls are listed first. This seems like an improvement if
anything, but in any case there's no defined order of urls that it's
supposed to use.

Sponsored-by: Dartmouth College's Datalad project
2021-09-01 15:33:38 -04:00
yarikoptic
89ace6f1da Added a comment 2021-09-01 18:05:26 +00:00
yarikoptic
61432f97cd Added a comment 2021-09-01 16:22:22 +00:00
Joey Hess
9cd1430fd6
comment 2021-09-01 11:43:51 -04:00
yarikoptic
5f78f96d2b Added a comment: clarification 2021-09-01 15:39:06 +00:00
Joey Hess
9595a247ae
fix test suite failure on windows
This was maybe a real bug too, although I don't know what circumstances
it would be a problem. See comment for analysis of this windows drive
letter wackyness issue.

Sponsored-by: Brock Spratlen on Patreon
2021-09-01 11:32:25 -04:00
Joey Hess
6db7079cb0
close 2021-09-01 10:47:33 -04:00
Joey Hess
afdca0aff4
skip test_readonly on windows
drop vs readonly repo failed (transcript follows)
      drop foo
      failed
      git-annex: .git\annex\objects\6cd\e82\SHA256E-s20--e394a389d787383843decc5d3d99b6d184ffa5fddeec23b911f9ee7fc8b9ea77\SHA256E-s20--e394a389d787383843decc5d3d99b6d184ffa5fddeec23b911f9ee7fc8b9ea77: DeleteFile "\\\\?\\C:\\Users\\runneradmin\\.t\\tmprepo3\\.git\\annex\\objects\\6cd\\e82\\SHA256E-s20--e394a389d787383843decc5d3d99b6d184ffa5fddeec23b911f9ee7fc8b9ea77\\SHA256E-s20--e394a389d787383843decc5d3d99b6d184ffa5fddeec23b911f9ee7fc8b9ea77": permission denied (Access is denied.)

This is a bit surprising, it seems that chmod -R -w on windows actually
had some effect. But also, on Windows lockContentShared
has to write to a lock file, so it cannot work in a readonly repo.
So git-annex cannot support readonly repos on Windows.

Sponsored-by: Dartmouth College's Datalad project
2021-09-01 10:42:50 -04:00
Joey Hess
6329997ac4
init: check for filesystem where write bit cannot be removed
This fixes a reversion caused by a99a84f342,
when git-annex init is run as root on a FAT filesystem mounted with
hdiutil on OSX. Such a mount point has file mode 777 for everything and
it cannot be changed. The existing crippled filesystem test tried to
write to a file after removing write bit, but that test does not run as
root (since root can write to unwritable files). So added a check of the
write permissions of the file, after attempting to remove them.

Sponsored-by: Dartmouth College's Datalad project
2021-09-01 10:27:28 -04:00
jkniiv@b330fc3a602d36a37a67b2a2d99d4bed3bb653cb
9a0eab9b0e prop_relPathDirToFileAbs_basics fails on occasion 2021-08-31 21:44:25 +00:00
jkniiv@b330fc3a602d36a37a67b2a2d99d4bed3bb653cb
8ebba0dc31 Added a comment 2021-08-31 20:47:24 +00:00
yarikoptic
07aa981379 initial TODO on improving 'get' errors reporting 2021-08-31 14:40:48 +00:00
yarikoptic
e2b52cb8af report on 2 tests fail on crippled FS on mac 2021-08-31 14:20:15 +00:00
yarikoptic
5ce0990017 initial report 2021-08-31 14:13:11 +00:00
yarikoptic
608593feca Added a comment: just an update on my testing 2021-08-30 22:17:53 +00:00
Joey Hess
ccf7e5b949
add test case for readonly remote
Tests for the reversion fixed in 837116ef1e
and similar problems.

Sponsored-by: Dartmouth College's Datalad project
2021-08-30 16:39:02 -04:00
Joey Hess
249d424b8a
comment 2021-08-30 14:42:09 -04:00
Joey Hess
d2f2a0e8c8
Merge branch 'master' of ssh://git-annex.branchable.com 2021-08-30 14:37:32 -04:00
Joey Hess
0418d54f28
update 2021-08-30 14:34:25 -04:00
Joey Hess
8208daaf17
idea for making more special remotes support importtree
Sponsored-by: Jack Hill on Patreon
2021-08-30 14:27:22 -04:00
yarikoptic
953baa0abd Added a comment: reproducer 2021-08-30 18:07:05 +00:00
yarikoptic
4a6915000e Added a comment 2021-08-30 17:24:25 +00:00
Joey Hess
e853ef3095
decorate openTempFile errors with the template name
This is to track down what file in .git/annex/ is being written to via a
temp file when the repository is read-only.

Sponsored-by: Dartmouth College's Datalad project
2021-08-30 13:05:02 -04:00
Joey Hess
837116ef1e
Fix support for readonly git remotes
Boolean blindness oops.

(Reversion in version 8.20210621)

Sponsored-by: Dartmouth College's Datalad project
2021-08-30 12:34:19 -04:00
Joey Hess
b083f9ec2f
comment 2021-08-30 12:08:01 -04:00
Joey Hess
e92434ce72
comment 2021-08-30 12:06:28 -04:00
Joey Hess
50bb5f9c47
comment 2021-08-30 12:02:52 -04:00
LazerEpilasyon
e2038ef412 2021-08-30 11:32:34 +00:00
strmd
63c2cfd470 2021-08-28 09:30:28 +00:00
yarikoptic
eb9abbd392 Added a comment 2021-08-27 18:46:39 +00:00
Joey Hess
a99a84f342
add: Detect when xattrs or perhaps ACLs prevent locking down a file's content
And fail with an informative message.

I don't think ACLs can prevent removing the write bit, but I'm not sure,
so kept it mentioning them as a possibility.

Should git-annex lock also check if the write bits are able to be removed?
Maybe, but the case I know about with xattrs involves cp -a copying NFS
xattrs, and it's the copy of the file that is the problem. So when locking
a file, I guess it will not be the copy.

Sponsored-by: Dartmouth College's Datalad project
2021-08-27 14:33:01 -04:00
Joey Hess
e17342b2a0
Run cp -a with --no-preserve=xattr, to avoid problems with copied xattrs
Including them breaking permissions setting on some NFS servers.

Sponsored-by: Dartmouth College's Datalad project
2021-08-27 13:09:34 -04:00
Joey Hess
7b1709105a
Merge branch 'master' of ssh://git-annex.branchable.com 2021-08-27 12:30:35 -04:00
Joey Hess
6d4a728455
Added annex.youtube-dl-command config
This can be used to run some forks of youtube-dl.

Sponsored-by: Brett Eisenberg on Patreon
2021-08-27 09:44:23 -04:00
Joey Hess
3093e8fcc3
thought 2021-08-27 00:59:24 -04:00
yarikoptic
16d35e04bf Added a comment 2021-08-27 01:38:54 +00:00
jkniiv@b330fc3a602d36a37a67b2a2d99d4bed3bb653cb
3007e3c177 Added a comment: it turns out I had to file this as a bug 2021-08-27 01:38:29 +00:00