* [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles @ 2013-10-24 15:35 Maxime Hadjinlian 2013-10-25 6:27 ` Thomas De Schampheleire 0 siblings, 1 reply; 4+ messages in thread From: Maxime Hadjinlian @ 2013-10-24 15:35 UTC (permalink / raw) To: buildroot --- package/Config.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/package/Config.in b/package/Config.in index ebbdeb5..8519f45 100644 --- a/package/Config.in +++ b/package/Config.in @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" source "package/jhead/Config.in" source "package/rrdtool/Config.in" -comment "graphic libraries" +comment "Graphic libraries" source "package/cegui06/Config.in" source "package/directfb/Config.in" source "package/directfb-examples/Config.in" @@ -196,7 +196,7 @@ source "package/sdl_net/Config.in" source "package/sdl_sound/Config.in" source "package/sdl_ttf/Config.in" -comment "other GUIs" +comment "Other GUIs" source "package/efl/Config.in" source "package/qt/Config.in" @@ -241,7 +241,7 @@ source "package/liberation/Config.in" source "package/libsexy/Config.in" source "package/x11r7/xkeyboard-config/Config.in" -comment "X Window managers" +comment "X window managers" source "package/blackbox/Config.in" source "package/enlightenment/Config.in" source "package/fluxbox/Config.in" -- 1.8.4.rc3 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles 2013-10-24 15:35 [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles Maxime Hadjinlian @ 2013-10-25 6:27 ` Thomas De Schampheleire 2013-10-25 8:06 ` Arnout Vandecappelle 0 siblings, 1 reply; 4+ messages in thread From: Thomas De Schampheleire @ 2013-10-25 6:27 UTC (permalink / raw) To: buildroot Hi Maxime, On Thu, Oct 24, 2013 at 5:35 PM, Maxime Hadjinlian <maxime.hadjinlian@gmail.com> wrote: > --- > package/Config.in | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/package/Config.in b/package/Config.in > index ebbdeb5..8519f45 100644 > --- a/package/Config.in > +++ b/package/Config.in > @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" > source "package/jhead/Config.in" > source "package/rrdtool/Config.in" > > -comment "graphic libraries" > +comment "Graphic libraries" > source "package/cegui06/Config.in" > source "package/directfb/Config.in" > source "package/directfb-examples/Config.in" > @@ -196,7 +196,7 @@ source "package/sdl_net/Config.in" > source "package/sdl_sound/Config.in" > source "package/sdl_ttf/Config.in" > > -comment "other GUIs" > +comment "Other GUIs" > source "package/efl/Config.in" > source "package/qt/Config.in" > > @@ -241,7 +241,7 @@ source "package/liberation/Config.in" > source "package/libsexy/Config.in" > source "package/x11r7/xkeyboard-config/Config.in" > > -comment "X Window managers" > +comment "X window managers" > source "package/blackbox/Config.in" > source "package/enlightenment/Config.in" > source "package/fluxbox/Config.in" Acked-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com> Maxime, when you send multiple versions of a patch, could you use v2, v3, ... in the subject line (PATCH v2) to make it easier for reviewers to distinghuish the different versions? Also below the changelog you should put a description of the changes compared to the previous version. See http://buildroot.uclibc.org/downloads/manual/manual.html#_patch_revision_changelog Also, I believe it is customary to resend the entire series when there is a revision of one of its patches. Thanks, Thomas ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles 2013-10-25 6:27 ` Thomas De Schampheleire @ 2013-10-25 8:06 ` Arnout Vandecappelle 2013-10-25 8:28 ` Maxime Hadjinlian 0 siblings, 1 reply; 4+ messages in thread From: Arnout Vandecappelle @ 2013-10-25 8:06 UTC (permalink / raw) To: buildroot On 25/10/13 08:27, Thomas De Schampheleire wrote: > Hi Maxime, > > On Thu, Oct 24, 2013 at 5:35 PM, Maxime Hadjinlian > <maxime.hadjinlian@gmail.com> wrote: >> --- >> package/Config.in | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/package/Config.in b/package/Config.in >> index ebbdeb5..8519f45 100644 >> --- a/package/Config.in >> +++ b/package/Config.in >> @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" >> source "package/jhead/Config.in" >> source "package/rrdtool/Config.in" >> >> -comment "graphic libraries" >> +comment "Graphic libraries" >> source "package/cegui06/Config.in" >> source "package/directfb/Config.in" >> source "package/directfb-examples/Config.in" >> @@ -196,7 +196,7 @@ source "package/sdl_net/Config.in" >> source "package/sdl_sound/Config.in" >> source "package/sdl_ttf/Config.in" >> >> -comment "other GUIs" >> +comment "Other GUIs" >> source "package/efl/Config.in" >> source "package/qt/Config.in" >> >> @@ -241,7 +241,7 @@ source "package/liberation/Config.in" >> source "package/libsexy/Config.in" >> source "package/x11r7/xkeyboard-config/Config.in" >> >> -comment "X Window managers" >> +comment "X window managers" >> source "package/blackbox/Config.in" >> source "package/enlightenment/Config.in" >> source "package/fluxbox/Config.in" > > Acked-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com> > > Maxime, when you send multiple versions of a patch, could you use v2, > v3, ... in the subject line (PATCH v2) to make it easier for reviewers Note that you can do this with git-send-email (or format-patch): git send-email -v2 # Send out PATCH v2, requires git >= 1.8.something git send-email --subject-prefix 'PATCH v2' Regards, Arnout > to distinghuish the different versions? Also below the changelog you > should put a description of the changes compared to the previous > version. > See http://buildroot.uclibc.org/downloads/manual/manual.html#_patch_revision_changelog > > Also, I believe it is customary to resend the entire series when there > is a revision of one of its patches. > > Thanks, > Thomas > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot > -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles 2013-10-25 8:06 ` Arnout Vandecappelle @ 2013-10-25 8:28 ` Maxime Hadjinlian 0 siblings, 0 replies; 4+ messages in thread From: Maxime Hadjinlian @ 2013-10-25 8:28 UTC (permalink / raw) To: buildroot On Fri, Oct 25, 2013 at 10:06 AM, Arnout Vandecappelle <arnout@mind.be> wrote: > On 25/10/13 08:27, Thomas De Schampheleire wrote: >> >> Hi Maxime, >> >> On Thu, Oct 24, 2013 at 5:35 PM, Maxime Hadjinlian >> <maxime.hadjinlian@gmail.com> wrote: >>> >>> --- >>> package/Config.in | 6 +++--- >>> 1 file changed, 3 insertions(+), 3 deletions(-) >>> >>> diff --git a/package/Config.in b/package/Config.in >>> index ebbdeb5..8519f45 100644 >>> --- a/package/Config.in >>> +++ b/package/Config.in >>> @@ -169,7 +169,7 @@ source "package/gnuplot/Config.in" >>> source "package/jhead/Config.in" >>> source "package/rrdtool/Config.in" >>> >>> -comment "graphic libraries" >>> +comment "Graphic libraries" >>> source "package/cegui06/Config.in" >>> source "package/directfb/Config.in" >>> source "package/directfb-examples/Config.in" >>> @@ -196,7 +196,7 @@ source "package/sdl_net/Config.in" >>> source "package/sdl_sound/Config.in" >>> source "package/sdl_ttf/Config.in" >>> >>> -comment "other GUIs" >>> +comment "Other GUIs" >>> source "package/efl/Config.in" >>> source "package/qt/Config.in" >>> >>> @@ -241,7 +241,7 @@ source "package/liberation/Config.in" >>> source "package/libsexy/Config.in" >>> source "package/x11r7/xkeyboard-config/Config.in" >>> >>> -comment "X Window managers" >>> +comment "X window managers" >>> source "package/blackbox/Config.in" >>> source "package/enlightenment/Config.in" >>> source "package/fluxbox/Config.in" >> >> >> Acked-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com> >> >> Maxime, when you send multiple versions of a patch, could you use v2, >> v3, ... in the subject line (PATCH v2) to make it easier for reviewers > > > Note that you can do this with git-send-email (or format-patch): > > git send-email -v2 # Send out PATCH v2, requires git >= 1.8.something > git send-email --subject-prefix 'PATCH v2' > Thanks, I did not know about that ! > > Regards, > Arnout > >> to distinghuish the different versions? Also below the changelog you >> should put a description of the changes compared to the previous >> version. >> See >> http://buildroot.uclibc.org/downloads/manual/manual.html#_patch_revision_changelog >> >> Also, I believe it is customary to resend the entire series when there >> is a revision of one of its patches. Yes, I though about sending the patch as an alone change instead of part of a serie but it was wrong of me since this one depends on the change introduced by the first patch. Anyway, sorry for theses mistakes, I should have known better. I'll wait on further comment on the first patch and resend the serie with a changelog. >> >> Thanks, >> Thomas >> _______________________________________________ >> buildroot mailing list >> buildroot at busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot >> > > > -- > Arnout Vandecappelle arnout at mind be > Senior Embedded Software Architect +32-16-286500 > Essensium/Mind http://www.mind.be > G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven > LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle > GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-10-25 8:28 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-10-24 15:35 [Buildroot] [PATCH] packages/Config.in: Fix case on entries titles Maxime Hadjinlian 2013-10-25 6:27 ` Thomas De Schampheleire 2013-10-25 8:06 ` Arnout Vandecappelle 2013-10-25 8:28 ` Maxime Hadjinlian
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox