dev-util/ragel fails to build with libc++ |
|||
Issue descriptionThis happens when testing the next version of llvm with libc++: https://logs.chromium.org/v/?s=chromeos%2Fbb%2Fchromiumos.tryserver%2Fchromiumos-sdk%2F2836%2F%2B%2Frecipes%2Fsteps%2FInitSDK%2F0%2Fstdout In the host sdk, dev-util/ragel build fails with new error: ragel-6.7-r3: rbxgoto.cpp:661:8: error: invalid operands to binary expression ragel-6.7-r3: ('basic_ostream<char, std::__1::char_traits<char> >' and ragel-6.7-r3: 'std::__1::ostream' (aka 'basic_ostream<char>')) ragel-6.7-r3: " " << rbxGoto(ret, "_out") << "\n" ragel-6.7-r3: ~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~ ragel-6.7-r3: /usr/bin/../include/c++/v1/ostream:220:20: note: candidate function not viable: ragel-6.7-r3: no known conversion from 'std::__1::ostream' (aka 'basic_ostream<char>') ragel-6.7-r3: to 'const void *' for 1st argument; take the address of the argument with ragel-6.7-r3: & ragel-6.7-r3: basic_ostream& operator<<(const void* __p); ragel-6.7-r3: ^ ragel-6.7-r3: /usr/bin/../include/c++/v1/ostream:196:20: note: candidate function not viable: ragel-6.7-r3: no known conversion from 'std::__1::ostream' (aka 'basic_ostream<char>') ragel-6.7-r3: to 'std::__1::basic_ostream<char, std::__1::char_traits<char> > ragel-6.7-r3: &(*)(std::__1::basic_ostream<char, std::__1::char_traits<char> > &)' for ragel-6.7-r3: 1st argument ragel-6.7-r3: basic_ostream& operator<<(basic_ostream& (*__pf)(basic_ostream&)) ragel-6.7-r3: ^ ... The other errors [-Wc++11-narrowing] are from llvm-next, but I suspect that this error is due to libc++.
,
Jan 23 2018
Let's do the upgrade. ragel-6.10 will fix crbug/804996 as well, by pulling in commit 8848bc25c8b943695cf9350332a215e655af2087, which is a backport of b36524576acfccf840842f4a615855e5b51c247e to ragel-6.* branch.
,
Jan 24 2018
,
Jan 26 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/77a4e26efca2693d68481768d6c95ce74cc2e84a commit 77a4e26efca2693d68481768d6c95ce74cc2e84a Author: Rahul Chaudhry <rahulchaudhry@chromium.org> Date: Fri Jan 26 03:23:38 2018 dev-util/ragel: add ragel-6.10 ebuild from gentoo. The ebuild is from: https://gitweb.gentoo.org/repo/gentoo.git/plain/dev-util/ragel/ragel-6.10.ebuild with one change: KEYWORDS="*" BUG= chromium:804996 BUG= chromium:805004 TEST='sudo emerge dev-util/ragel' works. Change-Id: I19defb380df1650e0b385eacd0febb3b92b9370b Reviewed-on: https://chromium-review.googlesource.com/884501 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [add] https://crrev.com/77a4e26efca2693d68481768d6c95ce74cc2e84a/dev-util/ragel/Manifest [add] https://crrev.com/77a4e26efca2693d68481768d6c95ce74cc2e84a/dev-util/ragel/ragel-6.10.ebuild
,
Jan 26 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/403f433a917b2d163e7c9c8dd2835d9b81025639 commit 403f433a917b2d163e7c9c8dd2835d9b81025639 Author: Rahul Chaudhry <rahulchaudhry@chromium.org> Date: Fri Jan 26 03:23:39 2018 dev-util/ragel: move ebuild to portage-stable. Import ragel-6.10 ebuild in portage-stable and remove from chromiumos-overlay. BUG= chromium:804996 BUG= chromium:805004 TEST='sudo emerge dev-util/ragel' works. CQ-DEPEND=CL:884501 Change-Id: I3748f6f1558f6b9b324b68459b9f0588cbd577f4 Reviewed-on: https://chromium-review.googlesource.com/882225 Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org> Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/ragel-6.7-r4.ebuild [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/files/ragel-6.7-narrowing.patch [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/Manifest [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/files/ragel-6.7-config.patch [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/files/ragel-6.7+gcc-4.7.patch [delete] https://crrev.com/437f356d9f785a4b539df7bd5b6fcc26fc6f4e98/dev-util/ragel/files/ragel-6.7-libcxx.patch
,
Jan 29 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by manojgupta@chromium.org
, Jan 23 2018