public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Richard Earnshaw <rearnsha@arm.com>
To: Maxim Kuvyrkov <maxim@codesourcery.com>
Cc: Jan Hubicka <hubicka@ucw.cz>,
	 Richard Sandiford <rdsandiford@googlemail.com>,
	"gcc@gcc.gnu.org" <gcc@gcc.gnu.org>
Subject: Re: [GCC Steering Committee] Android sub-port reviewer
Date: Mon, 02 Apr 2012 16:38:00 -0000	[thread overview]
Message-ID: <4F79D60F.4080006@arm.com> (raw)
In-Reply-To: <16285357-1E07-4D60-BD8D-2B2FE796609B@codesourcery.com>

On 01/04/12 20:57, Maxim Kuvyrkov wrote:
> On 29/03/2012, at 5:38 PM, Maxim Kuvyrkov wrote:
> 
>> I volunteer as the reviewer for Android sub-port.
>>
>> Android/Bionic support is an extension over Linux port and is being gradually added for more and more architectures.  I wrote the original Android GCC support for ARM (under a watchful design eye of Joseph Myers), and know how the bits fit together.
>>
>> As adding Android support to a new architecture requires changes to that architecture, the architecture maintainer will have the power of veto for the Android-related changes.
> 
> One of the members of SC raised a good point about whether architecture maintainers would prefer to handle the Android patches themselves.  My intention is to spare you the additional headache of dealing with Android, but, well, maybe you like it :-).
> 
> Richard E.,
> Jan,
> Richard S.,
> 
> Do you want to handle Android changes by yourself or do you want to delegate?
> 
> Thank you,
> 
> --
> Maxim Kuvyrkov
> CodeSourcery / Mentor Graphics
> 
> 
> 
> 
> 
> 

I have no desire to hold back android development, so I have no
objections to an Android-specific maintainer.

But the only file in the ARM back-end that is Android specific is
t-linux-android, a grand total of ten lines of code.

Any file shared with other arm ports would still need generic ARM
maintainer approval.

There are, however, some target independent android files in the config
directory and I guess it would make sense for a maintainer to be
appointed for those.

R.

  parent reply	other threads:[~2012-04-02 16:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-29  4:39 Maxim Kuvyrkov
2012-04-01 19:57 ` Maxim Kuvyrkov
2012-04-02  9:51   ` Jan Hubicka
2012-04-02 16:38   ` Richard Earnshaw [this message]
2012-04-02 18:45   ` Richard Sandiford
2012-04-02 20:55     ` Fu, Chao-Ying
2012-04-03  3:51       ` Andrew Pinski
2012-04-03 17:46         ` Fu, Chao-Ying
2012-04-03 18:08           ` Andrew Pinski
2012-04-03 18:28             ` Fu, Chao-Ying
2012-04-03 19:21               ` Maxim Kuvyrkov
2012-04-04 22:10                 ` Fu, Chao-Ying

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F79D60F.4080006@arm.com \
    --to=rearnsha@arm.com \
    --cc=gcc@gcc.gnu.org \
    --cc=hubicka@ucw.cz \
    --cc=maxim@codesourcery.com \
    --cc=rdsandiford@googlemail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).