Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Chris Johns
On 22/7/2022 8:59 am, Vijay Kumar Banerjee wrote:
> Patch pushed. The repository is working, and I see the commit email in vc.

Nice. Thanks for the feedback.

Chris
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Vijay Kumar Banerjee
Patch pushed. The repository is working, and I see the commit email in vc.


Thanks,
Vijay



On Thu, Jul 21, 2022 at 2:06 PM Vijay Kumar Banerjee  wrote:
>
> I will push this patch. The main intention of this patch is to test
> the new top-level repo, but it also provides some essential build
> instructions which will certainly be handy.
>
> On Thu, Jul 21, 2022 at 2:05 PM Vijay Kumar Banerjee  wrote:
> >
> > Hi Kinsey,
> >
> > thanks for the review
> >
> > On Thu, Jul 21, 2022 at 1:50 PM Kinsey Moore  
> > wrote:
> > >
> > > Looks good to me. I never realized that the list of BSPs and tools
> > > directory could be omitted if things were installed in particular 
> > > locations.
> > >
> > Yes, it picks up all the installed BSPs if there is that option is not
> > used. using the BSP argument only builds the mentioned bsps.
> >
> >
> > Best regards,
> > Vijay
> > > On 7/21/2022 14:16, Vijay Kumar Banerjee wrote:
> > > > ---
> > > >   README | 18 ++
> > > >   1 file changed, 18 insertions(+)
> > > >
> > > > diff --git a/README b/README
> > > > index 91e42ae..99800bf 100644
> > > > --- a/README
> > > > +++ b/README
> > > > @@ -12,3 +12,21 @@ The sources presented here originate in one of 
> > > > several locations described by
> > > >   the ORIGIN.* files and whose license is described by the COPYING.* 
> > > > files.
> > > >   Commits adding such files should include the hash of the target 
> > > > repository if
> > > >   applicable.
> > > > +
> > > > +Installation Instructions
> > > > +-
> > > > +1. Populate the git submodules:
> > > > +
> > > > +```
> > > > +git submodule init
> > > > +git submodule update
> > > > +```
> > > > +2. Configure and build
> > > > +```
> > > > +./waf configure --prefix=INSTALL_PREFIX
> > > > +./waf
> > > > +./waf install
> > > > +```
> > > > +
> > > > +More `waf` arguments can be found by using:
> > > > +`./waf --help`
> > > ___
> > > devel mailing list
> > > devel@rtems.org
> > > http://lists.rtems.org/mailman/listinfo/devel
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Vijay Kumar Banerjee
I will push this patch. The main intention of this patch is to test
the new top-level repo, but it also provides some essential build
instructions which will certainly be handy.

On Thu, Jul 21, 2022 at 2:05 PM Vijay Kumar Banerjee  wrote:
>
> Hi Kinsey,
>
> thanks for the review
>
> On Thu, Jul 21, 2022 at 1:50 PM Kinsey Moore  wrote:
> >
> > Looks good to me. I never realized that the list of BSPs and tools
> > directory could be omitted if things were installed in particular locations.
> >
> Yes, it picks up all the installed BSPs if there is that option is not
> used. using the BSP argument only builds the mentioned bsps.
>
>
> Best regards,
> Vijay
> > On 7/21/2022 14:16, Vijay Kumar Banerjee wrote:
> > > ---
> > >   README | 18 ++
> > >   1 file changed, 18 insertions(+)
> > >
> > > diff --git a/README b/README
> > > index 91e42ae..99800bf 100644
> > > --- a/README
> > > +++ b/README
> > > @@ -12,3 +12,21 @@ The sources presented here originate in one of several 
> > > locations described by
> > >   the ORIGIN.* files and whose license is described by the COPYING.* 
> > > files.
> > >   Commits adding such files should include the hash of the target 
> > > repository if
> > >   applicable.
> > > +
> > > +Installation Instructions
> > > +-
> > > +1. Populate the git submodules:
> > > +
> > > +```
> > > +git submodule init
> > > +git submodule update
> > > +```
> > > +2. Configure and build
> > > +```
> > > +./waf configure --prefix=INSTALL_PREFIX
> > > +./waf
> > > +./waf install
> > > +```
> > > +
> > > +More `waf` arguments can be found by using:
> > > +`./waf --help`
> > ___
> > devel mailing list
> > devel@rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Vijay Kumar Banerjee
Hi Kinsey,

