Don't try to link to AppStream-Qt if building with APPSTREAM=OFF
Details
Details
- Reviewers
dantti - Commits
- R549:29df2bd0bc9f: Fix build without AppStream
Builds and runs fine without AppStream
Diff Detail
Diff Detail
- Repository
- R549 Apper
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Thanks, I've been thinking into making this dependency mandatory, could you please tell me why would you like to build without AppStream or are you just fixing the bug? thanks
Comment Actions
I have no interest - in fact, I'm building it with AppStream on Arch. I'm just fixing it since it is currently optional (and disabled by default).
Comment Actions
OK, thanks I'll probably make it mandatory for 1.1.0 as it's a simple dep and the less ifdefs the better