[Buildroot] [PATCH 04/80 v1] package/angularjs: simplify modules selection
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Jan 18 21:57:08 UTC 2016
Thomas, All,
On 2016-01-18 22:48 +0100, Thomas Petazzoni spake thusly:
> On Sat, 16 Jan 2016 23:36:56 +0100, Yann E. MORIN wrote:
> > Remove the superfluous config option, and make all modules default to
> > 'y', which provides exactly the same functionality.
> >
> > simplify (and slightly beautify) the handling of modules in the .mk:
> > since the modules' symbols are always meaningful now, we can get rid of
> > the else-clause, and always build the list of modules using the Kconfig
> > options.
> >
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> >
> > ---
>
> You have an empty line at the end of many commit logs.
Yup, there's an empty line because I added a '---' comment and I always
separate them with an empty line so they are properly separated from the
commit log itself. But that's no a problem, since git-am (and thus
pwclient git-am does too):
- drops the --- comments
- drops emtpy trailing lines
> I agree with the principle of the patch, but sometimes is odd:
>
>
> > +# This list mirrors the list of available modules from Config.in.
> > +# Keep this list in sync and in the same (alphabetical) order as the
> > +# one in Config.in. However, always keep 'angular' first in the list.
>
> You say "angular" must be first in the list...
Damn, I forgot to remove that part when I moved back 'angular' out of
the modules list.
Thanks! :-)
Regards,
Yann E. MORIN.
> > +ANGULARJS_MODULES = \
> > + animate \
> > + aria \
> > + cookies \
> > + messages \
> > + message-format \
> > + resource \
> > + route \
> > + sanitize \
> > + touch
>
> But there is no "angular" in this list.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list