From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1294 invoked by alias); 30 Nov 2010 00:03:14 -0000 Received: (qmail 1283 invoked by uid 22791); 30 Nov 2010 00:03:12 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,SPF_HELO_PASS,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.44.51) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 30 Nov 2010 00:03:06 +0000 Received: from wpaz33.hot.corp.google.com (wpaz33.hot.corp.google.com [172.24.198.97]) by smtp-out.google.com with ESMTP id oAU034OK013139 for ; Mon, 29 Nov 2010 16:03:04 -0800 Received: from qwf6 (qwf6.prod.google.com [10.241.194.70]) by wpaz33.hot.corp.google.com with ESMTP id oAU024L5011801 for ; Mon, 29 Nov 2010 16:03:03 -0800 Received: by qwf6 with SMTP id 6so710934qwf.35 for ; Mon, 29 Nov 2010 16:03:02 -0800 (PST) MIME-Version: 1.0 Received: by 10.224.205.200 with SMTP id fr8mr5882481qab.198.1291075382864; Mon, 29 Nov 2010 16:03:02 -0800 (PST) Received: by 10.220.203.201 with HTTP; Mon, 29 Nov 2010 16:03:02 -0800 (PST) In-Reply-To: References: <20101129003949.502FC246199@ruffy.mtv.corp.google.com> Date: Tue, 30 Nov 2010 00:03:00 -0000 Message-ID: Subject: Re: [RFA, doc RFA] Don't install gdb.PYTHONDIR if -nx From: Doug Evans To: Tom Tromey , Eli Zaretskii Cc: gdb-patches@sourceware.org Content-Type: multipart/mixed; boundary=20cf300faae592ea82049639e927 X-System-Of-Record: true X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2010-11/txt/msg00486.txt.bz2 --20cf300faae592ea82049639e927 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Content-length: 1344 On Mon, Nov 29, 2010 at 1:38 PM, Tom Tromey wrote: >>>>>> "Doug" =3D=3D Doug Evans writes: > > Doug> I don't entirely like the patch, it's extending -nx into > Doug> new territory, e.g., -nx controlling whether certain commands > Doug> are available or not. =A0It shouldn't matter, from the user's > Doug> perspective, whether the commands are implemented in python. > > I agree. =A0In particular I think it is reasonable to want to use python > commands from a batch script, but such a script is likely to use -nx. > > Doug> However, the only other alternative I can think of is to > Doug> add a new option, and I'm guessing that's unacceptable. > > I think it would be fine to add one. > > > Either way I think the documentation needs to be updated. > > Jan pointed out that this is PR 12227. > > Tom > I will check this in then, pending doc RFA. 2010-11-29 Doug Evans PR python/12227 * main.c (captured_main): Recognize -np. * top.c (inhibit_pythoninit): New global. * top.h (inhibit_pythoninit): Declare. * python/python.c (finish_python_initialization): Only initialize gdb module if not -np. doc/ * gdb.texinfo (Mode Options): Document -np. testsuite/ * lib/gdb.exp: Add -np to INTERNAL_GDBFLAGS. --20cf300faae592ea82049639e927 Content-Type: text/plain; charset=US-ASCII; name="gdb-101129-pr-12227-1.patch.txt" Content-Disposition: attachment; filename="gdb-101129-pr-12227-1.patch.txt" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gh40w15o0 Content-length: 5897 MjAxMC0xMS0yOSAgRG91ZyBFdmFucyAgPGRqZUBnb29nbGUuY29tPgoKCVBS IHB5dGhvbi8xMjIyNwoJKiBtYWluLmMgKGNhcHR1cmVkX21haW4pOiBSZWNv Z25pemUgLW5wLgoJKiB0b3AuYyAoaW5oaWJpdF9weXRob25pbml0KTogTmV3 IGdsb2JhbC4KCSogdG9wLmggKGluaGliaXRfcHl0aG9uaW5pdCk6IERlY2xh cmUuCgkqIHB5dGhvbi9weXRob24uYyAoZmluaXNoX3B5dGhvbl9pbml0aWFs aXphdGlvbik6IE9ubHkgaW5pdGlhbGl6ZQoJZ2RiIG1vZHVsZSBpZiBub3Qg LW5wLgoKCWRvYy8KCSogZ2RiLnRleGluZm8gKE1vZGUgT3B0aW9ucyk6IERv Y3VtZW50IC1ucC4KCgl0ZXN0c3VpdGUvCgkqIGxpYi9nZGIuZXhwOiBBZGQg LW5wIHRvIElOVEVSTkFMX0dEQkZMQUdTLgoKSW5kZXg6IG1haW4uYwo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3ZzL3NyYy9zcmMvZ2Ri L21haW4uYyx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS44NwpkaWZmIC11IC1w IC1yMS44NyBtYWluLmMKLS0tIG1haW4uYwkyMiBTZXAgMjAxMCAxOTo1OTox NSAtMDAwMAkxLjg3CisrKyBtYWluLmMJMjkgTm92IDIwMTAgMjM6NDk6NTgg LTAwMDAKQEAgLTM5Niw2ICszOTYsNyBAQCBjYXB0dXJlZF9tYWluICh2b2lk ICpkYXRhKQogICAgICAgeyJzaWxlbnQiLCBub19hcmd1bWVudCwgJnF1aWV0 LCAxfSwKICAgICAgIHsibngiLCBub19hcmd1bWVudCwgJmluaGliaXRfZ2Ri aW5pdCwgMX0sCiAgICAgICB7Im4iLCBub19hcmd1bWVudCwgJmluaGliaXRf Z2RiaW5pdCwgMX0sCisgICAgICB7Im5wIiwgbm9fYXJndW1lbnQsICZpbmhp Yml0X3B5dGhvbmluaXQsIDF9LAogICAgICAgeyJiYXRjaC1zaWxlbnQiLCBu b19hcmd1bWVudCwgMCwgJ0InfSwKICAgICAgIHsiYmF0Y2giLCBub19hcmd1 bWVudCwgJmJhdGNoX2ZsYWcsIDF9LAogICAgICAgeyJlcG9jaCIsIG5vX2Fy Z3VtZW50LCAmZXBvY2hfaW50ZXJmYWNlLCAxfSwKSW5kZXg6IHRvcC5jCj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvc3JjL3NyYy9n ZGIvdG9wLmMsdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuMTg1CmRpZmYgLXUg LXAgLXIxLjE4NSB0b3AuYwotLS0gdG9wLmMJMiBOb3YgMjAxMCAxNjo0ODo0 MSAtMDAwMAkxLjE4NQorKysgdG9wLmMJMjkgTm92IDIwMTAgMjM6NDk6NTgg LTAwMDAKQEAgLTg4LDYgKzg4LDkgQEAgY2hhciBnZGJpbml0W1BBVEhfTUFY ICsgMV0gPSBHREJJTklUX0ZJTAogCiBpbnQgaW5oaWJpdF9nZGJpbml0ID0g MDsKIAorLyogSWYgbm9uemVybywgZG9uJ3QgbG9hZCBhbnkgaW5pdGlhbCBw eXRob24gc2NyaXB0cy4gICovCitpbnQgaW5oaWJpdF9weXRob25pbml0ID0g MDsKKwogLyogSWYgbm9uemVybywgYW5kIEdEQiBoYXMgYmVlbiBjb25maWd1 cmVkIHRvIGJlIGFibGUgdG8gdXNlIHdpbmRvd3MsCiAgICBhdHRlbXB0IHRv IG9wZW4gdGhlbSB1cG9uIHN0YXJ0dXAuICAqLwogCkluZGV4OiB0b3AuaAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3ZzL3NyYy9zcmMv Z2RiL3RvcC5oLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjIyCmRpZmYgLXUg LXAgLXIxLjIyIHRvcC5oCi0tLSB0b3AuaAk3IEFwciAyMDEwIDE2OjU0OjM5 IC0wMDAwCTEuMjIKKysrIHRvcC5oCTI5IE5vdiAyMDEwIDIzOjQ5OjU4IC0w MDAwCkBAIC0zMCw2ICszMCw3IEBAIGV4dGVybiBpbnQgaW5fdXNlcl9jb21t YW5kOwogZXh0ZXJuIGludCBjYXV0aW9uOwogZXh0ZXJuIGNoYXIgZ2RiX2Rp cmJ1ZlsxMDI0XTsKIGV4dGVybiBpbnQgaW5oaWJpdF9nZGJpbml0OworZXh0 ZXJuIGludCBpbmhpYml0X3B5dGhvbmluaXQ7CiBleHRlcm4gaW50IGVwb2No X2ludGVyZmFjZTsKIGV4dGVybiBjaGFyIGdkYmluaXRbXTsKIApJbmRleDog ZG9jL2dkYi50ZXhpbmZvCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZp bGU6IC9jdnMvc3JjL3NyYy9nZGIvZG9jL2dkYi50ZXhpbmZvLHYKcmV0cmll dmluZyByZXZpc2lvbiAxLjc3OApkaWZmIC11IC1wIC1yMS43NzggZ2RiLnRl eGluZm8KLS0tIGRvYy9nZGIudGV4aW5mbwkyOSBOb3YgMjAxMCAyMzoyMDo1 NyAtMDAwMAkxLjc3OAorKysgZG9jL2dkYi50ZXhpbmZvCTI5IE5vdiAyMDEw IDIzOjQ5OjU4IC0wMDAwCkBAIC0xMDI1LDYgKzEwMjUsMTQgQEAgRG8gbm90 IGV4ZWN1dGUgY29tbWFuZHMgZm91bmQgaW4gYW55IGluaQogb3B0aW9ucyBh bmQgYXJndW1lbnRzIGhhdmUgYmVlbiBwcm9jZXNzZWQuICBAeHJlZntDb21t YW5kIEZpbGVzLCxDb21tYW5kCiBGaWxlc30uCiAKK0BpdGVtIC1ucAorQGNp bmRleCBAY29kZXstLW5wfQorRG8gbm90IGluaXRpYWxpemUgdGhlIEBjb2Rl e2dkYn0gcHl0aG9uIG1vZHVsZS4KK05vcm1hbGx5IEB2YWx1ZXtHREJOfSwg YXMgcGFydCBvZiBpbml0aWFsaXphdGlvbiwgd2lsbCBpbml0aWFsaXplCit0 aGUgQGNvZGV7Z2RifSBweXRob24gbW9kdWxlLiAgQHhyZWZ7UHl0aG9uIEFQ SX0uCitUaGlzIGludGVyZmVyZXMgd2l0aCwgZm9yIGV4YW1wbGUsIHJ1bm5p bmcgdGhlIEdEQiB0ZXN0c3VpdGUgd2hpY2gKK3dhbnRzIHRvIHVzZSBpdHMg b3duIGNvcHkuCisKIEBpdGVtIC1xdWlldAogQGl0ZW14IC1zaWxlbnQKIEBp dGVteCAtcQpJbmRleDogcHl0aG9uL3B5dGhvbi5jCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvc3JjL3NyYy9nZGIvcHl0aG9uL3B5 dGhvbi5jLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjUzCmRpZmYgLXUgLXAg LXIxLjUzIHB5dGhvbi5jCi0tLSBweXRob24vcHl0aG9uLmMJMTIgTm92IDIw MTAgMjA6NDk6NDIgLTAwMDAJMS41MworKysgcHl0aG9uL3B5dGhvbi5jCTI5 IE5vdiAyMDEwIDIzOjQ5OjU4IC0wMDAwCkBAIC0xMDgwLDExICsxMDgwLDE1 IEBAIGRlZiBHZGJTZXRQeXRob25EaXJlY3RvcnkgKGRpcik6XG5cCiAgIGlw eSA9IGdkYi5QWVRIT05ESVIgKyAnL2dkYi9fX2luaXRfXy5weSdcblwKICAg aWYgb3MucGF0aC5leGlzdHMgKGlweSk6XG5cCiAgICAgZXhlY2ZpbGUgKGlw eSlcblwKLVxuXAotIyBJbnN0YWxsIHRoZSBkZWZhdWx0IGdkYi5QWVRIT05E SVIuXG5cCi1HZGJTZXRQeXRob25EaXJlY3RvcnkgKGdkYi5QWVRIT05ESVIp XG5cCiAiKTsKIAorICAvKiBEb24ndCBpbnN0YWxsIHRoZSBweXRob24gZGly ZWN0b3J5IGlmIC1ucC4KKyAgICAgV2UgZG9uJ3Qgd2FudCB0byBwaWNrIHVw LCBmb3IgZXhhbXBsZSwgcHl0aG9uLWJhc2VkIGNvbW1hbmRzIGZyb20gdGhl CisgICAgIGluc3RhbGwgZGlyZWN0b3J5IHdoZW4gcnVubmluZyB0aGUgdGVz dHN1aXRlLiAgKi8KKyAgaWYgKCEgaW5oaWJpdF9weXRob25pbml0KQorICAg IC8qIEluc3RhbGwgdGhlIGRlZmF1bHQgZ2RiLlBZVEhPTkRJUi4gICovCisg ICAgUHlSdW5fU2ltcGxlU3RyaW5nICgiR2RiU2V0UHl0aG9uRGlyZWN0b3J5 IChnZGIuUFlUSE9ORElSKSIpOworCiAgIGRvX2NsZWFudXBzIChjbGVhbnVw KTsKIH0KIApJbmRleDogdGVzdHN1aXRlL2xpYi9nZGIuZXhwCj09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvc3JjL3NyYy9nZGIvdGVz dHN1aXRlL2xpYi9nZGIuZXhwLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjE1 OQpkaWZmIC11IC1wIC1yMS4xNTkgZ2RiLmV4cAotLS0gdGVzdHN1aXRlL2xp Yi9nZGIuZXhwCTIzIE5vdiAyMDEwIDIyOjI1OjM3IC0wMDAwCTEuMTU5Cisr KyB0ZXN0c3VpdGUvbGliL2dkYi5leHAJMjkgTm92IDIwMTAgMjM6NDk6NTgg LTAwMDAKQEAgLTU2LDcgKzU2LDcgQEAgdmVyYm9zZSAidXNpbmcgR0RCRkxB R1MgPSAkR0RCRkxBR1MiIDIKICMgSU5URVJOQUxfR0RCRkxBR1MgY29udGFp bnMgZmxhZ3MgdGhhdCB0aGUgdGVzdHN1aXRlIHJlcXVpcmVzLgogZ2xvYmFs IElOVEVSTkFMX0dEQkZMQUdTCiBpZiAhW2luZm8gZXhpc3RzIElOVEVSTkFM X0dEQkZMQUdTXSB7Ci0gICAgc2V0IElOVEVSTkFMX0dEQkZMQUdTICItbncg LW54IgorICAgIHNldCBJTlRFUk5BTF9HREJGTEFHUyAiLW53IC1ueCAtbnAi CiB9CiAKICMgVGhlIHZhcmlhYmxlIGdkYl9wcm9tcHQgaXMgYSByZWdleHAg d2hpY2ggbWF0Y2hlcyB0aGUgZ2RiIHByb21wdC4K --20cf300faae592ea82049639e927--