Skip to content

Detail licences - #18

Merged
michaelherger merged 1 commit into
LMS-Community:masterfrom
mavit:licence
Sep 3, 2026
Merged

Detail licences#18
michaelherger merged 1 commit into
LMS-Community:masterfrom
mavit:licence

Conversation

@mavit

@mavit mavit commented Sep 3, 2026

Copy link
Copy Markdown

Comment thread README Outdated
located at
<http://svn.slimdevices.com/repos/slim/7.3/trunk/platforms/readynas/cont
rib/scanner>
http://svn.slimdevices.com/repos/slim/7.3/trunk/platforms/readynas/contrib/scanner

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we update this URL? I'd have to find out whether this survived the transition to Github.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Looks like it did survive: https://github.com/LMS-Community/slimserver-platforms/tree/public/7.3/readynas/contrib/scanner

I don't see why we wouldn't update it (in a separate pull request). I see that the directory is present in every branch from public/7.1 to public/8.5 inclusive, and I guess we'd point to either the earliest or the latest.

@michaelherger michaelherger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot for dealing with these details! I like it when someone takes action.

I have a few minor questions below.

Comment thread README Outdated

FLAC or OggFLAC
INFO
FLAC

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why no OggFLAC?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

It turns out the the README and lib/Audio/Scan.pm have drifted out of sync. I'll drop this commit from this pull request and raise a new one to fix that.

Comment thread README Outdated
@michaelherger

Copy link
Copy Markdown
Member

Thanks!

@michaelherger
michaelherger merged commit bbc0999 into LMS-Community:master Sep 3, 2026
@mavit
mavit deleted the licence branch September 4, 2026 07:55
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.

2 participants