-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
evolution optional dependency #207
Conversation
Hey thanks for your branch! I'm not certain if it makes sense to carry these changes since other parts of Pantheon use EDS as well. What is the value in adding a conditional compile for Evolution here? As it stands, there's no way to review or merge this branch. It has too many unrelated changes. A diff of +1,354 −1,205 is not realistic to review. If you'd like to propose several changes (like reorganizing files, etc), please use multiple branches |
You very quickly learned all the changes) unfortunately in smaller chunks long and more labor-intensive |
These were rejected upstream, sadly elementary/wingpanel-indicator-datetime#207.
These were rejected upstream, sadly elementary/wingpanel-indicator-datetime#207. (cherry picked from commit 7a164bf)
These were rejected upstream, sadly elementary/wingpanel-indicator-datetime#207. (cherry picked from commit 7a164bf)
I really did not like that the evolution-data-server was a mandatory dependency, I tried to make it an optional dependency, for those who do not need it) I think it worked. Several issues were corrected along the way.
issue #206 fix too