From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 85373 invoked by alias); 14 Jun 2017 16:32:20 -0000 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 Received: (qmail 85362 invoked by uid 89); 14 Jun 2017 16:32:19 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL,BAYES_00,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy= X-HELO: eggs.gnu.org Received: from eggs.gnu.org (HELO eggs.gnu.org) (208.118.235.92) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 14 Jun 2017 16:32:18 +0000 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dLBDM-0000LU-Sg for gdb-patches@sourceware.org; Wed, 14 Jun 2017 12:32:20 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:37149) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dLBDM-0000LQ-Pt; Wed, 14 Jun 2017 12:32:16 -0400 Received: from 84.94.185.246.cable.012.net.il ([84.94.185.246]:2376 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dLBDM-0005nk-3Z; Wed, 14 Jun 2017 12:32:16 -0400 Date: Wed, 14 Jun 2017 16:32:00 -0000 Message-Id: <83poe634za.fsf@gnu.org> From: Eli Zaretskii To: Yao Qi CC: gdb-patches@sourceware.org In-reply-to: <867f0ewne7.fsf@gmail.com> (message from Yao Qi on Wed, 14 Jun 2017 17:21:36 +0100) Subject: Re: [PATCH 06/25] Generate c for feature instead of tdesc Reply-to: Eli Zaretskii References: <1497256916-4958-1-git-send-email-yao.qi@linaro.org> <1497256916-4958-7-git-send-email-yao.qi@linaro.org> <837f0h5kjz.fsf@gnu.org> <86wp8gccqv.fsf@gmail.com> <83h8zk3pu3.fsf@gnu.org> <86k24fdhvf.fsf@gmail.com> <83efun520e.fsf@gnu.org> <867f0ewne7.fsf@gmail.com> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 2001:4830:134:3::e X-IsSubscribed: yes X-SW-Source: 2017-06/txt/msg00429.txt.bz2 > From: Yao Qi > Cc: gdb-patches@sourceware.org > Date: Wed, 14 Jun 2017 17:21:36 +0100 > > > But that means the produced file can only be used by rebuilding GDB, > > doesn't it? If so, the text should make this clear, I think. > > > > Yes. The last sentence is about it, "The created source file is built > into @value{GDBN}", but I didn't explicitly say GDB needs rebuild. I think we should say that. > > (In general, since GDB is a compiled program which cannot easily be > > modified without rebuilding it, I wonder how useful this feature will > > be. But I'll yield to your expertise on that.) > > This command is only used by developers after they add or modify XML > target descriptions. Then I think we should say that as well. > A question in general, do we really need to document these commands, > which are used for GDB development, in GDB user manual? These > commands are "maint print c-tdesc", "maint check xml-descriptions" > (added by one patch in this series), and "maint selftest". Can we > remove them from the user manual? If we remove them, there will be no place where they are documented, right? GDB developers are GDB users as well, e.g. I read the manual quite a lot. So I see no reason to remove these commands. We just need to indicate when a command makes sense only for GDB developers. Thanks.