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

libftdi: fix packagekit when useing cmake #690

Closed
wants to merge 1 commit into from

Conversation

lynxis
Copy link
Member

@lynxis lynxis commented Dec 22, 2014

cmake is missing the variable VERSION. VERSION is used in libftdi.pc.in. When other packages
using a min version they fail detecting libftdi.
[Please backport this to BB]

cmake is missing the variable VERSION. VERSION is used in libftdi.pc.in. When other packages
using a min version they fail detecting libftdi.

Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
@Noltari
Copy link
Member

Noltari commented Dec 22, 2014

Which packages are failing?
Do you know if this happens for libftdi1 too?

@Noltari Noltari self-assigned this Dec 22, 2014
@lynxis
Copy link
Member Author

lynxis commented Dec 22, 2014

OLA is failing to build libftdi module (open light architecture)
libftdi1 is not affected. I also sent the patch to the upstream mailinglist.
Take a look into libftdi.pc.in

Version: @Version@

@lynxis
Copy link
Member Author

lynxis commented Dec 27, 2014

Thomas Jarosch applied my patch on branch libftdi-0.x
git rev c8e975e613826577e4af54a14c76dcecac29b33d
git://developer.intra2net.com/libftdi

@Noltari
Copy link
Member

Noltari commented Dec 27, 2014

Noltari added a commit that referenced this pull request Jan 1, 2015
Based on upstream commit by Thomas Jarosch based on Alexander's patch.
More info: #690

Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
@Noltari
Copy link
Member

Noltari commented Jan 1, 2015

Fixed with d54b357

@Noltari Noltari closed this Jan 1, 2015
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.

None yet

2 participants