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

Library sonames no longer contain patch version (5.5. vs. 5.5.0) #1408

Closed
birdie-github opened this issue Jun 14, 2024 · 6 comments
Closed
Labels
Bug Something isn't working
Milestone

Comments

@birdie-github
Copy link

birdie-github commented Jun 14, 2024

./configure --prefix=/opt/audacious --disable-gtk --enable-qt5
Configuration:

  Install path:                           /opt/audacious

  D-Bus support:                          yes
  GTK support:                            no
  Qt support:                             yes
  libarchive support:                     no
  Valgrind analysis support:              no

make
make DESTDIR=/tmp/audacious install

Produces this:

ls -la /tmp/audacious/opt/audacious/lib
total 536
drwxr-xr-x. 3 root root    180 Jun 14 06:55 .
drwxr-xr-x. 6 root root    120 Jun 14 06:55 ..
lrwxrwxrwx. 1 root root     18 Jun 14 06:55 libaudcore.so -> libaudcore.so.5.5.
lrwxrwxrwx. 1 root root     18 Jun 14 06:55 libaudcore.so.5 -> libaudcore.so.5.5.
-rwxr-xr-x. 1 root root 476576 Jun 14 06:54 libaudcore.so.5.5.
lrwxrwxrwx. 1 root root     17 Jun 14 06:55 libaudtag.so -> libaudtag.so.3.0.
lrwxrwxrwx. 1 root root     17 Jun 14 06:55 libaudtag.so.3 -> libaudtag.so.3.0.
-rwxr-xr-x. 1 root root  67648 Jun 14 06:54 libaudtag.so.3.0.
drwxr-xr-x. 2 root root     60 Jun 14 06:55 pkgconfig

What's libaudcore.so.5.5. and libaudtag.so.3.0.?

@birdie-github birdie-github added the Bug Something isn't working label Jun 14, 2024
@birdie-github birdie-github reopened this Jun 14, 2024
@mschwendt
Copy link

Regression since beta1 introduced with the change to ./m4/buildsys.m4 which adds usage of an undefined ${LIB_PATCH} variable.

@radioactiveman
Copy link
Member

@birdie-github: When creating bug reports, please use the template. Your description is unclear, the expected behavior is totally missing and the header "make install is broken" is simply nonsense.

@birdie-github
Copy link
Author

birdie-github commented Jun 14, 2024

I've given the exact steps to reproduce the issue.

Sorry for bothering you. Not gonna file any bug reports against Audacious from now on. Do whatever you please. I am not a fan of being spat on. This is the first time in my 20+ years of open source work that I've been blamed for absolutely unrelated trifling stuff. Template. For Christ's sake. This is just despicable.

Oh, and it took me three attempts to specify configure flags properly. See, I filed #1377 and though I closed it, the way configure options are provided still makes absolutely no sense. It's 100% illogical and absurd as hell.

@kaniini kaniini reopened this Jun 14, 2024
@kaniini
Copy link
Member

kaniini commented Jun 14, 2024

@radioactiveman I don't see what the problem is with the bug report. The SONAMEs are being set wrong and this is a regression.

kaniini added a commit that referenced this issue Jun 14, 2024
Set LIB_PATCH in makefiles for correct SONAMEs. Closes: #1408
@mschwendt
Copy link

@birdie-github As a work-around, you can run "make install LIB_PATCH=0"

@radioactiveman radioactiveman changed the title make install is broken for 4.4 Library sonames no longer contain patch version (5.5. vs. 5.5.0) Jun 14, 2024
@radioactiveman
Copy link
Member

radioactiveman commented Jun 14, 2024

@radioactiveman I don't see what the problem is with the bug report. The SONAMEs are being set wrong and this is a regression.

And this could have been written in the bug report to make it clear. Simple as that. Same for the header. "make install is broken" is no proper error description and gives no insights if included in a changelog.

The issue is fixed however, so let's forget it. And using Meson is the preferred way to build Audacious anyway (as mentioned in our release notes). 😉

@radioactiveman radioactiveman added this to the 4.4.1 milestone Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants