[igt-dev] [PATCH i-g-t] README: Add missing dependencies

[Petri Latvala]

On Wed, Jul 31, 2019 at 10:01:23AM +0300, Ser, Simon wrote:
> On Tue, 2019-07-30 at 18:00 +0000, Souza, Jose wrote:
> > On Tue, 2019-07-30 at 14:15 +0300, Petri Latvala wrote:
> > > On Mon, Jul 29, 2019 at 03:26:06PM -0700, José Roberto de Souza
> > > wrote:
> > > > Doing a fresh Ubuntu 18.10 install this dependencies were also
> > > > needed to build IGT.
> > > > 
> > > > Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
> > > > ---
> > > >  README.md | 3 +++
> > > >  1 file changed, 3 insertions(+)
> > > > 
> > > > diff --git a/README.md b/README.md
> > > > index 865ed6e0..e9798419 100644
> > > > --- a/README.md
> > > > +++ b/README.md
> > > > @@ -139,13 +139,16 @@ Requirements
> > > >  This is a non-exhaustive list of package dependencies required for
> > > > building
> > > >  the default configuration (package names may vary):
> > > >  
> > > > +	bison	
> > > 
> > > You have extra whitespace at the end of this line.
> > 
> > Thanks for spotting this.
> > 
> > > 
> > > With that fixed,
> > > Reviewed-by: Petri Latvala <petri.latvala at intel.com>
> > 
> > Thanks, fixed and pushed.
> > 
> > > I wonder if it's soon time to remove the list of packages and
> > > emphasise the reference to dockerfiles...
> > 
> > Well I don't plan to run IGT inside of docker my one cent vote is to
> > keep it.
> 
> I guess the idea would be to state "refer to Dockerfiles for a list of
> dependencies" (as opposed to "run IGT inside Docker").


Yes, that's what I meant. There is already this line in README after
the package list:

 See Dockerfiles.* for package names in different distributions.



-- 
Petri Latvala