[SeaBIOS] [PATCH] build: Require "make VERSION=x" parameter to be on command line

Kevin O'Connor kevin at koconnor.net
Thu Aug 13 13:06:39 CET 2015


On Thu, Aug 13, 2015 at 07:59:42AM +0200, Paul Menzel wrote:
> Am Mittwoch, den 12.08.2015, 18:32 -0400 schrieb Kevin O'Connor:
> > Don't pull in VERSION from the build environment - there is too great
> > a potential for a variable conflict.
> > 
> > Signed-off-by: Kevin O'Connor <kevin at koconnor.net>
> > ---
> >  Makefile | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Makefile b/Makefile
> > index e287530..220a9f7 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -26,6 +26,7 @@ PYTHON=python
> >  CPP=cpp
> >  IASL:=iasl
> >  LD32BIT_FLAG:=-melf_i386
> > +VERSION:=
> 
> What about changing the name to `SEABIOS_VERSION` instead to avoid
> possible conflicts?

Thanks for reviewing.

I prefer to avoid pulling in settings from the environment because I
find the extra complexity confusing.

That said, I don't plan to commit this patch.  After further thought,
I think the 'make VERSION=x' support was a mistake and should be
reverted.

-Kevin



More information about the SeaBIOS mailing list