KIT-ILIAS-downloader icon indicating copy to clipboard operation
KIT-ILIAS-downloader copied to clipboard

Make the sync-url flag respect iliasignore

Open funnym0nk3y opened this issue 2 years ago • 4 comments

It seems to me that the .iliasignore file is not respected when using the --sync-url flag. I use the flag for syncing a course before the lecture on my notebook instead of crawling all other courses. Thus, I'd like it to respect excluded folders from the .iliasignore.

funnym0nk3y avatar Nov 22 '22 12:11 funnym0nk3y

The .iliasignore file is not ignored. Do you have an additional .iliasignore file for the course?

For example, if you usually download all courses into ./ILIAS and want to download a specific course in ./ILIAS/Analysis I, the downloader will certainly read ./ILIAS/Analysis I/.iliasignore. Did you expect the downloader to consider ./ILIAS/.iliasignore?

FliegendeWurst avatar Nov 22 '22 18:11 FliegendeWurst

Exactly, I did expect it to use ./ILIAS/.iliasignore, even though I invoked the command for a specific course folder.

funnym0nk3y avatar Nov 22 '22 20:11 funnym0nk3y

Ah, got it. Your suggestion is implemented in the latest commit. Feel free to install it using cargo install --git 'https://github.com/FliegendeWurst/KIT-ILIAS-downloader. I'll probably release a new version soon.

FliegendeWurst avatar Nov 24 '22 11:11 FliegendeWurst

Thank you very much! I'll try it soon.

funnym0nk3y avatar Nov 24 '22 12:11 funnym0nk3y