Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

libqb 1.0.3 PACKAGE_VERSION is UNKNOW #312

Closed
sasadangelo opened this issue Jul 5, 2018 · 7 comments
Closed

libqb 1.0.3 PACKAGE_VERSION is UNKNOW #312

sasadangelo opened this issue Jul 5, 2018 · 7 comments

Comments

@sasadangelo
Copy link

Hi,

I am trying to use libqb 1.0.3 with pacemaker and corosync to create a cluster and I think I found a bug. I compiled libqb as per doc but in /usr/lib/pkgconfig/libqb.pc in the Version field I found UNKNOWN.
This arise an issue in Pacemaker when I launch ./configure because it check with pkg-config if the library it is greater than 0.13. The check fails reporting that 0.13 was expected but UNKNOWN was found. I simply fixed the issue replacing the correct value in /usr/lib/pkgconfig/libqb.pc, but I think code must be fixed.

@edwintorok
Copy link

edwintorok commented Jul 5, 2018

Have you built libqb from the git tag or from the released tarball. I think the tarball differs from the git tag in that it has a .tarball-version inside it.

I had the same problem and I worked it around by adding this in the .spec file:
echo %{version} >.tarball-version

Perhaps the right way of fixing this would be to enable export-subst in .gitattributes on the .pc file and use git archive substitution formats to get the version number substituted into the file.
That way when you generate a tarball from a git tag using git archive you would get the correct version inside it.

@jnpkrn
Copy link
Contributor

jnpkrn commented Jul 5, 2018

Sorry, but using "native on-the-fly archives" is not a supported
use case at this time. Sadly, GitHub is obnoxious enough that we
cannot turn that "feature" (making for unintended misfeature here;
accidentally, it was discussed recently at the developers mailing
list that patchset review process leaves rather too much to be
desired, too). In turn, we cannot be held accountable for something
we don't want to endorse in the first place, though we can probably
better document the matter.

Your options:

  1. use proper (and stable and signed!) archives (you can recognize
    these for having a static file size that's displayed along the
    respective "Assets" per-release items)

  2. git clone [...]; git checkout $TAG; ./autogen.sh && ./configure [...]
    possibly followed with, e.g., make dist-xz

Closing, though feel free to reopen if the above options don't suite
you for some reason.

@jnpkrn jnpkrn closed this as completed Jul 5, 2018
@sasadangelo
Copy link
Author

Hi,

Thanks for the reply. I downloaded the tar.gz file from git releases page. I understand the issue but I don't fully understand how I can fix it. Can you please elaborate better without assuming I know what you are talking about. If possible I will appreciate if you can share the exact step I need to execute.
Thanks again for support.

@jnpkrn
Copy link
Contributor

jnpkrn commented Jul 5, 2018

hopethishelps

@chrissie-c
Copy link
Contributor

Annoyingly github insists on putting the useless "Source Code" archives in the release list. I really wish they wouldn't.

@sasadangelo
Copy link
Author

Ok it worked. Thanks.

@jnpkrn
Copy link
Contributor

jnpkrn commented Sep 4, 2018

This may change, see #321, but read the commit message carefully.
What I said above still applies, the exception is meant to support CI
and other deliberate "I am perfectly aware what I am doing" cases,
and that's it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants