build timestamps don't work

build timestamps don't work

[Antony Pavlov]

Hi!

  I have found that on my Debian testing build system generation of
timestamps don't work correctly.

$ cd barebox.git
$ git checkout next
$ make mrproper
$ make qemu-malta_defconfig
$ make
$ cat .version
0
$ edit ...
$ edit ...
$ edit ...
$ make
$ cat .version
0
$ echo 4 > .version
$ make
$ edit ...
$ edit ...
$ edit ...
$ make
$ cat .version
4

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Antony Pavlov]

ping

On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
> Hi!
>
>  I have found that on my Debian testing build system generation of
> timestamps don't work correctly.
>
> $ cd barebox.git
> $ git checkout next
> $ make mrproper
> $ make qemu-malta_defconfig
> $ make
> $ cat .version
> 0
> $ edit ...
> $ edit ...
> $ edit ...
> $ make
> $ cat .version
> 0
> $ echo 4 > .version
> $ make
> $ edit ...
> $ edit ...
> $ edit ...
> $ make
> $ cat .version
> 4
>
> --
> Best regards,
>   Antony Pavlov



-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Sascha Hauer]

On Thu, May 17, 2012 at 09:52:36PM +0400, Antony Pavlov wrote:
> ping
> 
> On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
> > Hi!
> >
> >  I have found that on my Debian testing build system generation of
> > timestamps don't work correctly.

I just tried and it works with ARM builds. Tried again with MIPS, and
yes, indeed it does not work. Maybe worth looking at the architecture
specific Makefile.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Antony Pavlov]

On 17 May 2012 22:19, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> On Thu, May 17, 2012 at 09:52:36PM +0400, Antony Pavlov wrote:
>> ping
>>
>> On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
>> > Hi!
>> >
>> >  I have found that on my Debian testing build system generation of
>> > timestamps don't work correctly.
>
> I just tried and it works with ARM builds. Tried again with MIPS, and
> yes, indeed it does not work. Maybe worth looking at the architecture
> specific Makefile.

Ok, I'll look.

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Antony Pavlov]

On 17 May 2012 22:19, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> On Thu, May 17, 2012 at 09:52:36PM +0400, Antony Pavlov wrote:
>> ping
>>
>> On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
>> > Hi!
>> >
>> >  I have found that on my Debian testing build system generation of
>> > timestamps don't work correctly.
>
> I just tried and it works with ARM builds. Tried again with MIPS, and
> yes, indeed it does not work. Maybe worth looking at the architecture
> specific Makefile.
>

I tried another debian linux. I tried ARM & MIPS: the both don't work.

I checked linux-3.1 build for mips: it work. At least  there is one differrence:
in linux if i make 'rm .version && make', then scripts/mkcompile_h
started _two_ times, but only _one_ time in barebox.

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Sascha Hauer]

On Fri, May 18, 2012 at 12:28:52AM +0400, Antony Pavlov wrote:
> On 17 May 2012 22:19, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> > On Thu, May 17, 2012 at 09:52:36PM +0400, Antony Pavlov wrote:
> >> ping
> >>
> >> On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
> >> > Hi!
> >> >
> >> >  I have found that on my Debian testing build system generation of
> >> > timestamps don't work correctly.
> >
> > I just tried and it works with ARM builds. Tried again with MIPS, and
> > yes, indeed it does not work. Maybe worth looking at the architecture
> > specific Makefile.
> >
> 
> I tried another debian linux. I tried ARM & MIPS: the both don't work.
> 
> I checked linux-3.1 build for mips: it work. At least  there is one differrence:
> in linux if i make 'rm .version && make', then scripts/mkcompile_h
> started _two_ times, but only _one_ time in barebox.

Just to make sure, you have the following in your tree, right?

commit 55c81081e577922a8d7d0a1d633efb7feb3560d4
Author: Sascha Hauer <s.hauer@pengutronix.de>
Date:   Sun Apr 15 12:05:21 2012 +0200

    Makefile: fix build timestamps
    
    This patch adds the scripts/mkcompile_h from the Kernel. This gives
    us additional information about the build like who built this binary
    and with which compiler. Currently the only information used is the
    build timestamp and the generation number.
    This also fixes the build timestamp in the banner which is now
correct
    even with consecutive builds without making clean in between.
    
    Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: build timestamps don't work

[Antony Pavlov]

On 18 May 2012 12:49, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> On Fri, May 18, 2012 at 12:28:52AM +0400, Antony Pavlov wrote:
>> On 17 May 2012 22:19, Sascha Hauer <s.hauer@pengutronix.de> wrote:
>> > On Thu, May 17, 2012 at 09:52:36PM +0400, Antony Pavlov wrote:
>> >> ping
>> >>
>> >> On 16 May 2012 08:27, Antony Pavlov <antonynpavlov@gmail.com> wrote:
>> >> > Hi!
>> >> >
>> >> >  I have found that on my Debian testing build system generation of
>> >> > timestamps don't work correctly.
>> >
>> > I just tried and it works with ARM builds. Tried again with MIPS, and
>> > yes, indeed it does not work. Maybe worth looking at the architecture
>> > specific Makefile.
>> >
>>
>> I tried another debian linux. I tried ARM & MIPS: the both don't work.
>>
>> I checked linux-3.1 build for mips: it work. At least  there is one differrence:
>> in linux if i make 'rm .version && make', then scripts/mkcompile_h
>> started _two_ times, but only _one_ time in barebox.
>
> Just to make sure, you have the following in your tree, right?
>
> commit 55c81081e577922a8d7d0a1d633efb7feb3560d4
> Author: Sascha Hauer <s.hauer@pengutronix.de>
> Date:   Sun Apr 15 12:05:21 2012 +0200
>
>    Makefile: fix build timestamps
>
>    This patch adds the scripts/mkcompile_h from the Kernel. This gives
>    us additional information about the build like who built this binary
>    and with which compiler. Currently the only information used is the
>    build timestamp and the generation number.
>    This also fixes the build timestamp in the banner which is now
> correct
>    even with consecutive builds without making clean in between.
>
>    Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Yes, I have this commit in my tree. I used the 'next' branch.

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox