bugGNU GRUB - Bugs: bug #27361, Minimum version of Bison not...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #27361: Minimum version of Bison not advised

Submitted by:  Neil Cafferkey <ncafferkey>
Submitted on:  Tue 01 Sep 2009 01:02:39 AM UTC  
 
Category: CompilationSeverity: Major
Priority: 5 - NormalItem Group: Non-software Error
Status: FixedPrivacy: Public
Assigned to: NoneOriginator Name: 
Open/Closed: ClosedRelease: r2563
Release: Bazaar - trunkReproducibility: Every Time
Planned Release: None

(Jump to the original submission Jump to the original submission)

Thu 03 Sep 2009 03:37:39 PM UTC, comment #6:

Yep that's what I only did

Felix Zielcke <fzielcke>
Project Member
Thu 03 Sep 2009 03:30:02 PM UTC, comment #5:

If you're just changing the INSTALL instead of forcing the version in configure.ac, then it's fine by me.

Bean <bean>
Thu 03 Sep 2009 03:24:43 PM UTC, comment #4:

well it's just INSTALL so people are still free to try older versions.
But if even CentOS 5.3 ships Bison 2.3 I think that's recent enough for everyone.
RHEL is using mostly older software.
INSTALL also says Ruby 1.6 or Binutils 2.9.1.0.23 is required
and this isn't either checked in configure.ac.
Just documented.

Felix Zielcke <fzielcke>
Project Member
Thu 03 Sep 2009 03:03:19 PM UTC, comment #3:

I don't like the idea of bumping up software version to solve certain problem, this may break other system, I don't think all os ships with bison 2.3 by default. In fact, you should check if the specific feature is missing. In the case of bison 1.28, it's these two lines that cause the problem:

%lex-param { struct grub_parser_param *state };
%parse-param { struct grub_parser_param *state };

%lex-param and %parse-param is not supported in bison 1.28, it uses the now deprecated YYLEX_PARAM and YYPARSE_PARAM macro to set parameter. It's better to write a configure.ac test to check for this feature explicitly.

Bean <bean>
Thu 03 Sep 2009 02:13:22 PM UTC, comment #2:

Ok I just edited with approval from Robert INSTALL so it says Bison 2.3 or later is required.
I didn't bother to check if earlier versions work too.
All recent Linux distros should have at least 2.3

Felix Zielcke <fzielcke>
Project Member
Thu 03 Sep 2009 09:00:39 AM UTC, comment #1:

Uhm Bison 1.28 was released 1999
Bison 2.3 which I think should still work fine with current trunk was released 2006
Hm ok Mac OS X 10.4 was released in 2005 but then they shouldn't use a 6 year old Bison.

Felix Zielcke <fzielcke>
Project Member
Tue 01 Sep 2009 01:02:39 AM UTC, original submission:

The INSTALL file mentions that Bison is required, but doesn't state the minimum version. Under Mac OS X 10.4, I found that GRUB 2 (SVN revision 2525) couldn't be compiled with Bison 1.28.

Neil Cafferkey <ncafferkey>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bean (Posted a comment)
  • -unavailable- added by fzielcke (Posted a comment)
  • -unavailable- added by ncafferkey (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 03 Sep 2009 02:13:22 PM UTCfzielckeStatusNone=>Fixed
      Open/ClosedOpen=>Closed
      Release=>r2563

    Back to the top


    Powered by Savane 3.1-cleanup1