Skip to content

RPM spec minor fixes#112

Open
mavit wants to merge 9 commits into
LMS-Community:public/9.2from
mavit:rpmlint
Open

RPM spec minor fixes#112
mavit wants to merge 9 commits into
LMS-Community:public/9.2from
mavit:rpmlint

Conversation

@mavit

@mavit mavit commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Here are a bunch of minor changes that fix warning or errors from rpmlint or rpmbuild.

mavit added 8 commits June 18, 2026 16:36
…pt /etc/logrotate.d/lyrionmusicserver”

Signed-off-by: Peter Oliver <git@mavit.org.uk>
```
lyrionmusicserver.noarch: W: obsolete-not-provided SliMP3
lyrionmusicserver.noarch: W: obsolete-not-provided logitechmediaserver
lyrionmusicserver.noarch: W: obsolete-not-provided slimserver
lyrionmusicserver.noarch: W: obsolete-not-provided squeezeboxserver
lyrionmusicserver.noarch: W: obsolete-not-provided squeezecenter
```

Signed-off-by: Peter Oliver <git@mavit.org.uk>
Signed-off-by: Peter Oliver <git@mavit.org.uk>
It’s not referenced anywhere.

Signed-off-by: Peter Oliver <git@mavit.org.uk>
Signed-off-by: Peter Oliver <git@mavit.org.uk>
Fixes the following warning from `rpmlint`:

```
lyrionmusicserver.spec: E: superfluous-%clean-section
```

Signed-off-by: Peter Oliver <git@mavit.org.uk>
Fixes the following `rpmlint` warning:

```
lyrionmusicserver.spec:402: W: macro-in-comment %{shortname}
```

Signed-off-by: Peter Oliver <git@mavit.org.uk>
… RPM

Fixes the following warnings from `rpmbuild`:

```
    absolute symlink: /etc/lyrionmusicserver/server.conf -> /var/lib/lyrionmusicserver/prefs/server.prefs
    absolute symlink: /usr/share/lyrionmusicserver/Plugins -> /var/lib/lyrionmusicserver/Plugins
```

Signed-off-by: Peter Oliver <git@mavit.org.uk>
@michaelherger michaelherger requested a review from JohanSaaw June 24, 2026 04:25
@michaelherger

Copy link
Copy Markdown
Member

Adding @JohanSaaw as another expert eye - thanks!

Fixes the following warning from `rpmbuild`:

```
*** WARNING: ./etc/sysconfig/lyrionmusicserver is executable but has no shebang, removing executable bit
```

Signed-off-by: Peter Oliver <git@mavit.org.uk>
@michaelherger

Copy link
Copy Markdown
Member

Could you please sync this PR with the public/9.2 branch? Thanks!

@JohanSaaw

Copy link
Copy Markdown
Contributor

Hi,

I was away, so I did not check github for some time. If there still is a need, I'll be happy to look at the RPM spec file.

@JohanSaaw JohanSaaw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think all these changes are OK. I noticed that some were due to errors on my side, but there were also issues due to very old code snippets in the spec file.

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.

3 participants