datalad / git-annex

A non-official clone of git-annex established for DataLad purposes. No PRs will be merged, but could be used to test perspective git-annex patches. Official git-annex repository: https://git.kitenet.net/index.cgi/git-annex.git/
16 stars 3 forks source link

Add new matrix run with custom git annex configuration #133

Closed yarikoptic closed 1 year ago

yarikoptic commented 1 year ago
yarikoptic commented 1 year ago

according to @joeyh in http://git-annex.branchable.com/bugs/reports_file___34__modified__34___whenever_it_is_not/#comment-281950c9f3863fb613877acd170a1d64 it would not work until ... damn -- it is done https://git-annex.branchable.com/todo/specify_gitconfig_for_test_suite/ -- so need to add --test-git-config option to invocation... later

yarikoptic commented 1 year ago

well, there is no indication that it was applied etc, but seems to work. there is only one error and only on crippled-tmp on mac with

/Users/runner/work/_temp/519d70c6-efac-44fd-8a9d-01b70ba8532f.sh: line 57: test_opts[@]: unbound variable

which really does not make much sense why only on that run and why at all since codeblock is

          # Do it after we possibly setup HOME
          git config --global user.email "test@github.land"
          git config --global user.name "GitHub Almighty"
          test_opts=( )

          case "${{ matrix.flavor }}" in
            # For git-annex it causes only few temporary directories to be on the crippled FS,
            # while the main ones produced by git annex test  reside in CWD, for which we use
            # $HOME
            custom-config1)
              test_opts=( --test-git-config annex.stalldetection=1KB/120s )
              ;;
            *)
              ;;
          esac

          cd $HOME
          export | grep -e crippledfs || :

          timeout 1800 git annex test "${test_opts[@]}"

I will try adding :- for empty default but still -- situation doesn't make sense to me.

yarikoptic commented 1 year ago

not clear to me why some were canceled. I will force push a rewrite to see if it comes back to its senses.

yarikoptic commented 1 year ago

windows build is failing but it can't relate, so all is good -- merging