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

build fails if QTDIR= is set and libpng != 1.2.x #95

Closed
GoogleCodeExporter opened this issue Apr 22, 2015 · 4 comments
Closed

build fails if QTDIR= is set and libpng != 1.2.x #95

GoogleCodeExporter opened this issue Apr 22, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Set environment variable QTDIR (e.g. QTDIR=/usr).
2. Run `build_mozc.py gyp'.

What is the expected output? What do you see instead?
Build fails due to the libpng12 is not found (build log attached).

In gui/qt_libraries.gypi, libraries file name is hard coded such as 'libpng12' 
and if installed libpng is not ver. 1.2.x, pkg-config fails.

What version of the product are you using? On what operating system?
Mozc-1.1.773.102
libpng 1.4.8
on Arch Linux (x86_64)

Original issue reported on code.google.com by cabezon....@gmail.com on 3 Aug 2011 at 3:57

Attachments:

@GoogleCodeExporter
Copy link
Author

Thank you for the info.

Could you tell me if you can build and use Mozc by just removing libpng12 from 
the pkg-config list?
If everything is fine, we will just remove libpng12 from the list.

Original comment by yukawa@google.com on 4 Aug 2011 at 7:57

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

> Could you tell me if you can build and use Mozc by just removing libpng12 
from the pkg-config list?
Works fine.
In my distro (Arch), libpng is depended on qt so it should be installed when 
installing mozc in either case (except with --no-qt).

Original comment by cabezon....@gmail.com on 4 Aug 2011 at 12:18

@GoogleCodeExporter
Copy link
Author

Thank you for the info again.
libpng12 is removed from the list in our internal code tree. This change will 
be synced in the next code push.

Original comment by yukawa@google.com on 5 Aug 2011 at 7:28

@GoogleCodeExporter
Copy link
Author

r61 contains this fix.

Please verify if it works for you.

Original comment by yukawa@google.com on 17 Aug 2011 at 2:13

  • Changed state: Fixed

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