thanks for the review

On Thu, Jul 21, 2022 at 1:50 PM Kinsey Moore  wrote:
>
> Looks good to me. I never realized that the list of BSPs and tools
> directory could be omitted if things were installed in particular locations.
>
Yes, it picks up all the installed BSPs if there is that option is not
used. using the BSP argument only builds the mentioned bsps.


Best regards,
Vijay
> On 7/21/2022 14:16, Vijay Kumar Banerjee wrote:
> > ---
> >   README | 18 ++
> >   1 file changed, 18 insertions(+)
> >
> > diff --git a/README b/README
> > index 91e42ae..99800bf 100644
> > --- a/README
> > +++ b/README
> > @@ -12,3 +12,21 @@ The sources presented here originate in one of several 
> > locations described by
> >   the ORIGIN.* files and whose license is described by the COPYING.* files.
> >   Commits adding such files should include the hash of the target 
> > repository if
> >   applicable.
> > +
> > +Installation Instructions
> > +-
> > +1. Populate the git submodules:
> > +
> > +```
> > +git submodule init
> > +git submodule update
> > +```
> > +2. Configure and build
> > +```
> > +./waf configure --prefix=INSTALL_PREFIX
> > +./waf
> > +./waf install
> > +```
> > +
> > +More `waf` arguments can be found by using:
> > +`./waf --help`
> ___
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Kinsey Moore
Looks good to me. I never realized that the list of BSPs and tools 
directory could be omitted if things were installed in particular locations.


On 7/21/2022 14:16, Vijay Kumar Banerjee wrote:

---
  README | 18 ++
  1 file changed, 18 insertions(+)

diff --git a/README b/README
index 91e42ae..99800bf 100644
--- a/README
+++ b/README
@@ -12,3 +12,21 @@ The sources presented here originate in one of several 
locations described by
  the ORIGIN.* files and whose license is described by the COPYING.* files.
  Commits adding such files should include the hash of the target repository if
  applicable.
+
+Installation Instructions
+-
+1. Populate the git submodules:
+
+```
+git submodule init
+git submodule update
+```
+2. Configure and build
+```
+./waf configure --prefix=INSTALL_PREFIX
+./waf
+./waf install
+```
+
+More `waf` arguments can be found by using:
+`./waf --help`

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


Re: [PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Kinsey Moore

On 7/21/2022 14:16, Vijay Kumar Banerjee wrote:

---
  README | 18 ++
  1 file changed, 18 insertions(+)

diff --git a/README b/README
index 91e42ae..99800bf 100644
--- a/README
+++ b/README
@@ -12,3 +12,21 @@ The sources presented here originate in one of several 
locations described by
  the ORIGIN.* files and whose license is described by the COPYING.* files.
  Commits adding such files should include the hash of the target repository if
  applicable.
+
+Installation Instructions
+-
+1. Populate the git submodules:
+
+```
+git submodule init
+git submodule update
+```
+2. Configure and build
+```
+./waf configure --prefix=INSTALL_PREFIX
Can this be run without specifying which BSPs it's being configured for? 
I've only ever configured it with an explicit list.

+./waf
+./waf install
+```
+
+More `waf` arguments can be found by using:
+`./waf --help`

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel


[PATCH rtems-lwip] README: Add build instruction

2022-07-21 Thread Vijay Kumar Banerjee
---
 README | 18 ++
 1 file changed, 18 insertions(+)

diff --git a/README b/README
index 91e42ae..99800bf 100644
--- a/README
+++ b/README
@@ -12,3 +12,21 @@ The sources presented here originate in one of several 
locations described by
 the ORIGIN.* files and whose license is described by the COPYING.* files.
 Commits adding such files should include the hash of the target repository if
 applicable.
+
+Installation Instructions
+-
+1. Populate the git submodules:
+
+```
+git submodule init
+git submodule update
+```
+2. Configure and build
+```
+./waf configure --prefix=INSTALL_PREFIX
+./waf
+./waf install
+```
+
+More `waf` arguments can be found by using:
+`./waf --help`
-- 
2.34.3

___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel