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

Submitter:  Neil Cafferkey <ncafferkey>
Submitted:  Tue 01 Sep 2009 01:02:39 AM UTC
   
 
Category:  Compilation Severity:  Major
Priority:  5 - Normal Item Group:  Non-software Error
Status:  Fixed Privacy:  Public
Assigned to:  None Originator Name: 
Open/Closed:  Closed Release:  Bazaar - trunk
Release:  r2563 Reproducibility:  Every Time
Planned Release:  None

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>
Group 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>
Group 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>
Group 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>
Group 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>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-09-03 fzielcke StatusNone Fixed
        Open/ClosedOpen Closed
        Release r2563

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code