Added a comment

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

assistant: Fix a crash on startup by avoiding using forkProcess

ghc 8.8.4 seems to have changed something that broke code that has been
successfully using forkProcess since 2012. Likely a change to GC internals.

Since forkProcess has never had clear documentation about how to
use it safely, avoid using it at all. Instead, when git-annex needs to
daemonize itself, re-run the git-annex command, in a new process group
and session.

This commit was sponsored by Luke Shumaker on Patreon.

comment

removed

Added a comment

Added a comment

already implemented

comment

comment

comment

comment

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

remove inode cache in unannex

Similar to what commit 675556fd9ae682ffcb034fdf454a15fe6e7da979 did for
adding a non-annexed file, this prevents the smudge clean filter
recognising the inode if git add is later run on the unannexed file.

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

idea

Added a comment

Added a comment: D'oh!

mention index mode explicitly

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

comment

Added a comment

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

split out common options to its own page and mention it on each subcommand page

Sometimes users would get confused because an option they were looking
for was not mentioned on a subcommand's man page, and they had not
noticed that the main git-annex man page had a list of common options.
This change lets each subcommand mention the common options, similarly
to how the matching options are handled.

This commit was sponsored by Svenne Krap on Patreon.

comment

add test cases for git/annexed conversions

Make sure that tip keeps working.

I tried to go futher and touch the file and make sure it stayed what it
was converted to, but struggled with some weird and not entirely
reproducable behavior, so kept the tests simple for now.

smudge: check for known annexed inodes before checking annex.largefiles

smudge: Fix a case where an unlocked annexed file that annex.largefiles
does not match could get its unchanged content checked into git, due to git
running the smudge filter unecessarily.

When the file has the same inodecache as an already annexed file,
we can assume that the user is not intending to change how it's stored in
git.

Note that checkunchangedgitfile already handled the inverse case, where the
file was added to git previously. That goes further and actually sha1
hashes the new file and checks if it's the same hash in the index.

It would be possible to generate a key for the file and see if it's the
same as the old key, however that could be considerably more expensive than
sha1 of a small file is, and it is not necessary for the case I have, at
least, where the file is not modified or touched, and so its inode will
match the cache.

git-annex add was changed, when adding a small file, to remove the inode
cache for it. This is necessary to keep the recipe in
doc/tips/largefiles.mdwn for converting from annex to git working.
It also avoids bugs/case_where_using_pathspec_with_git-commit_leaves_s.mdwn
which the earlier try at this change introduced.

comment

comment

Revert "smudge: check for known annexed inodes before checking annex.largefiles"

This reverts commit 424bef6b6f43ba9e09b61faa337fc6c9104e1a1f.

This commit caused other buggy behavior unfortunately.

mention --all on fsck man page, and repurpose todo

comment and close

comment

comment

Added a comment

Added a comment

user error, not a bug

bug report

Added a comment

Added a comment: thanks

reinject: Error out when run on a file that is not annexed

rather than silently skipping it

fileRef: make paths relative and simplified

Fix behavior of several commands, including reinject, addurl, and rmurl
when given an absolute path to an unlocked file, or a relative path that
leaves and re-enters the repository.

To avoid slowing down all the cases where the paths are already ok
with an unncessary call to getCurrentDirectory, put in an optimisation
in relPathCwdToFile. That will probably also speed up other parts of
git-annex by some small amount, but I have not benchmarked.

Note that I did not convert branchFileRef, because it seems likely that
it will be used with a file that is not provided by the user, so is already
in a sane format. This is certainly true for the way git-annex uses it,
though maybe arguable to the extent Git.Ref is a reusable library.

comment

document an important property of relPathCwdToFile

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

patch review

Added a comment: different keys for the same content

remove now redundant -O0

That is now enabled by -f-Production, since commit
cab398c945d6e7a890fa64071d37ddbabaaf6619

Merge remote-tracking branch 'atemu/misc-fixes'

applied patch

fromkey: create directory for pointer files too

fromkey creates leading directories for symbolic links.  Do the same
for pointer files.

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

reinject: silent failure with absolute path to pointer file

patch: fromkey: create directory for pointer files too

Added a comment

update

Added a comment

Added a comment

Added a comment

Added a comment

Cabal: Use -O0 for development builds

Can be configured with `--flags=-production`

Time for full build on my machine: 2m -> 45s

Added a comment

git-commit pathspec change due to 424bef6b6

Added a comment

Added a comment

Added a comment

Added a comment

removed

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

comment

comment

response

comment and todo

remove my comment as someone else already answered it well

response

Added a comment

Added a comment

Command/Multicast: use proper hyphen

GHC was complaining about it possibly being a homoglyph:

Command/Multicast.hs:111:36: error:
     warning: treating Unicode character <U+2212> as identifier character rather than as '-' symbol [-Wunicode-homoglyph]
            -- using a nice prime, namely 2521−1  but the sheer size of this
                                              ^
    |
111 |         -- using a nice prime, namely 2521−1  but the sheer size of this
    |                                    ^
1 warning generated.