7081 2014-12-12 23:14:53 +0000 glibc test suite fails to build some programs 2015-04-03 16:00:48 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core unspecified x86 Multiple RESOLVED FIXED Medium normal 1.8 M4 1 juro.bystricky juro.bystricky meta.mr.watcher meta.watcher --- --- No (bug/feature does not impact docs) oldest_to_newest 47549 0 2297 juro.bystricky 2014-12-12 23:14:53 +0000 Created attachment 2297 Patch to detect g++ system header files The following programs from the glibc test suite fail to build: /elf/tst-unique3cc /elf/tst-unique4cc /elf/tst-unique3lib.cc /elf/tst-unique3lib2.cc in all cases the error is: fatal error: cstdio: No such file or directory #include <cstdio> The reason is the c++ header files were not found by the g++. g++ was invoked with -nostdinc flag, and in that case the path to system include files must be specified explicitly. The glibc "configure" contains some code to detect the g++ system headers file location, but it does not work correctly. For example, it uses g++ itself to query for the location of the header files, but g++ may not be present at all at the cofigure time. Also, the algorithm used to detect the header file location does not seem to work with g++ anymore. Even if g++ is present the resulting config.log shows: CXX_SYSINCLUDES='' To fix the problem, the location of the include headers should be determined before running the test suite, not before building glibc. The proposed solution modifies /meta/recipes-core/glibc/glibc-testing.inc. The new code cracks "configparms" to get the actual CXX and sysroot used for the glibc test suite, then uses CXX to determine CXX version and CXX machine. Based on the results, it composes a new value for c++-sysincludes and appends "configparms" with the new value. The header locations are assumed to be: sysroot/usr/include/c++/<version> sysroot/usr/include/c++/<version>/<machine> This was so far successfully tested on qemux86-64. 47550 1 juro.bystricky 2014-12-13 02:59:18 +0000 Also tested fine on qemumips. 47570 2 juro.bystricky 2014-12-15 19:04:17 +0000 The fix I described previously (but a bit more paranoid) is being tested. The fix is quite safe, it does not affect glibc build at all, it only modifies the testing script. If there are no c++-sysincludes specified in config.make, then the location of the headers is determined by querying the CXX compiler as specified in configparms. If the corresponding folders do exist in the <sysroot> (as specified by configparms as well) then tew value of c++-sysincludes is written to the configparms as well, overriding the previous empty definition. If the config.make contains a value for c++-sysincludes, then it is assumed to be correct and the testing is done as before without any changes. 49750 3 juro.bystricky 2015-03-25 15:07:00 +0000 The problem remains also with glibc 2.21. Posted a patch that fixes the problem. 49751 4 2453 juro.bystricky 2015-03-25 15:10:03 +0000 Created attachment 2453 Patch to determine location of g++ system header files 50079 5 juro.bystricky 2015-04-03 16:00:48 +0000 Merged to trunk commit: 47dedbcb52702539b124ecf43cf53e4c3b7b3b33 2297 2014-12-12 23:14:53 +0000 2014-12-12 23:14:53 +0000 Patch to detect g++ system header files glibc-testing.diff text/plain 1394 juro.bystricky ZGlmZiAtLWdpdCBhL21ldGEvcmVjaXBlcy1jb3JlL2dsaWJjL2dsaWJjLXRlc3RpbmcuaW5jIGIv bWV0YS9yZWNpcGVzLWNvcmUvZ2xpYmMvZ2xpYmMtdGVzdGluZy5pbmMKaW5kZXggZWMxNmZlMS4u MmU4ZWUwMiAxMDA2NDQKLS0tIGEvbWV0YS9yZWNpcGVzLWNvcmUvZ2xpYmMvZ2xpYmMtdGVzdGlu Zy5pbmMKKysrIGIvbWV0YS9yZWNpcGVzLWNvcmUvZ2xpYmMvZ2xpYmMtdGVzdGluZy5pbmMKQEAg LTY1LDkgKzY1LDIyIEBAIHRoZW4KIGZpCiAJZWNobyAiIyB3ZSB0ZXN0IHVzaW5nIGNyb3NzIGNv bXBpbGVyIGZyb20gcmVhbCBzeXNyb290IHRoZXJlZm9yZSBvdmVycmlkZSB0aGUiID4gJHtCfS9j b25maWdwYXJtcwogCWVjaG8gIiMgZGVmaW5pdGlvbnMgdGhhdCBjb21lIGZyb20gJHtCfS9jb25m aWcubWFrZSIgPj4gJHtCfS9jb25maWdwYXJtcwotCiAJZmdyZXAgdGNib290c3RyYXAgJHtCfS9j b25maWcubWFrZSA+ICR7Qn0vY29uZmlncGFybXMKIAlzZWQgLWkgLWUgInMvXC10Y2Jvb3RzdHJh cC8vZyIgJHtCfS9jb25maWdwYXJtcworCisjIGcrKyB1c2VzIGZsYWcgLW5vc3RkaW5jLCBzbyB0 aGUgbG9jYXRpb25zIG9mIHN5c3RlbSBpbmNsdWRlIGhlYWRlcnMgbXVzdCBiZSBleHBsaWNpdGx5 IHNwZWNpZmllZAorIyBUaGUgbG9jYXRpb25zIGFyZSBhc3N1bWVkIHRvIGJlOgorIyAgICBzeXNy b290L3Vzci9pbmNsdWRlL2MrKy88ZysrIHZlcnNpb24+CisjICAgIHN5c3Jvb3QvdXNyL2luY2x1 ZGUvYysrLzxnKysgdmVyc2lvbj4vPG1hY2hpbmU+CisKK3N5c3Jvb3Q9XGBjYXQgJHtCfS9jb25m aWdwYXJtcyB8IHNlZCAtbiAiL0NYWC9wIiB8IHNlZCAtZSAicy9eLiotLXN5c3Jvb3Q9Ly8iXGAK K2N4eD1cYGNhdCAke0J9L2NvbmZpZ3Bhcm1zIHwgYXdrICdcJDEgfiAvXkNYWC8nIHwgYXdrIC1G Ij0iICd7cHJpbnQgXCQyfScgfCBhd2sgJ3twcmludCBcJDF9J1xgCitjeHhtYWNoaW5lPVxgXCRj eHggLWR1bXBtYWNoaW5lXGAKK2N4eHZlcnNpb249XGBcJGN4eCAtZHVtcHZlcnNpb25cYAorCisj IHBhc3MgdGhlIG5ldyB2YWx1ZSBvZiBjKystc3lzaW5jbHVkZXMgdmlhIGNvbmZpZ3Bhcm1zCitl Y2hvICJjKystc3lzaW5jbHVkZXMgPSAtaXN5c3RlbSBcJHN5c3Jvb3QvdXNyL2luY2x1ZGUvYysr L1wkY3h4dmVyc2lvbiAtaXN5c3RlbSBcJHN5c3Jvb3QvdXNyL2luY2x1ZGUvYysrL1wkY3h4dmVy c2lvbi9cJGN4eG1hY2hpbmUiID4+ICR7Qn0vY29uZmlncGFybXMKKwogd3JhcHBlcj0iJHtTfS9z Y3JpcHRzL2Nyb3NzLXRlc3Qtc3NoLnNoIFwkdGFyZ2V0IgogbG9jYWxlZGVmPSIke1NUQUdJTkdf QklORElSX05BVElWRX0vY3Jvc3MtbG9jYWxlZGVmIC0tbGl0dGxlLWVuZGlhbiAtLXVpbnQzMi1h bGlnbj00IgogbWFrZSB0ZXN0cy1jbGVhbgo= 2453 2015-03-25 15:10:03 +0000 2015-03-25 15:10:03 +0000 Patch to determine location of g++ system header files file_7081.txt text/plain 1748 juro.bystricky ZGlmZiAtLWdpdCBhL21ldGEvcmVjaXBlcy1jb3JlL2dsaWJjL2dsaWJjLXRlc3RpbmcuaW5jIGIv bWV0YS9yZWNpcGVzLWNvcmUvZ2xpYmMvZ2xpYmMtdGVzdGluZy5pbmMNCmluZGV4IGVjMTZmZTEu LmZkMWFmMzkgMTAwNjQ0DQotLS0gYS9tZXRhL3JlY2lwZXMtY29yZS9nbGliYy9nbGliYy10ZXN0 aW5nLmluYw0KKysrIGIvbWV0YS9yZWNpcGVzLWNvcmUvZ2xpYmMvZ2xpYmMtdGVzdGluZy5pbmMN CkBAIC02NSw5ICs2NSwyNiBAQCB0aGVuDQogZmkNCiAgICAgICAgZWNobyAiIyB3ZSB0ZXN0IHVz aW5nIGNyb3NzIGNvbXBpbGVyIGZyb20gcmVhbCBzeXNyb290IHRoZXJlZm9yZSBvdmVycmlkZSB0 aGUiID4gJHtCfS9jb25maWdwYXJtcw0KICAgICAgICBlY2hvICIjIGRlZmluaXRpb25zIHRoYXQg Y29tZSBmcm9tICR7Qn0vY29uZmlnLm1ha2UiID4+ICR7Qn0vY29uZmlncGFybXMNCi0NCiAgICAg ICAgZmdyZXAgdGNib290c3RyYXAgJHtCfS9jb25maWcubWFrZSA+ICR7Qn0vY29uZmlncGFybXMN CiAgICAgICAgc2VkIC1pIC1lICJzL1wtdGNib290c3RyYXAvL2ciICR7Qn0vY29uZmlncGFybXMN CisNCisjIGcrKyB1c2VzIGZsYWcgLW5vc3RkaW5jLCBzbyB0aGUgbG9jYXRpb25zIG9mIHN5c3Rl bSBpbmNsdWRlIGhlYWRlcnMgbXVzdCBiZSBleHBsaWNpdGx5IHNwZWNpZmllZA0KKyMgSWYgdGhl IGxvY2F0aW9ucyBhcmUgbm90IGFscmVhZHkgc3BlY2lmaWVkIGluIGNvbmZpZy5tYWtlLCB0aGVu IHdlIHByb3ZpZGUgdGhlIGZvbGxvd2luZyBsb2NhdGlvbnM6DQorIyAgICA8c3lzcm9vdD4vdXNy L2luY2x1ZGUvYysrLzxnKysgdmVyc2lvbj4NCisjICAgIDxzeXNyb290Pi91c3IvaW5jbHVkZS9j KysvPGcrKyB2ZXJzaW9uPi88bWFjaGluZT4NCisNCitjeHhpbmNsdWRlcz1cYGNhdCAke0J9L2Nv bmZpZy5tYWtlIHwgZ2F3ayAnXCQxID09ICJjKystc3lzaW5jbHVkZXMiJyB8IGdhd2sgLUYiPSIg J3twcmludCBcJDJ9JyB8IHNlZCAicy9bIFx0XVw/Ly9nIlxgDQorDQoraWYgWyAteiAiXCRjeHhp bmNsdWRlcyIgXTsgdGhlbg0KKyAgc3lzcm9vdD1cYGNhdCAke0J9L2NvbmZpZ3Bhcm1zIHwgc2Vk IC1uICIvQ1hYL3AiIHwgc2VkIC1lICJzL14uKi0tc3lzcm9vdD0vLyJcYA0KKyAgY3h4PVxgY2F0 ICR7Qn0vY29uZmlncGFybXMgfCBnYXdrICdcJDEgfiAvXkNYWC8nIHwgZ2F3ayAtRiI9IiAne3By aW50IFwkMn0nIHwgZ2F3ayAne3ByaW50IFwkMX0nXGANCisgIGN4eG1hY2hpbmU9XGBcJGN4eCAt ZHVtcG1hY2hpbmVcYA0KKyAgY3h4dmVyc2lvbj1cYFwkY3h4IC1kdW1wdmVyc2lvblxgDQorICAj IHBhc3MgdGhlIG5ldyB2YWx1ZSBvZiBjKystc3lzaW5jbHVkZXMgdmlhIGNvbmZpZ3Bhcm1zDQor ICBlY2hvICIjIGMrKy1zeXNpbmNsdWRlcyBhZGRlZDoiID4+ICR7Qn0vY29uZmlncGFybXMNCisg IGVjaG8gImMrKy1zeXNpbmNsdWRlcyA9IC1pc3lzdGVtIFwkc3lzcm9vdC91c3IvaW5jbHVkZS9j KysvXCRjeHh2ZXJzaW9uIC1pc3lzdGVtIFwkc3lzcm9vdC91c3IvaW5jbHVkZS9jKysvXCRjeHh2 ZXJzaW9uL1wkY3h4bWFjaGluZSIgPj4gJHtCfS9jb25maWdwYXJtcw0KK2ZpDQorDQogd3JhcHBl cj0iJHtTfS9zY3JpcHRzL2Nyb3NzLXRlc3Qtc3NoLnNoIFwkdGFyZ2V0Ig0KIGxvY2FsZWRlZj0i JHtTVEFHSU5HX0JJTkRJUl9OQVRJVkV9L2Nyb3NzLWxvY2FsZWRlZiAtLWxpdHRsZS1lbmRpYW4g LS11aW50MzItYWxpZ249NCINCiBtYWtlIHRlc3RzLWNsZWFuDQo=