From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20688 invoked by alias); 27 Oct 2002 03:33:43 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 20621 invoked from network); 27 Oct 2002 03:33:42 -0000 Received: from unknown (HELO dair.pair.com) (209.68.1.49) by sources.redhat.com with SMTP; 27 Oct 2002 03:33:42 -0000 Received: (qmail 28370 invoked by uid 20157); 27 Oct 2002 03:33:41 -0000 Received: from localhost (sendmail-bs@127.0.0.1) by localhost with SMTP; 27 Oct 2002 03:33:41 -0000 Date: Sat, 26 Oct 2002 20:33:00 -0000 From: Hans-Peter Nilsson X-X-Sender: hp@dair.pair.com To: bug-dejagnu@gnu.org cc: gdb-patches@sources.redhat.com Subject: [RFA:] DejaGNU patch: Change -I to -isystem for testsuite. Message-ID: MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="0-845395881-1035689621=:28032" X-SW-Source: 2002-10/txt/msg00571.txt.bz2 This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. Send mail to mime@docserver.cac.washington.edu for more info. --0-845395881-1035689621=:28032 Content-Type: TEXT/PLAIN; charset=US-ASCII Content-length: 1640 This patch fixes, for uninstalled newlib targets in a unified tree, the listed spurious GCC testsuite excess-errors. They occur because the supposedly-to-be-installed newlib limits.h uses #include_next. That directive makes gcc emit a warning when used in a non-system-header context. While that can be "fixed" for non-installed headers by adding "#pragma GCC system_header", it shouldn't be needed; it'd be better if gcc treated it as a system header while running the test-suite. Done by changing -I to -isystem for include directives for the test-suite. The -isystem option has been there since at least gcc-2.7.2 so it should be safe for all uses. This patch is against the sources.redhat.com unified-tree repository but the patch applied fine (with offset) to the savannah.gnu.org DejaGNU repository. FAIL: gcc.dg/c90-intconst-1.c (test for excess errors) FAIL: gcc.dg/c99-intconst-1.c (test for excess errors) FAIL: g++.dg/abi/bitfield4.C (test for excess errors) FAIL: g++.benjamin/bool01.C (test for excess errors) FAIL: g++.benjamin/p12475.C (test for excess errors) FAIL: g++.jason/rfg10.C (test for excess errors) BTW, does anybody have an idea why gcc/testsuite/site.exp is completely ineffective? Changing s/-I/-isystem/g in the site.exp in gcc/Makefile.in has no effect; in fact it doesn't seem to be used at all. Ok to commit to sources.redhat.com/dejagnu? dejagnu: * lib/libgloss.exp (newlib_include_flags): Use -isystem, not -I. (libio_include_flags, g++_include_flags, libstdc++_include_flags, winsup_include_flags): Ditto. * doc/user.sgml (Local Config File): Use -isystem, not -I, in example. brgds, H-P --0-845395881-1035689621=:28032 Content-Type: TEXT/PLAIN; charset=US-ASCII; name="is1.txt" Content-Transfer-Encoding: BASE64 Content-ID: Content-Description: Content-Disposition: attachment; filename="is1.txt" Content-length: 6170 SW5kZXg6IGRvYy91c2VyLnNnbWwNCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0N ClJDUyBmaWxlOiAvY3ZzL3NyYy9zcmMvZGVqYWdudS9kb2MvdXNlci5zZ21s LHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS4yDQpkaWZmIC1wIC1jIC11IC1w IC1yMS4yIHVzZXIuc2dtbA0KLS0tIGRvYy91c2VyLnNnbWwJMjEgQXByIDIw MDIgMDg6NDc6MDMgLTAwMDAJMS4yDQorKysgZG9jL3VzZXIuc2dtbAkyNyBP Y3QgMjAwMiAwMjozNTo0MCAtMDAwMA0KQEAgLTg3OSw3ICs4NzksNyBAQA0K ICAgICAgIHNldCB0YXJnZXRfdHJpcGxldCBpNTg2LXBjLWxpbnV4LWdudWxp YmMxDQogICAgICAgc2V0IHRhcmdldF9hbGlhcyBpNTg2LXBjLWxpbnV4LWdu dWxpYmMxDQogICAgICAgc2V0IENGTEFHUyAiIg0KLSAgICAgIHNldCBDWFhG TEFHUyAiLUkvYnVpbGQvZGV2by1idWlsZHMvaTU4Ni1wYy1saW51eC1nbnVs aWJjMS9nY2MvLi4vbGliaW8gLUkkc3JjZGlyLy4uL2xpYmcrKy9zcmMgLUkk c3JjZGlyLy4uL2xpYmlvIC1JJHNyY2Rpci8uLi9saWJzdGRjKysgLUkkc3Jj ZGlyLy4uL2xpYnN0ZGMrKy9zdGwgLUwvYnVpbGQvZGV2by1idWlsZHMvaTU4 Ni1wYy1saW51eC1nbnVsaWJjMS9nY2MvLi4vbGliZysrIC1ML2J1aWxkL2Rl dm8tYnVpbGRzL2k1ODYtcGMtbGludXgtZ251bGliYzEvZ2NjLy4uL2xpYnN0 ZGMrKyINCisgICAgICBzZXQgQ1hYRkxBR1MgIi1pc3lzdGVtIC9idWlsZC9k ZXZvLWJ1aWxkcy9pNTg2LXBjLWxpbnV4LWdudWxpYmMxL2djYy8uLi9saWJp byAtaXN5c3RlbSAkc3JjZGlyLy4uL2xpYmcrKy9zcmMgLWlzeXN0ZW0gJHNy Y2Rpci8uLi9saWJpbyAtaXN5c3RlbSAkc3JjZGlyLy4uL2xpYnN0ZGMrKyAt aXN5c3RlbSAkc3JjZGlyLy4uL2xpYnN0ZGMrKy9zdGwgLUwvYnVpbGQvZGV2 by1idWlsZHMvaTU4Ni1wYy1saW51eC1nbnVsaWJjMS9nY2MvLi4vbGliZysr IC1ML2J1aWxkL2Rldm8tYnVpbGRzL2k1ODYtcGMtbGludXgtZ251bGliYzEv Z2NjLy4uL2xpYnN0ZGMrKyINCiAgICAgICBhcHBlbmQgTERGTEFHUyAiIC1M L2J1aWxkL2Rldm8tYnVpbGRzL2k1ODYtcGMtbGludXgtZ251bGliYzEvZ2Nj Ly4uL2xkIg0KICAgICAgIHNldCB0bXBkaXIgL2J1aWxkL2Rldm8tYnVpbGRz L2k1ODYtcGMtbGludXgtZ251bGliYzEvZ2NjL3Rlc3RzdWl0ZQ0KICAgICAg IHNldCBzcmNkaXIgIiR7c3JjZGlyfS90ZXN0c3VpdGUiDQpJbmRleDogbGli L2xpYmdsb3NzLmV4cA0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KUkNTIGZp bGU6IC9jdnMvc3JjL3NyYy9kZWphZ251L2xpYi9saWJnbG9zcy5leHAsdg0K cmV0cmlldmluZyByZXZpc2lvbiAxLjkNCmRpZmYgLXAgLWMgLXUgLXAgLXIx LjkgbGliZ2xvc3MuZXhwDQotLS0gbGliL2xpYmdsb3NzLmV4cAkyNiBTZXAg MjAwMiAwOTo1MDowMyAtMDAwMAkxLjkNCisrKyBsaWIvbGliZ2xvc3MuZXhw CTI3IE9jdCAyMDAyIDAyOjM1OjQwIC0wMDAwDQpAQCAtMTU5LDcgKzE1OSw3 IEBAIHByb2MgbmV3bGliX2luY2x1ZGVfZmxhZ3MgeyBhcmdzIH0gew0KIAlp ZiB7ICR7bmV3bGliX2Rpcn0gIT0gIiIgfSB7DQogCSAgICBzZXQgbmV3bGli X2RpciBbZmlsZSBkaXJuYW1lICR7bmV3bGliX2Rpcn1dDQogCX0NCi0JcmV0 dXJuICIgLUkkZ2NjcGF0aC9uZXdsaWIvdGFyZy1pbmNsdWRlIC1JJHtuZXds aWJfZGlyfSINCisJcmV0dXJuICIgLWlzeXN0ZW0gJGdjY3BhdGgvbmV3bGli L3RhcmctaW5jbHVkZSAtaXN5c3RlbSAke25ld2xpYl9kaXJ9Ig0KICAgICB9 IGVsc2Ugew0KIAl2ZXJib3NlICJObyBuZXdsaWIgc3VwcG9ydCBmb3IgdGhp cyB0YXJnZXQiDQogICAgIH0NCkBAIC0xOTEsNyArMTkxLDcgQEAgcHJvYyBs aWJpb19pbmNsdWRlX2ZsYWdzIHsgYXJncyB9IHsNCiAgICAgaWYgeyAkbGli aW9fYmluX2RpciAhPSAiIiAmJiAkbGliaW9fc3JjX2RpciAhPSAiIiB9IHsN CiAJc2V0IGxpYmlvX3NyY19kaXIgW2ZpbGUgZGlybmFtZSAke2xpYmlvX3Ny Y19kaXJ9XQ0KIAlzZXQgbGliaW9fYmluX2RpciBbZmlsZSBkaXJuYW1lICR7 bGliaW9fYmluX2Rpcn1dOw0KLQlyZXR1cm4gIiAtSSR7bGliaW9fc3JjX2Rp cn0gLUkke2xpYmlvX2Jpbl9kaXJ9Ig0KKwlyZXR1cm4gIiAtaXN5c3RlbSAk e2xpYmlvX3NyY19kaXJ9IC1pc3lzdGVtICR7bGliaW9fYmluX2Rpcn0iDQog ICAgIH0gZWxzZSB7DQogCXJldHVybiAiIg0KICAgICB9DQpAQCAtMjI2LDIy ICsyMjYsMjMgQEAgcHJvYyBnKytfaW5jbHVkZV9mbGFncyB7IGFyZ3MgfSB7 DQogDQogICAgIHNldCBkaXIgW2xvb2tmb3JfZmlsZSAke3NyY2Rpcn0gbGli ZysrXQ0KICAgICBpZiB7ICR7ZGlyfSAhPSAiIiB9IHsNCi0JYXBwZW5kIGZs YWdzICIgLUkke2Rpcn0gLUkke2Rpcn0vc3JjIg0KKwlhcHBlbmQgZmxhZ3Mg IiAtaXN5c3RlbSAke2Rpcn0gLWlzeXN0ZW0gJHtkaXJ9L3NyYyINCiAgICAg fQ0KIA0KICAgICBzZXQgZGlyIFtsb29rZm9yX2ZpbGUgJHtzcmNkaXJ9IGxp YnN0ZGMrKy12M10NCiAgICAgaWYgeyAke2Rpcn0gIT0gIiIgfSB7DQotCWFw cGVuZCBmbGFncyAiIC1JJHtkaXJ9L2luY2x1ZGUgLUkke2Rpcn0vaW5jbHVk ZS9zdGQgLUkke2Rpcn0vaW5jbHVkZS9jX3N0ZCAtSSR7ZGlyfS9saWJzdXBj KysiDQorCWFwcGVuZCBmbGFncyAiIC1pc3lzdGVtICR7ZGlyfS9pbmNsdWRl IC1pc3lzdGVtICR7ZGlyfS9pbmNsdWRlL3N0ZCINCisJYXBwZW5kIGZsYWdz ICIgLWlzeXN0ZW0gJHtkaXJ9L2luY2x1ZGUvY19zdGQgLWlzeXN0ZW0gJHtk aXJ9L2xpYnN1cGMrKyINCiAgICAgfQ0KIA0KICAgICBzZXQgZGlyIFtsb29r Zm9yX2ZpbGUgJHtnY2NwYXRofSBsaWJzdGRjKystdjNdDQogICAgIGlmIHsg JHtkaXJ9ICE9ICIiIH0gew0KLQlhcHBlbmQgZmxhZ3MgIiAtSSR7ZGlyfS9p bmNsdWRlIC1JJHtkaXJ9L2luY2x1ZGUvJHt0YXJnZXRfYWxpYXN9Ig0KKwlh cHBlbmQgZmxhZ3MgIiAtaXN5c3RlbSAke2Rpcn0vaW5jbHVkZSAtaXN5c3Rl bSAke2Rpcn0vaW5jbHVkZS8ke3RhcmdldF9hbGlhc30iDQogICAgIH0NCiAN CiAgICAgc2V0IGRpciBbbG9va2Zvcl9maWxlICR7c3JjZGlyfSBsaWJzdGRj KytdDQogICAgIGlmIHsgJHtkaXJ9ICE9ICIiIH0gew0KLQlhcHBlbmQgZmxh Z3MgIiAtSSR7ZGlyfSAtSSR7ZGlyfS9zdGwiDQorCWFwcGVuZCBmbGFncyAi IC1pc3lzdGVtICR7ZGlyfSAtaXN5c3RlbSAke2Rpcn0vc3RsIg0KICAgICB9 DQogDQogICAgIHJldHVybiAiJGZsYWdzIg0KQEAgLTMxNywxOSArMzE4LDIw IEBAIHByb2MgbGlic3RkYysrX2luY2x1ZGVfZmxhZ3MgeyBhcmdzIH0gew0K IA0KICAgICBzZXQgZGlyIFtsb29rZm9yX2ZpbGUgJHtzcmNkaXJ9IGxpYnN0 ZGMrKy12M10NCiAgICAgaWYgeyAke2Rpcn0gIT0gIiIgfSB7DQotCWFwcGVu ZCBmbGFncyAiIC1JJHtkaXJ9L2luY2x1ZGUgLUkke2Rpcn0vaW5jbHVkZS9z dGQgLUkke2Rpcn0vaW5jbHVkZS9jX3N0ZCAtSSR7ZGlyfS9saWJzdXBjKysi DQorCWFwcGVuZCBmbGFncyAiIC1pc3lzdGVtICR7ZGlyfS9pbmNsdWRlIC1p c3lzdGVtICR7ZGlyfS9pbmNsdWRlL3N0ZCINCisJYXBwZW5kIGZsYWdzICIg LWlzeXN0ZW0gJHtkaXJ9L2luY2x1ZGUvY19zdGQgLWlzeXN0ZW0gJHtkaXJ9 L2xpYnN1cGMrKyINCiAgICAgfQ0KIA0KICAgICBzZXQgZ2NjcGF0aCBbZ2V0 X211bHRpbGlic10NCiANCiAgICAgc2V0IGRpciBbbG9va2Zvcl9maWxlICR7 Z2NjcGF0aH0gbGlic3RkYysrLXYzXQ0KICAgICBpZiB7ICR7ZGlyfSAhPSAi IiB9IHsNCi0JYXBwZW5kIGZsYWdzICIgLUkke2Rpcn0vaW5jbHVkZSAtSSR7 ZGlyfS9pbmNsdWRlLyR7dGFyZ2V0X2FsaWFzfSINCisJYXBwZW5kIGZsYWdz ICIgLWlzeXN0ZW0gJHtkaXJ9L2luY2x1ZGUgLWlzeXN0ZW0gJHtkaXJ9L2lu Y2x1ZGUvJHt0YXJnZXRfYWxpYXN9Ig0KICAgICB9DQogDQogICAgIHNldCBk aXIgW2xvb2tmb3JfZmlsZSAke3NyY2Rpcn0gbGlic3RkYysrXQ0KICAgICBp ZiB7ICR7ZGlyfSAhPSAiIiB9IHsNCi0JYXBwZW5kIGZsYWdzICIgLUkke2Rp cn0gLUkke2Rpcn0vc3RsIg0KKwlhcHBlbmQgZmxhZ3MgIiAtaXN5c3RlbSAk e2Rpcn0gLWlzeXN0ZW0gJHtkaXJ9L3N0bCINCiAgICAgfQ0KIA0KICAgICBy ZXR1cm4gIiRmbGFncyINCkBAIC04NjAsNyArODYyLDcgQEAgcHJvYyB3aW5z dXBfaW5jbHVkZV9mbGFncyB7IGFyZ3MgfSB7DQogCXNldCB3aW5zdXBfZGly IFtsb29rZm9yX2ZpbGUgJHtzcmNkaXJ9IHdpbnN1cC9pbmNsdWRlL3dpbmRv d3MuaF0NCiAJaWYgeyAke3dpbnN1cF9kaXJ9ICE9ICIiIH0gew0KIAkgICAg c2V0IHdpbnN1cF9kaXIgW2ZpbGUgZGlybmFtZSAke3dpbnN1cF9kaXJ9XQ0K LQkgICAgcmV0dXJuICIgLUkke3dpbnN1cF9kaXJ9Ig0KKwkgICAgcmV0dXJu ICIgLWlzeXN0ZW0gJHt3aW5zdXBfZGlyfSINCiAJfQ0KICAgICB9DQogICAg IHZlcmJvc2UgIk5vIHdpbnN1cCBzdXBwb3J0IGZvciB0aGlzIHRhcmdldCIN CiANCg== --0-845395881-1035689621=:28032--