From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17696 invoked by alias); 6 Jan 2014 03:46:42 -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 17684 invoked by uid 89); 6 Jan 2014 03:46:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.5 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 06 Jan 2014 03:46:40 +0000 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s063kaDb005936 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Sun, 5 Jan 2014 22:46:37 -0500 Received: from psique ([10.3.113.9]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id s063kXft001030 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Sun, 5 Jan 2014 22:46:35 -0500 From: Sergio Durigan Junior To: Honggyu Kim Cc: gdb-patches@sourceware.org Subject: Re: [PATCH v2] Remove duplicated include in gdb directory References: <1388973207-31270-1-git-send-email-hong.gyu.kim@lge.com> X-URL: http://www.redhat.com Date: Mon, 06 Jan 2014 03:46:00 -0000 In-Reply-To: <1388973207-31270-1-git-send-email-hong.gyu.kim@lge.com> (Honggyu Kim's message of "Mon, 6 Jan 2014 10:53:27 +0900") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-IsSubscribed: yes X-SW-Source: 2014-01/txt/msg00042.txt.bz2 On Sunday, January 05 2014, Honggyu Kim wrote: > This patch simply removes duplicated include statements in gdb directory. > If there are two duplicated include statements, this patch keeps the first > include and removes the second include. > Those are found by checkincludes.pl tool in linux kernel and double checked > manually once again if the include statements are affected by ifdef macro. Thanks for the patch, Honggyu. I believe you have tested it by building GDB with --enable-targets=all, right? It needs a ChangeLog (mechanical), and it seems to be simple enough to be committed without needing a copyright assignment (I don't see your name in the MAINTAINERS file). Other than that, let's wait for some maintainer to give the green light. BTW, if you want to get started on the process to obtain the copyright assignment, mail me offlist and I can send you the instructions. Thanks, -- Sergio