Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] boot/at91bootstrap3: Add the option to download via git
Date: Sat, 7 Mar 2015 12:21:28 +0100	[thread overview]
Message-ID: <20150307112128.GA4186@free.fr> (raw)
In-Reply-To: <CA+TH9VkkqdJtyWBFGu2Jc13grJeBYO8pvEjX4XUi2YU1oL+RbQ@mail.gmail.com>

Angelo, All,

On 2015-03-07 10:08 +0100, Angelo Compagnucci spake thusly:
> 2015-03-06 23:53 GMT+01:00 Yann E. MORIN <yann.morin.1998@free.fr>:
> > Angelo, All,
> >
> > On 2015-03-04 15:55 +0100, Angelo Compagnucci spake thusly:
> >> >> Adding support for git downloading of a custom repository.
> > [--SNIP--]
> >> Any news on this?
> >
> > Sorry it takes so long, but we are just getting out of a release cycle
> > (i.e. the last month was a feature-freeze), and we're trying to catch up
> > with the currently 363-or-so patches pending in our patch-tracker:
> >     http://patchwork.ozlabs.org/project/buildroot/list/
> >
> > One reason that it takes so much time is that there are very
> > reviewers/testers that give feedback on patches (I have been myself not
> > very active in that activity recently), and Peter and Thomas have to do
> > the grunt work of the triaging, reviewing, testing and applying almost
> > all on their own...
> >
> > A little bit of help in reviewing those patches, testing them, triaging
> > the duplicates and so on... is very welcome.
> 
> I usually contribute to several Open Source software and I usually
> tend to lend a hand because it's fun!
> Unfortunately I'm still grasping the internal of Buildroot and I've
> not the necessary knowledge to help reviewing patches.
> I'm monitorin the mailing list everyday and in case I find something
> to work on, you can count on me!

That's great, and it is appreciated! :-)

> I'm open to some sort of mentorship if someone wants to help me became
> proficient and helpful!

Yes, sure. First, don't be afraid! Even a review that just looks at
typoes is much welcome. Start with simple changes, such as a version
bump and build-test it, in various configurations (mips, arm, ppc...
with uclibc or glibc, with or without threads...); for example, I try to
build for arm and i386with the "default" toolchain options then with a
"fat" toolchain (LFS, threads...)

If you find typoes, just reply to the patch, pointing the typoes.

If you find the change to be OK, you can reply to the patch with a mail
stating:

    Reviewed-by: Firstname NAME <your.name@example.net>

If you tested it, you can say something like (no need to be too verbose):

    [Firstname: build-tested for mips; run-tested on armel]
    Tested-by: Firstname NAME <your.name@example.net>

Also, you can have a look at our manual, where we have a full section
about how to contribute:
    http://buildroot.net/downloads/manual/manual.html#_contributing_to_buildroot

And again, each contribution counts! So thanks for yours! :-)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2015-03-07 11:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-30 16:52 [Buildroot] [PATCH] boot/at91bootstrap3: Add the option to download via git Angelo Compagnucci
2015-02-16  9:12 ` Angelo Compagnucci
2015-03-04 14:55   ` Angelo Compagnucci
2015-03-06 22:53     ` Yann E. MORIN
2015-03-07  9:08       ` Angelo Compagnucci
2015-03-07 11:21         ` Yann E. MORIN [this message]
2015-03-07 16:01           ` Arnout Vandecappelle
  -- strict thread matches above, loose matches on Subject: below --
2015-03-06 11:24 Giuseppe Marocchio

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=20150307112128.GA4186@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /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