From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17211 invoked by alias); 20 Apr 2014 12:51:57 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 17178 invoked by uid 89); 20 Apr 2014 12:51:56 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-Spam-User: qpsmtpd, 4 recipients X-HELO: mail-la0-f41.google.com Received: from mail-la0-f41.google.com (HELO mail-la0-f41.google.com) (209.85.215.41) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Sun, 20 Apr 2014 12:51:53 +0000 Received: by mail-la0-f41.google.com with SMTP id gl10so2584977lab.0 for ; Sun, 20 Apr 2014 05:51:49 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.152.1.199 with SMTP id 7mr22701365lao.24.1397998309559; Sun, 20 Apr 2014 05:51:49 -0700 (PDT) Received: by 10.112.125.202 with HTTP; Sun, 20 Apr 2014 05:51:49 -0700 (PDT) In-Reply-To: <5352D100.9040108@marino.st> References: <5352D100.9040108@marino.st> Date: Sun, 20 Apr 2014 19:05:00 -0000 Message-ID: Subject: Re: Contributing new gcc targets: i386-*-dragonfly and x86-64-*-dragonfly From: Jonathan Wakely To: John Marino Cc: gcc-patches , Gerald Pfeifer , "manu at gcc dot gnu.org" , "Eric Botcazou (gnu.org)" , "libstdc++" Content-Type: text/plain; charset=UTF-8 X-SW-Source: 2014-04/txt/msg01138.txt.bz2 On 19 April 2014 20:39, John Marino wrote: > Hello GCC developers, > > For the last few years, I have been maintaining a large set of patches > that add support for the DragonFly BSD target and also complete Ada > frontend support on all four major BSDs among other things. Before I > can submit patches for Ada or testsuite cases, DragonFly must be a > recognized, working target. The patches attached here will provide > out-of-the-box support for the C, C++, Objective-C and Fortran frontends. Thanks for the patch - I only have a few general, minor comments. As noted at http://gcc.gnu.org/lists.html C++ library patches should go to the libstdc++ list as well as gcc-patches, so I've CC'd that list (original mail and patch are at http://gcc.gnu.org/ml/gcc-patches/2014-04/msg01128.html) Patches should not include generated files such as configure, as the diffs don't always apply cleanly and the changes are implied by the patches to files such as acinclude.m4 and configure.ac. The regenerated versions should of course be committed, and the ChangeLog should mention they are regenerated, as you've done. The changelog text should be correctly capitalised and sentences ended with a period (e.g. "New target." and "New." not "New target" and "new"). The individual ChangeLog entries at http://leaf.dragonflybsd.org/~marino/gcc-df-target/changelog_entries/ would generally be used as the commit message, grouped and prefixed by the name of the sub-directory: gcc/ * config.gcc (*-*-dragonfly*): New target etc. etc. libcilkrts/ * runtime/os-unix.c (__DragonFly__): New target etc. etc. libgcc/ * libgcc/config.host (*-*-dragonfly*): New target etc. etc. The libstdc++ changes are OK for trunk if the rest gets approved.