Re: no-preinstall: Ready for final review

2018-01-19 Thread Sebastian Huber
On 18/01/18 17:34, Gedare Bloom wrote: After installation. The BSP builds fine I fixed this for most BSPs. The rest will be fixed on Monday. -- Sebastian Huber, embedded brains GmbH Address : Dornierstr. 4, D-82178 Puchheim, Germany Phone : +49 89 189 47 41-16 Fax : +49 89 189 47 41-09

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
On 18/01/18 21:57, Gedare Bloom wrote: On Thu, Jan 18, 2018 at 3:34 PM, Chris Johns wrote: On 19/1/18 3:37 am, Gedare Bloom wrote: On Thu, Jan 18, 2018 at 11:36 AM, Sebastian Huber wrote: - Am 18. Jan 2018 um 16:06 schrieb Gedare bloomged...@rtems.org: [...] I also took a quick look t

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
On Thu, Jan 18, 2018 at 3:34 PM, Chris Johns wrote: > On 19/1/18 3:37 am, Gedare Bloom wrote: >> On Thu, Jan 18, 2018 at 11:36 AM, Sebastian Huber >> wrote: >>> >>> >>> - Am 18. Jan 2018 um 16:06 schrieb Gedare Bloom ged...@rtems.org: >>> >>> [...] I also took a quick look through the co

Re: no-preinstall: Ready for final review

2018-01-18 Thread Chris Johns
On 19/1/18 3:37 am, Gedare Bloom wrote: > On Thu, Jan 18, 2018 at 11:36 AM, Sebastian Huber > wrote: >> >> >> - Am 18. Jan 2018 um 16:06 schrieb Gedare Bloom ged...@rtems.org: >> >> [...] >>> I also took a quick look through the commits, and I have a small nit. >>> Might I suggest "headers.sh"

Re: no-preinstall: Ready for final review

2018-01-18 Thread Chris Johns
On 19/1/18 3:35 am, Sebastian Huber wrote: > v1 of the commit message: Looks good. Best to limit the number of problems with preinstalling to just 5 :) Chris ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
- Gedare Bloom schrieb: > On Thu, Jan 18, 2018 at 11:36 AM, Sebastian Huber > wrote: > > > > > > - Am 18. Jan 2018 um 16:06 schrieb Gedare Bloom ged...@rtems.org: > > > > [...] > >> I also took a quick look through the commits, and I have a small nit. > >> Might I suggest "headers.sh" an

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
On Thu, Jan 18, 2018 at 11:36 AM, Sebastian Huber wrote: > > > - Am 18. Jan 2018 um 16:06 schrieb Gedare Bloom ged...@rtems.org: > > [...] >> I also took a quick look through the commits, and I have a small nit. >> Might I suggest "headers.sh" and "headers.am" instead of "header.sh" >> and "he

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
- Am 18. Jan 2018 um 16:06 schrieb Gedare Bloom ged...@rtems.org: [...] > I also took a quick look through the commits, and I have a small nit. > Might I suggest "headers.sh" and "headers.am" instead of "header.sh" > and "header.am"? I want to add this stuff to the bootstrap script as a -p

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
After installation. The BSP builds fine On Thu, Jan 18, 2018 at 11:33 AM, Sebastian Huber wrote: > Is this after installation or during the BSP build? > > - Am 18. Jan 2018 um 17:19 schrieb Gedare Bloom ged...@rtems.org: > >> On Thu, Jan 18, 2018 at 10:34 AM, Gedare Bloom wrote: >>> On Thu,

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
v1 of the commit message: Remove make preinstall A speciality of the RTEMS build system was the make preinstall step. It copied header files from arbitrary locations into the build tree. The header files were included via the -Bsome/build/tree/path GCC command line option. This has at least fi

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
Is this after installation or during the BSP build? - Am 18. Jan 2018 um 17:19 schrieb Gedare Bloom ged...@rtems.org: > On Thu, Jan 18, 2018 at 10:34 AM, Gedare Bloom wrote: >> On Thu, Jan 18, 2018 at 10:20 AM, Sebastian Huber >> wrote: >>> I fixed the cpuopts.h problem two minutes ago. >>>

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
On Thu, Jan 18, 2018 at 10:34 AM, Gedare Bloom wrote: > On Thu, Jan 18, 2018 at 10:20 AM, Sebastian Huber > wrote: >> I fixed the cpuopts.h problem two minutes ago. >> > Thanks. > > I just noticed that actually when I run tester on the installed tree I > get a lot of timeouts. I'm not sure if thi

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
On Thu, Jan 18, 2018 at 10:20 AM, Sebastian Huber wrote: > I fixed the cpuopts.h problem two minutes ago. > Thanks. I just noticed that actually when I run tester on the installed tree I get a lot of timeouts. I'm not sure if this is my host problem. > - Gedare Bloom schrieb: >> On Wed, Jan

Re: no-preinstall: Ready for final review

2018-01-18 Thread Sebastian Huber
I fixed the cpuopts.h problem two minutes ago. - Gedare Bloom schrieb: > On Wed, Jan 17, 2018 at 2:24 AM, Sebastian Huber > wrote: > > Hello, > > > > the no-preinstall branch is now in a good shape. > > > > https://git.rtems.org/sebh/rtems.git/log/?h=no-preinstall > > > > Please review this

Re: no-preinstall: Ready for final review

2018-01-18 Thread Gedare Bloom
On Wed, Jan 17, 2018 at 2:24 AM, Sebastian Huber wrote: > Hello, > > the no-preinstall branch is now in a good shape. > > https://git.rtems.org/sebh/rtems.git/log/?h=no-preinstall > > Please review this work. The next step would be to squash all commits on > this branch, write a proper commit mess

no-preinstall: Ready for final review

2018-01-16 Thread Sebastian Huber
Hello, the no-preinstall branch is now in a good shape. https://git.rtems.org/sebh/rtems.git/log/?h=no-preinstall Please review this work. The next step would be to squash all commits on this branch, write a proper commit message and push it to the master. -- Sebastian Huber, embedded brains