comment

adjusted report with more of not so deep thoughts/observation ;)

initial plea for considering credentials upon 404 of /config

Added a comment

Added a comment

Merge branch 'master' of ssh://git-annex.branchable.com

omit inode from ContentIdentifier for directory special remote

Directory special remotes with importtree=yes now avoid unncessary overhead
when inodes of files have changed, as happens whenever a FAT filesystem
gets remounted.

A few unusual edge cases of modifications won't be detected and
imported. I think they're unusual enough not to be a concern. It would
be possible to add a config setting that controls whether to compare
inodes too, but does not seem worth bothering the user about currently.

I chose to continue to use the InodeCache serialization, just with the
inode zeroed. This way, if I later change my mind or make it
configurable, can parse it back to an InodeCache and operate on it. The
overhead of storing a 0 in the content identifier log seems worth it.

There is a one-time cost to this change; all directory special remotes
with importtree=yes will re-hash all files once, and will update the
content identifier logs with zeroed inodes.

This commit was sponsored by Brett Eisenberg on Patreon.

removed

Added a comment

Added a comment

display exception as part of warnings

and comment that led to this change

Added a comment

comment

comment

comment and todo

removed

Added a comment

Added a comment

Added a comment: Went away with fsck?

Added a comment: Issue when using flag --recurse-submodules with git-checkout and git-switch

avoid partial functions in Git.Url

After the last commit, it was able to throw errors just due to an
unparseable url. This avoids needing to worry about that, as long
as the call site has already checked that it has a parseable url.

avoid crashing when there are remotes using unparseable urls

Including the non-standard URI form that git-remote-gcrypt uses for rsync.

Eg, "ook://foo:bar" cannot be parsed because "bar" is not a valid port
number. But git could have a remote with that, it would try to run
git-remote-ook to handle it. So, git-annex has to allow for such things,
rather than crashing.

This commit was sponsored by Luke Shumaker on Patreon.

comment

comment

Merge branch 'master' of ssh://git-annex.branchable.com

fix prop_relPathDirToFileAbs_basics fail on windows

It was just slapping on a path separator to the front of the path to
make it absolute, but on windows, a path like "//foo/bar" actually
has a network "drive" of "//foo" and so that broke the test case.

Since "a:foo" is a somehow relative path on windows
(who knows how), drop any drive from the input. But dropDrive also drops
any leading path separator, making the input path relative. So now
it should be safe to slapp on a leading path separator.

Added a comment

update

comment

Bug fix: Fix tilde expansion in ssh urls when the tilde is the last character in the url. Thanks, Grond for the patch.

Merge branch 'master' of ssh://git-annex.branchable.com

initial report on prop_relPathDirToFileAbs_basics windows FAIL

Added a comment

Added a comment

Added a comment: Works fine after changing mount point

improve

Added a comment

removed

removed

Added a comment

close as not a git-annex bug

Bug fix: export with -J could fail when two files had the same content.

Exporting is done inside a call to writeLockDbWhile which guarantees there
is only one process uploading to a given ExportLocation.

fix build warning

Windows: Work around win32 length limits when dealing with lock files

export TestableFilePath constructor

Useful for eg, replicating failures in ghci. No need for this to be a
smart constructor, as long as it's used with valid filepaths, it's ok
and if not the test breaks.

rewrite prop_relPathDirToFileAbs_basics

This was not a good test, it broke the requirement that
relPathDirToFileAbs take absolute paths. And it failed when the two
input paths were eg, the same but differently normalized.

Replaced with some tests of the real basics of that function.

close

comment

improve comment

comment

Merge branch 'master' of ssh://git-annex.branchable.com

include bugs/todos tagged datalad

sometimes they are not signed with an author

initial report about fresh test fail

merge from git-repair

Added a comment

Added a comment

Added a comment: git-annex uninit --fast

removed myself as the author, damn cut/paste ;)

assigned to datalad project

fixed

comment

Merge branch 'master' of ssh://git-annex.branchable.com

test: avoid unnecessary tests of variants of git remote

Configuring chunking and encryption for a git remote has no effect, so
skip testing those variants in the TestRemote call.

It would be better if TestRemote itself could do this, but it
doesn't seem possible there. There is no way to look at a Remote and
tell if it supports chunking or encryption.

Note that, while the test suite displays output as it it's testing
exporting, it actually skips doing anything for the tests when run on
the git remote. So at least does not waste time even though the output
is not ideal.

This commit was sponsored by Noam Kremen on Patreon.

remove blank

update test suite after git-annex trust behavior change

fix format of message

newlines are eaten

add testremote of a git remote to test suite

The test suite contains its own tests that test a lot of basic stuff
about git remotes, mostly in passing to set up other situations.
But testremote does try some unusual edge cases, which may as
well be tried for git remotes as well as directory, especially since
it's so little code to add it.

This commit was sponsored by Kevin Mueller on Patreon.

close

idea

Added a comment: Specific to repository location in /private

Added a comment

Added a comment: checkout borg remote

report on fresh test fails

merged fix from kyle

avoid using wildcard near bug kyle fixed

adjustTree: Consider submodule deletions

In addition to regular file deletions, the removefiles argument passed
to adjustTree may contain removed submodules.  When making the new
tree, filter these out in the same way that is done for regular files
so that the deletion is propagated.

Merge branch 'master' of ssh://git-annex.branchable.com

Behavior change: --trust-glacier option no longer overrides trust

Since that can lead to data loss, which should never be enabled by an
option other than --force.

This commit was sponsored by Jake Vosloo on Patreon.

Behavior change: --trust option no longer overrides trust

Since that can lead to data loss, which should never be enabled by an
option other than --force.

I suppose that using --trust was in some situation, safer than --force,
because it doesn't entirely disable checking for data loss, but only
disables checking involving data that is on the specified repository.
But it seems better to be able to say that data loss only happens with
--force.

This commit was sponsored by Graham Spencer on Patreon.

Behavior change: git-annex trust now needs --force

Since unconsidered use of trusted repositories can lead to data loss.

Trusted has always been this way, but it used to be acceptable for
git-annex to be set up so that data could be lost without using --force,
and most or all other ways that can happen have already been eliminated.

This commit was sponsored by Mark Reidenbach on Patreon.

Merge branch 'master' of ssh://git-annex.branchable.com

wording

improve wording

Merge branch 'master' into requirednumcopies

mincopies

This is conceptually very simple, just making a 1 that was hard coded be
exposed as a config option. The hard part was plumbing all that, and
dealing with complexities like reading it from git attributes at the
same time that numcopies is read.

Behavior change: When numcopies is set to 0, git-annex used to drop
content without requiring any copies. Now to get that (highly unsafe)
behavior, mincopies also needs to be set to 0. It seemed better to
remove that edge case, than complicate mincopies by ignoring it when
numcopies is 0.

This commit was sponsored by Denis Dzyubenko on Patreon.

Added a comment

Added a comment

branch

docs for requirednumcopies

Not implemented yet.