Added a comment: Unless the key is the argument to set

Added a comment: -- works between the value and the option

Remove broken link.

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

update

add

Added a comment: Other benefits of proposed improvements to the smudge/clean API

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

fix breakage caused by recent commit

34a535ebeaf483f6643f949ae774939650d74837 broke the test suite.
Getting a file started failing in one case, because the annex object did
not have its inode cached, so was not trusted to be unmodified.

This adds something very similar to what was added to linkAnnex
in commit 2e9341a47d437b5a7b14824d3e5b2864b059a850 -- if there are not
yet any inodes cached for a key, add the inode of the annex object when
adding the inode of the unlocked file.

Feels like this should be handled in a more principled way. How
do we know the addInodeCaches call in getMoveRaceRecovery just above
this change is currently correct? It doesn't add the annex object inode
cache. Ah well, maybe sometime when I've not had my entire evening eaten
by a reversion that the test suite caught as I was cooking dinner.

wording

support modifying and running git add on an unlocked file that used an URL key

Avoids the smudge --clean filter failing because URL keys do not support
genKey. Instead the modified content will be added using the default
backend.

This commit was sponsored by Jochen Bartl on Patreon.

adjust: Fix some bad behavior when unlocked files use URL keys.

This avoids the smudge --clean filter failing on the URL keys.

git checkout runs the post-checkout hook, which runs smudge --update.
That populates all the pointer files, but it neglected to store their inode
caches in the keys db. With that done, and the keys db flushed before
smudge --clean gets run (by restagePointerFile), the isUnmodifiedCheap
check can tell the file is not modified, so will not try to re-ingest it,
which does not work with URL keys because they do not support genKey.

It also seems possible that the isUnmodifiedCheap was also failing for
non-URL keys, which would cause them to be re-ingested, leading to a lot of
extra work. I have not verified that, but don't see why it wouldn't have
happened. So this probably also speeds up checking out adjusted branches.

This commit was sponsored by Boyd Stephen Smith Jr. on Patreon.

bug

fix export overwrite on FAT

Don't accept the cid of the temp file that the content has just been
written to as something we will accept if another file has that same
content. There's no reason to, and on FAT, due to mtime resolution,
the test suite hit just such a case.

This fixes a reversion from 73df633a6215faa093e4a7524c6d328aa988aed1
which removed inode from the ContentIdentifier.

Added a comment

comment

comment

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

notabug

Added a comment

Added a comment

Add positive stuff

Added a comment: bug report

comment

followup

fix problem on windows with newly rewritten prop_relPathDirToFileAbs_basics

Seems that dropDrive on windows only drops eg c:/ but not a leading /
while on linux, it does drop a leading / (which is what it considers
to be equivilant to a drive letter. I had been relying on it to drop
both. So need to drop leading directory separators.

Also, if the quickcheck generated input is eg "c:c:c:c:foo",
dropDrive will only drop the first one, leaving a path that's
still not relative. So instead of using dropDrive, just remove the
colons from the path.

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

Fix a bug in view filename generation when a metadata value ended with "/"

Or ":" or "\" on Windows, eg "c:" again.

update copyright year

initial report on export_import fail

Added a comment

Added a comment

Added a comment

Added a comment

fix a bug that prevented git-annex init from working in a submodule

This is probably a reversion, but not sure what caused it. By the time
Annex.Init runs fixupUnusualReposAfterInit, another git-annex process has
at least sometimes already done the necessary fixups. (Eg, one run
indirectly by a git command.) But since the Repo is cached, it doesn't
realize and does them again. So, avoid crashing when git config --unset
fails.

This commit was sponsored by Jack Hill on Patreon.

comment

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

Added a comment

comment typo

change user-visible error to giveup

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