-
Notifications
You must be signed in to change notification settings - Fork 84
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
Remove unneeded HAVE_TRACKER conditional #664
Conversation
I have refactored the Spotlight macro so it meets all the following conditions:
This should help users identify which missing packages need to be installed for Spotlight support if required |
Refactor Spotlight macro Warn if any Spotlight support components are missing at configure time
690e506
to
fa20af7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like that we’re being more verbose now when other spotlight dependencies are missing!
#ifdef HAVE_TRACKER | ||
printf( " tracker manager:\t%s\n", TRACKER_PREFIX "/bin/" TRACKER_MANAGING_COMMAND); | ||
|
||
#ifdef WITH_SPOTLIGHT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn’t dbus strictly speaking used for other functionality too, even if you’re not building spotlight? E.g. afpstats
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well spotted. I'll look into it as just moving bus-daemon path outside of the Spotlight conditional causes compilation failure.
dbus-session.conf is Spotlight only.
|
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional * Refactor Spotlight macro * Warn if any Spotlight support components are missing at configure time
• Remove unneeded HAVE_TRACKER conditional • Refactor Spotlight macro • Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional • Refactor Spotlight macro • Warn if any Spotlight support components are missing at configure time
* Remove unneeded HAVE_TRACKER conditional • Refactor Spotlight macro • Warn if any Spotlight support components are missing at configure time
Refactor Spotlight macro
Warn if any Spotlight support components are missing at configure time