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

TravisCI broken for "conky-all" but not "conky" #19

Open
npyl opened this issue Jan 6, 2019 · 11 comments
Open

TravisCI broken for "conky-all" but not "conky" #19

npyl opened this issue Jan 6, 2019 · 11 comments
Assignees

Comments

@npyl
Copy link
Member

npyl commented Jan 6, 2019

There are some problems, like building the toluapp target and installing cairo-xlib with homebrew.

@npyl npyl self-assigned this Jan 6, 2019
@npyl
Copy link
Member Author

npyl commented Jun 28, 2019

Problems right now:

  • (1) build toluapp (checked because it has been amended by installing pre-compiled binaries in the right place before build process needs them.)
  • (2) cairo-conky target doesn't use cairo-xlib thus it fails because cairo is not compiled with xlib support enabled.
  • (3) conky or maybe cairo-imlib2-helper (?) fails to link because there are some undefined references related to imlib2 and cairo. (I haven't figured out how yet.)

@npyl
Copy link
Member Author

npyl commented Jun 28, 2019

For (2), although we include cairo-xlib in PKG_CONFIG_PATH, it doesn't seem like xcode is able to locate it.

Force-linking cairo-xlib with homebrew fixes the problem but we shouldn't rely on this solution because on a user's mac it could break cairo.

@npyl
Copy link
Member Author

npyl commented Jun 28, 2019

For now I am thinking we should finish with (2) and then go to (3).
Lets also try a TravisCI build with (2) only.

@npyl
Copy link
Member Author

npyl commented Jun 28, 2019

Without (2) and (3) TravisCI worked nicely. (after a mind-numbingly long time.)
This could be used as a ground to work on (2) and finally (3).

@npyl npyl changed the title TravisCI script awfully broken TravisCI broken for "conky-all" but not "conky" Jun 28, 2019
@npyl
Copy link
Member Author

npyl commented Jun 30, 2019

More troubleshooting:

A notable question would be:

  • Can I enable just LUA_CAIRO without LUA_IMLIB2 and everything build fine? (case senario 1)

If this works, can I do the opposite:

  • Enable LUA_IMLIB2 without LUA_CAIRO and everything build fine? (case senario 2)

@npyl
Copy link
Member Author

npyl commented Jun 30, 2019

Case senario (1) worked as can be observed in f63bf46 . (I've even taken a stab at enabling LUA_RSVG and it worked.)

Now for case senario (2).

@npyl
Copy link
Member Author

npyl commented Jun 30, 2019

Case senario 2 failed.
That is were we should focus our debugging.

@npyl
Copy link
Member Author

npyl commented Jun 30, 2019

Why is this happening?
Screenshot 2019-07-01 at 02 30 06

@npyl
Copy link
Member Author

npyl commented Jun 30, 2019

Based on this we need to ship a version of imlib2 with xlib support enabled.

@npyl
Copy link
Member Author

npyl commented Jul 2, 2019

The cairo-xlib (falsely called conky-all) branch has been merged to master.
Cairo-xlib problem was unrelated to imlib2 problem.

@npyl
Copy link
Member Author

npyl commented Jul 10, 2019

This can be solved if we explicitly add /usr/local/opt/imlib2-xlib/lib/libImlib2.dylib in conky's linker flags. But why is this required? It shouldn't be...

@npyl npyl added this to the ManageConky v1.0 milestone Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant