All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel
@ 2014-05-02 12:45 Yann E. MORIN
  2014-05-02 13:09 ` Peter Korsgaard
  2014-05-02 19:18 ` Yann E. MORIN
  0 siblings, 2 replies; 4+ messages in thread
From: Yann E. MORIN @ 2014-05-02 12:45 UTC (permalink / raw)
  To: buildroot

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Previously, when we were using the rpi-3.10.y branch, we coiuld use a sha1 to
refer to a specific changeset, and that would work and give reproducible
builds, thanks to upstream merging linux-stable into their branch.

Now they have declared rpi-3.12.y the new stable branch, and deprecated the
rpi-3.10.y branch.

Unfortunately, they are now rebasing the rpi-3.12.y branch instead of merging
linux-stable as they used to do. This means we can no longer use a sha1 as the
tree will change as soon as upstream rebases onto a new linux-stable.

There are two possibilities here:
  - revert back to using a rpi-3.10.y cset
  - use the rpi-3.12.y branch name

This patch goes with the second option, since rpi-3.10.y is no longer maitained,
and we prefer to use a stable branch, at the cost of reproducible builds.

Reported-by: Jeremy Rosen <jeremy.rosen@openwide.fr>
Signed-off-by: yann.morin.1998 at free.fr
Cc: Jeremy Rosen <jeremy.rosen@openwide.fr>

---
Note: before the next release, we should revisit this, and if upstream has
not switched to merging over rebasing, then we will just revert to using
the latest cset from the rpi-3.10.y branch for the release.
---
 configs/raspberrypi_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/raspberrypi_defconfig b/configs/raspberrypi_defconfig
index 61e23c0..28ca5ac 100644
--- a/configs/raspberrypi_defconfig
+++ b/configs/raspberrypi_defconfig
@@ -16,7 +16,7 @@ BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_3_12=y
 BR2_LINUX_KERNEL=y
 BR2_LINUX_KERNEL_CUSTOM_GIT=y
 BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://github.com/raspberrypi/linux.git"
-BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="746069c0ac473cfb4fd70415149b4cfdea193119"
+BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rpi-3.12.y"
 BR2_LINUX_KERNEL_USE_DEFCONFIG=y
 BR2_LINUX_KERNEL_DEFCONFIG="bcmrpi_quick"
 BR2_LINUX_KERNEL_ZIMAGE=y
-- 
1.8.3.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel
  2014-05-02 12:45 [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel Yann E. MORIN
@ 2014-05-02 13:09 ` Peter Korsgaard
  2014-05-02 13:12   ` Yann E. MORIN
  2014-05-02 19:18 ` Yann E. MORIN
  1 sibling, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2014-05-02 13:09 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Previously, when we were using the rpi-3.10.y branch, we coiuld use a sha1 to
 > refer to a specific changeset, and that would work and give reproducible
 > builds, thanks to upstream merging linux-stable into their branch.

 > Now they have declared rpi-3.12.y the new stable branch, and deprecated the
 > rpi-3.10.y branch.

 > Unfortunately, they are now rebasing the rpi-3.12.y branch instead of merging
 > linux-stable as they used to do. This means we can no longer use a sha1 as the
 > tree will change as soon as upstream rebases onto a new linux-stable.

 > There are two possibilities here:
 >   - revert back to using a rpi-3.10.y cset
 >   - use the rpi-3.12.y branch name

 > This patch goes with the second option, since rpi-3.10.y is no longer maitained,
 > and we prefer to use a stable branch, at the cost of reproducible builds.

Hmm, but that means that the builds are not reproducable :/

Can't we convince them to create a new branch whenever they rebase or
something like that?

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel
  2014-05-02 13:09 ` Peter Korsgaard
@ 2014-05-02 13:12   ` Yann E. MORIN
  0 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2014-05-02 13:12 UTC (permalink / raw)
  To: buildroot

Peter, All,

On 2014-05-02 15:09 +0200, Peter Korsgaard spake thusly:
> >>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> 
>  > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>  > Previously, when we were using the rpi-3.10.y branch, we coiuld use a sha1 to
>  > refer to a specific changeset, and that would work and give reproducible
>  > builds, thanks to upstream merging linux-stable into their branch.
> 
>  > Now they have declared rpi-3.12.y the new stable branch, and deprecated the
>  > rpi-3.10.y branch.
> 
>  > Unfortunately, they are now rebasing the rpi-3.12.y branch instead of merging
>  > linux-stable as they used to do. This means we can no longer use a sha1 as the
>  > tree will change as soon as upstream rebases onto a new linux-stable.
> 
>  > There are two possibilities here:
>  >   - revert back to using a rpi-3.10.y cset
>  >   - use the rpi-3.12.y branch name
> 
>  > This patch goes with the second option, since rpi-3.10.y is no longer maitained,
>  > and we prefer to use a stable branch, at the cost of reproducible builds.
> 
> Hmm, but that means that the builds are not reproducable :/

Indeed, that's a drawback.

> Can't we convince them to create a new branch whenever they rebase or
> something like that?

Already done:
    https://github.com/raspberrypi/linux/issues/584

I forgot to add this in my commit log. Can you do that before you apply,
please, or do you want me to re-submit?

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel
  2014-05-02 12:45 [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel Yann E. MORIN
  2014-05-02 13:09 ` Peter Korsgaard
@ 2014-05-02 19:18 ` Yann E. MORIN
  1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2014-05-02 19:18 UTC (permalink / raw)
  To: buildroot

All,

On 2014-05-02 14:45 +0200, Yann E. MORIN spake thusly:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> Previously, when we were using the rpi-3.10.y branch, we coiuld use a sha1 to
> refer to a specific changeset, and that would work and give reproducible
> builds, thanks to upstream merging linux-stable into their branch.
> 
> Now they have declared rpi-3.12.y the new stable branch, and deprecated the
> rpi-3.10.y branch.
> 
> Unfortunately, they are now rebasing the rpi-3.12.y branch instead of merging
> linux-stable as they used to do. This means we can no longer use a sha1 as the
> tree will change as soon as upstream rebases onto a new linux-stable.
> 
> There are two possibilities here:
>   - revert back to using a rpi-3.10.y cset
>   - use the rpi-3.12.y branch name
> 
> This patch goes with the second option, since rpi-3.10.y is no longer maitained,
> and we prefer to use a stable branch, at the cost of reproducible builds.
> 
> Reported-by: Jeremy Rosen <jeremy.rosen@openwide.fr>
> Signed-off-by: yann.morin.1998 at free.fr
> Cc: Jeremy Rosen <jeremy.rosen@openwide.fr>
> 
> ---
> Note: before the next release, we should revisit this, and if upstream has
> not switched to merging over rebasing, then we will just revert to using
> the latest cset from the rpi-3.10.y branch for the release.

OK, upstream has agreed to merge the stable branch:
    https://github.com/raspberrypi/linux/issues/584#issuecomment-42057891

So, I'll send an updated patch to use the latest cset instead of the
branch name.

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-05-02 19:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-02 12:45 [Buildroot] [PATCH] configs/raspberrypi: use the branch name instead of the sha1 for the linux kernel Yann E. MORIN
2014-05-02 13:09 ` Peter Korsgaard
2014-05-02 13:12   ` Yann E. MORIN
2014-05-02 19:18 ` Yann E. MORIN

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.