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

Pass the name defined inside the .pc file to the Meson Build System's dependency() #284

Closed
wants to merge 1 commit into from

Conversation

ikitayama
Copy link

Seen on Ubuntu 24.10. The systemd package in the corresponding .pc file has a valid name "libsystemd".
Pass the proper name that meson expects in dependency(), otherwise meson setup build step fails complaining the (canonical) package name is not found.

dependencu(), otherwise meson setup build fails complaining the (canonical
package name is not found.
@ikitayama
Copy link
Author

Could you @AlisonSchofield have a look if you have time? Looks like haven't changed for a while though.

@stellarhopper
Copy link
Member

See comment here: #285

@marc-hb
Copy link

marc-hb commented Apr 4, 2025

I don't know why ndctl has this dependency but I can see that nothing seems to have changed in 24.10: libsystemd0 and systemd have always been two different packages for a very long time:

Same in Debian, no recent change:

Can you please file a new bug with:

  • The error message that you get when you locally and temporarily revert this change
  • The content of the each .pc file.
  • The versions of everything in use

@marc-hb
Copy link

marc-hb commented Apr 4, 2025

Also, would you mind testing with Ubuntu LTS 24.04? Non-LTS versions are short lived and not very well tested.

@marc-hb
Copy link

marc-hb commented Apr 5, 2025

Can you please file a new bug with...

Wait, could someone with enough permissions just transfer #285 here? I tried but I'm not allowed.

@stellarhopper
Copy link
Member

Can you please file a new bug with...

Wait, could someone with enough permissions just transfer #285 here? I tried but I'm not allowed.

Done!

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