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

Hide library folders from scanning with a sentinel file. #429

Merged
merged 4 commits into from Oct 1, 2020
Merged

Hide library folders from scanning with a sentinel file. #429

merged 4 commits into from Oct 1, 2020

Conversation

sciurius
Copy link
Contributor

I have a vast collection of music, mostly ripped CDs, but not all of them are eligible to be included in the LMS library. So many years ago (2007 to be precise) I added a small tweak to function folderFilter (module Slim::Utils::Misc) to stop scanning a folder when a sentinel file .slimskip is present.

Since this may be interesting for other users as well (and because I always forget to re-apply this tweak after an upgrade) I hereby present it as a formal pull request.

Copy link
Member

@michaelherger michaelherger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've had this request before...

Slim/Utils/Misc.pm Outdated Show resolved Hide resolved
Copy link
Member

@michaelherger michaelherger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea of making this not only optional, but configurable, too. But please move the pref from basic to My Music, as it's a rather advanced feature.

Slim/Utils/Misc.pm Outdated Show resolved Hide resolved
@@ -89,6 +89,12 @@
[% END %]
[% END %]

[% WRAPPER settingSection %]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please move this out of basic settings to My Music instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved $skipsentinel into folderFilter() sub.

I'm not sure if My Music is the right place for the skipsentinel setting. My Music deals with music (meta)data content. Skipsentinel operates on the scanning level, not the content level.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a better place than Basic Settings, as it's an advanced setting. Please :-).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, done.

@mherger mherger merged commit aa87f35 into LMS-Community:public/8.0 Oct 1, 2020
@mherger
Copy link
Contributor

mherger commented Oct 1, 2020

Thanks!

@sciurius sciurius deleted the slimskip branch October 4, 2020 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants