[Buildroot] [PATCH] glibc: bump version to 2.21

Yann E. MORIN yann.morin.1998 at free.fr
Sun Feb 8 14:40:48 UTC 2015


Vicente, All,

On 2015-02-08 15:22 +0100, Vicente Olivert Riera spake thusly:
> when glibc-2.20 was added, it was done in the same way: adding 2.20,
> removing 2.18 and setting 2.19 as default. All in the same patch, and
> nobody complained. See:
> 
> http://git.buildroot.net/buildroot/commit/?id=f660cfd043b898c9b9eb5fc5ccc90335d6b334f5
> 
> Why should I do it in separate patches now?

Hehe. :-)

Well, the fact that a previous commit is not correct does not mean we
can continue to commit incorrect changes.

Q. Why do I suggest we do it in separate patches?

A. Because they are *semantically* different changes. Doing semantically
   unrelated changes means:
    1) they are easier to review (granted, your patch is pretty simple),
    2) each can be easily reverted in case we discover an issue later on,
       without reverting the full change.

Q. Why are those *semantically* separate changes?

A. Because each is doing different things:
    1) it is adding a new version,
    2) it is changing a default,
    3) it is removing a version.

A review if also to give one's opinion about how things should be done.
Different people may see "The Way" differently. ;-)

That's why I said I would _prefer_ they be done in separate changes. Of
course, others may disagree, and the maintainer will have the final say
(i.e. accept your patch as-is if it suits him, or ask for it to be
split).

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.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list