Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wrong path being used on aur sync. #363

Closed
rafasc opened this issue Apr 18, 2018 · 2 comments
Closed

wrong path being used on aur sync. #363

rafasc opened this issue Apr 18, 2018 · 2 comments

Comments

@rafasc
Copy link
Collaborator

rafasc commented Apr 18, 2018

Recent changes to aur-fetch{,-git} are interacting badly with 03a1747

$ aur sync aurutils-git --no-ver 
  -> Using [aur] repository
fatal: not a git repository: '/home/rafasc/.cache/aurutils/sync//tmp/sync.Ya14PEiF/patch/aurutils-git.patch/.git'

b04ee03 seems to have dropped the first argument.

-if [[ -d $log_dir ]]; then
-    merge() { merge_across_fs "$1" "$log_dir/$1".patch; }
-elif [[ $log_dir == /dev/stdout ]]; then
-    merge() { merge_across_fs "$1"; }
-else
-    printf -- >&2 '%s: %q: not a directory or /dev/stdout\n' "$argv0" "$log_dir"
-    exit 1
-fi
+# set filters
+case $log in
+    directory) merge() { merge_across_fs "$log_dir/$1".patch; } ;;
+    stdout)    merge() { merge_across_fs; } ;;
+esac

Didn't have much time to investigate this further.

@AladW
Copy link
Member

AladW commented Apr 18, 2018

I think I should stop commiting if I can't actually test things... thanks for keeping track of these changes.

@rafasc
Copy link
Collaborator Author

rafasc commented Apr 18, 2018

You're welcome.
Tested on the same scenario and seems to be working.
Thanks for the quick fix.
closed via ebc3671

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants